Posted by tbenice on November 13, 2010 at 7:10pm
4 followers
| Project: | Messaging |
| Version: | 6.x-4.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | critical |
| Assigned: | Unassigned |
| Status: | needs review |
Issue Summary
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.
Comments
#1
What'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.
#2
this is a truly annoying bug. i'm changing it to urgent. not sure how to address it.
#3
See 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.
#4
Here it is a normal patch as well.
#5
Actually 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.
#6
Thanks - Attached is a rerolled patch against latest dev.
Tested and works for user email changes. Is anyone able to test the the SMS amends?