My Moodle: MDL-19089 Course limit for My Moodle not propperly obeyed. Fixing so that...

diff --git a/my/index.php b/my/index.php
index ebdd718503c8618dd6473dd5ad6c961ac1385ae3..64bd4ea33d594d7dfa2623821e0e4b6eac7e654b 100644 (file)
--- a/my/index.php
+++ b/my/index.php
@@ -66,13 +66,26 @@
 
     // limits the number of courses showing up
     $courses_limit = 21;
-    if (!empty($CFG->mycoursesperpage)) {
+    if (isset($CFG->mycoursesperpage)) {
         $courses_limit = $CFG->mycoursesperpage;
     }
+    
+    $morecourses = false;
+    if ($courses_limit > 0) {
+        $courses_limit = $courses_limit + 1;
+    }
+    
     $courses = get_my_courses($USER->id, 'visible DESC,sortorder ASC', '*', false, $courses_limit);
     $site = get_site();
     $course = $site; //just in case we need the old global $course hack
 
+    if (($courses_limit > 0) && (count($courses) >= $courses_limit)) {
+        //remove the 'marker' course that we retrieve just to see if we have more than $courses_limit
+        array_pop($courses);
+        $morecourses = true;
+    }
+
+
     if (array_key_exists($site->id,$courses)) {
         unset($courses[$site->id]);
     }
@@ -92,7 +105,7 @@
     }
 
     // if more than 20 courses
-    if (count($courses) > 20) {
+    if ($morecourses) {
         echo '<br />...';
     }