Filter the subject within _wp_privacy_send_request_confirmation_notification

[garrett-eclipse]

Hello,

Looks like a filter was missed for the $subject found within _wp_privacy_send_request_confirmation_notification(). It should match the filter found in wp_send_user_request

Reference - ​https://github.com/WordPress/WordPress/blob/3ee58b55b14683af4c568bce1845c37408c88fbb/wp-includes/user.php#L3400

/**
	 * Filters the subject of the email sent when an account action is attempted.
	 *
	 * @since 4.9.6
	 *
	 * @param string $subject    The email subject.
	 * @param string $blogname   The name of the site.
	 * @param array  $email_data {
	 *     Data relating to the account action email.
	 *
	 *     @type WP_User_Request $request     User request object.
	 *     @type string          $email       The email address this is being sent to.
	 *     @type string          $description Description of the action being performed so the user knows what the email is for.
	 *     @type string          $confirm_url The link to click on to confirm the account action.
	 *     @type string          $sitename    The site name sending the mail.
	 *     @type string          $siteurl     The site URL sending the mail.
	 * }
	 */
	$subject = apply_filters( 'user_request_action_email_subject', $subject, $blogname, $email_data );

Cheers

[birgire]

Yes, it looks like it's missing.

44382.diff​ is a suggestion that adds the filter user_request_confirmed_email_subject for the subject of the user request confirmation email.

It also moves the $subject assigning down, to have it near the new filter, similar to wp_send_user_request(). It looks save to me. I will though most likely revert this to keep the patch as simple as possible. Just wanted to get some opinion on this ;-)

@garrett-eclipse What do you think about that?

PS: A related ticket is #44265, that adds an email subject filter within _wp_privacy_send_erasure_fulfillment_notification()

[garrett-eclipse]

Thanks @birgire I like it and did notice the difference in placement good catch there.

[desrosj]

This looks good to me. In 44382.2.diff​ I added documentation for $admin_email in the $email_data array for the new filter. This was also missing from the user_confirmed_action_email_content filter directly above.

One thing I noticed looking at this is an inconsistent use of blogname and sitename in these emails. I am going to open a new ticket for this though.

@garrett-eclipse for future reference, the Version field in Trac should be the earliest affected version of WordPress (4.9.6 in this instance), and you don't have to assign the privacy focus if the component is also Privacy. Thanks for all the great tickets!

[garrett-eclipse]

Thanks @desrosj appreciate the tip. And that patch looks pretty here, thanks

[azaozz]

In 43373:

Privacy: filter the email subject in _wp_privacy_send_request_confirmation_notification().

Props garrett-eclipse, birgire, desrosj.
Fixes #44382.

[azaozz]

Reopen for 4.9.7.

[ocean90]

4.9.7 has been released, moving to next milestone.

[SergeyBiryukov]

In 43451:

Privacy: Change @since entry for user_request_confirmed_email_subject filter added in [43373] to 4.9.8.

See #44382.

[SergeyBiryukov]

[43373] was accidentally backported with [43375] in [43450], I guess it's fine to leave that as is.

[SergeyBiryukov]

In 43452:

Privacy: Change @since entry for user_request_confirmed_email_subject filter added in [43373] to 4.9.8.

Merges [43451] to the 4.9 branch.
Fixes #44382.