Index: includes/database/database.inc =================================================================== RCS file: /cvs/drupal/drupal/includes/database/database.inc,v retrieving revision 1.10 diff -u -p -r1.10 database.inc --- includes/database/database.inc 28 Sep 2008 21:52:08 -0000 1.10 +++ includes/database/database.inc 30 Sep 2008 20:54:39 -0000 @@ -746,10 +746,6 @@ abstract class Database { /** * Process the configuration file for database information. - * - * Because the config file accepts various "fallback" configurations, we have - * to parse the configuration array out into a standardized "complete" form, - * applying defaults where necessary. */ final protected static function parseConnectionInfo() { global $databases; @@ -759,17 +755,7 @@ abstract class Database { } $databaseInfo = $databases; - // If no database key is specified, default to default. - if (!is_array($databaseInfo)) { - $databaseInfo = array('default' => $databaseInfo); - } - foreach ($databaseInfo as $index => $info) { - // If no targets are specified, default to one default. - if (!is_array($databaseInfo[$index])) { - $databaseInfo[$index] = array('default' => $info); - } - foreach ($databaseInfo[$index] as $target => $value) { // If there is no "driver" property, then we assume it's an array of // possible connections for this target. Pick one at random. That Index: sites/default/default.settings.php =================================================================== RCS file: /cvs/drupal/drupal/sites/default/default.settings.php,v retrieving revision 1.14 diff -u -p -r1.14 default.settings.php --- sites/default/default.settings.php 21 Aug 2008 19:36:39 -0000 1.14 +++ sites/default/default.settings.php 30 Sep 2008 21:03:49 -0000 @@ -60,6 +60,7 @@ * 'username' => 'username', * 'password' => 'password', * 'host' => 'localhost', + * 'port' => 3306, * ); * * The "driver" property indicates what Drupal database driver the @@ -87,28 +88,17 @@ * $databases['default']['default'] = $info_array; * $databases['default']['slave'][] = $info_array; * $databases['default']['slave'][] = $info_array; - * $databases['extra'] = $info_array; + * $databases['extra']['default'] = $info_array; * * In the above example, $info_array is an array of settings described above. * The first line sets a "default" database that has one master database * (the second level default). The second and third lines create an array * of potential slave databases. Drupal will select one at random for a given * request as needed. The fourth line creates a new database with a name of - * "extra". Since no target is specified, it is assumed to be "default", that - * is, only one server. + * "extra". * * For a single database configuration, the following is sufficient: * - * $databases = array( - * 'driver' => 'mysql', - * 'database' => 'databasename', - * 'username' => 'username', - * 'password' => 'password', - * 'host' => 'localhost', - * ); - * - * That is equivalent to: - * * $databases['default']['default'] = array( * 'driver' => 'mysql', * 'database' => 'databasename', @@ -141,21 +131,21 @@ * ); * * Database configuration format: - * $databases = array( + * $databases['default']['default'] = array( * 'driver' => 'mysql', * 'database' => 'databasename', * 'username' => 'username', * 'password' => 'password', * 'host' => 'localhost', * ); - * $databases = array( + * $databases['default']['default'] = array( * 'driver' => 'pgsql', * 'database' => 'databasename', * 'username' => 'username', * 'password' => 'password', * 'host' => 'localhost', * ); - * $databases = array( + * $databases['default']['default'] = array( * 'driver' => 'sqlite', * 'database' => 'databasefilename', * );