Opened 15 years ago
Closed 15 years ago
#1015 closed defect (bug) (fixed)
Opt out for new message notifications doesn't work (fix provided)
Reported by: | rebootnow | Owned by: | apeatling |
---|---|---|---|
Milestone: | 1.1 | Priority: | major |
Severity: | Version: | ||
Component: | Keywords: | has_patch | |
Cc: | rebootnow |
Description
Email notifications still get sent when a user opts out using the notification settings form. The bug is in the function that sends the mail, where there are two errors on the line that checks whether the user's 'notification_messages_new_message' setting == "no".
Around line 12 in file "bp-messages-notifications.php", in function "messages_notification_new_message", the line is currently:
'no' == get_userdata( $recipient_ids[$i], 'notification-messages-new-message' ) ) continue; |
That should be changed to read:
'no' == get_usermeta( $recipient_ids[$i], 'notification_messages_new_message' ) ) continue; |
The two changes:
- call function get_usermeta instead of get_userdata
- underscores instead of dashes in the metadata string
Change History (2)
Note: See
TracTickets for help on using
tickets.
(In [1892]) Fixes #1015 props rebootnow