Moodle
  1. Moodle
  2. MDL-20421

Add new quizz: notice is displayed

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.9.5
    • Fix Version/s: 1.9.6
    • Component/s: Quiz
    • Labels:
      None
    • Affected Branches:
      MOODLE_19_STABLE
    • Fixed Branches:
      MOODLE_19_STABLE

      Description

      This notice is displayed: Notice: Undefined property: stdClass::$enablesafebrowserintegration in /Users/jerome/Documents/Projects/Moodle_19/moodle/mod/quiz/mod_form.php on line 200

        Gliffy Diagrams

          Activity

          Hide
          Jérôme Mouneyrac added a comment -

          Fix committed. Tested on 1.9, "Enable Safe Exam Browser integration" option still well displayed (enabled and disabled).

          Show
          Jérôme Mouneyrac added a comment - Fix committed. Tested on 1.9, "Enable Safe Exam Browser integration" option still well displayed (enabled and disabled).
          Hide
          Tim Hunt added a comment -

          Jerome, did you visit the admin page after CVS updating?

          Show
          Tim Hunt added a comment - Jerome, did you visit the admin page after CVS updating?
          Hide
          Jérôme Mouneyrac added a comment -

          Good question, actually I went to the admin pahe and I had a downgrade problem last week during an update. I guess it was at the same moment that your update was committed, so Moodle didn't go on the page to save the new variables. It could explain that it didn't work for me. If it's why you though about.

          Show
          Jérôme Mouneyrac added a comment - Good question, actually I went to the admin pahe and I had a downgrade problem last week during an update. I guess it was at the same moment that your update was committed, so Moodle didn't go on the page to save the new variables. It could explain that it didn't work for me. If it's why you though about.
          Hide
          Tim Hunt added a comment -

          Yes. $CFG->enablesafebrowserintegration is created on upgrade. However, no harm in making the code do !empty instead.

          Show
          Tim Hunt added a comment - Yes. $CFG->enablesafebrowserintegration is created on upgrade. However, no harm in making the code do !empty instead.
          Hide
          Jérôme Mouneyrac added a comment -

          Yes, thanks Tim for your review

          Show
          Jérôme Mouneyrac added a comment - Yes, thanks Tim for your review

            People

            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: