Merge branch 'check-connection-on-install' into 'master'

Check DB connection before any possible use

This is to avoid an exception when enabling 'pgsql' backend

See merge request !121
This commit is contained in:
mmn 2016-06-17 16:30:25 -04:00
commit 48efdc3593
1 changed files with 5 additions and 5 deletions

View File

@ -285,6 +285,11 @@ abstract class Installer
$this->updateStatus("Checking database...");
$conn = $this->connectDatabase($dsn);
if (!$conn instanceof DB_common) {
// Is not the right instance
throw new Exception('Cannot connect to database: ' . $conn->getMessage());
}
// ensure database encoding is UTF8
if ($this->dbtype == 'mysql') {
// @fixme utf8m4 support for mysql 5.5?
@ -299,11 +304,6 @@ abstract class Installer
}
}
if (!$conn instanceof DB_common) {
// Is not the right instance
throw new Exception('Cannot connect to database: ' . $conn->getMessage());
}
$res = $this->updateStatus("Creating database tables...");
if (!$this->createCoreTables($conn)) {
$this->updateStatus("Error creating tables.", true);