[wp-trac] Re: [WordPress Trac] #9682: Move wp-login.php function definitions to pluggable.php

WordPress Trac wp-trac at lists.automattic.com
Thu Apr 30 14:59:19 GMT 2009


#9682: Move wp-login.php function definitions to pluggable.php
--------------------------+-------------------------------------------------
 Reporter:  misterbisson  |       Owner:           
     Type:  enhancement   |      Status:  new      
 Priority:  normal        |   Milestone:  2.8      
Component:  Users         |     Version:  2.7.1    
 Severity:  normal        |    Keywords:  has-patch
--------------------------+-------------------------------------------------

Comment(by Denis-de-Bernardy):

 I've updated my patch based on your first feedback. looking into your
 patch and the rest of the feedback right away.

 reset password is extended with:

 {{{
 do_action('password_reset', $user, $new_pass);
 }}}

 ... so you can send the new password to wherever you want it to go.

 both of the emails can be now cancelled by returning an empty message. in
 other words, you can disable the WP functionality in the sense that, WP
 will still do its stuff but it won't show to the user.

 this hook seems valid to generate a cell phone key on the fly:

 {{{
 do_action('retrieve_password_key', $user_login, $key);
 }}}

 just ignore the $key and generate yours.

 the last change is at the very end of the file: I've changed the default
 case in the switch statement, so as to make the entire thing pluggable.

 it needs a little testing on that last point. please give it a roll
 quickly, so we know if anything else is needed. I'd love to see this one
 get into 2.8 as well, as a matter of fact, because the last point makes
 the entire screen overriddable.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/9682#comment:7>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list