When a user changes their email address, the messaging destination address does not track. You have to change the send method in the account settings to something else, save, then change back.
IMHO the email addresses should track or messaging should always pull the account email address for any mail based messages.
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | messaging-update-destination-970936-6.patch | 3.72 KB | JoeMcGuire |
| #5 | messaging-970936-5.patch | 4.29 KB | univate |
| #4 | messaging-970936-4.patch | 5.94 KB | univate |
| #3 | 0001-970936-update-destination-address-for-mail-mimemail-.patch | 7.09 KB | univate |
Comments
Comment #1
tbenice commentedWhat's also not right is that the mail option test in messaging settings will send to the user's account address, but notifications messages are sent to the messaging_destinations address. There's a disconnect there someplace.
Comment #2
tbenice commentedthis is a truly annoying bug. i'm changing it to urgent. not sure how to address it.
Comment #3
univate commentedSee attached patch for a solution.
I have added a function to update addresses in the message_destination table for any message type and then in messaging_mail, messaging_mime_mail, messaging_phpmailer & messaging_sms added function to hook into user updating these details and update these.
I haven't used another other message types, so they may need to still be updated.
Comment #4
univate commentedHere it is a normal patch as well.
Comment #5
univate commentedActually seems the fix is a little simpler then I first thought and don't need to repeat the changes for mime_mail and phpmailer as they inherit the messaging_mail destination, so I can remove half the changes from the above patch.
Comment #6
JoeMcGuire commentedThanks - Attached is a rerolled patch against latest dev.
Tested and works for user email changes. Is anyone able to test the the SMS amends?
Comment #7
ryan_courtnage commented#5 and #6 are good patches that fix the issues described in the bug description. I did not test sms.