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
    • Rank:
      26228

      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

        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: