Ticket #5495 (closed defect: fixed)

Opened 1 year ago

Last modified 1 year ago

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

Reported by: Potter_System Assigned to: 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

wpdb-error.diff (2.1 kB) - added by andy on 12/20/07 00:22:06.
install.diff (1.9 kB) - added by Potter_System on 12/20/07 00:58:38.
Updated to detect if MySQL fails

Change History

12/20/07 00:22:06 changed by andy

  • attachment wpdb-error.diff added.

12/20/07 00:22:51 changed by andy

  • keywords changed from install mysql to install mysql has-patch.
  • owner changed from anonymous to andy.
  • status changed from new to assigned.

12/20/07 00:58:38 changed by Potter_System

  • attachment install.diff added.

Updated to detect if MySQL fails

12/20/07 02:10:09 changed by ryan

  • status changed from assigned to closed.
  • resolution set to fixed.

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

12/21/07 01:35:27 changed by ryan

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

12/21/07 03:18:12 changed by lloydbudd

  • version changed from 2.4 to 2.3.1.
  • milestone changed from 2.4 to 2.3.2.

12/21/07 19:28:21 changed by ryan

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

12/21/07 19:30:08 changed by ryan

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

12/21/07 19:54:46 changed by ryan

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

12/21/07 19:55:21 changed by ryan

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