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

Messaging over email ignores messaging->email setting

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.0.2
    • Fix Version/s: STABLE backlog
    • Component/s: Messages
    • Labels:
    • Difficulty:
      Moderate
    • Affected Branches:
      MOODLE_20_STABLE

      Description

      While testing PULL-513, it was detected that messages sent over email are always being sent to user->email and not to the email setting available in the messaging configuration.

      Nothing else and nothing more

      To Reproduce:

      • set one user to have different email addresses in user->email and messaging->email and enable email for direct messages.
      • sent one message to him/her.
      • error: the message is sent to user->email, ignoring completely the messaging->email.

      Ciao

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            blepoxp Glenn Ansley added a comment -

            This commit should fix it. Let me know if you want it fixed differently and I can modify:
            https://github.com/glennansley/moodle/commit/2a5a47472f2268ba9efb3e360d98629e55da9bed

            Show
            blepoxp Glenn Ansley added a comment - This commit should fix it. Let me know if you want it fixed differently and I can modify: https://github.com/glennansley/moodle/commit/2a5a47472f2268ba9efb3e360d98629e55da9bed
            Hide
            andyjdavis Andrew Davis added a comment -

            Hi Glen. I think this issue was resolved as part of MDL-26185. Let me know if I'm wrong and I'll reopen this.

            Show
            andyjdavis Andrew Davis added a comment - Hi Glen. I think this issue was resolved as part of MDL-26185 . Let me know if I'm wrong and I'll reopen this.

              People

              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: