[wp-trac] [WordPress Trac] #11881: setup-config.php step 2 broken

WordPress Trac wp-trac at lists.automattic.com
Fri Jan 15 21:19:37 UTC 2010


#11881: setup-config.php step 2 broken
-----------------------------+----------------------------------------------
 Reporter:  sivel            |       Owner:           
     Type:  defect (bug)     |      Status:  new      
 Priority:  normal           |   Milestone:  3.0      
Component:  Upgrade/Install  |     Version:  3.0      
 Severity:  major            |    Keywords:  has-patch
-----------------------------+----------------------------------------------

Comment(by nacin):

 I have questions about two variables losing global scope and whether we
 need to globalize them:

  - {{{$mu_plugins}}}, an array of PHP files from the must use plugins
 folder. We unset the corresponding array holding plain WP plugins to
 include. It is used nowhere in core and I highly doubt it needs to remain
 a global.

  - {{{$prefix}}}, as in {{{$prefix = $wpdb->set_prefix($table_prefix);}}}
 -- which is then checked for is_wp_error().

 I think the answer to both is no (just being cautious), in which case
 [attachment:11881.5.diff] is good on my end.

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


More information about the wp-trac mailing list