From: Mark A. Hershberger Date: Fri, 4 Feb 2011 00:21:59 +0000 (+0000) Subject: re r81495 — you know, I've completely forgotten why I wanted to return a status objec... X-Git-Tag: 1.31.0-rc.0~32215 X-Git-Url: http://git.cyclocoop.org/%22.%24match%5B1%5D.%22?a=commitdiff_plain;h=ee3e5163c4165ee8f64572bd58f1f49224df5034;p=lhc%2Fweb%2Fwiklou.git re r81495 — you know, I've completely forgotten why I wanted to return a status object. Fix that and not the busted status message. --- diff --git a/includes/installer/PostgresInstaller.php b/includes/installer/PostgresInstaller.php index 311e7d23dc..9efd0f4e10 100644 --- a/includes/installer/PostgresInstaller.php +++ b/includes/installer/PostgresInstaller.php @@ -138,10 +138,10 @@ class PostgresInstaller extends DatabaseInstaller { ); if( !$rights || ( $rights != 1 && $rights != 3 ) ) { - $status = Status::newFatal("can't create"); + return false; } - return $status; + return true; } public function getSettingsForm() { @@ -164,7 +164,7 @@ class PostgresInstaller extends DatabaseInstaller { // Validate the create checkbox $create = true; $canCreate = $this->canCreateAccounts(); - if ( !$canCreate->isOK() ) { + if ( $canCreate ) { $this->setVar( '_CreateDBAccount', false ); $create = false; } else {