[wp-trac] [WordPress Trac] #15682: Entering invalid DB information causes WP installer to silently die()
WordPress Trac
wp-trac at lists.automattic.com
Wed Dec 8 05:59:16 UTC 2010
#15682: Entering invalid DB information causes WP installer to silently die()
-----------------------------+----------------------------------------------
Reporter: itsananderson | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: 3.1
Component: Upgrade/Install | Version: 3.1
Severity: major | Keywords:
-----------------------------+----------------------------------------------
Comment(by nacin):
Replying to [comment:3 zippykid]:
> I think a more elegant solution may be to set show_errors(true) in the
constructor of wpdb, if WP_INSTALLING === true.
That sounds fine.
We don't need to check for === true. In fact, we can't, because sometimes
it won't be defined, and that'll trigger a notice.
We instead want to just check `defined( 'WP_INSTALLING' )`. Line 499,
where it says `if ( WP_DEBUG )` can change to `if ( WP_DEBUG || defined(
'WP_INSTALLING' ) )`.
Actually, we want that to be `defined( 'WP_SETUP_CONFIG' )` instead. We'll
call show_errors() ourselves during the install process.
I doubt this has been caught previously since we test typically with
WP_DEBUG on.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/15682#comment:4>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list