[wp-trac] [WordPress Trac] #43869: Do not initialize current user too early in `get_user_locale()`

WordPress Trac noreply at wordpress.org
Sun Sep 23 15:05:31 UTC 2018


#43869: Do not initialize current user too early in `get_user_locale()`
-------------------------------------------------+-------------------------
 Reporter:  flixos90                             |       Owner:
                                                 |  SergeyBiryukov
     Type:  defect (bug)                         |      Status:  reviewing
 Priority:  normal                               |   Milestone:  4.9.9
Component:  Bootstrap/Load                       |     Version:
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch has-unit-tests 2nd-        |     Focuses:
  opinion                                        |
-------------------------------------------------+-------------------------

Comment (by tazotodua):

 as that last topic is opened by me, i will add my observation here:

 the issue still exists, and i've wrote in that topic, how to reproduce.
 After further check, i've found that , if `wp-includes/user.php`, line
 2605

 in front of :

 `wp_set_current_user( 0 );`

 if we add:

 `if(did_action('plugins_loaded')) `

 the problem is fixed.  Will that be accepted as patch or it has negative
 effects?  How to overcome te problem, as many plugins call those functions
 before `plugins_loaded` hook, and our dashboard pages are broken.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/43869#comment:5>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list