[wp-trac] [WordPress Trac] #54874: [New User Notification] Allow conditional mail supression

WordPress Trac noreply at wordpress.org
Fri Jan 21 12:30:32 UTC 2022


#54874: [New User Notification] Allow conditional mail supression
-------------------------------------------------+-------------------------
 Reporter:  janthiel                             |       Owner:  (none)
     Type:  feature request                      |      Status:  new
 Priority:  normal                               |   Milestone:  6.0
Component:  Users                                |     Version:
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch needs-unit-tests needs-    |     Focuses:
  dev-note 2nd-opinion                           |
-------------------------------------------------+-------------------------
Changes (by costdev):

 * keywords:  has-patch => has-patch needs-unit-tests needs-dev-note 2nd-
     opinion
 * type:  enhancement => feature request
 * severity:  minor => normal
 * milestone:  Awaiting Review => 6.0


Comment:

 Looks like a candidate for a similar filter to
 `send_retrieve_password_email`, one of the filters introduced as part of
 #54690.

 - The admin filter could go on the line just after the `'user' !==
 $notify` condition.

 - The user filter could go just after `$key` is defined and validated.

 I believe these locations would allow for the earliest short-circuit.

 Pinging @SergeyBiryukov @johnbillion @audrasjb for `2nd-opinion`.

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


More information about the wp-trac mailing list