From cfb993f5b96f40287471acf521ed2a2ce57a440c Mon Sep 17 00:00:00 2001 From: Greg Sabino Mullane Date: Mon, 21 Mar 2011 21:36:21 +0000 Subject: [PATCH] $safedb needs to be set *outside* the if/else block, not just inside the if Fixes bug 28170 --- includes/installer/PostgresInstaller.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/includes/installer/PostgresInstaller.php b/includes/installer/PostgresInstaller.php index bf6d8d4086..1a3a420f7d 100644 --- a/includes/installer/PostgresInstaller.php +++ b/includes/installer/PostgresInstaller.php @@ -224,8 +224,8 @@ class PostgresInstaller extends DatabaseInstaller { $SQL = "SELECT 1 FROM pg_catalog.pg_database WHERE datname = " . $conn->addQuotes( $dbName ); $rows = $conn->numRows( $conn->query( $SQL ) ); + $safedb = $conn->addIdentifierQuotes( $dbName ); if( !$rows ) { - $safedb = $conn->addIdentifierQuotes( $dbName ); $conn->query( "CREATE DATABASE $safedb OWNER $safeuser", __METHOD__ ); } else { $conn->query( "ALTER DATABASE $safedb OWNER TO $safeuser", __METHOD__ ); -- 2.20.1