[wp-trac] [WordPress Trac] #12184: Move and add filters to wp_new_user_notification

WordPress Trac wp-trac at lists.automattic.com
Tue Feb 9 15:57:29 UTC 2010


#12184: Move and add filters to wp_new_user_notification
-------------------------+--------------------------------------------------
 Reporter:  jfarthing84  |       Owner:              
     Type:  enhancement  |      Status:  new         
 Priority:  normal       |   Milestone:  Unassigned  
Component:  General      |     Version:  2.9.1       
 Severity:  normal       |    Keywords:  dev-feedback
-------------------------+--------------------------------------------------
 I think that we should add actions and filters to the functions
 wp_new_user_notification() and while we're at it,
 wp_password_change_notification() instead of having them pluggable.php.

 The reason I feel this way is because NUMEROUS plugins operate by
 overtaking the function wp_new_user_notification to add user settings and
 such when a user registers. However, as we know, only one of them can
 actually define this function, and that will naturally be the first plugin
 in alphabetical order.

 So, why not make the function static by removing it from pluggable.php and
 adding it to, maybe wp-login.php, with all of the other login-related
 functions? Then we can add hooks and filters into the function in order to
 modify data and trigger events.

 I have attached modified a diff with the functions removed from
 pluggable.php and added them to wp-login.php with the appropriate (I
 believe) actions and filters.

 Let's get this change done so that many useful registration plugins that
 utilize this function can be used in conjunction with each other (once
 their respective authors update them, of course).

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/12184>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list