From: tjhunt <tjhunt>
Date: Wed, 7 Jan 2009 07:12:37 +0000 (+0000)
Subject: quiz editing: MDL-17284 fix infinite redirect bug.
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=b4a2e413135b947d1c9924e341edd5db5877dd3e;p=moodle.git

quiz editing: MDL-17284 fix infinite redirect bug.
---

diff --git a/mod/quiz/edit.php b/mod/quiz/edit.php
index 4b91178349..3a87d7000c 100644
--- a/mod/quiz/edit.php
+++ b/mod/quiz/edit.php
@@ -62,7 +62,7 @@ function module_specific_actions($pageurl, $questionid, $cmid, $canuse, $cmoptio
  * (which is called from showbank())
  * Displays button in form with checkboxes for each question.
  */
-function module_specific_buttons($cmid,$cmoptions){
+function module_specific_buttons($cmid, $cmoptions){
     if($cmoptions->hasattempts){
         $disabled='disabled="disabled"';
     }else{
@@ -540,7 +540,7 @@ if ($significantchangemade) {
         }
     }
     quiz_update_sumgrades($quiz);
-    redirect($qcobject->pageurl->out());
+    redirect($thispageurl->out());
 }
 
 question_showbank_actions($thispageurl, $cm);
diff --git a/question/editlib.php b/question/editlib.php
index 60afa8e56c..006e92d300 100644
--- a/question/editlib.php
+++ b/question/editlib.php
@@ -604,6 +604,8 @@ function question_edit_setup($edittab, $requirecmid = false, $requirecourseid =
     //$thispageurl is used to construct urls for all question edit pages we link to from this page. It contains an array
     //of parameters that are passed from page to page.
     $thispageurl = new moodle_url();
+    $thispageurl->remove_params(); // We are going to explicity add back everything important - this avoids unwanted params from being retained.
+
     if ($requirecmid){
         $cmid =required_param('cmid', PARAM_INT);
     } else {