setup-config.php should check for existence of db.php

[ComputerGuru]

wp-admin/setup-config.php should check for wp-content/db.php and load that if it exists instead of blindly loading wp-db.php. Searching for db.php first is the correct behavior as can be seen in wp-settings.php

[westi]

Adding to my review and checkin todo list

[westi]

Alternative patch - move common code to a function.

[westi]

I've added a new patch which moves the common code to a new function instead of duplicating it.

What does everyone think?

[ComputerGuru]

Looks good to me :-)

I was originally planning on adding a check in the actual wp-db.php at the very top, which would load the alternate db.php if it exists; but I felt it was a bit ugly of a hack.. A function is a nice compromise.

[westi]

(In [6196]) Move database class loading to a shared function to ensure all of WordPress is wp-content/wp-db.php aware. Fixes #5128 props ComputerGuru.

[westi]

Arg.. the change went into the wrong place.

[westi]

(In [6198]) Move database class loading to a shared function to ensure all of WordPress is wp-content/wp-db.php aware. Fixes #5128 props ComputerGuru.