[wp-trac] [WordPress Trac] #44314: `user_confirmed_action_email_content` filter run on two different strings

WordPress Trac noreply at wordpress.org
Wed Oct 2 05:25:15 UTC 2019


#44314: `user_confirmed_action_email_content` filter run on two different strings
-------------------------------------------------+-------------------------
 Reporter:  desrosj                              |       Owner:  garrett-
                                                 |  eclipse
     Type:  defect (bug)                         |      Status:  reviewing
 Priority:  normal                               |   Milestone:  5.4
Component:  Privacy                              |     Version:  4.9.6
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch has-unit-tests needs-      |     Focuses:
  refresh needs-dev-note                         |
-------------------------------------------------+-------------------------
Changes (by garrett-eclipse):

 * keywords:  has-patch has-unit-tests => has-patch has-unit-tests needs-
     refresh needs-dev-note
 * owner:  SergeyBiryukov => garrett-eclipse
 * milestone:  5.3 => 5.4


Comment:

 Thanks @SergeyBiryukov I appreciate the thorough feedback and agree with
 your proposal.

 While I would love to resolve the duplication sooner than later I would be
 more comfortable landing a change involving deprecations early and not in
 beta2. I've moved this to 5.4 early and taken ownership of the refresh and
 accompaniment of a dev-note.

 I also wanted to flag a related #46303 which was committed in
 [https://core.trac.wordpress.org/changeset/46265 ChangeSet#46265] that
 introduced `wp_privacy_personal_data_email_to` and
 `wp_privacy_personal_data_email_subject` to the
 `wp_privacy_send_personal_data_export_email`. *These will need to be taken
 into account in the proposal here.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/44314#comment:29>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list