mbk-lab
/
nextcloud
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Use const for dbname and dbtableprefix defaults 

Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
This commit is contained in:
Daniel Kesselberg 2018-09-23 18:15:57 +02:00
parent a5f0cc6539
commit 0ee191bf44
No known key found for this signature in database
GPG Key ID: 36E3664E099D0614
2 changed files with 42 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
lib/private/DB/ConnectionFactory.php
View File

@ -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

41
lib/private/Setup/Sqlite.php
View File

@ -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 setupDatabase($username) {
$datadir = $this->config->getValue('datadirectory', \OC::$SERVERROOT . '/data');
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.
*/
//delete the old sqlite database first, might cause infinte loops otherwise
if (file_exists("$datadir/owncloud.db")) {
unlink("$datadir/owncloud.db");
$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'
);
$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");
}
}