diff --git a/mod/quiz/report/statistics/statistics_table.php b/mod/quiz/report/statistics/statistics_table.php index ae54a697984..508da380d42 100644 --- a/mod/quiz/report/statistics/statistics_table.php +++ b/mod/quiz/report/statistics/statistics_table.php @@ -91,7 +91,7 @@ class quiz_report_statistics_table extends flexible_table { if (!$this->is_downloading()){ if ($question->qtype!='random'){ $tooltip = get_string('detailedanalysis', 'quiz_statistics'); - $url = $this->baseurl .'&qid='.$question->id; + $url = $this->baseurl .'qid='.$question->id; $html = "".$question->name.""; } else { $html = $question->name; @@ -132,7 +132,11 @@ class quiz_report_statistics_table extends flexible_table { } } function col_actions($question){ - return quiz_question_action_icons($this->quiz, $this->cmid, $question, $this->baseurl); + global $CFG; + $editreturnurl = str_replace($CFG->wwwroot, '', $this->baseurl); + $editreturnurl = str_replace('&', '&', $editreturnurl); + $editreturnurl = preg_replace('/&$/', '', $editreturnurl); + return quiz_question_action_icons($this->quiz, $this->cmid, $question, $editreturnurl); } function col_qtype($question){ return get_string($question->qtype,'quiz'); diff --git a/question/question.php b/question/question.php index 3ce621119ba..8402cbb9eef 100644 --- a/question/question.php +++ b/question/question.php @@ -206,11 +206,10 @@ if ($mform->is_cancelled()){ if ($inpopup) { close_window(); } else { - $nexturl = new moodle_url($returnurl); if (!empty($question->id)) { - $nexturl->param('lastchanged', $question->id); + $returnurl->param('lastchanged', $question->id); } - redirect($nexturl->out()); + redirect($returnurl->out(false)); } } elseif ($fromform = $mform->get_data()) { /// If we are saving as a copy, break the connection to the old question.