[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:47:05 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 misterbisson):

 My latest diff still moves the functions to pluggable.php, but the real
 point is the following addition just above the main switch( $action ):

 {{{
 global $wp_filter;
 if( isset( $wp_filter[ 'wp_login_'. $action ] )):
         do_action( 'wp_login_'. $action );
 else:
 switch( $action ) {...
 }}}

 This allows plugins to hook into that and, perhaps, override the existing
 functionality. It has to check if there's an action attached to prevent
 leaving the user with a blank screen because somebody added an invalid
 action to the get vars.

 That solution could eliminate my need to move the functions, as I could
 replace them with the action switch.

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


More information about the wp-trac mailing list