[wp-trac] [WordPress Trac] #10938: AtomPub redefines pluggable wp_set_current_user()

WordPress Trac wp-trac at lists.automattic.com
Fri Oct 9 22:38:47 UTC 2009


#10938: AtomPub redefines pluggable wp_set_current_user()
--------------------------+-------------------------------------------------
 Reporter:  mdawaffe      |       Owner:  josephscott 
     Type:  defect (bug)  |      Status:  new         
 Priority:  normal        |   Milestone:  2.9         
Component:  AtomPub       |     Version:  2.8.4       
 Severity:  normal        |    Keywords:  dev-feedback
--------------------------+-------------------------------------------------
 wp-app.php redefines the pluggable wp_set_current_user().

 This means that plugins which redefine wp_set_current_user() don't work
 during a APP call.

 Also the redefined function is identical to the core function except that
 it calls neither setup_userdata() nor do_action( 'set_current_user' ).

 setup_userdata() defines some globals that plugins might rely on.  And
 defining globals is basically all it does, so it's not very expensive.

 The do_action( 'set_current_user' ) is important for plugins that need to
 sync user data, change caps, etc.

 I have no insight into why wp-app.php needs to redefine the function.

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


More information about the wp-trac mailing list