MDL-42060 block_course_list: fixed sorting of mycourses
authorSam Hemelryk <sam@moodle.com>
Sun, 17 Nov 2013 20:37:01 +0000 (09:37 +1300)
committerSam Hemelryk <sam@moodle.com>
Sun, 17 Nov 2013 22:15:02 +0000 (11:15 +1300)
blocks/course_list/block_course_list.php

index aaf6570..d459982 100644 (file)
@@ -35,7 +35,14 @@ class block_course_list extends block_list {
 
         if (empty($CFG->disablemycourses) and isloggedin() and !isguestuser() and
           !(has_capability('moodle/course:update', context_system::instance()) and $adminseesall)) {    // Just print My Courses
-            if ($courses = enrol_get_my_courses(NULL, 'visible DESC, fullname ASC')) {
+            // As this is producing navigation sort order should default to $CFG->navsortmycoursessort instead
+            // of using the default.
+            if (!empty($CFG->navsortmycoursessort)) {
+                $sortorder = 'visible DESC, ' . $CFG->navsortmycoursessort . ' ASC';
+            } else {
+                $sortorder = 'visible DESC, sortorder ASC';
+            }
+            if ($courses = enrol_get_my_courses(NULL, $sortorder)) {
                 foreach ($courses as $course) {
                     $coursecontext = context_course::instance($course->id);
                     $linkcss = $course->visible ? "" : " class=\"dimmed\" ";