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

Quiz throws debugging message when sending notification messages

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 2.8.8, 3.0
    • 2.9.4, 3.0.2
    • Quiz
    • MOODLE_28_STABLE, MOODLE_30_STABLE
    • MOODLE_29_STABLE, MOODLE_30_STABLE
    • Hide
      1. You need a course with a student and a teacher.
      2. The teacher account needs to be set up so you can tell if a message was sent to it.
      3. Create a quiz.
      4. Edit or override the teacher role so that it has the mod/quiz:emailnotifysubmission capability.
      5. Attempt the quiz as the student.
      6. Verify that when the quiz is submitted, there are no notices.
      7. Verify that the teacher gets an email saying that the quiz was submitted.
      Show
      You need a course with a student and a teacher. The teacher account needs to be set up so you can tell if a message was sent to it. Create a quiz. Edit or override the teacher role so that it has the mod/quiz:emailnotifysubmission capability. Attempt the quiz as the student. Verify that when the quiz is submitted, there are no notices. Verify that the teacher gets an email saying that the quiz was submitted.

    Description

      This is a very minor issue as the code in messagelib.php adds the missing fields after the debugging message:

      Necessary properties missing in userto object, fetching full record
      line 95 of /lib/messagelib.php: call to debugging()
      line 1453 of /mod/quiz/locallib.php: call to message_send()
      line 1534 of /mod/quiz/locallib.php: call to quiz_send_notification()
      line 1649 of /mod/quiz/locallib.php: call to quiz_send_notification_messages()
      line ? of unknownfile: call to quiz_attempt_submitted_handler()
      line 155 of /lib/classes/event/manager.php: call to call_user_func()
      line 89 of /lib/classes/event/manager.php: call to core\event\manager::process_buffers()
      line 2411 of /lib/dml/moodle_database.php: call to core\event\manager::database_transaction_commited()
      line 92 of /lib/dml/moodle_transaction.php: call to moodle_database->commit_delegated_transaction()
      line 184 of /mod/quiz/processattempt.php: call to moodle_transaction->allow_commit()

      FIX
      mod/quiz/locallib.php #1486

      — $notifyfields = 'u.id, u.username, u.idnumber, u.email, u.emailstop, u.lang, u.timezone, u.mailformat, u.maildisplay, ';
      +++ $notifyfields = 'u.id, u.username, u.idnumber, u.email, u.emailstop, u.lang, u.timezone, u.mailformat, u.maildisplay, u.auth, u.suspended, u.deleted,';

      Attachments

        Issue Links

          Activity

            People

              amanda.doughty Amanda Doughty
              amanda.doughty Amanda Doughty
              Tim Hunt Tim Hunt
              Dan Poltawski Dan Poltawski
              Andrew Lyons Andrew Lyons
              Tim Hunt, Ilya Tregubov, Kevin Percy, Mathew May, Mihail Geshoski, Shamim Rezaie
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                11/Jan/16