diff options
author | Daniel Kesselberg <mail@danielkesselberg.de> | 2018-09-23 18:15:57 +0200 |
---|---|---|
committer | Daniel Kesselberg <mail@danielkesselberg.de> | 2018-09-23 18:15:57 +0200 |
commit | 0ee191bf44a4204dae0dd5ee641ccbf6720e1341 (patch) | |
tree | 446549a9d5dbda58765e20c1e3419bb272da8bdb | |
parent | a5f0cc653963210228b1992568286f6e4330aff3 (diff) | |
download | nextcloud-server-0ee191bf44a4204dae0dd5ee641ccbf6720e1341.tar.gz nextcloud-server-0ee191bf44a4204dae0dd5ee641ccbf6720e1341.zip |
Use const for dbname and dbtableprefix defaults
Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
-rw-r--r-- | lib/private/DB/ConnectionFactory.php | 10 | ||||
-rw-r--r-- | lib/private/Setup/Sqlite.php | 39 |
2 files changed, 41 insertions, 8 deletions
diff --git a/lib/private/DB/ConnectionFactory.php b/lib/private/DB/ConnectionFactory.php index 9fc2ed42c33..d6526a14b7a 100644 --- a/lib/private/DB/ConnectionFactory.php +++ b/lib/private/DB/ConnectionFactory.php @@ -38,6 +38,12 @@ use OC\SystemConfig; * Takes care of creating and configuring Doctrine connections. */ class ConnectionFactory { + /** @var string default database name */ + const DEFAULT_DBNAME = 'owncloud'; + + /** @var string default database table prefix */ + const DEFAULT_DBTABLEPREFIX = 'oc_'; + /** * @var array * @@ -186,7 +192,7 @@ class ConnectionFactory { 'user' => $this->config->getValue('dbuser', ''), 'password' => $this->config->getValue('dbpassword', ''), ]; - $name = $this->config->getValue('dbname', 'owncloud'); + $name = $this->config->getValue('dbname', self::DEFAULT_DBNAME); if ($this->normalizeType($type) === 'sqlite3') { $dataDir = $this->config->getValue("datadirectory", \OC::$SERVERROOT . '/data'); @@ -197,7 +203,7 @@ class ConnectionFactory { $connectionParams['dbname'] = $name; } - $connectionParams['tablePrefix'] = $this->config->getValue('dbtableprefix', 'oc_'); + $connectionParams['tablePrefix'] = $this->config->getValue('dbtableprefix', self::DEFAULT_DBTABLEPREFIX); $connectionParams['sqlite.journal_mode'] = $this->config->getValue('sqlite.journal_mode', 'WAL'); //additional driver options, eg. for mysql ssl diff --git a/lib/private/Setup/Sqlite.php b/lib/private/Setup/Sqlite.php index 406c1e6fc9a..f0682b807ec 100644 --- a/lib/private/Setup/Sqlite.php +++ b/lib/private/Setup/Sqlite.php @@ -23,6 +23,8 @@ namespace OC\Setup; +use OC\DB\ConnectionFactory; + class Sqlite extends AbstractDatabase { public $dbprettyname = 'Sqlite'; @@ -30,14 +32,39 @@ class Sqlite extends AbstractDatabase { return array(); } + public function initialize($config) { + /* + * Web: When using web based installer its not possible to set dbname + * or dbtableprefix. Defaults used from ConnectionFactory and dbtype = 'sqlite' + * is written to config.php. + * + * Cli: When --database-name or --database-table-prefix empty or default + * dbtype = 'sqlite' is written to config.php. If you choose a value different + * from default these values are written to config.php. This is required because + * in connection factory configuration is obtained from config.php. + */ + + $this->dbName = $config['dbname'] ?? ConnectionFactory::DEFAULT_DBNAME; + $this->tablePrefix = $config['dbtableprefix'] ?? ConnectionFactory::DEFAULT_DBTABLEPREFIX; + + if ($this->dbName !== ConnectionFactory::DEFAULT_DBNAME) { + $this->config->setValue('dbname', $this->dbName); + } + + if ($this->tablePrefix !== ConnectionFactory::DEFAULT_DBTABLEPREFIX) { + $this->config->setValue('dbtableprefix', $this->tablePrefix); + } + } + public function setupDatabase($username) { - $datadir = $this->config->getValue('datadirectory', \OC::$SERVERROOT . '/data'); + $datadir = $this->config->getValue( + 'datadirectory', + \OC::$SERVERROOT . '/data' + ); - //delete the old sqlite database first, might cause infinte loops otherwise - if (file_exists("$datadir/owncloud.db")) { - unlink("$datadir/owncloud.db"); + $sqliteFile = $datadir . '/' . $this->dbName . 'db'; + if (file_exists($sqliteFile)) { + unlink($sqliteFile); } - //in case of sqlite, we can always fill the database - error_log("creating sqlite db"); } } |