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

Add new quizz: notice is displayed

    Details

    • Type: Bug
    • Status: Closed
    • Priority: 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

          Attachments

            Activity

            Hide
            jerome 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
            jerome 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
            timhunt Tim Hunt added a comment -

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

            Show
            timhunt Tim Hunt added a comment - Jerome, did you visit the admin page after CVS updating?
            Hide
            jerome 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
            jerome 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
            timhunt Tim Hunt added a comment -

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

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

            Yes, thanks Tim for your review

            Show
            jerome 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:
                  Fix Release Date:
                  21/Oct/09