Opened 5 years ago

Closed 5 years ago

Last modified 5 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
Priority: high Milestone: 2.3.2
Component: Administration Version: 2.3.1
Severity: normal Keywords: install mysql has-patch
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 5 years ago.
install.diff​ (1.9 KB) - added by Potter_System 5 years ago.
Updated to detect if MySQL fails

Download all attachments as: .zip

Change History (10)

andy — 5 years ago

comment:1   andy — 5 years ago

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

Potter_System — 5 years ago

Updated to detect if MySQL fails

comment:2   ryan — 5 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

comment:3   ryan — 5 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

comment:4   lloydbudd — 5 years ago

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

comment:5   ryan — 5 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

comment:6   ryan — 5 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

comment:7   ryan — 5 years ago

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

comment:8   ryan — 5 years ago

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

Note: See TracTickets for help on using tickets.