Make WordPress Core

Opened 6 years ago

Closed 6 years ago

Last modified 4 years ago

#44380 closed defect (bug) (duplicate)

The two instances of user_confirmed_action_email_content should be unique

Reported by: garrett-eclipse's profile garrett-eclipse Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: Privacy Keywords:
Focuses: Cc:

Description

Hello,

I noticed that the user_confirmed_action_email_content filter is used in two places will fully different contents expected and feel they should be unique to avoid accidentally affected the other instance unintentionally.

They can be found here;
https://github.com/WordPress/WordPress/blob/3ee58b55b14683af4c568bce1845c37408c88fbb/wp-includes/user.php#L3054
https://github.com/WordPress/WordPress/blob/3ee58b55b14683af4c568bce1845c37408c88fbb/wp-includes/user.php#L3172

The biggest difference I noticed was the $email_data and placeholders are all different between each instance.

Cheers

Change History (4)

#1 @desrosj
6 years ago

  • Milestone Awaiting Review deleted
  • Resolution set to duplicate
  • Status changed from new to closed
  • Version trunk deleted

Good find, @garrett-eclipse. This is already being tracked in #44314. Patch testing is appreciated!

This ticket was mentioned in Slack in #core-privacy by desrosj. View the logs.


6 years ago

#3 @garrett-eclipse
6 years ago

Thanks @desrosj I'll check it out

#4 @garrett-eclipse
4 years ago

  • Focuses privacy removed

Dropping privacy focus as it's already in the Privacy component.

Note: See TracTickets for help on using tickets.