[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