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, Jun Pataleta, 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