WebSVN – Moodle – Diff – /mod/quiz/classes/output/renderer.php

  * @package   mod_quiz
  * @category  output
  * @copyright 2011 The Open University
  * @license   http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-class renderer extends plugin_renderer_base {
+class renderer extends plugin_renderer_base
+{
     /**
      * Builds the review page
+     *
      * @param quiz_attempt $attemptobj an instance of quiz_attempt.
      * @param array $slots of slots to be displayed.
      * @param display_options $displayoptions instance of display_options.
      * @param attempt_summary_information|array $summarydata summary information about the attempt.
      *      Passing an array is deprecated.
      * @return string HTML to display.
      */
-    public function review_page(quiz_attempt $attemptobj, $slots, $page, $showall,
+    public function review_page(
+        quiz_attempt $attemptobj,
+        $slots,
+        $page,
+        $showall,
+        $lastpage,
-            $lastpage, display_options $displayoptions, $summarydata) {
+        display_options $displayoptions,
+        $summarydata
+    ) {
         if (is_array($summarydata)) {
+            debugging(
-            debugging('Since Moodle 4.4, $summarydata passed to review_page should be a attempt_summary_information.',
+                'Since Moodle 4.4, $summarydata passed to review_page should be a attempt_summary_information.',
-                DEBUG_DEVELOPER);
+                DEBUG_DEVELOPER
+            );
             $summarydata = $this->filter_review_summary_table($summarydata, $page);
             $summarydata = attempt_summary_information::create_from_legacy_array($summarydata);
+        }
         $output = '';
         $output .= $this->header();
         $output .= $this->review_attempt_summary($summarydata, $page);
+        $output .= $this->review_form(
+            $page,
+            $showall,
-        $output .= $this->review_form($page, $showall, $displayoptions,
+            $displayoptions,
-                $this->questions($attemptobj, true, $slots, $page, $showall, $displayoptions),
+            $this->questions($attemptobj, true, $slots, $page, $showall, $displayoptions),
+            $attemptobj
-                $attemptobj);
+        );
         $output .= $this->review_next_navigation($attemptobj, $page, $lastpage, $showall);
         $output .= $this->footer();
      * @param display_options $displayoptions instance of display_options.
      * @param attempt_summary_information|array $summarydata summary information about the attempt.
      *      Passing an array is deprecated.
      * @return string HTML to display.
      */
-    public function review_question_page(quiz_attempt $attemptobj, $slot, $seq,
+    public function review_question_page(
+        quiz_attempt $attemptobj,
+        $slot,
+        $seq,
-            display_options $displayoptions, $summarydata) {
+        display_options $displayoptions,
+        $summarydata
+    ) {
         if (is_array($summarydata)) {
+            debugging(
-            debugging('Since Moodle 4.4, $summarydata passed to review_question_page should be a attempt_summary_information.',
+                'Since Moodle 4.4, $summarydata passed to review_question_page should be a attempt_summary_information.',
-                DEBUG_DEVELOPER);
+                DEBUG_DEVELOPER
+            );
             $summarydata = attempt_summary_information::create_from_legacy_array($summarydata);
+        }
         $output = '';
+     *
      * @param quiz_attempt $attemptobj an instance of quiz_attempt.
      * @param string $message Why the review is not allowed.
      * @return string html to output.
      */
-    public function review_question_not_allowed(quiz_attempt $attemptobj, $message) {
+    public function review_question_not_allowed(quiz_attempt $attemptobj, $message)
+    {
         $output = '';
         $output .= $this->header();
-        $output .= $this->heading(format_string($attemptobj->get_quiz_name(), true,
+        $output .= $this->heading(format_string(
+            $attemptobj->get_quiz_name(),
+            true,
-                ["context" => $attemptobj->get_quizobj()->get_context()]));
+            ["context" => $attemptobj->get_quizobj()->get_context()]
+        ));
         $output .= $this->notification($message);
         $output .= $this->close_window_button();
         $output .= $this->footer();
         return $output;
+    }
      * @param array $summarydata contains row data for table
      * @param int $page the current page number
      * @return array updated version of the $summarydata array.
      * @deprecated since Moodle 4.4. Replaced by filter_review_attempt_summary.
      */
-    protected function filter_review_summary_table($summarydata, $page) {
+    protected function filter_review_summary_table($summarydata, $page)
+    {
         debugging('filter_review_summary_table() is deprecated. Replaced by filter_review_attempt_summary().', DEBUG_DEVELOPER);
         if ($page == 0) {
             return $summarydata;
+        }
      * @param array $summarydata contains row data for table
      * @param int $page contains the current page number
      * @return string HTML to display.
      * @deprecated since Moodle 4.4. Replaced by review_attempt_summary.
      */
-    public function review_summary_table($summarydata, $page) {
+    public function review_summary_table($summarydata, $page)
+    {
         debugging('review_summary_table() is deprecated. Please use review_attempt_summary() instead.', DEBUG_DEVELOPER);
         $summarydata = $this->filter_review_summary_table($summarydata, $page);
         $this->render(attempt_summary_information::create_from_legacy_array($summarydata));
+    }
      * @param array $slots array of integers relating to questions
      * @param int $page current page number
      * @param bool $showall if true shows attempt on single page
      * @param display_options $displayoptions instance of display_options
      */
+    public function questions(
-    public function questions(quiz_attempt $attemptobj, $reviewing, $slots, $page, $showall,
+        quiz_attempt $attemptobj,
+        $reviewing,
+        $slots,
+        $page,
+        $showall,
-            display_options $displayoptions) {
+        display_options $displayoptions
+    ) {
         $output = '';
         foreach ($slots as $slot) {
-            $output .= $attemptobj->render_question($slot, $reviewing, $this,
+            $output .= $attemptobj->render_question(
+                $slot,
+                $reviewing,
+                $this,
-                    $attemptobj->review_url($slot, $page, $showall));
+                $attemptobj->review_url($slot, $page, $showall)
+            );
+        }
         return $output;
+    }
      * @param display_options $displayoptions instance of display_options
      * @param string $content the rendered display of each question
      * @param quiz_attempt $attemptobj instance of quiz_attempt
      * @return string HTML to display.
      */
-    public function review_form($page, $showall, $displayoptions, $content, $attemptobj) {
+    public function review_form($page, $showall, $displayoptions, $content, $attemptobj)
+    {
         if ($displayoptions->flags != question_display_options::EDITABLE) {
             return $content;
+        }
+        $this->page->requires->js_init_call(
+            'M.mod_quiz.init_review_form',
+            null,
-        $this->page->requires->js_init_call('M.mod_quiz.init_review_form', null, false,
+            false,
+            quiz_get_js_module()
-                quiz_get_js_module());
+        );
+        $output = '';
+        $output .= html_writer::start_tag('form', ['action' => $attemptobj->review_url(
+            null,
-        $output = '';
+            $page,
-        $output .= html_writer::start_tag('form', ['action' => $attemptobj->review_url(null,
+            $showall
-                $page, $showall), 'method' => 'post', 'class' => 'questionflagsaveform']);
+        ), 'method' => 'post', 'class' => 'questionflagsaveform']);
         $output .= html_writer::start_tag('div');
+        $output .= $content;
+        $output .= html_writer::empty_tag('input', [
-        $output .= $content;
+            'type' => 'hidden',
+            'name' => 'sesskey',
-        $output .= html_writer::empty_tag('input', ['type' => 'hidden', 'name' => 'sesskey',
+            'value' => sesskey()
-                'value' => sesskey()]);
+        ]);
+        $output .= html_writer::start_tag('div', ['class' => 'submitbtns']);
-        $output .= html_writer::start_tag('div', ['class' => 'submitbtns']);
+        $output .= html_writer::empty_tag('input', [
+            'type' => 'submit',
-        $output .= html_writer::empty_tag('input', ['type' => 'submit',
+            'class' => 'questionflagsavebutton btn btn-secondary',
+            'name' => 'savingflags',
-                'class' => 'questionflagsavebutton btn btn-secondary', 'name' => 'savingflags',
+            'value' => get_string('saveflags', 'question')
-                'value' => get_string('saveflags', 'question')]);
+        ]);
         $output .= html_writer::end_tag('div');
         $output .= html_writer::end_tag('div');
     /**
      * Returns either a link or button.
+     *
      * @param quiz_attempt $attemptobj instance of quiz_attempt
      */
-    public function finish_review_link(quiz_attempt $attemptobj) {
+    public function finish_review_link(quiz_attempt $attemptobj)
+    {
         $url = $attemptobj->view_url();
+        if ($attemptobj->get_access_manager(time())->attempt_must_be_in_popup()) {
-        if ($attemptobj->get_access_manager(time())->attempt_must_be_in_popup()) {
+            $this->page->requires->js_init_call(
+                'M.mod_quiz.secure_window.init_close_button',
+                [$url->out(false)],
-            $this->page->requires->js_init_call('M.mod_quiz.secure_window.init_close_button',
+                false,
+                quiz_get_js_module()
-                    [$url->out(false)], false, quiz_get_js_module());
+            );
+            return html_writer::empty_tag('input', [
-            return html_writer::empty_tag('input', ['type' => 'button',
+                'type' => 'button',
-                    'value' => get_string('finishreview', 'quiz'),
+                'value' => get_string('finishreview', 'quiz'),
-                    'id' => 'secureclosebutton',
+                'id' => 'secureclosebutton',
-                    'class' => 'mod_quiz-next-nav btn btn-primary']);
+                'class' => 'mod_quiz-next-nav btn btn-primary'
             ]);
+        } else {
+            return html_writer::link(
-        } else {
+                $url,
-            return html_writer::link($url, get_string('finishreview', 'quiz'),
+                get_string('finishreview', 'quiz'),
+                ['class' => 'mod_quiz-next-nav']
-                    ['class' => 'mod_quiz-next-nav']);
+            );
+        }
+    }
      * @param bool|null $showall if true, the URL will be to review the entire attempt on one page,
      *      and $page will be ignored. If null, a sensible default will be chosen.
+     *
      * @return string HTML fragment.
      */
-    public function review_next_navigation(quiz_attempt $attemptobj, $page, $lastpage, $showall = null) {
+    public function review_next_navigation(quiz_attempt $attemptobj, $page, $lastpage, $showall = null)
+    {
         $nav = '';
         if ($page > 0) {
+            $nav .= link_arrow_left(
-            $nav .= link_arrow_left(get_string('navigateprevious', 'quiz'),
+                get_string('navigateprevious', 'quiz'),
-                    $attemptobj->review_url(null, $page - 1, $showall), false, 'mod_quiz-prev-nav');
+                $attemptobj->review_url(null, $page - 1, $showall),
+                false,
+                'mod_quiz-prev-nav'
+            );
+        }
         if ($lastpage) {
             $nav .= $this->finish_review_link($attemptobj);
         } else {
+            $nav .= link_arrow_right(
-            $nav .= link_arrow_right(get_string('navigatenext', 'quiz'),
+                get_string('navigatenext', 'quiz'),
-                    $attemptobj->review_url(null, $page + 1, $showall), false, 'mod_quiz-next-nav');
+                $attemptobj->review_url(null, $page + 1, $showall),
+                false,
+                'mod_quiz-next-nav'
+            );
+        }
         return html_writer::tag('div', $nav, ['class' => 'submitbtns']);
+    }
+     *
      * @param quiz_attempt $attemptobj instance of quiz_attempt
      * @param int $timenow timestamp to use as 'now'.
      * @return string HTML content.
      */
-    public function countdown_timer(quiz_attempt $attemptobj, $timenow) {
+    public function countdown_timer(quiz_attempt $attemptobj, $timenow)
+    {
         $timeleft = $attemptobj->get_time_left_display($timenow);
         if ($timeleft !== false) {
             $ispreview = $attemptobj->is_preview();
     /**
      * Create a preview link
+     *
      * @param moodle_url $url URL to restart the attempt.
      */
-    public function restart_preview_button($url) {
+    public function restart_preview_button($url)
+    {
         return $this->single_button($url, get_string('startnewpreview', 'quiz'));
+    }
     /**
      * Outputs the navigation block panel
+     *
      * @param navigation_panel_base $panel
      */
+    public function navigation_panel(navigation_panel_base $panel)
     public function navigation_panel(navigation_panel_base $panel) {
         $output = '';
         $userpicture = $panel->user_picture();
         if ($userpicture) {
             $fullname = fullname($userpicture->user);
             if ($userpicture->size) {
+                $fullname = html_writer::div($fullname);
+            }
-                $fullname = html_writer::div($fullname);
+            $output .= html_writer::tag(
-            }
+                'div',
+                $this->render($userpicture) . $fullname,
-            $output .= html_writer::tag('div', $this->render($userpicture) . $fullname,
+                ['id' => 'user-picture', 'class' => 'clearfix']
-                    ['id' => 'user-picture', 'class' => 'clearfix']);
+            );
+        }
         $output .= $panel->render_before_button_bits($this);
         $bcc = $panel->get_button_container_class();
         $output .= html_writer::start_tag('div', ['class' => "qn_buttons clearfix $bcc"]);
         foreach ($panel->get_question_buttons() as $button) {
+            $output .= $this->render($button);
+        }
-            $output .= $this->render($button);
+        $output .= html_writer::end_tag('div');
-        }
+        $output .= html_writer::tag(
-        $output .= html_writer::end_tag('div');
+            'div',
+            $panel->render_end_bits($this),
+            ['class' => 'othernav']
+        );
+        $this->page->requires->js_init_call(
-        $output .= html_writer::tag('div', $panel->render_end_bits($this),
+            'M.mod_quiz.nav.init',
-                ['class' => 'othernav']);
+            null,
             false,
-        $this->page->requires->js_init_call('M.mod_quiz.nav.init', null, false,
+            quiz_get_js_module()
-                quiz_get_js_module());
+        );
         return $output;
+    }
+    /**
-    /**
+     * Display a quiz navigation button.
      * Display a quiz navigation button.
      * @param navigation_question_button $button
-     * @param navigation_question_button $button
+     * @return string HTML fragment.
         $a = new stdClass();
         $a->number = s($button->number);
         $a->attributes = implode(' ', $extrainfo);
         $tagcontents = html_writer::tag('span', '', ['class' => 'thispageholder']) .
-                html_writer::tag('span', '', ['class' => 'trafficlight']) .
+            html_writer::tag('span', '', ['class' => 'trafficlight']) .
+            get_string($qnostring, 'quiz', $a);
-                get_string($qnostring, 'quiz', $a);
+        $tagattributes = [
+            'class' => implode(' ', $classes),
+            'id' => $button->id,
-        $tagattributes = ['class' => implode(' ', $classes), 'id' => $button->id,
+            'title' => $tooltip,
+            'data-quiz-page' => $button->page
-                'title' => $tooltip, 'data-quiz-page' => $button->page];
+        ];
         if ($button->url) {
             return html_writer::link($button->url, $tagcontents, $tagattributes);
      * Display a quiz navigation heading.
+     *
      * @param navigation_section_heading $heading the heading.
      * @return string HTML fragment.
      */
-    protected function render_navigation_section_heading(navigation_section_heading $heading) {
+    protected function render_navigation_section_heading(navigation_section_heading $heading)
+    {
         if (empty($heading->heading)) {
             $headingtext = get_string('sectionnoname', 'quiz');
             $class = ' dimmed_text';
         } else {
             $headingtext = $heading->heading;
+     *
      * @param links_to_other_attempts $links
      * @return string HTML fragment.
      */
     protected function render_links_to_other_attempts(
-            links_to_other_attempts $links) {
+        links_to_other_attempts $links
+    ) {
         $attemptlinks = [];
         foreach ($links->links as $attempt => $url) {
             if (!$url) {
                 $attemptlinks[] = html_writer::tag('strong', $attempt);
             } else {
      * having this renderer method allows themes to override the default rendering.
+     *
      * @param grade_out_of $grade
      * @return string HTML to output.
      */
-    protected function render_grade_out_of(grade_out_of $grade): string {
+    protected function render_grade_out_of(grade_out_of $grade): string
+    {
+        return get_string(
-        return get_string($grade->get_string_key(), 'quiz',
+            $grade->get_string_key(),
+            'quiz',
-                $grade->style_formatted_values($grade->get_formatted_values()));
+            $grade->style_formatted_values($grade->get_formatted_values())
+        );
+    }
     /**
      * Render the 'start attempt' page.
+     *
      * @param \mod_quiz\quiz_settings $quizobj
      * @param preflight_check_form $mform
      * @return string
      */
-    public function start_attempt_page(quiz_settings $quizobj, preflight_check_form $mform) {
+    public function start_attempt_page(quiz_settings $quizobj, preflight_check_form $mform)
+    {
         $output = '';
         $output .= $this->header();
         $output .= $this->during_attempt_tertiary_nav($quizobj->view_url());
-        $output .= $this->heading(format_string($quizobj->get_quiz_name(), true,
+        $output .= $this->heading(format_string(
+            $quizobj->get_quiz_name(),
+            true,
-                ["context" => $quizobj->get_context()]));
+            ["context" => $quizobj->get_context()]
+        ));
         $output .= $this->quiz_intro($quizobj->get_quiz(), $quizobj->get_cm());
         $output .= $mform->render();
         $output .= $this->footer();
         return $output;
+    }
      * @param array $slots Contains an array of integers that relate to questions
      * @param int $id The ID of an attempt
      * @param int $nextpage The number of the next page
      * @return string HTML to output.
      */
-    public function attempt_page($attemptobj, $page, $accessmanager, $messages, $slots, $id,
+    public function attempt_page(
+        $attemptobj,
+        $page,
+        $accessmanager,
+        $messages,
+        $slots,
+        $id,
-            $nextpage) {
+        $nextpage
+    ) {
         $output = '';
         $output .= $this->header();
         $output .= $this->during_attempt_tertiary_nav($attemptobj->view_url());
         $output .= $this->quiz_notices($messages);
         $output .= $this->countdown_timer($attemptobj, time());
      * Render the tertiary navigation for pages during the attempt.
+     *
      * @param string|moodle_url $quizviewurl url of the view.php page for this quiz.
      * @return string HTML to output.
      */
-    public function during_attempt_tertiary_nav($quizviewurl): string {
+    public function during_attempt_tertiary_nav($quizviewurl): string
+    {
         $output = '';
-        $output .= html_writer::start_div('container-fluid tertiary-navigation');
+        $output .= html_writer::start_div('container-fluid tertiary-navigation', ['style' => 'width: fit-content;']);
         $output .= html_writer::start_div('row');
         $output .= html_writer::start_div('navitem');
-        $output .= html_writer::link($quizviewurl, get_string('back'),
+        $output .= html_writer::link(
+            $quizviewurl,
+            get_string('back'),
-                ['class' => 'btn btn-secondary']);
+            ['class' => 'btn btn-secondary']
+        );
         $output .= html_writer::end_div();
         $output .= html_writer::end_div();
         $output .= html_writer::end_div();
         return $output;
+    }
     /**
      * Returns any notices.
+     *
      * @param array $messages
      */
-    public function quiz_notices($messages) {
+    public function quiz_notices($messages)
+    {
         if (!$messages) {
             return '';
+        }
         return $this->notification(
-                html_writer::tag('p', get_string('accessnoticesheader', 'quiz')) . $this->access_messages($messages),
+            html_writer::tag('p', get_string('accessnoticesheader', 'quiz')) . $this->access_messages($messages),
-                'warning',
+            'warning',
-                false
+            false
         );
+    }
     /**
      * @param int $page Current page number
      * @param array $slots Array of integers relating to questions
      * @param int $id ID of the attempt
      * @param int $nextpage Next page number
      */
-    public function attempt_form($attemptobj, $page, $slots, $id, $nextpage) {
+    public function attempt_form($attemptobj, $page, $slots, $id, $nextpage)
+    {
         $output = '';
         // Start the form.
+        $output .= html_writer::start_tag(
+            'form',
+            [
-        $output .= html_writer::start_tag('form',
+                'action' => new moodle_url(
-                ['action' => new moodle_url($attemptobj->processattempt_url(),
+                    $attemptobj->processattempt_url(),
+                    ['cmid' => $attemptobj->get_cmid()]
+                ),
-                        ['cmid' => $attemptobj->get_cmid()]), 'method' => 'post',
+                'method' => 'post',
+                'enctype' => 'multipart/form-data',
-                        'enctype' => 'multipart/form-data', 'accept-charset' => 'utf-8',
+                'accept-charset' => 'utf-8',
+                'id' => 'responseform'
+            ]
-                        'id' => 'responseform']);
+        );
         $output .= html_writer::start_tag('div');
         // Print all the questions.
+        foreach ($slots as $slot) {
+            $output .= $attemptobj->render_question(
+                $slot,
-        foreach ($slots as $slot) {
+                false,
+                $this,
-            $output .= $attemptobj->render_question($slot, false, $this,
+                $attemptobj->attempt_url($slot, $page)
-                    $attemptobj->attempt_url($slot, $page));
+            );
+        }
         $navmethod = $attemptobj->get_quiz()->navmethod;
+        $output .= $this->attempt_navigation_buttons($page, $attemptobj->is_last_page($page), $navmethod);
-        $output .= $this->attempt_navigation_buttons($page, $attemptobj->is_last_page($page), $navmethod);
+        // Some hidden fields to track what is going on.
+        $output .= html_writer::empty_tag('input', [
+            'type' => 'hidden',
+            'name' => 'attempt',
+            'value' => $attemptobj->get_attemptid()
+        ]);
-        // Some hidden fields to track what is going on.
+        $output .= html_writer::empty_tag('input', [
+            'type' => 'hidden',
-        $output .= html_writer::empty_tag('input', ['type' => 'hidden', 'name' => 'attempt',
+            'name' => 'thispage',
+            'value' => $page,
+            'id' => 'followingpage'
-                'value' => $attemptobj->get_attemptid()]);
+        ]);
+        $output .= html_writer::empty_tag('input', [
-        $output .= html_writer::empty_tag('input', ['type' => 'hidden', 'name' => 'thispage',
+            'type' => 'hidden',
+            'name' => 'nextpage',
-                'value' => $page, 'id' => 'followingpage']);
+            'value' => $nextpage
+        ]);
+        $output .= html_writer::empty_tag('input', [
+            'type' => 'hidden',
-        $output .= html_writer::empty_tag('input', ['type' => 'hidden', 'name' => 'nextpage',
+            'name' => 'timeup',
+            'value' => '0',
+            'id' => 'timeup'
-                'value' => $nextpage]);
+        ]);
+        $output .= html_writer::empty_tag('input', [
-        $output .= html_writer::empty_tag('input', ['type' => 'hidden', 'name' => 'timeup',
+            'type' => 'hidden',
+            'name' => 'sesskey',
-                'value' => '0', 'id' => 'timeup']);
+            'value' => sesskey()
+        ]);
+        $output .= html_writer::empty_tag('input', [
+            'type' => 'hidden',
-        $output .= html_writer::empty_tag('input', ['type' => 'hidden', 'name' => 'sesskey',
+            'name' => 'mdlscrollto',
-                'value' => sesskey()]);
+            'value' => '',
-        $output .= html_writer::empty_tag('input', ['type' => 'hidden', 'name' => 'mdlscrollto',
+            'id' => 'mdlscrollto'
-                'value' => '', 'id' => 'mdlscrollto']);
+        ]);
+        // Add a hidden field with questionids. Do this at the end of the form, so
+        // if you navigate before the form has finished loading, it does not wipe all
+        // the student's answers.
-        // Add a hidden field with questionids. Do this at the end of the form, so
+        $output .= html_writer::empty_tag('input', [
-        // if you navigate before the form has finished loading, it does not wipe all
+            'type' => 'hidden',
-        // the student's answers.
+            'name' => 'slots',
      * @param int $page the page number. Starts at 0 for the first page.
      * @param bool $lastpage is this the last page in the quiz?
      * @param string $navmethod Optional quiz attribute, 'free' (default) or 'sequential'
      * @return string HTML fragment.
      */
-    protected function attempt_navigation_buttons($page, $lastpage, $navmethod = 'free') {
+    protected function attempt_navigation_buttons($page, $lastpage, $navmethod = 'free')
+    {
         $output = '';
         $output .= html_writer::start_tag('div', ['class' => 'submitbtns']);
         if ($page > 0 && $navmethod == 'free') {
+            $output .= html_writer::empty_tag('input', [
+                'type' => 'submit',
-            $output .= html_writer::empty_tag('input', ['type' => 'submit', 'name' => 'previous',
+                'name' => 'previous',
+                'value' => get_string('navigateprevious', 'quiz'),
-                    'value' => get_string('navigateprevious', 'quiz'), 'class' => 'mod_quiz-prev-nav btn btn-secondary',
+                'class' => 'mod_quiz-prev-nav btn btn-secondary',
+                'id' => 'mod_quiz-prev-nav'
-                    'id' => 'mod_quiz-prev-nav']);
+            ]);
             $this->page->requires->js_call_amd('core_form/submit', 'init', ['mod_quiz-prev-nav']);
+        }
         if ($lastpage) {
             $nextlabel = get_string('endtest', 'quiz');
         } else {
             $nextlabel = get_string('navigatenext', 'quiz');
         }
+        $output .= html_writer::empty_tag('input', [
+            'type' => 'submit',
+            'name' => 'next',
-        $output .= html_writer::empty_tag('input', ['type' => 'submit', 'name' => 'next',
+            'value' => $nextlabel,
+            'class' => 'mod_quiz-next-nav btn btn-primary',
+            'id' => 'mod_quiz-next-nav'
-                'value' => $nextlabel, 'class' => 'mod_quiz-next-nav btn btn-primary', 'id' => 'mod_quiz-next-nav']);
+        ]);
         $output .= html_writer::end_tag('div');
         $this->page->requires->js_call_amd('core_form/submit', 'init', ['mod_quiz-next-nav']);
+     *
      * @param int $slot the number of the slot to generate the button for.
      * @param bool $disabled if true, output the button disabled.
      * @return string HTML fragment.
      */
-    public function redo_question_button($slot, $disabled) {
+    public function redo_question_button($slot, $disabled)
+    {
+        $attributes = [
+            'type' => 'submit',
-        $attributes = ['type' => 'submit', 'name' => 'redoslot' . $slot,
+            'name' => 'redoslot' . $slot,
-                'value' => get_string('redoquestion', 'quiz'),
+            'value' => get_string('redoquestion', 'quiz'),
-                'class' => 'mod_quiz-redo_question_button btn btn-secondary',
+            'class' => 'mod_quiz-redo_question_button btn btn-secondary',
-                'id' => 'redoslot' . $slot . '-submit',
+            'id' => 'redoslot' . $slot . '-submit',
-                'data-savescrollposition' => 'true',
+            'data-savescrollposition' => 'true',
-            ];
+        ];
         if ($disabled) {
             $attributes['disabled'] = 'disabled';
         } else {
             $this->page->requires->js_call_amd('core_question/question_engine', 'initSubmitButton', [$attributes['id']]);
+        }
      * Initialise the JavaScript required to initialise the countdown timer.
+     *
      * @param int $timerstartvalue time remaining, in seconds.
      * @param bool $ispreview true if this is a preview attempt.
      */
-    public function initialise_timer($timerstartvalue, $ispreview) {
+    public function initialise_timer($timerstartvalue, $ispreview)
+    {
         $options = [$timerstartvalue, (bool) $ispreview];
         $this->page->requires->js_init_call('M.mod_quiz.timer.init', $options, false, quiz_get_js_module());
+    }
+     *
      * @param moodle_url $url the URL to redirect the parent window to.
      * @param string $message message to display before closing the window. (optional)
      * @return string HTML to output.
      */
-    public function close_attempt_popup($url, $message = '') {
+    public function close_attempt_popup($url, $message = '')
+    {
         $output = '';
         $output .= $this->header();
         $output .= $this->box_start();
             $delay = 5;
         } else {
             $output .= html_writer::tag('p', get_string('pleaseclose', 'quiz'));
             $delay = 0;
+        }
-        $this->page->requires->js_init_call('M.mod_quiz.secure_window.close',
+        $this->page->requires->js_init_call(
+            'M.mod_quiz.secure_window.close',
-            [$url->out(false), $delay], false, quiz_get_js_module());
+            [$url->out(false), $delay],
+            false,
+            quiz_get_js_module()
+        );
         $output .= $this->box_end();
         $output .= $this->footer();
         return $output;
      * Print each message in an array, surrounded by &lt;p>, &lt;/p> tags.
+     *
      * @param array $messages the array of message strings.
      * @return string HTML to output.
      */
-    public function access_messages($messages) {
+    public function access_messages($messages)
+    {
         $output = '';
         foreach ($messages as $message) {
             $output .= html_writer::tag('p', $message, ['class' => 'text-left']);
+        }
         return $output;
      * Create the summary page
+     *
      * @param quiz_attempt $attemptobj
      * @param display_options $displayoptions
      */
-    public function summary_page($attemptobj, $displayoptions) {
+    public function summary_page($attemptobj, $displayoptions)
+    {
         $output = '';
         $output .= $this->header();
         $output .= $this->during_attempt_tertiary_nav($attemptobj->view_url());
         $output .= $this->heading(format_string($attemptobj->get_quiz_name()));
         $output .= $this->heading(get_string('summaryofattempt', 'quiz'), 3);
      * Generates the table of summarydata
+     *
      * @param quiz_attempt $attemptobj
      * @param display_options $displayoptions
      */
-    public function summary_table($attemptobj, $displayoptions) {
+    public function summary_table($attemptobj, $displayoptions)
+    {
         // Prepare the summary table header.
         $table = new html_table();
         $table->attributes['class'] = 'generaltable quizsummaryofattempt boxaligncenter';
         $table->head = [get_string('question', 'quiz'), get_string('status', 'quiz')];
         $table->align = ['left', 'left'];
             // Real question, show it.
             $flag = '';
             if ($attemptobj->is_question_flagged($slot)) {
+                // Quiz has custom JS manipulating these image tags - so we can't use the pix_icon method here.
-                // Quiz has custom JS manipulating these image tags - so we can't use the pix_icon method here.
+                $flag = html_writer::empty_tag('img', [
-                $flag = html_writer::empty_tag('img', ['src' => $this->image_url('i/flagged'),
+                    'src' => $this->image_url('i/flagged'),
+                    'alt' => get_string('flagged', 'question'),
+                    'class' => 'questionflag icon-post'
-                        'alt' => get_string('flagged', 'question'), 'class' => 'questionflag icon-post']);
+                ]);
+            }
+            if ($attemptobj->can_navigate_to($slot)) {
+                $row = [
-            if ($attemptobj->can_navigate_to($slot)) {
+                    html_writer::link(
-                $row = [html_writer::link($attemptobj->attempt_url($slot),
+                        $attemptobj->attempt_url($slot),
+                        $attemptobj->get_question_number($slot) . $flag
-                        $attemptobj->get_question_number($slot) . $flag),
+                    ),
+                    $attemptobj->get_question_status($slot, $displayoptions->correctness)
-                        $attemptobj->get_question_status($slot, $displayoptions->correctness)];
+                ];
+            } else {
-            } else {
+                $row = [
-                $row = [$attemptobj->get_question_number($slot) . $flag,
+                    $attemptobj->get_question_number($slot) . $flag,
+                    $attemptobj->get_question_status($slot, $displayoptions->correctness)
-                        $attemptobj->get_question_status($slot, $displayoptions->correctness)];
+                ];
+            }
             if ($markscolumn) {
                 $row[] = $attemptobj->get_question_mark($slot);
+            }
             $table->data[] = $row;
+            $table->rowclasses[] = 'quizsummary' . $slot . ' ' . $attemptobj->get_question_state_class(
-            $table->rowclasses[] = 'quizsummary' . $slot . ' ' . $attemptobj->get_question_state_class(
+                $slot,
+                $displayoptions->correctness
-                            $slot, $displayoptions->correctness);
+            );
+        }
         // Print the summary table.
     /**
      * Creates any controls the page should have.
+     *
      * @param quiz_attempt $attemptobj
      */
-    public function summary_page_controls($attemptobj) {
+    public function summary_page_controls($attemptobj)
+    {
         $output = '';
         // Return to place button.
         if ($attemptobj->get_state() == quiz_attempt::IN_PROGRESS) {
             $button = new single_button(
-                    new moodle_url($attemptobj->attempt_url(null, $attemptobj->get_currentpage())),
+                new moodle_url($attemptobj->attempt_url(null, $attemptobj->get_currentpage())),
+                get_string('returnattempt', 'quiz')
-                    get_string('returnattempt', 'quiz'));
+            );
+            $output .= $this->container($this->container(
+                $this->render($button),
-            $output .= $this->container($this->container($this->render($button),
+                'controls'
-                    'controls'), 'submitbtns mdl-align');
+            ), 'submitbtns mdl-align');
+        }
         // Finish attempt button.
         $options = [
-                'attempt' => $attemptobj->get_attemptid(),
+            'attempt' => $attemptobj->get_attemptid(),
-                'finishattempt' => 1,
+            'finishattempt' => 1,
-                'timeup' => 0,
+            'timeup' => 0,
-                'slots' => '',
+            'slots' => '',
-                'cmid' => $attemptobj->get_cmid(),
+            'cmid' => $attemptobj->get_cmid(),
-                'sesskey' => sesskey(),
+            'sesskey' => sesskey(),
         ];
+        $button = new single_button(
-        $button = new single_button(
+            new moodle_url($attemptobj->processattempt_url(), $options),
-                new moodle_url($attemptobj->processattempt_url(), $options),
+            get_string('submitallandfinish', 'quiz')
-                get_string('submitallandfinish', 'quiz'));
+        );
         $button->class = 'btn-finishattempt';
         $button->formid = 'frm-finishattempt';
         $duedate = $attemptobj->get_due_date();
         $message = '';
         if ($attemptobj->get_state() == quiz_attempt::OVERDUE) {
-            $message = get_string('overduemustbesubmittedby', 'quiz', userdate($duedate));
             $message = get_string('overduemustbesubmittedby', 'quiz', userdate($duedate));
         } else {
             if ($duedate) {
                 $message = get_string('mustbesubmittedby', 'quiz', userdate($duedate));
+            }
+        }
+        $output .= $this->countdown_timer($attemptobj, time());
+        $output .= $this->container($message . $this->container(
-        $output .= $this->countdown_timer($attemptobj, time());
+            $this->render($button),
-        $output .= $this->container($message . $this->container(
+            'controls'
-                        $this->render($button), 'controls'), 'submitbtns mdl-align');
+        ), 'submitbtns mdl-align');
      * @param stdClass $cm the course_module settings row from the database.
      * @param context_module $context the quiz context.
      * @param view_page $viewobj
      * @return string HTML to display
      */
-    public function view_page($course, $quiz, $cm, $context, $viewobj) {
+    public function view_page($course, $quiz, $cm, $context, $viewobj)
+    {
         $output = '';
         $output .= $this->view_page_tertiary_nav($viewobj);
         $output .= $this->view_information($quiz, $cm, $context, $viewobj->infomessages);
      * Render the tertiary navigation for the view page.
+     *
      * @param view_page $viewobj the information required to display the view page.
      * @return string HTML to output.
      */
-    public function view_page_tertiary_nav(view_page $viewobj): string {
+    public function view_page_tertiary_nav(view_page $viewobj): string
+    {
         $content = '';
         if ($viewobj->buttontext) {
+            $attemptbtn = $this->start_attempt_button(
+                $viewobj->buttontext,
-            $attemptbtn = $this->start_attempt_button($viewobj->buttontext,
+                $viewobj->startattempturl,
-                    $viewobj->startattempturl, $viewobj->preflightcheckform,
+                $viewobj->preflightcheckform,
+                $viewobj->popuprequired,
+                $viewobj->popupoptions
-                    $viewobj->popuprequired, $viewobj->popupoptions);
+            );
             $content .= $attemptbtn;
+        }
+        if ($viewobj->canedit && !$viewobj->quizhasquestions) {
+            $content .= html_writer::link(
-        if ($viewobj->canedit && !$viewobj->quizhasquestions) {
+                $viewobj->editurl,
+                get_string('addquestion', 'quiz'),
-            $content .= html_writer::link($viewobj->editurl, get_string('addquestion', 'quiz'),
+                ['class' => 'btn btn-secondary']
-                    ['class' => 'btn btn-secondary']);
+            );
+        }
      * at the end of the review page.
+     *
      * @param view_page $viewobj the information required to display the view page.
      * @return string HTML to output.
      */
-    public function view_page_buttons(view_page $viewobj) {
+    public function view_page_buttons(view_page $viewobj)
+    {
         $output = '';
         if (!$viewobj->quizhasquestions) {
             $output .= html_writer::div(
-                    $this->notification(get_string('noquestions', 'quiz'), 'warning', false),
+                $this->notification(get_string('noquestions', 'quiz'), 'warning', false),
+                'text-left mb-3'
-                    'text-left mb-3');
+            );
+        }
         $output .= $this->access_messages($viewobj->preventmessages);
+        if ($viewobj->showbacktocourse) {
-        if ($viewobj->showbacktocourse) {
+            $output .= $this->single_button(
+                $viewobj->backtocourseurl,
-            $output .= $this->single_button($viewobj->backtocourseurl,
+                get_string('backtocourse', 'quiz'),
+                'get',
-                    get_string('backtocourse', 'quiz'), 'get',
+                ['class' => 'continuebutton']
-                    ['class' => 'continuebutton']);
+            );
+        }
      * @param preflight_check_form|null $preflightcheckform deprecated.
      * @param bool $popuprequired whether the attempt needs to be opened in a pop-up.
      * @param array $popupoptions the options to use if we are opening a popup.
      * @return string HTML fragment.
      */
-    public function start_attempt_button($buttontext, moodle_url $url,
+    public function start_attempt_button(
+        $buttontext,
+        moodle_url $url,
-            preflight_check_form $preflightcheckform = null,
+        preflight_check_form $preflightcheckform = null,
-            $popuprequired = false, $popupoptions = null) {
+        $popuprequired = false,
+        $popupoptions = null
+    ) {
         $button = new single_button($url, $buttontext, 'post', single_button::BUTTON_PRIMARY);
         $button->class .= ' quizstartbuttondiv';
         if ($popuprequired) {
         if ($popuprequired && $popupoptions) {
             $action = new popup_action('click', $url, 'popup', $popupoptions);
             $popupjsoptions = $action->get_js_options();
+        }
+        $this->page->requires->js_call_amd(
+            'mod_quiz/preflightcheck',
+            'init',
-        $this->page->requires->js_call_amd('mod_quiz/preflightcheck', 'init',
+            [
+                '.quizstartbuttondiv [type=submit]',
-                ['.quizstartbuttondiv [type=submit]', get_string('startattempt', 'quiz'),
+                get_string('startattempt', 'quiz'),
+                '#mod_quiz_preflight_form',
+                $popupjsoptions
+            ]
-                        '#mod_quiz_preflight_form', $popupjsoptions]);
+        );
         return $this->render($button) . ($preflightcheckform ? $preflightcheckform->render() : '');
      * @param moodle_url $editurl URL of the edit quiz page.
      * @return string HTML to output.
+     *
      * @deprecated since Moodle 4.0 MDL-71915 - please do not use this function any more.
      */
-    public function no_questions_message($canedit, $editurl) {
+    public function no_questions_message($canedit, $editurl)
+    {
         debugging('no_questions_message() is deprecated, please use generate_no_questions_message() instead.', DEBUG_DEVELOPER);
         $output = html_writer::start_tag('div', ['class' => 'card text-center mb-3']);
      * @param stdClass $cm the course_module settings row from the database.
      * @param context_module $context the quiz context.
      * @param array $messages Array containing any messages
      * @param view_page $viewobj
      */
-    public function view_page_guest($course, $quiz, $cm, $context, $messages, $viewobj) {
+    public function view_page_guest($course, $quiz, $cm, $context, $messages, $viewobj)
+    {
         $output = '';
         $output .= $this->view_page_tertiary_nav($viewobj);
         $output .= $this->view_information($quiz, $cm, $context, $messages);
         $guestno = html_writer::tag('p', get_string('guestsno', 'quiz'));
         $liketologin = html_writer::tag('p', get_string('liketologin'));
      * @param stdClass $cm the course_module settings row from the database.
      * @param context_module $context the quiz context.
      * @param array $messages Array containing any messages
      * @param view_page $viewobj
      */
-    public function view_page_notenrolled($course, $quiz, $cm, $context, $messages, $viewobj) {
+    public function view_page_notenrolled($course, $quiz, $cm, $context, $messages, $viewobj)
+    {
         global $CFG;
         $output = '';
         $output .= $this->view_page_tertiary_nav($viewobj);
         $output .= $this->view_information($quiz, $cm, $context, $messages);
         $youneedtoenrol = html_writer::tag('p', get_string('youneedtoenrol', 'quiz'));
-        $button = html_writer::tag('p',
+        $button = html_writer::tag(
+            'p',
-                $this->continue_button($CFG->wwwroot . '/course/view.php?id=' . $course->id));
+            $this->continue_button($CFG->wwwroot . '/course/view.php?id=' . $course->id)
+        );
         $output .= $this->box($youneedtoenrol . "\n\n" . $button . "\n", 'generalbox', 'notice');
         return $output;
+    }
      * @param context $context the quiz context.
      * @param array $messages any access messages that should be described.
      * @param bool $quizhasquestions does quiz has questions added.
      * @return string HTML to output.
      */
-    public function view_information($quiz, $cm, $context, $messages, bool $quizhasquestions = false) {
+    public function view_information($quiz, $cm, $context, $messages, bool $quizhasquestions = false)
+    {
         $output = '';
         // Output any access messages.
         if ($messages) {
             $output .= $this->box($this->access_messages($messages), 'quizinfo');
+        }
         // Show number of attempts summary to those who can view reports.
+        if (has_capability('mod/quiz:viewreports', $context)) {
+            if ($strattemptnum = $this->quiz_attempt_summary_link_to_reports(
-        if (has_capability('mod/quiz:viewreports', $context)) {
+                $quiz,
+                $cm,
-            if ($strattemptnum = $this->quiz_attempt_summary_link_to_reports($quiz, $cm,
+                $context
+            )) {
+                $output .= html_writer::tag(
-                    $context)) {
+                    'div',
+                    $strattemptnum,
-                $output .= html_writer::tag('div', $strattemptnum,
+                    ['class' => 'quizattemptcounts']
-                        ['class' => 'quizattemptcounts']);
+                );
+            }
+        }
+     *
      * @param stdClass $quiz the quiz settings.
      * @param stdClass $cm the course_module object.
      * @return string HTML to output.
      */
-    public function quiz_intro($quiz, $cm) {
+    public function quiz_intro($quiz, $cm)
+    {
         if (html_is_blank($quiz->intro)) {
             return '';
+        }
         return $this->box(format_module_intro('quiz', $quiz, $cm->id), 'generalbox', 'intro');
+    }
     /**
      * Generates the table heading.
+     */
-     */
+    public function view_table_heading()
     public function view_table_heading() {
         return $this->heading(get_string('summaryofattempts', 'quiz'), 3);
+    }
      * @param stdClass $quiz the quiz settings.
      * @param context_module $context the quiz context.
      * @param view_page $viewobj
      * @deprecated Since 4.4 please use the {@see list_of_attempts} renderable instead.
      */
-    public function view_table($quiz, $context, $viewobj) {
+    public function view_table($quiz, $context, $viewobj)
+    {
         debugging('view_table has been deprecated since 4.4 please use the list_of_attempts renderable instead.');
         if (!$viewobj->attempts) {
             return '';
+        }
         $table->head[] = get_string('attemptstate', 'quiz');
         $table->align[] = 'left';
         $table->size[] = '';
         if ($viewobj->markcolumn) {
             $table->head[] = get_string('marks', 'quiz') . ' / ' .
-                    quiz_format_grade($quiz, $quiz->sumgrades);
+                quiz_format_grade($quiz, $quiz->sumgrades);
             $table->align[] = 'center';
             $table->size[] = '';
+        }
         if ($viewobj->gradecolumn) {
             $table->head[] = get_string('gradenoun') . ' / ' .
-                    quiz_format_grade($quiz, $quiz->grade);
+                quiz_format_grade($quiz, $quiz->grade);
             $table->align[] = 'center';
             $table->size[] = '';
+        }
         if ($viewobj->canreviewmine) {
             $table->head[] = get_string('review', 'quiz');
+            }
             $row[] = $this->attempt_state($attemptobj);
+            if ($viewobj->markcolumn) {
-            if ($viewobj->markcolumn) {
+                if (
+                    $attemptoptions->marks >= question_display_options::MARK_AND_MAX &&
-                if ($attemptoptions->marks >= question_display_options::MARK_AND_MAX &&
+                    $attemptobj->is_finished()
-                        $attemptobj->is_finished()) {
+                ) {
                     $row[] = quiz_format_grade($quiz, $attemptobj->get_sum_marks());
                 } else {
                     $row[] = '';
+                }
+            }
+            // Outside the if because we may be showing feedback but not grades.
-            // Outside the if because we may be showing feedback but not grades.
+            $attemptgrade = quiz_rescale_grade($attemptobj->get_sum_marks(), $quiz, false);
-            $attemptgrade = quiz_rescale_grade($attemptobj->get_sum_marks(), $quiz, false);
+            if ($viewobj->gradecolumn) {
                 if (
+                    $attemptoptions->marks >= question_display_options::MARK_AND_MAX &&
-            if ($viewobj->gradecolumn) {
+                    $attemptobj->is_finished()
-                if ($attemptoptions->marks >= question_display_options::MARK_AND_MAX &&
+                ) {
-                        $attemptobj->is_finished()) {
+                    // Highlight the highest grade if appropriate.
-                    // Highlight the highest grade if appropriate.
+                    if (
+                        $viewobj->overallstats && !$attemptobj->is_preview()
-                    if ($viewobj->overallstats && !$attemptobj->is_preview()
+                        && $viewobj->numattempts > 1 && !is_null($viewobj->mygrade)
-                            && $viewobj->numattempts > 1 && !is_null($viewobj->mygrade)
+                        && $attemptobj->get_state() == quiz_attempt::FINISHED
-                            && $attemptobj->get_state() == quiz_attempt::FINISHED
+                        && $attemptgrade == $viewobj->mygrade
-                            && $attemptgrade == $viewobj->mygrade
+                        && $quiz->grademethod == QUIZ_GRADEHIGHEST
-                            && $quiz->grademethod == QUIZ_GRADEHIGHEST) {
+                    ) {
                         $table->rowclasses[$attemptobj->get_attempt_number()] = 'bestrow';
+                    }
                     $row[] = quiz_format_grade($quiz, $attemptgrade);
+                } else {
-                } else {
+                    $row[] = '';
+                }
+            }
                     $row[] = '';
             if ($viewobj->canreviewmine) {
                 $row[] = $viewobj->accessmanager->make_review_link(
                     $attemptobj->get_attempt(),
      * Generate a brief textual description of the current state of an attempt.
+     *
      * @param quiz_attempt $attemptobj the attempt
      * @return string the appropriate lang string to describe the state.
      */
-    public function attempt_state($attemptobj) {
+    public function attempt_state($attemptobj)
+    {
         switch ($attemptobj->get_state()) {
             case quiz_attempt::IN_PROGRESS:
                 return get_string('stateinprogress', 'quiz');
             case quiz_attempt::OVERDUE:
+                return get_string('stateoverdue', 'quiz') . html_writer::tag(
+                    'span',
-                return get_string('stateoverdue', 'quiz') . html_writer::tag('span',
+                    get_string(
+                        'stateoverduedetails',
-                                get_string('stateoverduedetails', 'quiz',
+                        'quiz',
+                        userdate($attemptobj->get_due_date())
-                                        userdate($attemptobj->get_due_date())),
+                    ),
+                    ['class' => 'statedetails']
-                                ['class' => 'statedetails']);
+                );
+            case quiz_attempt::FINISHED:
+                return get_string('statefinished', 'quiz') . html_writer::tag(
-            case quiz_attempt::FINISHED:
+                    'span',
+                    get_string(
-                return get_string('statefinished', 'quiz') . html_writer::tag('span',
+                        'statefinisheddetails',
+                        'quiz',
-                                get_string('statefinisheddetails', 'quiz',
+                        userdate($attemptobj->get_submitted_date())
+                    ),
-                                        userdate($attemptobj->get_submitted_date())),
+                    ['class' => 'statedetails']
-                                ['class' => 'statedetails']);
+                );
      * @param context_module $context The quiz context.
      * @param stdClass|cm_info $cm The course module information.
      * @param view_page $viewobj
      * @return string HTML to display.
      */
-    public function view_result_info($quiz, $context, $cm, $viewobj) {
+    public function view_result_info($quiz, $context, $cm, $viewobj)
+    {
         $output = '';
         if (!$viewobj->numattempts && !$viewobj->gradecolumn && is_null($viewobj->mygrade)) {
             return $output;
+        }
         $resultinfo = '';
+            }
+        }
+        if ($viewobj->mygradeoverridden) {
-        if ($viewobj->mygradeoverridden) {
+            $resultinfo .= html_writer::tag(
+                'p',
+                get_string('overriddennotice', 'grades'),
-            $resultinfo .= html_writer::tag('p', get_string('overriddennotice', 'grades'),
+                ['class' => 'overriddennotice']
-                            ['class' => 'overriddennotice']) . "\n";
+            ) . "\n";
+        }
         if ($viewobj->gradebookfeedback) {
             $resultinfo .= $this->heading(get_string('comment', 'quiz'), 3);
             $resultinfo .= html_writer::div($viewobj->gradebookfeedback, 'quizteacherfeedback') . "\n";
+        }
         if ($viewobj->feedbackcolumn) {
             $resultinfo .= $this->heading(get_string('overallfeedback', 'quiz'), 3);
             $resultinfo .= html_writer::div(
+                quiz_feedback_for_grade($viewobj->mygrade, $quiz, $context),
-                            quiz_feedback_for_grade($viewobj->mygrade, $quiz, $context),
+                'quizgradefeedback'
-                            'quizgradefeedback') . "\n";
+            ) . "\n";
+        }
      * @param moodle_url $url of the target page.
      * @param bool $reviewinpopup whether a pop-up is required.
      * @param array $popupoptions options to pass to the popup_action constructor.
      * @return string HTML to output.
      */
-    public function review_link($url, $reviewinpopup, $popupoptions) {
+    public function review_link($url, $reviewinpopup, $popupoptions)
+    {
         if ($reviewinpopup) {
             $button = new single_button($url, get_string('review', 'quiz'));
             $button->add_action(new popup_action('click', $url, 'quizpopup', $popupoptions));
             return $this->render($button);
         } else {
+            return html_writer::link(
+                $url,
-            return html_writer::link($url, get_string('review', 'quiz'),
+                get_string('review', 'quiz'),
-                    ['title' => get_string('reviewthisattempt', 'quiz')]);
+                ['title' => get_string('reviewthisattempt', 'quiz')]
+            );
+        }
+    }
     /**
      * Displayed where there might normally be a review link, to explain why the
      * review is not available at this time.
+     *
      * @param string $message optional message explaining why the review is not possible.
      * @return string HTML to output.
      */
+    public function no_review_message($message)
-    public function no_review_message($message) {
+    {
+        return html_writer::nonempty_tag(
+            'span',
-        return html_writer::nonempty_tag('span', $message,
+            $message,
+            ['class' => 'noreviewmessage']
-                ['class' => 'noreviewmessage']);
+        );
+    }
     /**
      *      instead of 'Attempts: 0'.
      * @param int $currentgroup if there is a concept of current group where this method is being
      *      called (e.g. a report) pass it in here. Default 0 which means no current group.
      * @return string HTML fragment for the link.
      */
-    public function quiz_attempt_summary_link_to_reports($quiz, $cm, $context,
+    public function quiz_attempt_summary_link_to_reports(
+        $quiz,
+        $cm,
+        $context,
-            $returnzero = false, $currentgroup = 0) {
+        $returnzero = false,
+        $currentgroup = 0
+    ) {
         global $CFG;
         $summary = quiz_num_attempt_summary($quiz, $cm, $returnzero, $currentgroup);
         if (!$summary) {
             return '';
+        }
         require_once($CFG->dirroot . '/mod/quiz/report/reportlib.php');
+        $url = new moodle_url('/mod/quiz/report.php', [
-        $url = new moodle_url('/mod/quiz/report.php', [
+            'id' => $cm->id,
+            'mode' => quiz_report_default_report($context)
-                'id' => $cm->id, 'mode' => quiz_report_default_report($context)]);
+        ]);
         return html_writer::link($url, $summary);
+    }
      * @param stdClass $quiz the quiz settings.
      * @param cm_info|stdClass $cm the cm object.
      * @param int $currentgroup currently selected group, if there is one.
      * @return string HTML fragment for the link.
      */
-    public function quiz_override_summary_links(stdClass $quiz, cm_info|stdClass $cm, $currentgroup = 0): string {
+    public function quiz_override_summary_links(stdClass $quiz, cm_info|stdClass $cm, $currentgroup = 0): string
+    {
         $baseurl = new moodle_url('/mod/quiz/overrides.php', ['cmid' => $cm->id]);
         $counts = quiz_override_summary($quiz, $cm, $currentgroup);
+        $links = [];
-        $links = [];
+        if ($counts['group']) {
-        if ($counts['group']) {
+            $links[] = html_writer::link(
+                new moodle_url($baseurl, ['mode' => 'group']),
-            $links[] = html_writer::link(new moodle_url($baseurl, ['mode' => 'group']),
+                get_string('overridessummarygroup', 'quiz', $counts['group'])
-                    get_string('overridessummarygroup', 'quiz', $counts['group']));
+            );
+        }
-        }
+        if ($counts['user']) {
-        if ($counts['user']) {
+            $links[] = html_writer::link(
+                new moodle_url($baseurl, ['mode' => 'user']),
-            $links[] = html_writer::link(new moodle_url($baseurl, ['mode' => 'user']),
+                get_string('overridessummaryuser', 'quiz', $counts['user'])
-                    get_string('overridessummaryuser', 'quiz', $counts['user']));
+            );
+        }
      * @param \core\chart_base $chart The chart.
      * @param string $title The title to display above the graph.
      * @param array $attrs extra container html attributes.
      * @return string HTML of the graph.
      */
-    public function chart(\core\chart_base $chart, $title, $attrs = []) {
+    public function chart(\core\chart_base $chart, $title, $attrs = [])
+    {
-        return $this->heading($title, 3) . html_writer::tag('div',
+        return $this->heading($title, 3) . html_writer::tag(
+            'div',
+            $this->render($chart),
-                        $this->render($chart), array_merge(['class' => 'graph'], $attrs));
+            array_merge(['class' => 'graph'], $attrs)
+        );
+    }
     /**
      * Output a graph, or a message saying that GD is required.
+     *
      * @param moodle_url $url the URL of the graph.
      * @param string $title the title to display above the graph.
      * @return string HTML of the graph.
      */
+    public function graph(moodle_url $url, $title)
     public function graph(moodle_url $url, $title) {
         $graph = html_writer::empty_tag('img', ['src' => $url, 'alt' => $title]);
         return $this->heading($title, 3) . html_writer::tag('div', $graph, ['class' => 'graph']);
+    }
     /**
      * Output the connection warning messages, which are initially hidden, and
+     * only revealed by JavaScript if necessary.
-     * only revealed by JavaScript if necessary.
+     */
-     */
+    public function connection_warning()
     public function connection_warning() {
         $options = ['filter' => false, 'newlines' => false];
+        $warning = format_text(get_string('connectionerror', 'quiz'), FORMAT_MARKDOWN, $options);
+        $ok = format_text(get_string('connectionok', 'quiz'), FORMAT_MARKDOWN, $options);
-        $warning = format_text(get_string('connectionerror', 'quiz'), FORMAT_MARKDOWN, $options);
+        return html_writer::tag(
+            'div',
-        $ok = format_text(get_string('connectionok', 'quiz'), FORMAT_MARKDOWN, $options);
+            $warning,
-        return html_writer::tag('div', $warning,
+            ['id' => 'connection-error', 'style' => 'display: none;', 'role' => 'alert']
-                        ['id' => 'connection-error', 'style' => 'display: none;', 'role' => 'alert']) .
+        ) .
-                html_writer::tag('div', $ok, ['id' => 'connection-ok', 'style' => 'display: none;', 'role' => 'alert']);
+            html_writer::tag('div', $ok, ['id' => 'connection-ok', 'style' => 'display: none;', 'role' => 'alert']);
+    }
     /**
      * Deprecated version of render_links_to_other_attempts.
+     *
      * @param links_to_other_attempts $links
      * @return string HTML fragment.
+     * @deprecated since Moodle 4.2. Please use render_links_to_other_attempts instead.
-     * @deprecated since Moodle 4.2. Please use render_links_to_other_attempts instead.
+     * @todo MDL-76612 Final deprecation in Moodle 4.6
-     * @todo MDL-76612 Final deprecation in Moodle 4.6
+     */
-     */
+    protected function render_mod_quiz_links_to_other_attempts(links_to_other_attempts $links)
     protected function render_mod_quiz_links_to_other_attempts(links_to_other_attempts $links) {
      * @param navigation_question_button $button
      * @return string HTML fragment.
      * @deprecated since Moodle 4.2. Please use render_links_to_other_attempts instead.
      * @todo MDL-76612 Final deprecation in Moodle 4.6
      */
-    protected function render_quiz_nav_question_button(navigation_question_button $button) {
+    protected function render_quiz_nav_question_button(navigation_question_button $button)
+    {
         return $this->render_navigation_question_button($button);
+    }
     /**
      * @param navigation_section_heading $heading the heading.
      * @return string HTML fragment.
      * @deprecated since Moodle 4.2. Please use render_links_to_other_attempts instead.
      * @todo MDL-76612 Final deprecation in Moodle 4.6
      */
-    protected function render_quiz_nav_section_heading(navigation_section_heading $heading) {
+    protected function render_quiz_nav_section_heading(navigation_section_heading $heading)
+    {
         return $this->render_navigation_section_heading($heading);
+    }
+}

Proyectos de Subversion Moodle

(root)/mod/quiz/classes/output/renderer.php – Rev 11 → 704