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

[WordPress Trac]

#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):

 That's the default value of AUTH_KEY, SECURE_AUTH_KEY, et al., in wp-
 config-sample.php, not what the user changes it to. See how it is used in
 wp_salt() in [http://core.trac.wordpress.org/browser/trunk/wp-
 includes/pluggable.php&rev=12725#L1257 pluggable.php].

 Also, changing it to default to SECRET_KEY would require not only logic
 changes in wp_salt() but it would break any pluggable implementations of
 wp_salt() as well.

 If anyone has a PHP 4 environment, what especially needs testing is I
 think you no longer assign the return value of a new object by reference
 in non global scope, because the global keyword would create a reference
 to the global variable. [Relevant docs,
 http://www.php.net/manual/en/language.variables.scope.php#language.variables.scope.references
 Relevant PHP docs].

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