Use DatabaseConfigurationProvider to get DB info

Summary:
remove accessing the db config info directly. Use
DatabaseConfigurationProvider instead. Also fixed a minor issue where
different number of newlines are output in PhabricatorSetup.php's output.

Test Plan:
executed upgrade_schema.php; executed PhabricatorSetup.php by
setting 'phabricator.setup' to true.

Reviewed By: epriestley
Reviewers: epriestley
CC: aran, jungejason, epriestley
Differential Revision: 443

Author: Jason Ge

scripts/sql/upgrade_schema.php

@@ -59,15 +59,16 @@
 // Use always the version from the commandline if it is defined
 $next_version = isset($options['v']) ? (int)$options['v'] : null;
 
-// TODO: Get this stuff from DatabaseConfigurationProvider?
+$conf = DatabaseConfigurationProvider::getConfiguration();
+
 if ($options['u']) {
   $conn_user = $options['u'];
   $conn_pass = $options['p'];
 } else {
-  $conn_user = PhabricatorEnv::getEnvConfig('mysql.user');
-  $conn_pass = PhabricatorEnv::getEnvConfig('mysql.pass');
+  $conn_user = $conf->getUser();
+  $conn_pass = $conf->getPassword();
 }
-$conn_host = PhabricatorEnv::getEnvConfig('mysql.host');
+$conn_host = $conf->getHost();
 
 // Split out port information, since the command-line client requires a
 // separate flag for the port.

src/applications/base/storage/configuration/DatabaseConfigurationProvider.php

@@ -48,4 +48,15 @@ final protected function getDao() {
   final protected function getMode() {
     return $this->mode;
   }
+
+  public static function getConfiguration() {
+    // Get DB info. Note that we are using a dummy PhabricatorUser object in
+    // creating the DatabaseConfigurationProvider, which is not used at all.
+    $conf_provider = PhabricatorEnv::getEnvConfig(
+      'mysql.configuration_provider', 'DatabaseConfigurationProvider');
+    PhutilSymbolLoader::loadClass($conf_provider);
+    $conf = newv($conf_provider, array(new PhabricatorUser(), 'r'));
+    return $conf;
+  }
+
 }

src/applications/base/storage/configuration/__init__.php

@@ -6,7 +6,11 @@
 
 
 
+phutil_require_module('phabricator', 'applications/people/storage/user');
 phutil_require_module('phabricator', 'infrastructure/env');
 
+phutil_require_module('phutil', 'symbols');
+phutil_require_module('phutil', 'utils');
+
 
 phutil_require_source('DatabaseConfigurationProvider.php');

src/infrastructure/setup/PhabricatorSetup.php

@@ -68,7 +68,7 @@ public static function runSetup() {
     } else {
       if (trim($stdout) == 'YES') {
         self::write(" okay  pcntl is available from the command line.\n");
-        self::write("[OKAY] All extensions OKAY\n\n");
+        self::write("[OKAY] All extensions OKAY\n");
       } else {
         self::write(" warn  pcntl is not available!\n");
         self::write("[WARN] *** WARNING *** pcntl extension not available. ".
@@ -120,7 +120,7 @@ public static function runSetup() {
         }
       }
     }
-    self::write("[OKAY] All submodules OKAY.");
+    self::write("[OKAY] All submodules OKAY.\n");
 
     self::writeHeader("BASIC CONFIGURATION");
 
@@ -241,9 +241,10 @@ public static function runSetup() {
 
     self::writeHeader("MySQL DATABASE CONFIGURATION");
 
-    $conn_user = PhabricatorEnv::getEnvConfig('mysql.user');
-    $conn_pass = PhabricatorEnv::getEnvConfig('mysql.pass');
-    $conn_host = PhabricatorEnv::getEnvConfig('mysql.host');
+    $conf = DatabaseConfigurationProvider::getConfiguration();
+    $conn_user = $conf->getUser();
+    $conn_pass = $conf->getPassword();
+    $conn_host = $conf->getHost();
 
     $timeout = ini_get('mysql.connect_timeout');
     if ($timeout > 5) {

src/infrastructure/setup/__init__.php

@@ -4,6 +4,9 @@
  * @generated
  */
 
+
+
+phutil_require_module('phabricator', 'applications/base/storage/configuration');
 phutil_require_module('phabricator', 'infrastructure/env');
 phutil_require_module('phabricator', 'infrastructure/setup/sql');
 phutil_require_module('phabricator', 'storage/connection/mysql');
@@ -15,4 +18,5 @@
 phutil_require_module('phutil', 'parser/uri');
 phutil_require_module('phutil', 'utils');
 
-phutil_require_source('PhabricatorSetup.php');
+
+phutil_require_source('PhabricatorSetup.php');