Streamline account templates

i-lateral · Feb 22, 2018 · b7577ba · b7577ba
1 parent debec2f
commit b7577ba
Show file tree

Hide file tree

Showing 6 changed files with 45 additions and 63 deletions.
diff --git a/code/control/Users_Account_Controller.php b/code/control/Users_Account_Controller.php
@@ -139,14 +139,33 @@ public function RelativeLink($action = null)
      */
     public function index()
     {
+        // Setup default profile summary sections
+        $sections = ArrayList::create();
+
+        $sections->push(ArrayData::create(array(
+            "Title" => "",
+            "Content" => $this->renderWith(
+                "UsersProfileSummary",
+                array("CurrentUser" => Member::currentUser())
+            )
+        )));
+
+        // Allow users to add extra content sections to the
+        // summary
+        $this->extend("updateContentSections", $sections);
+
         $this->customise(array(
-            "ClassName" => "AccountPage"
+            "Title" => _t('Users.ProfileSummary', 'Profile Summary'),
+            "MetaTitle" => _t('Users.ProfileSummary', 'Profile Summary'),
+            "Content" => $this->renderWith(
+                "UsersAccountSections",
+                array("Sections" => $sections)
+            )
         ));
 
         $this->extend("onBeforeIndex");
 
         return $this->renderWith(array(
-            "UserAccount",
             "UserAccount",
             "Page"
         ));
@@ -162,7 +181,8 @@ public function edit()
         }
 
         $this->customise(array(
-            "ClassName" => "AccountPage",
+            "Title" => _t("Users.EditAccountDetails", "Edit account details"),
+            "MetaTitle" => _t("Users.EditAccountDetails", "Edit account details"),
             "Form"  => $form
         ));
 
@@ -197,7 +217,8 @@ public function changepassword()
         }
 
         $this->customise(array(
-            "ClassName" => "AccountPage",
+            "Title" => _t("Security.ChangeYourPassword", "Change your password"),
+            "MetaTitle" => _t("Security.ChangeYourPassword", "Change your password"),
             "Form"  => $form
         ));
 
@@ -263,25 +284,25 @@ public function ChangePasswordForm()
      */
     public function getAccountMenu()
     {
-        $menu = new ArrayList();
+        $menu = ArrayList::create();
 
         $curr_action = $this->request->param("Action");
 
-        $menu->add(new ArrayData(array(
+        $menu->add(ArrayData::create(array(
             "ID"    => 0,
             "Title" => _t('Users.PROFILESUMMARY', "Profile Summary"),
             "Link"  => $this->Link(),
             "LinkingMode" => (!$curr_action) ? "current" : "link"
         )));
 
-        $menu->add(new ArrayData(array(
+        $menu->add(ArrayData::create(array(
             "ID"    => 10,
             "Title" => _t('Users.EDITDETAILS', "Edit account details"),
             "Link"  => $this->Link("edit"),
             "LinkingMode" => ($curr_action == "edit") ? "current" : "link"
         )));
 
-        $menu->add(new ArrayData(array(
+        $menu->add(ArrayData::create(array(
             "ID"    => 30,
             "Title" => _t('Users.CHANGEPASSWORD', "Change password"),
             "Link"  => $this->Link("changepassword"),

diff --git a/templates/Includes/UsersAccountSections.ss b/templates/Includes/UsersAccountSections.ss
@@ -0,0 +1,5 @@
+<% loop $Sections %>
+    <h2>$Title</h2>
+
+    <div class="section-content">$Content</div>
+<% end_loop %>
diff --git a/templates/Includes/UsersProfileSummary.ss b/templates/Includes/UsersProfileSummary.ss
@@ -0,0 +1,9 @@
+<% with $CurrentUser %>
+    <div class="users-profile-summary">
+        <p>
+            <strong><% _t('Member.FIRSTNAME',"First Name") %></strong> $FirstName<br/>
+            <strong><% _t('Member.SURNAME',"Surname") %></strong> $Surname<br/>
+            <strong><% _t("Member.EMAIL","Email") %></strong> $Email<br/>
+        </p>
+    </div>
+<% end_with %>
diff --git a/templates/Layout/UserAccount.ss b/templates/Layout/UserAccount.ss
@@ -14,18 +14,9 @@
                 </div>
             <% end_if %>
 
-            <h1><%t Users.ProfileSummary "Profile Summary" %></h1>
-
-            <% with $CurrentUser %>
-                <div class="users-profile-summary">
-                    <p>
-                        <strong><% _t('Member.FIRSTNAME',"First Name") %></strong> $FirstName<br/>
-                        <strong><% _t('Member.SURNAME',"Surname") %></strong> $Surname<br/>
-                        <strong><% _t("Member.EMAIL","Email") %></strong> $Email<br/>
-                    </p>
-                </div>
-            <% end_with %>
+            <h1>$Title</h1>
 
+            <div class="content">$Content</div>
 
             $Form
         </div>

diff --git a/templates/Layout/UserAccount_changepassword.ss b/templates/Layout/UserAccount_changepassword.ss
diff --git a/templates/Layout/UserAccount_edit.ss b/templates/Layout/UserAccount_edit.ss