Skip to:

Opened 7 weeks ago

Closed 6 weeks ago

#9171 closed defect (bug) (fixed)

bp_user_has_opted_out() is misused in bp_email_unsubscribe_handler()

Reported by: dcavins's profile dcavins Owned by: djpaul's profile DJPaul
Milestone: 14.0.0 Priority: normal
Severity: normal Version: 12.5.0
Component: Emails Keywords:


In bp_email_unsubscribe_handler(), bp_user_has_opted_out() is called without passing an email, which must be supplied.

Attachments (1)

9171.1.diff (643 bytes) - added by dcavins 7 weeks ago.
Pass email to bp_user_has_opted_out().

Download all attachments as: .zip

Change History (2)

7 weeks ago

Pass email to bp_user_has_opted_out().

#1 @dcavins
6 weeks ago

  • Resolution set to fixed
  • Status changed from new to closed

In 13908:

Make sure to pass an email address to bp_user_has_opted_out().

In bp_email_unsubscribe_handler() we missed passing the incoming email to bp_user_has_opted_out().
It's hard to figure out whether a user has opted out if you don't check the user's email address.

Props emeralive.

Fixes #9171.

Note: See TracTickets for help on using tickets.