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

Quiz throws debugging message when sending notification messages

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.8.8, 3.0
    • Fix Version/s: 2.9.4, 3.0.2
    • Component/s: Quiz
    • Labels:
    • Testing Instructions:
      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.
    • Affected Branches:
      MOODLE_28_STABLE, MOODLE_30_STABLE
    • Fixed Branches:
      MOODLE_29_STABLE, MOODLE_30_STABLE
    • Pull from Repository:
    • Pull Master Branch:

      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

              • Assignee:
                amanda.doughty Amanda Doughty
                Reporter:
                amanda.doughty Amanda Doughty
                Peer reviewer:
                Tim Hunt
                Integrator:
                Dan Poltawski
                Tester:
                Andrew Nicols
                Participants:
                Component watchers:
                Tim Hunt, Andrew Nicols, Mathew May, Michael Hawkins, Shamim Rezaie, Simey Lameze
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  11/Jan/16