MDL-28945 course_reports: Fixed up pagination issue
authorSam Hemelryk <sam@moodle.com>
Wed, 19 Oct 2011 03:50:23 +0000 (16:50 +1300)
committerSam Hemelryk <sam@moodle.com>
Wed, 19 Oct 2011 03:50:23 +0000 (16:50 +1300)
course/report/completion/index.php
course/report/progress/index.php

index 0118cf1..af07f65 100644 (file)
@@ -274,9 +274,13 @@ if($total > COMPLETION_REPORT_PAGE) {
     $pagingbar .= get_string('page').': ';
 
     $sistrings = array();
-    $sistrings[] = $sifirst != 'all' ? "sifirst={$sifirst}" : null;
-    $sistrings[] = $silast != 'all' ? "silast={$silast}" : null;
-    $sistring = !empty($sistrings) ? implode('&amp;', $sistrings) : '';
+    if ($sifirst != 'all') {
+        $sistrings[] =  "sifirst={$sifirst}";
+    }
+    if ($silast != 'all') {
+        $sistrings[] =  "silast={$silast}";
+    }
+    $sistring = !empty($sistrings) ? '&amp;'.implode('&amp;', $sistrings) : '';
 
     // Display previous link
     if ($start > 0) {
index a45a643..323d2b8 100644 (file)
@@ -206,9 +206,13 @@ if($total > COMPLETION_REPORT_PAGE) {
     $pagingbar .= get_string('page').': ';
 
     $sistrings = array();
-    $sistrings[] = $sifirst != 'all' ? "sifirst={$sifirst}" : null;
-    $sistrings[] = $silast != 'all' ? "silast={$silast}" : null;
-    $sistring = !empty($sistrings) ? implode('&amp;', $sistrings) : '';
+    if ($sifirst != 'all') {
+        $sistrings[] =  "sifirst={$sifirst}";
+    }
+    if ($silast != 'all') {
+        $sistrings[] =  "silast={$silast}";
+    }
+    $sistring = !empty($sistrings) ? '&amp;'.implode('&amp;', $sistrings) : '';
 
     // Display previous link
     if ($start > 0) {