Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-40213

Clean theme layout "secure.php" missing settings.

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: 2.5, 2.6
    • Fix Version/s: 2.5.4, 2.6.1, 2.7, FRONTEND
    • Component/s: Themes
    • Labels:
    • Testing Instructions:
      Hide
      1. Select the 'Clean' theme.
      2. Turn on developer level debugging.
      3. Create a course.
      4. Add a quiz and in 'Extra restrictions on attempts' -> 'Browser security' set to 'Full screen pop-up with some JavaScript security'. See http://docs.moodle.org/26/en/Quiz_settings#Extra_restrictions_on_attempts for more information.
      5. Add one question.
      6. Go to the quiz and click on the 'Attempt quiz now'.
      7. Observe that the error occurs and that the course name is not displayed at the top.
      8. Apply the patch.
      9. Refresh the page and observe that the error no longer occurs and the course name is displayed at the top.
      Show
      Select the 'Clean' theme. Turn on developer level debugging. Create a course. Add a quiz and in 'Extra restrictions on attempts' -> 'Browser security' set to 'Full screen pop-up with some JavaScript security'. See http://docs.moodle.org/26/en/Quiz_settings#Extra_restrictions_on_attempts for more information. Add one question. Go to the quiz and click on the 'Attempt quiz now'. Observe that the error occurs and that the course name is not displayed at the top. Apply the patch. Refresh the page and observe that the error no longer occurs and the course name is displayed at the top.
    • Affected Branches:
      MOODLE_25_STABLE, MOODLE_26_STABLE
    • Fixed Branches:
      MOODLE_25_STABLE, MOODLE_26_STABLE, MOODLE_27_STABLE
    • Epic Link:
    • Pull from Repository:
    • Pull Master Branch:
      wip-MDL-40213_master
    • Pull Master Diff URL:

      Description

      I noticed a bug in the secure page layout for the clean theme. It is missing this code from the head:

      // Get the HTML for the settings bits.
      $html = theme_clean_get_html_for_settings($OUTPUT, $PAGE);

      Without this - it should get an undefined variable warning later in the page when it writes the page header.

      With developer level debugging on, the following errors are reported:

      Notice: Undefined variable: html in F:\moodledev\moodlegjb\theme\clean\layout\secure.php on line 51

      Notice: Trying to get property of non-object in F:\moodledev\moodlegjb\theme\clean\layout\secure.php on line 51

        Attachments

          Issue Links

          has been marked as being related by

          Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-31365 Safe Exam Browser window is not safe

          • Minor - Minor loss of function where workaround is possible
          • Closed

            Activity

              People

              • Votes:
                1 Vote for this issue
                Watchers:
                8 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  13/Jan/14