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

Email digest may send out messages that have already been read

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 3.7
    • Fix Version/s: 3.7
    • Component/s: Messages
    • Labels:

      Description

      The issue MDL-64017 added a message digest which is the cause of this issue.

      The scenario.

      1. The messaging email processor is enabled.
      2. User A sends user B a message.
      3. This gets added to the message_email_messages table.
      4. User B then logs in and reads the message.
      5. The task message_email\task\send_email_task is run and sends the message (that has already been read) to user B.

      We can avoid this by removing the message from the message_email_messages table once it has been read.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              markn Mark Nelson
              Reporter:
              markn Mark Nelson
              Peer reviewer:
              Carlos Escobedo
              Integrator:
              Jake Dallimore
              Tester:
              CiBoT
              Participants:
              Component watchers:
              Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Sara Arjona (@sarjona)
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                20/May/19

                  Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 5 hours, 5 minutes
                  5h 5m