deblan/php-censor
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

When phpci is default, it should be used.

Browse source 
The installer displays `phpci` as placeholders for the database settings (database and username).

When I do not change them the installer fails with this error:

`Fatal error: Uncaught exception 'PDOException' with message 'SQLSTATE[3D000]: Invalid catalog name: 1046 No database selected' in /Users/tobias/Sites/PHPCI/vendor/block8/b8framework/b8/Database/Map.php on line 40`

I assume it is simply because they are not set. This pull requests uses the given values, or phpci as default.

In theory I would assume all the configuration values should be checked somehow. Is this just not implemented, or is it not desired?
This commit is contained in:
Tobias Tom 2014-01-25 14:22:15 +01:00
parent 54694d197b
commit 37248fcd5b
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
public/install.php
View file

@ -101,8 +101,8 @@ if ($installOK && strtoupper($_SERVER['REQUEST_METHOD']) == 'POST') {
* Create database:
*/
$dbhost = $config['b8']['database']['servers']['write'][0];
$dbname = $config['b8']['database']['name'];
$dbuser = $config['b8']['database']['username'];
$dbname = $config['b8']['database']['name'] ?: 'phpci';
$dbuser = $config['b8']['database']['username'] ?: 'phpci';
$dbpass = $config['b8']['database']['password'];
$pdo = new PDO('mysql:host=' . $dbhost, $dbuser, $dbpass);