WordPress.org

Make WordPress Core

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#5495 closed defect (bug) (fixed)

setup-config.php, install.php don't check for a valid MySQL connection

Reported by: Potter_System Owned by: andy
Milestone: 2.3.2 Priority: high
Severity: normal Version: 2.3.1
Component: Administration Keywords: install mysql has-patch
Focuses: Cc:

Description

Pointing to a non-existent MySQL DB and/or user on setup-config.php doesn't show even a warning alert, but creates the config file anyway.

Running install.php with the same broken config file allows to get a "Success" message, but going into a never-ending install loop

Attachments (2)

wpdb-error.diff (2.1 KB) - added by andy 11 years ago.
install.diff (1.9 KB) - added by Potter_System 11 years ago.
Updated to detect if MySQL fails

Download all attachments as: .zip

Change History (10)

@andy
11 years ago

#1 @andy
11 years ago

  • Keywords has-patch added
  • Owner changed from anonymous to andy
  • Status changed from new to assigned

@Potter_System
11 years ago

Updated to detect if MySQL fails

#2 @ryan
11 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

(In [6426]) Set error if there is a problem connecting to the DB. Props skeltoac and Potter_System. fixes #5495

#3 @ryan
11 years ago

(In [6444]) Set error if there is a problem connecting to the DB. Props skeltoac and Potter_System. fixes #5495 for 2.3

#4 @lloydbudd
11 years ago

  • Milestone changed from 2.4 to 2.3.2
  • Version changed from 2.4 to 2.3.1

#5 @ryan
11 years ago

(In [6458]) Return after bailing since bail doesn't always die now. Don't do queries if constructor didn't complete. see #5495

#6 @ryan
11 years ago

(In [6459]) Return after bailing since bail doesn't always die now. Don't do queries if constructor didn't complete. see #5495

#7 @ryan
11 years ago

(In [6460]) Forgot this. see #5495

#8 @ryan
11 years ago

(In [6461]) Forgot this. see #5495

Note: See TracTickets for help on using tickets.