From: tjhunt <tjhunt>
Date: Mon, 12 Jun 2006 13:54:01 +0000 (+0000)
Subject: Bug #5793: remember whether Show all has been clicked in the question bank. Merged... 
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=f92cf442db218cdcef819c9b8f84ab2b2da638a5;p=moodle.git

Bug #5793: remember whether Show all has been clicked in the question bank. Merged from MOODLE_16_STABLE.
---

diff --git a/question/editlib.php b/question/editlib.php
index 5135123f4f..85b5643ae1 100644
--- a/question/editlib.php
+++ b/question/editlib.php
@@ -15,6 +15,8 @@
 
 require_once($CFG->libdir.'/questionlib.php');
 
+define('DEFAULT_QUESTIONS_PER_PAGE', 20);
+
 /**
 * Function to read all questions for category into big array
 *
@@ -366,8 +368,12 @@ function question_list($course, $categoryid, $quizid=0,
     }
     echo '<tr><td align="center" colspan="3">';
     print_paging_bar($totalnumber, $page, $perpage, "edit.php?courseid={$course->id}&amp;perpage=$perpage&amp;");
-    if ($totalnumber > $perpage) {
-        echo '<a href="edit.php?courseid='.$course->id.'&amp;perpage=1000">'.get_string('showall', 'moodle', $totalnumber).'</a>';
+    if ($totalnumber > DEFAULT_QUESTIONS_PER_PAGE) {
+        if ($perpage == DEFAULT_QUESTIONS_PER_PAGE) {
+            echo '<a href="edit.php?courseid='.$course->id.'&amp;perpage=1000">'.get_string('showall', 'moodle', $totalnumber).'</a>';
+        } else {
+            echo '<a href="edit.php?courseid='.$course->id.'&amp;perpage=' . DEFAULT_QUESTIONS_PER_PAGE . '">'.get_string('showperpage', 'moodle', DEFAULT_QUESTIONS_PER_PAGE).'</a>';
+        }
     }
     echo "</td></tr></table>\n";
     print_simple_box_end();
diff --git a/question/showbank.php b/question/showbank.php
index cf23d4a721..786fef9e74 100644
--- a/question/showbank.php
+++ b/question/showbank.php
@@ -24,7 +24,7 @@
     require_once($CFG->dirroot.'/question/editlib.php');
 
     $page      = optional_param('page', -1, PARAM_INT);
-    $perpage   = optional_param('perpage', 20, PARAM_INT);
+    $perpage   = optional_param('perpage', -1, PARAM_INT);
     $sortorder = optional_param('sortorder', '');
     if (preg_match("/[';]/", $sortorder)) {
         error("Incorrect use of the parameter 'sortorder'");
@@ -36,6 +36,12 @@
         $page = isset($SESSION->questionpage) ? $SESSION->questionpage : 0;
     }
 
+    if ($perpage > -1) {
+        $SESSION->questionperpage = $perpage;
+    } else {
+        $perpage = isset($SESSION->questionperpage) ? $SESSION->questionperpage : DEFAULT_QUESTIONS_PER_PAGE;
+    }
+
     if ($sortorder) {
         $SESSION->questionsortorder = $sortorder;
     } else {