Let's use the default (cached) list of fields here. MDL-10571

Merged from MOODLE_18_STABLE

diff --git a/blocks/course_list/block_course_list.php b/blocks/course_list/block_course_list.php
index 5f237c3eb0caac8c173675d2087a6237c439be5f..cf3fac55a0465d8916a04ec147a6793350fec55e 100644 (file)
--- a/blocks/course_list/block_course_list.php
+++ b/blocks/course_list/block_course_list.php
@@ -38,7 +38,7 @@ class block_course_list extends block_list {
             !empty($USER->id) and 
             !(has_capability('moodle/course:update', get_context_instance(CONTEXT_SYSTEM)) and $adminseesall) and
             !isguest()) {    // Just print My Courses
-            if ($courses = get_my_courses($USER->id, NULL, 'id, visible, shortname, fullname')) {
+            if ($courses = get_my_courses($USER->id)) {
                 foreach ($courses as $course) {
                     if ($course->id == SITEID) {
                         continue;

diff --git a/calendar/lib.php b/calendar/lib.php
index a4571c9e7b7daa9c0c30374d5cdfa646b10a9e4e..caef701d280ad41b913e7a1a0896b6c99c82012a 100644 (file)
--- a/calendar/lib.php
+++ b/calendar/lib.php
@@ -1396,10 +1396,10 @@ function calendar_get_default_courses($ignoreref = false) {
     }
     
     if (isset($CFG->adminseesall)) {
-        $courses = get_my_courses($USER->id, 'visible DESC', null, $CFG->adminseesall);
+        $courses = get_my_courses($USER->id, null, null, $CFG->adminseesall);
     }
     else {
-        $courses = get_my_courses($USER->id, 'visible DESC', null, false);
+        $courses = get_my_courses($USER->id, null, null, false);
     }
 
     return $courses;

diff --git a/mod/forum/lib.php b/mod/forum/lib.php
index ae86036dd70674aaad3be9ef17fc34489aa05d29..e3ff8cff33fe1283d4d5f031fb93a6ac49dd11d5 100644 (file)
--- a/mod/forum/lib.php
+++ b/mod/forum/lib.php
@@ -1342,7 +1342,7 @@ function forum_get_readable_forums($userid, $courseid=0) {
         // If no course is specified, then the user can see SITE + his courses. 
         // And admins can see all courses, so pass the $doanything flag enabled
         $courses1 = get_records('course', 'id', SITEID);
-        $courses2 = get_my_courses($userid, 'visible DESC,sortorder ASC', null, true);
+        $courses2 = get_my_courses($userid, null, null, true);
         $courses = array_merge($courses1, $courses2);
     }
     if (!$courses) {

diff --git a/user/view.php b/user/view.php
index 86836ee4e3ed18a8a0655121b3e825e7f9250f26..8dc2f807e35df34b5f793637102e04642ccff9a7 100644 (file)
--- a/user/view.php
+++ b/user/view.php
@@ -321,7 +321,7 @@
     profile_display_fields($user->id);
 
 
-    if ($mycourses = get_my_courses($user->id,'visible DESC,sortorder ASC', null, false, 21)) {
+    if ($mycourses = get_my_courses($user->id, null, null, false, 21)) {
         $shown=0;
         $courselisting = '';
         foreach ($mycourses as $mycourse) {