Moodle

Add new quizz: notice is displayed

Details

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

Activity

Hide
Jerome Mouneyrac added a comment -

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

Show
Jerome 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
Jerome 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 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
Jerome Mouneyrac added a comment -

Yes, thanks Tim for your review

Show
Jerome Mouneyrac added a comment - Yes, thanks Tim for your review

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: