Permalink
Browse files

MDL-38647 quiz review question issues

1. Page URL should contain 'step=0' if necessary, but not 'step='.
2. Review specific step was not using the right display options.
  • Loading branch information...
1 parent 373a8e0 commit 2515be0e8035bb1473da93ebaab645cb79a34110 @timhunt timhunt committed Mar 22, 2013
Showing with 2 additions and 2 deletions.
  1. +1 −1 mod/quiz/attemptlib.php
  2. +1 −1 mod/quiz/reviewquestion.php
View
@@ -1192,7 +1192,7 @@ public function render_question($slot, $reviewing, $thispageurl = null) {
*/
public function render_question_at_step($slot, $seq, $reviewing, $thispageurl = '') {
return $this->quba->render_question_at_step($slot, $seq,
- $this->get_display_options($reviewing),
+ $this->get_display_options_with_edit_link($reviewing, $slot, $thispageurl),
$this->get_question_number($slot));
}
@@ -35,7 +35,7 @@
$baseurl = new moodle_url('/mod/quiz/reviewquestion.php',
array('attempt' => $attemptid, 'slot' => $slot));
$currenturl = new moodle_url($baseurl);
-if ($seq !== 0) {
+if (!is_null($seq)) {
$currenturl->param('step', $seq);
}
$PAGE->set_url($currenturl);

0 comments on commit 2515be0

Please sign in to comment.