diff --git a/mod/quiz/addrandom.php b/mod/quiz/addrandom.php
index 5568bd0fd50..e08c1d1c517 100644
--- a/mod/quiz/addrandom.php
+++ b/mod/quiz/addrandom.php
@@ -54,6 +54,15 @@ if ($newquestioninfo) {
$quiz_page = optional_param('quiz_page', 0, PARAM_SEQUENCE);
$returnurl = optional_param('returnurl', 0, PARAM_LOCALURL);
+$url = new moodle_url($CFG->wwwroot.'/mod/quiz/addrandom.php');
+if ($quiz_page != 0) {
+ $url->param('quiz_page', $quiz_page);
+}
+if ($returnurl != 0) {
+ $url->param('returnurl', $returnurl);
+}
+$PAGE->set_url($url);
+
$strquizzes = get_string('modulenameplural', 'quiz');
$strquiz = get_string('modulename', 'quiz');
$streditingquestions = get_string('editquestions', 'quiz');
diff --git a/mod/quiz/attempt.php b/mod/quiz/attempt.php
index d7f22c33f07..051df81edb7 100644
--- a/mod/quiz/attempt.php
+++ b/mod/quiz/attempt.php
@@ -100,8 +100,9 @@
$accessmanager->setup_secure_page($attemptobj->get_course()->shortname . ': ' .
format_string($attemptobj->get_quiz_name()), $headtags);
} elseif ($accessmanager->safebrowser_required($attemptobj->is_preview_user())) {
- print_header($attemptobj->get_course()->shortname . ': '.
- format_string($attemptobj->get_quiz_name()), '', '', '', $headtags, false, '', '', false, '');
+ $PAGE->set_title($attemptobj->get_course()->shortname . ': '.format_string($attemptobj->get_quiz_name()));
+ $PAGE->set_cacheable(false);
+ echo $OUTPUT->header();
} else {
$PAGE->set_title(format_string($attemptobj->get_quiz_name()));
$PAGE->set_button($attemptobj->update_module_button());
diff --git a/mod/quiz/comment.php b/mod/quiz/comment.php
index fe3abc3fd50..9419e02ace6 100644
--- a/mod/quiz/comment.php
+++ b/mod/quiz/comment.php
@@ -1,4 +1,4 @@
-set_url(new moodle_url($CFG->wwwroot.'/mod/quiz/comment.php', array('attempt'=>$attemptid, 'question'=>$questionid)));
+
$attemptobj = new quiz_attempt($attemptid);
/// Can only grade finished attempts.
diff --git a/mod/quiz/index.php b/mod/quiz/index.php
index 99213c5777f..53cfcddb3f8 100644
--- a/mod/quiz/index.php
+++ b/mod/quiz/index.php
@@ -1,4 +1,4 @@
-set_url(new moodle_url($CFG->wwwroot.'/mod/quiz/index.php', array('id'=>$id)));
if (!$course = $DB->get_record('course', array('id' => $id))) {
print_error('invalidcourseid');
}
diff --git a/mod/quiz/review.php b/mod/quiz/review.php
index 847b562999e..7eb76063ce2 100644
--- a/mod/quiz/review.php
+++ b/mod/quiz/review.php
@@ -93,7 +93,9 @@
if ($accessmanager->securewindow_required($attemptobj->is_preview_user())) {
$accessmanager->setup_secure_page($attemptobj->get_course()->shortname.': '.format_string($attemptobj->get_quiz_name()), $headtags);
} elseif ($accessmanager->safebrowser_required($attemptobj->is_preview_user())) {
- print_header($attemptobj->get_course()->shortname . ': '.format_string($attemptobj->get_quiz_name()), '', '', '', $headtags, false, '', '', false, '');
+ $PAGE->set_title($attemptobj->get_course()->shortname . ': '.format_string($attemptobj->get_quiz_name()));
+ $PAGE->set_cacheable(false);
+ echo $OUTPUT->header();
} else {
$attemptobj->navigation($strreviewtitle);
$PAGE->set_title(format_string($attemptobj->get_quiz_name()));
diff --git a/mod/quiz/reviewquestion.php b/mod/quiz/reviewquestion.php
index 75f21de62be..6198176c6f4 100644
--- a/mod/quiz/reviewquestion.php
+++ b/mod/quiz/reviewquestion.php
@@ -1,4 +1,4 @@
-wwwroot.'/mod/quiz/reviewquestion.php', array('attempt'=>$attemptid,'question'=>$questionid));
+ if ($stateid !== 0) {
+ $url->param('state', $stateid);
+ }
+ $PAGE->set_url($url);
+
$attemptobj = new quiz_attempt($attemptid);
/// Check login.
diff --git a/mod/quiz/summary.php b/mod/quiz/summary.php
index 3140f6a7a09..999894d0887 100644
--- a/mod/quiz/summary.php
+++ b/mod/quiz/summary.php
@@ -1,4 +1,4 @@
-dirroot . '/mod/quiz/locallib.php');
$attemptid = required_param('attempt', PARAM_INT); // The attempt to summarise.
+
+$PAGE->set_url(new moodle_url($CFG->wwwroot.'/mod/quiz/summary.php', array('attempt'=>$attemptid)));
+
$attemptobj = new quiz_attempt($attemptid);
/// Check login.
@@ -50,8 +53,9 @@ if ($accessmanager->securewindow_required($attemptobj->is_preview_user())) {
$accessmanager->setup_secure_page($attemptobj->get_course()->shortname . ': ' .
format_string($attemptobj->get_quiz_name()), '');
} elseif ($accessmanager->safebrowser_required($attemptobj->is_preview_user())) {
- print_header($attemptobj->get_course()->shortname . ': '.
- format_string($attemptobj->get_quiz_name()), '', '', '', '', false, '', '', false, '');
+ $PAGE->set_title($attemptobj->get_course()->shortname . ': '.format_string($attemptobj->get_quiz_name()));
+ $PAGE->set_cacheable(false);
+ echo $OUTPUT->header();
} else {
$attemptobj->navigation($title);
$PAGE->set_title(format_string($attemptobj->get_quiz_name()));
diff --git a/mod/quiz/view.php b/mod/quiz/view.php
index 909f8612c5d..eccfc5f5977 100644
--- a/mod/quiz/view.php
+++ b/mod/quiz/view.php
@@ -74,7 +74,7 @@
// The code will be much nicer than this eventually.
$title = $course->shortname . ': ' . format_string($quiz->name);
- $buttons = '
| '.update_module_button($cm->id, $course->id, get_string('modulename', 'quiz')).' | ';
+ $buttons = '| '.$OUTPUT->update_module_button($cm->id, $cm->modname).' | ';
if ($PAGE->user_allowed_editing() && !empty($CFG->showblocksonmodpages)) {
$buttons .= ' |