MDL-49688 navigation: First review fixes and version bump.
authorAdrian Greeve <adrian@moodle.com>
Fri, 27 Mar 2015 08:17:30 +0000 (16:17 +0800)
committerAdrian Greeve <adrian@moodle.com>
Fri, 10 Apr 2015 09:18:03 +0000 (17:18 +0800)
Part of MDL-45774.

22 files changed:
admin/roles/assign.php
blog/edit.php
blog/index.php
course/user.php
lib/db/upgrade.php
lib/myprofilelib.php
lib/navigationlib.php
lib/outputrenderers.php
message/index.php
message/output/email/message_output_email.php
mod/forum/lib.php
mod/forum/user.php
my/index.php
user/edit.php
user/editadvanced.php
user/editlib.php
user/editor.php
user/forum.php
user/forum_form.php
user/language.php
user/tests/fixtures/myprofile_fixtures.php
user/view.php

index 78b496a..fee876b 100644 (file)
@@ -135,6 +135,11 @@ if ($roleid) {
     }
 }
 
+if ($user->id != $USER->id) {
+    $PAGE->navigation->extend_for_user($user);
+    $PAGE->navbar->includesettingsbase = true;
+}
+
 $PAGE->set_pagelayout('admin');
 $PAGE->set_title($title);
 
index 1963c28..24612dd 100644 (file)
@@ -48,8 +48,20 @@ if (!empty($id) && $action == 'add') {
     $id = null;
 }
 
+$entry = new stdClass();
+$entry->id = null;
+
+if ($id) {
+    if (!$entry = new blog_entry($id)) {
+        print_error('wrongentryid', 'blog');
+    }
+    $userid = $entry->userid;
+} else {
+    $userid = $USER->id;
+}
+
 $sitecontext = context_system::instance();
-$usercontext = context_user::instance($USER->id);
+$usercontext = context_user::instance($userid);
 $PAGE->set_context($usercontext);
 $blognode = $PAGE->settingsnav->find('blogadd', null);
 $blognode->make_active();
@@ -85,24 +97,15 @@ if (!has_capability('moodle/blog:create', $sitecontext) && !has_capability('mood
 
 // Make sure that the person trying to edit has access right.
 if ($id) {
-    if (!$entry = new blog_entry($id)) {
-        print_error('wrongentryid', 'blog');
-    }
-
     if (!blog_user_can_edit_entry($entry)) {
         print_error('notallowedtoedit', 'blog');
     }
-    $userid = $entry->userid;
     $entry->subject      = clean_text($entry->subject);
     $entry->summary      = clean_text($entry->summary, $entry->format);
-
 } else {
     if (!has_capability('moodle/blog:create', $sitecontext)) {
         print_error('noentry', 'blog'); // The capability "manageentries" is not enough for adding.
     }
-    $entry  = new stdClass();
-    $entry->id = null;
-    $userid = $USER->id;
 }
 $returnurl->param('userid', $userid);
 
index 0f43d24..35bab0c 100644 (file)
@@ -57,6 +57,12 @@ if (!empty($tag)) {
     }
 }
 
+// Set the userid to the entry author if we have the entry ID.
+if ($entryid and !isset($userid)) {
+    $entry = new blog_entry($entryid);
+    $userid = $entry->userid;
+}
+
 if (isset($userid)) {
     $context = context_user::instance($userid);
 } else {
index 9aedcef..c1f0c69 100644 (file)
@@ -154,7 +154,7 @@ switch ($mode) {
                 // Add the user name.
                 $PAGE->navbar->add(fullname($user), $userurl, navigation_node::TYPE_SETTING);
             }
-            $gradeurl = new moodle_url('/course/user.php', array('id'=>$id, 'user'=>$user->id, 'mode'=>$mode));
+            $gradeurl = new moodle_url('/course/user.php', array('id' => $id, 'user' => $user->id, 'mode' => $mode));
             // Add the 'grades' node to the navbar.
             $navbar = $PAGE->navbar->add(get_string('grades', 'grades'), $gradeurl, navigation_node::TYPE_SETTING);
         }
index 1873ce0..782c84c 100644 (file)
@@ -4270,6 +4270,7 @@ function xmldb_main_upgrade($oldversion) {
         upgrade_main_savepoint(true, 2015032000.00);
     }
 
+<<<<<<< HEAD
     if ($oldversion < 2015040200.01) {
         // Force uninstall of deleted tool.
         if (!file_exists("$CFG->dirroot/$CFG->admin/tool/timezoneimport")) {
@@ -4315,7 +4316,7 @@ function xmldb_main_upgrade($oldversion) {
         upgrade_main_savepoint(true, 2015040800.01);
     }
 
-    if ($oldversion < 2015030400.02) {
+    if ($oldversion < 2015040800.02) {
         // Update the default user menu (add preferences, remove my files and my badges).
         $oldconfig = get_config('core', 'customusermenuitems');
 
@@ -4334,7 +4335,7 @@ function xmldb_main_upgrade($oldversion) {
         $newconfig = preg_replace('/(\r\n)+/', "\n", $newconfig);
         set_config('customusermenuitems', $newconfig);
 
-        upgrade_main_savepoint(true, 2015030400.02);
+        upgrade_main_savepoint(true, 2015040800.02);
     }
 
     return true;
index d436e70..8c12a46 100644 (file)
@@ -83,7 +83,11 @@ function core_myprofile_navigation(core_user\output\myprofile\tree $tree, $user,
             if ($userauthplugin && $userauthplugin->can_edit_profile()) {
                 $url = $userauthplugin->edit_profile_url();
                 if (empty($url)) {
-                    $url = new moodle_url('/user/edit.php', array('userid' => $user->id, 'course' => $id));
+                    if (empty($course)) {
+                        $url = new moodle_url('/user/edit.php', array('userid' => $user->id));
+                    } else {
+                        $url = new moodle_url('/user/edit.php', array('userid' => $user->id, 'course' => $course->id));
+                    }
                 }
                 $node = new core_user\output\myprofile\node('administration', 'editprofile',
                         get_string('editmyprofile'), null, $url);
index 98eaff5..2e18723 100644 (file)
@@ -2162,7 +2162,7 @@ class global_navigation extends navigation_node {
             // This is the site so add a users node to the root branch.
             $usersnode = $this->rootnodes['users'];
             if (has_capability('moodle/course:viewparticipants', $coursecontext)) {
-                $usersnode->action = new moodle_url('/user/index.php', array('id'=>$course->id));
+                $usersnode->action = new moodle_url('/user/index.php', array('id' => $course->id));
             }
             $userviewurl = new moodle_url('/user/profile.php', $baseargs);
         }
@@ -2189,9 +2189,9 @@ class global_navigation extends navigation_node {
             if ($iscurrentuser || has_capability('moodle/user:viewdetails', $coursecontext) ||
                     has_capability('moodle/user:viewdetails', $usercontext)) {
                 if ($issitecourse || ($iscurrentuser && !$forceforcontext)) {
-                    $usernode->add(get_string('viewprofile'), new moodle_url('/user/profile.php',$baseargs));
+                    $usernode->add(get_string('viewprofile'), new moodle_url('/user/profile.php', $baseargs));
                 } else {
-                    $usernode->add(get_string('viewprofile'), new moodle_url('/user/view.php',$baseargs));
+                    $usernode->add(get_string('viewprofile'), new moodle_url('/user/view.php', $baseargs));
                 }
             }
 
@@ -2202,7 +2202,7 @@ class global_navigation extends navigation_node {
                 $forumtab = $usernode->add(get_string('forumposts', 'forum'));
                 $forumtab->add(get_string('posts', 'forum'), new moodle_url('/mod/forum/user.php', $baseargs));
                 $forumtab->add(get_string('discussions', 'forum'), new moodle_url('/mod/forum/user.php',
-                        array_merge($baseargs, array('mode'=>'discussions'))));
+                        array_merge($baseargs, array('mode' => 'discussions'))));
             }
 
             // Add blog nodes.
@@ -2239,7 +2239,7 @@ class global_navigation extends navigation_node {
                 if ($course->id != $SITE->id) {
                     $messageargs['viewing'] = MESSAGE_VIEW_COURSE. $course->id;
                 }
-                $url = new moodle_url('/message/index.php',$messageargs);
+                $url = new moodle_url('/message/index.php', $messageargs);
                 $usernode->add(get_string('messages', 'message'), $url, self::TYPE_SETTING, null, 'messages');
             }
 
@@ -2259,7 +2259,7 @@ class global_navigation extends navigation_node {
             // Add a node to view the users notes if permitted.
             if (!empty($CFG->enablenotes) &&
                     has_any_capability(array('moodle/notes:manage', 'moodle/notes:view'), $coursecontext)) {
-                $url = new moodle_url('/notes/index.php',array('user'=>$user->id));
+                $url = new moodle_url('/notes/index.php', array('user' => $user->id));
                 if ($coursecontext->instanceid != SITEID) {
                     $url->param('course', $coursecontext->instanceid);
                 }
@@ -2291,7 +2291,7 @@ class global_navigation extends navigation_node {
 
                 // Add view grade report is permitted.
                 $reports = core_component::get_plugin_list('gradereport');
-                arsort($reports); // user is last, we want to test it first.
+                arsort($reports); // User is last, we want to test it first.
 
                 $userscourses = enrol_get_users_courses($user->id);
                 $userscoursesnode = $usernode->add(get_string('courses'));
@@ -2307,13 +2307,13 @@ class global_navigation extends navigation_node {
                     $usercoursecontext = context_course::instance($usercourse->id);
                     $usercourseshortname = format_string($usercourse->shortname, true, array('context' => $usercoursecontext));
                     $usercoursenode = $userscoursesnode->add($usercourseshortname, new moodle_url('/user/view.php',
-                            array('id'=>$user->id, 'course'=>$usercourse->id)), self::TYPE_CONTAINER);
+                            array('id' => $user->id, 'course' => $usercourse->id)), self::TYPE_CONTAINER);
 
                     $gradeavailable = has_capability('moodle/grade:viewall', $usercoursecontext);
                     if (!$gradeavailable && !empty($usercourse->showgrades) && is_array($reports) && !empty($reports)) {
                         foreach ($reports as $plugin => $plugindir) {
                             if (has_capability('gradereport/'.$plugin.':view', $usercoursecontext)) {
-                                //stop when the first visible plugin is found.
+                                // Stop when the first visible plugin is found.
                                 $gradeavailable = true;
                                 break;
                             }
@@ -2321,7 +2321,7 @@ class global_navigation extends navigation_node {
                     }
 
                     if ($gradeavailable) {
-                        $url = new moodle_url('/grade/report/index.php', array('id'=>$usercourse->id));
+                        $url = new moodle_url('/grade/report/index.php', array('id' => $usercourse->id));
                         $usercoursenode->add(get_string('grades'), $url, self::TYPE_SETTING, null, null,
                                 new pix_icon('i/grades', ''));
                     }
@@ -2329,13 +2329,13 @@ class global_navigation extends navigation_node {
                     // Add a node to view the users notes if permitted.
                     if (!empty($CFG->enablenotes) &&
                             has_any_capability(array('moodle/notes:manage', 'moodle/notes:view'), $usercoursecontext)) {
-                        $url = new moodle_url('/notes/index.php',array('user'=>$user->id, 'course'=>$usercourse->id));
+                        $url = new moodle_url('/notes/index.php', array('user' => $user->id, 'course' => $usercourse->id));
                         $usercoursenode->add(get_string('notes', 'notes'), $url, self::TYPE_SETTING);
                     }
 
                     if (can_access_course($usercourse, $user->id)) {
                         $usercoursenode->add(get_string('entercourse'), new moodle_url('/course/view.php',
-                                array('id'=>$usercourse->id)), self::TYPE_SETTING, null, null, new pix_icon('i/course', ''));
+                                array('id' => $usercourse->id)), self::TYPE_SETTING, null, null, new pix_icon('i/course', ''));
                     }
 
                     $reporttab = $usercoursenode->add(get_string('activityreports'));
@@ -2623,7 +2623,7 @@ class global_navigation extends navigation_node {
           and ($CFG->bloglevel == BLOG_GLOBAL_LEVEL or ($CFG->bloglevel == BLOG_SITE_LEVEL and (isloggedin() and !isguestuser())))
           and has_capability('moodle/blog:view', context_system::instance())) {
             $blogsurls = new moodle_url('/blog/index.php', array('courseid' => $filterselect));
-            $coursenode->add(get_string('blogssite','blog'), $blogsurls->out(), self::TYPE_SYSTEM, null, 'siteblog');
+            $coursenode->add(get_string('blogssite', 'blog'), $blogsurls->out(), self::TYPE_SYSTEM, null, 'siteblog');
         }
 
         //Badges
@@ -2634,13 +2634,14 @@ class global_navigation extends navigation_node {
 
         // Notes
         if (!empty($CFG->enablenotes) && (has_capability('moodle/notes:manage', $this->page->context) || has_capability('moodle/notes:view', $this->page->context))) {
-            $coursenode->add(get_string('notes','notes'), new moodle_url('/notes/index.php',
-                array('filtertype'=>'course', 'filterselect'=>$filterselect)), self::TYPE_SETTING, null, 'notes');
+            $coursenode->add(get_string('notes', 'notes'), new moodle_url('/notes/index.php',
+                array('filtertype' => 'course', 'filterselect' => $filterselect)), self::TYPE_SETTING, null, 'notes');
         }
 
         // Tags
         if (!empty($CFG->usetags) && isloggedin()) {
-            $node = $coursenode->add(get_string('tags', 'tag'), new moodle_url('/tag/search.php'), self::TYPE_SETTING, null, 'tags');
+            $node = $coursenode->add(get_string('tags', 'tag'), new moodle_url('/tag/search.php'),
+                    self::TYPE_SETTING, null, 'tags');
         }
 
         if (isloggedin()) {
@@ -4198,7 +4199,7 @@ class settings_navigation extends navigation_node {
 
             $iscurrentuser = ($user->id == $USER->id);
 
-            $baseargs = array('id'=>$user->id);
+            $baseargs = array('id' => $user->id);
             if ($course->id != $SITE->id && !$iscurrentuser) {
                 $baseargs['course'] = $course->id;
                 $issitecourse = false;
@@ -4218,7 +4219,7 @@ class settings_navigation extends navigation_node {
                 $forumtab = $dashboard->add(get_string('forumposts', 'forum'));
                 $forumtab->add(get_string('posts', 'forum'), new moodle_url('/mod/forum/user.php', $baseargs), null, 'myposts');
                 $forumtab->add(get_string('discussions', 'forum'), new moodle_url('/mod/forum/user.php',
-                        array_merge($baseargs, array('mode'=>'discussions'))), null, 'mydiscussions');
+                        array_merge($baseargs, array('mode' => 'discussions'))), null, 'mydiscussions');
             }
 
             // Add blog nodes.
@@ -4236,7 +4237,7 @@ class settings_navigation extends navigation_node {
                     $blogs = $dashboard->add(get_string('blogs', 'blog'), null, navigation_node::TYPE_CONTAINER);
                     foreach ($options as $type => $option) {
                         if ($type == "rss") {
-                            $blogs->add($option['string'], $option['link'], settings_navigation::TYPE_SETTING, null, null,
+                            $blogs->add($option['string'], $option['link'], self::TYPE_SETTING, null, null,
                                     new pix_icon('i/rss', ''));
                         } else {
                             $blogs->add($option['string'], $option['link'], self::TYPE_SETTING, null, 'blog' . $type);
@@ -4255,7 +4256,7 @@ class settings_navigation extends navigation_node {
                 if ($course->id != $SITE->id) {
                     $messageargs['viewing'] = MESSAGE_VIEW_COURSE. $course->id;
                 }
-                $url = new moodle_url('/message/index.php',$messageargs);
+                $url = new moodle_url('/message/index.php', $messageargs);
                 $dashboard->add(get_string('messages', 'message'), $url, self::TYPE_SETTING, null, 'messages');
             }
 
@@ -4275,7 +4276,7 @@ class settings_navigation extends navigation_node {
             // Add a node to view the users notes if permitted.
             if (!empty($CFG->enablenotes) &&
                     has_any_capability(array('moodle/notes:manage', 'moodle/notes:view'), $coursecontext)) {
-                $url = new moodle_url('/notes/index.php',array('user'=>$user->id));
+                $url = new moodle_url('/notes/index.php', array('user' => $user->id));
                 if ($coursecontext->instanceid != SITEID) {
                     $url->param('course', $coursecontext->instanceid);
                 }
@@ -4360,8 +4361,8 @@ class settings_navigation extends navigation_node {
         if (isloggedin() && !isguestuser($user) && !is_mnet_remote_user($user)) {
             if ($currentuser && has_capability('moodle/user:editownprofile', $systemcontext) ||
                     has_capability('moodle/user:editprofile', $usercontext)) {
-                $url = new moodle_url('/user/language.php', array('id'=>$user->id, 'course'=>$course->id));
-                $useraccount->add(get_string('preferredlanguage'), $url, self::TYPE_SETTING);
+                $url = new moodle_url('/user/language.php', array('id' => $user->id, 'course' => $course->id));
+                $useraccount->add(get_string('preferredlanguage'), $url, self::TYPE_SETTING, null, 'preferredlanguage');
             }
         }
         $pluginmanager = core_plugin_manager::instance();
@@ -4369,7 +4370,7 @@ class settings_navigation extends navigation_node {
         if (isset($enabled['forum']) && isloggedin() && !isguestuser($user) && !is_mnet_remote_user($user)) {
             if ($currentuser && has_capability('moodle/user:editownprofile', $systemcontext) ||
                     has_capability('moodle/user:editprofile', $usercontext)) {
-                $url = new moodle_url('/user/forum.php', array('id'=>$user->id, 'course'=>$course->id));
+                $url = new moodle_url('/user/forum.php', array('id' => $user->id, 'course' => $course->id));
                 $useraccount->add(get_string('forumpreferences'), $url, self::TYPE_SETTING);
             }
         }
@@ -4378,15 +4379,15 @@ class settings_navigation extends navigation_node {
             if (isloggedin() && !isguestuser($user) && !is_mnet_remote_user($user)) {
                 if ($currentuser && has_capability('moodle/user:editownprofile', $systemcontext) ||
                         has_capability('moodle/user:editprofile', $usercontext)) {
-                    $url = new moodle_url('/user/editor.php', array('id'=>$user->id, 'course'=>$course->id));
+                    $url = new moodle_url('/user/editor.php', array('id' => $user->id, 'course' => $course->id));
                     $useraccount->add(get_string('editorpreferences'), $url, self::TYPE_SETTING);
                 }
             }
         }
 
         // View the roles settings.
-        if (has_any_capability(array('moodle/role:assign', 'moodle/role:safeoverride','moodle/role:override', 'moodle/role:manage'),
-                $usercontext)) {
+        if (has_any_capability(array('moodle/role:assign', 'moodle/role:safeoverride', 'moodle/role:override',
+                'moodle/role:manage'), $usercontext)) {
             $roles = $usersetting->add(get_string('roles'), null, self::TYPE_SETTING);
 
             $url = new moodle_url('/admin/roles/usersroles.php', array('userid'=>$user->id, 'courseid'=>$course->id));
@@ -4396,18 +4397,18 @@ class settings_navigation extends navigation_node {
 
             if (!empty($assignableroles)) {
                 $url = new moodle_url('/admin/roles/assign.php',
-                        array('contextid'=>$usercontext->id,'userid'=>$user->id, 'courseid'=>$course->id));
+                        array('contextid' => $usercontext->id, 'userid' => $user->id, 'courseid' => $course->id));
                 $roles->add(get_string('assignrolesrelativetothisuser', 'role'), $url, self::TYPE_SETTING);
             }
 
             if (has_capability('moodle/role:review', $usercontext) || count(get_overridable_roles($usercontext, ROLENAME_BOTH))>0) {
                 $url = new moodle_url('/admin/roles/permissions.php',
-                        array('contextid'=>$usercontext->id,'userid'=>$user->id, 'courseid'=>$course->id));
+                        array('contextid' => $usercontext->id, 'userid' => $user->id, 'courseid' => $course->id));
                 $roles->add(get_string('permissions', 'role'), $url, self::TYPE_SETTING);
             }
 
             $url = new moodle_url('/admin/roles/check.php',
-                    array('contextid'=>$usercontext->id,'userid'=>$user->id, 'courseid'=>$course->id));
+                    array('contextid' => $usercontext->id, 'userid' => $user->id, 'courseid' => $course->id));
             $roles->add(get_string('checkpermissions', 'role'), $url, self::TYPE_SETTING);
         }
 
@@ -4466,7 +4467,8 @@ class settings_navigation extends navigation_node {
         if ($currentuser && !empty($CFG->enableblogs)) {
             $blog = $usersetting->add(get_string('blogs', 'blog'), null, navigation_node::TYPE_CONTAINER, null, 'blogs');
             if (has_capability('moodle/blog:view', $systemcontext)) {
-                $blog->add(get_string('preferences', 'blog'), new moodle_url('/blog/preferences.php'), navigation_node::TYPE_SETTING);
+                $blog->add(get_string('preferences', 'blog'), new moodle_url('/blog/preferences.php'),
+                        navigation_node::TYPE_SETTING);
             }
             if (!empty($CFG->useexternalblogs) && $CFG->maxexternalblogsperuser > 0 &&
                     has_capability('moodle/blog:manageexternal', $systemcontext)) {
@@ -4482,7 +4484,8 @@ class settings_navigation extends navigation_node {
         // Badges.
         if ($currentuser && !empty($CFG->enablebadges)) {
             $badges = $usersetting->add(get_string('badges'), null, navigation_node::TYPE_CONTAINER, null, 'badges');
-            $badges->add(get_string('preferences', 'badges'), new moodle_url('/badges/preferences.php'), navigation_node::TYPE_SETTING);
+            $badges->add(get_string('preferences', 'badges'), new moodle_url('/badges/preferences.php'),
+                    navigation_node::TYPE_SETTING);
             if (!empty($CFG->badges_allowexternalbackpack)) {
                 $badges->add(get_string('backpackdetails', 'badges'), new moodle_url('/badges/mybackpack.php'),
                         navigation_node::TYPE_SETTING);
index 059e27c..ca4e70a 100644 (file)
@@ -3960,6 +3960,7 @@ EOD;
      */
     public function render_preferences_groups(preferences_groups $renderable) {
         $html = '';
+        $html .= html_writer::start_div('row-fluid');
         $html .= html_writer::start_tag('div', array('class' => 'span12 preferences-groups'));
         $i = 0;
         $open = false;
@@ -3968,7 +3969,7 @@ EOD;
                 if ($open) {
                     $html .= html_writer::end_tag('div');
                 }
-                $html .= html_writer::start_tag('div', array('class' => 'row'));
+                $html .= html_writer::start_tag('div', array('class' => 'row-fluid'));
                 $open = true;
             }
             $html .= $this->render($group);
@@ -3979,6 +3980,7 @@ EOD;
 
         $html .= html_writer::end_tag('ul');
         $html .= html_writer::end_tag('div');
+        $html .= html_writer::end_div();
         return $html;
     }
 
@@ -4122,11 +4124,13 @@ EOD;
 
     /**
      * Wrapper for header elements.
+     *
+     * @param string $heading Heading to be used for the main header.
+     * @return string HTML to display the main header.
      */
     public function full_header($heading = null) {
         $html = html_writer::start_tag('header', array('id' => 'page-header', 'class' => 'clearfix'));
-        // This is to ensure that the logo completely overwrites the header. MDL-49536 has been created to investigate
-        // whether the logo should work this way or not.
+        // This is to ensure that the logo completely overwrites the header if set.
         if (isset($heading) && $heading == '<div class="logo"></div>') {
             $html .= $heading;
         } else {
@@ -4136,7 +4140,7 @@ EOD;
         $html .= html_writer::tag('nav', $this->navbar(), array('class' => 'breadcrumb-nav'));
         $html .= html_writer::div($this->page_heading_button(), 'breadcrumb-button');
         $html .= html_writer::end_div();
-        $html .= html_writer::tag('div', $this->course_header(), array('class' => 'course-header'));
+        $html .= html_writer::tag('div', $this->course_header(), array('id' => 'course-header'));
         $html .= html_writer::end_tag('header');
         return $html;
     }
index 9423419..4c6514c 100644 (file)
@@ -198,7 +198,7 @@ if ($user2realuser) {
 } else {
     $PAGE->set_title("{$SITE->shortname}: $strmessages");
 }
-$PAGE->set_heading(fullname($USER));
+$PAGE->set_heading(fullname($user1));
 
 // Remove the user node from the main navigation for this page.
 $usernode = $PAGE->navigation->find('users', null);
index dcc5b46..2f8195e 100644 (file)
@@ -134,8 +134,9 @@ class message_output_email extends message_output {
         }
 
         if (!empty($CFG->messagingallowemailoverride)) {
-            $inputattributes = array('size'=>'30', 'name'=>'email_email', 'value'=>$preferences->email_email, 'id'=>'email_email');
-            $string .= html_writer::label(get_string('email','message_email'), 'email_email');
+            $inputattributes = array('size' => '30', 'name' => 'email_email', 'value' => $preferences->email_email,
+                    'id' => 'email_email');
+            $string .= html_writer::label(get_string('email', 'message_email'), 'email_email');
             $string .= $OUTPUT->container(html_writer::empty_tag('input', $inputattributes));
 
             if (empty($preferences->email_email) && !empty($preferences->userdefaultemail)) {
index 78f843d..d24030c 100644 (file)
@@ -7831,7 +7831,7 @@ function mod_forum_myprofile_navigation(core_user\output\myprofile\tree $tree, $
     }
     $postsurl = new moodle_url('/mod/forum/user.php', array('id' => $user->id));
     if (!empty($course)) {
-        $postsurl->param('coursre', $course->id);
+        $postsurl->param('course', $course->id);
     }
     $string = $iscurrentuser ? get_string('myprofileownpost', 'mod_forum') :
             get_string('myprofileotherpost', 'mod_forum', fullname($user));
@@ -7840,7 +7840,7 @@ function mod_forum_myprofile_navigation(core_user\output\myprofile\tree $tree, $
 
     $discussionssurl = new moodle_url('/mod/forum/user.php', array('id' => $user->id, 'mode' => 'discussions'));
     if (!empty($course)) {
-        $postsurl->param('coursre', $course->id);
+        $postsurl->param('course', $course->id);
     }
     $string = $iscurrentuser ? get_string('myprofileowndis', 'mod_forum') :
             get_string('myprofileotherdis', 'mod_forum', fullname($user));
index de07281..8ad6b9d 100644 (file)
@@ -193,7 +193,7 @@ if (empty($result->posts)) {
                         array('id' => $user->id, 'course' => $courseid)));
             } else {
                 $navbar = $PAGE->navbar->add(get_string('discussions', 'forum'), new moodle_url('/mod/forum/user.php',
-                        array('id' => $user->id, 'course' => $courseid, 'mode'=>'discussions')));
+                        array('id' => $user->id, 'course' => $courseid, 'mode' => 'discussions')));
             }
         }
     } else if ($canviewuser) {
@@ -211,7 +211,7 @@ if (empty($result->posts)) {
                         array('id' => $user->id, 'course' => $courseid)));
             } else {
                 $navbar = $PAGE->navbar->add(get_string('discussions', 'forum'), new moodle_url('/mod/forum/user.php',
-                        array('id' => $user->id, 'course' => $courseid, 'mode'=>'discussions')));
+                        array('id' => $user->id, 'course' => $courseid, 'mode' => 'discussions')));
             }
         }
 
@@ -384,7 +384,7 @@ if (isset($courseid) && $courseid != SITEID) {
                 array('id' => $user->id, 'course' => $courseid)));
     } else {
         $navbar = $PAGE->navbar->add(get_string('discussions', 'forum'), new moodle_url('/mod/forum/user.php',
-                array('id' => $user->id, 'course' => $courseid, 'mode'=>'discussions')));
+                array('id' => $user->id, 'course' => $courseid, 'mode' => 'discussions')));
     }
 }
 
index ce73fd3..4b6e730 100644 (file)
@@ -93,9 +93,6 @@ if (!isguestuser()) {   // Skip default home page for guests
         }
     }
 }
-$mycoursenode = $PAGE->navigation->get('mycourses');
-$mycoursenode->collapse = true;
-$mycoursenode->make_inactive();
 
 // Toggle the editing state and switches
 if ($PAGE->user_allowed_editing()) {
index 49da9f2..bd3d5db 100644 (file)
@@ -194,7 +194,8 @@ if ($usernew = $userform->get_data()) {
             $a->oldemail = $usernew->email = $user->email;
 
             $emailchangedhtml = $OUTPUT->box(get_string('auth_changingemailaddress', 'auth', $a), 'generalbox', 'notice');
-            $emailchangedhtml .= $OUTPUT->continue_button("$CFG->wwwroot/user/preferences.php?userid=$user->id&amp;course=$course->id");
+            $emailchangedhtml .=
+                    $OUTPUT->continue_button("$CFG->wwwroot/user/preferences.php?userid=$user->id&amp;course=$course->id");
             $emailchanged = true;
         }
     }
index 976ccd5..024b9f5 100644 (file)
@@ -338,7 +338,7 @@ if ($user->id == -1 or ($user->id != $USER->id)) {
     $PAGE->set_heading($userfullname);
 
     echo $OUTPUT->header();
-    echo $OUTPUT->heading($userfullname);
+    echo $OUTPUT->heading($streditmyprofile);
 }
 
 // Finally display THE form.
index 205f6df..0890d4c 100644 (file)
@@ -96,10 +96,7 @@ function useredit_setup_preference_page($userid, $courseid) {
     } else {
         // Teachers, parents, etc.
         require_capability('moodle/user:editprofile', $personalcontext);
-        // No editing of guest user account.
-        if (isguestuser($user->id)) {
-            print_error('guestnoeditprofileother');
-        }
+
         // No editing of primary admin!
         if (is_siteadmin($user) and !is_siteadmin($USER)) {  // Only admins may edit other admins.
             print_error('useradmineditadmin');
index 57ebaaf..7a3267d 100644 (file)
@@ -56,6 +56,8 @@ if ($data = $editorform->get_data()) {
 $streditmyeditor = get_string('editorpreferences');
 $userfullname     = fullname($user, true);
 
+$PAGE->navbar->includesettingsbase = true;
+
 $PAGE->set_title("$course->shortname: $streditmyeditor");
 $PAGE->set_heading($userfullname);
 
index 4947d2c..88b8628 100644 (file)
@@ -57,6 +57,8 @@ if ($data = $forumform->get_data()) {
 $streditmyforum = get_string('forumpreferences');
 $userfullname     = fullname($user, true);
 
+$PAGE->navbar->includesettingsbase = true;
+
 $PAGE->set_title("$course->shortname: $streditmyforum");
 $PAGE->set_heading($userfullname);
 
index 2a037f1..3a4b04d 100644 (file)
@@ -15,7 +15,9 @@
 // along with Moodle.  If not, see <http://www.gnu.org/licenses/>.
 
 /**
- * Form to edit a users forum preferences. These are stored as columns in the user table, which
+ * Form to edit a users forum preferences.
+ *
+ * These are stored as columns in the user table, which
  * is why they are in /user and not /mod/forum.
  *
  * @copyright 1999 Martin Dougiamas  http://dougiamas.com
index 2feb2c2..b49f69b 100644 (file)
@@ -60,11 +60,13 @@ if ($data = $languageform->get_data()) {
 $streditmylanguage = get_string('preferredlanguage');
 $userfullname     = fullname($user, true);
 
+$PAGE->navbar->includesettingsbase = true;
+
 $PAGE->set_title("$course->shortname: $streditmylanguage");
 $PAGE->set_heading($userfullname);
 
 echo $OUTPUT->header();
-echo $OUTPUT->heading($userfullname);
+echo $OUTPUT->heading($streditmylanguage);
 
 // Finally display THE form.
 $languageform->display();
index 126932e..30ed596 100644 (file)
@@ -36,6 +36,9 @@ defined('MOODLE_INTERNAL') || die();
 class phpunit_fixture_myprofile_category extends \core_user\output\myprofile\category {
     /**
      * Make protected method public for testing.
+     *
+     * @param node $node
+     * @return node Nodes after the specified node.
      */
     public function find_nodes_after($node) {
         return parent::find_nodes_after($node);
@@ -51,10 +54,18 @@ class phpunit_fixture_myprofile_category extends \core_user\output\myprofile\cat
 
 /**
  * Class phpunit_fixture_myprofile_tree
+ *
+ * @package   core_user
+ * @category  test
+ * @copyright 2015 onwards Ankit Agarwal
+ * @license   http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
 class phpunit_fixture_myprofile_tree extends \core_user\output\myprofile\tree {
     /**
      * Make protected method public for testing.
+     *
+     * @param category $cat Category object
+     * @return array An array of category objects.
      */
     public function find_categories_after($cat) {
         return parent::find_categories_after($cat);
index b72a2f2..cbbb29d 100644 (file)
@@ -190,7 +190,7 @@ if ($node = $PAGE->settingsnav->get('courseadmin')) {
 echo $OUTPUT->header();
 
 echo '<div class="userprofile">';
-$headerinfo = array('heading' => fullname($user) ,'user' => $user, 'usercontext' => $usercontext);
+$headerinfo = array('heading' => fullname($user)'user' => $user, 'usercontext' => $usercontext);
 echo $OUTPUT->context_header($headerinfo, 2);
 
 if ($user->deleted) {