Follow-up to #2562847: [meta] Messaging

Filling in incorrect database credentials presents raw PDO error messages to the user (or no messages at all) - it would be ideal if we could re-use the same error handling as the installer, which prints much user-friendlier messages.

CommentFileSizeAuthor
#4 migrate_upgrade-improve-handling-of-database-errors-2564049-4.patch932 bytesaleksip

Comments

mikeryan created an issue. See original summary.

mikeryan’s picture

mikeryan
he/him
Primary language English
Location Pittsfield, MA, USA
commented
Issue tags: +Barcelona2015, +Novice

Tagging for Barcelona sprint.

ultimike’s picture

ultimike
Primary language English
Location Florida, USA
commented
Issue summary: View changes
aleksip’s picture

aleksip
Primary language Finnish
Location Finland
commented
StatusFileSize
new migrate_upgrade-improve-handling-of-database-errors-2564049-4.patch932 bytes

Should now show an error in similar fashion to install.core.inc

  • mikeryan committed 5c79bb2 on 8.x-1.x authored by aleksip 
    Issue #2564049 by aleksip: Improve handling of database errors
mikeryan’s picture

mikeryan
he/him
Primary language English
Location Pittsfield, MA, USA
commented
Status: Active » Fixed
Issue tags: -Barcelona2015, -Novice

Forgot to provide my custom MySQL port number:

Resolve the issue below to continue the upgrade.

    SQLSTATE[HY000] [2003] Can't connect to MySQL server on '127.0.0.1' (61)

Fixed port number, bad database: 

Resolve the issue below to continue the upgrade.

    SQLSTATE[42000] [1049] Unknown database 'asdf'

Bad username:

Resolve the issue below to continue the upgrade.

    SQLSTATE[42000] [1044] Access denied for user ''@'localhost' to database 'mryan1_dev'

Committed, thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.