Skip to content

Commit

Permalink
DbResourcePage: Don't fail validation if connecting to pgsql wasn't s…
Browse files Browse the repository at this point in the history
…uccessfull
  • Loading branch information
yhabteab committed Oct 26, 2023
1 parent bcdad8c commit ae9575e
Showing 1 changed file with 8 additions and 8 deletions.
16 changes: 8 additions & 8 deletions modules/setup/application/forms/DbResourcePage.php
Original file line number Diff line number Diff line change
Expand Up @@ -141,14 +141,14 @@ protected function validateConfiguration()

if ($this->getValue('db') === 'pgsql') {
if ($connectionError !== null) {
$this->warning(sprintf(
$this->translate('Unable to check the server\'s version. This is usually not a critical error'
. ' as there is probably only access to the database permitted which does not exist yet. If you are'
. ' absolutely sure you are running PostgreSQL in a version equal to or newer than 9.1,'
. ' you can skip the validation and safely proceed to the next step. The error was: %s'),
$connectionError->getMessage()
));
$state = false;
// $this->warning(sprintf(
// $this->translate('Unable to check the server\'s version. This is usually not a critical error'
// . ' as there is probably only access to the database permitted which does not exist yet. If you are'
// . ' absolutely sure you are running PostgreSQL in a version equal to or newer than 9.1,'
// . ' you can skip the validation and safely proceed to the next step. The error was: %s'),
// $connectionError->getMessage()
// ));
// $state = false;
} else {
$version = $db->getServerVersion();
if (version_compare($version, '9.1', '<')) {
Expand Down

0 comments on commit ae9575e

Please sign in to comment.