@@ -38,118 +38,118 @@ |
||
| 38 | 38 | |
| 39 | 39 | abstract class AbstractDatabase { |
| 40 | 40 | |
| 41 | - /** @var IL10N */ |
|
| 42 | - protected $trans; |
|
| 43 | - /** @var string */ |
|
| 44 | - protected $dbUser; |
|
| 45 | - /** @var string */ |
|
| 46 | - protected $dbPassword; |
|
| 47 | - /** @var string */ |
|
| 48 | - protected $dbName; |
|
| 49 | - /** @var string */ |
|
| 50 | - protected $dbHost; |
|
| 51 | - /** @var string */ |
|
| 52 | - protected $dbPort; |
|
| 53 | - /** @var string */ |
|
| 54 | - protected $tablePrefix; |
|
| 55 | - /** @var SystemConfig */ |
|
| 56 | - protected $config; |
|
| 57 | - /** @var LoggerInterface */ |
|
| 58 | - protected $logger; |
|
| 59 | - /** @var ISecureRandom */ |
|
| 60 | - protected $random; |
|
| 41 | + /** @var IL10N */ |
|
| 42 | + protected $trans; |
|
| 43 | + /** @var string */ |
|
| 44 | + protected $dbUser; |
|
| 45 | + /** @var string */ |
|
| 46 | + protected $dbPassword; |
|
| 47 | + /** @var string */ |
|
| 48 | + protected $dbName; |
|
| 49 | + /** @var string */ |
|
| 50 | + protected $dbHost; |
|
| 51 | + /** @var string */ |
|
| 52 | + protected $dbPort; |
|
| 53 | + /** @var string */ |
|
| 54 | + protected $tablePrefix; |
|
| 55 | + /** @var SystemConfig */ |
|
| 56 | + protected $config; |
|
| 57 | + /** @var LoggerInterface */ |
|
| 58 | + protected $logger; |
|
| 59 | + /** @var ISecureRandom */ |
|
| 60 | + protected $random; |
|
| 61 | 61 | |
| 62 | - public function __construct(IL10N $trans, SystemConfig $config, LoggerInterface $logger, ISecureRandom $random) { |
|
| 63 | - $this->trans = $trans; |
|
| 64 | - $this->config = $config; |
|
| 65 | - $this->logger = $logger; |
|
| 66 | - $this->random = $random; |
|
| 67 | - } |
|
| 62 | + public function __construct(IL10N $trans, SystemConfig $config, LoggerInterface $logger, ISecureRandom $random) { |
|
| 63 | + $this->trans = $trans; |
|
| 64 | + $this->config = $config; |
|
| 65 | + $this->logger = $logger; |
|
| 66 | + $this->random = $random; |
|
| 67 | + } |
|
| 68 | 68 | |
| 69 | - public function validate($config) { |
|
| 70 | - $errors = []; |
|
| 71 | - if (empty($config['dbuser']) && empty($config['dbname'])) { |
|
| 72 | - $errors[] = $this->trans->t("Enter the database username and name for %s", [$this->dbprettyname]); |
|
| 73 | - } elseif (empty($config['dbuser'])) { |
|
| 74 | - $errors[] = $this->trans->t("Enter the database username for %s", [$this->dbprettyname]); |
|
| 75 | - } elseif (empty($config['dbname'])) { |
|
| 76 | - $errors[] = $this->trans->t("Enter the database name for %s", [$this->dbprettyname]); |
|
| 77 | - } |
|
| 78 | - if (substr_count($config['dbname'], '.') >= 1) { |
|
| 79 | - $errors[] = $this->trans->t("You cannot use dots in the database name %s", [$this->dbprettyname]); |
|
| 80 | - } |
|
| 81 | - return $errors; |
|
| 82 | - } |
|
| 69 | + public function validate($config) { |
|
| 70 | + $errors = []; |
|
| 71 | + if (empty($config['dbuser']) && empty($config['dbname'])) { |
|
| 72 | + $errors[] = $this->trans->t("Enter the database username and name for %s", [$this->dbprettyname]); |
|
| 73 | + } elseif (empty($config['dbuser'])) { |
|
| 74 | + $errors[] = $this->trans->t("Enter the database username for %s", [$this->dbprettyname]); |
|
| 75 | + } elseif (empty($config['dbname'])) { |
|
| 76 | + $errors[] = $this->trans->t("Enter the database name for %s", [$this->dbprettyname]); |
|
| 77 | + } |
|
| 78 | + if (substr_count($config['dbname'], '.') >= 1) { |
|
| 79 | + $errors[] = $this->trans->t("You cannot use dots in the database name %s", [$this->dbprettyname]); |
|
| 80 | + } |
|
| 81 | + return $errors; |
|
| 82 | + } |
|
| 83 | 83 | |
| 84 | - public function initialize($config) { |
|
| 85 | - $dbUser = $config['dbuser']; |
|
| 86 | - $dbPass = $config['dbpass']; |
|
| 87 | - $dbName = $config['dbname']; |
|
| 88 | - $dbHost = !empty($config['dbhost']) ? $config['dbhost'] : 'localhost'; |
|
| 89 | - $dbPort = !empty($config['dbport']) ? $config['dbport'] : ''; |
|
| 90 | - $dbTablePrefix = isset($config['dbtableprefix']) ? $config['dbtableprefix'] : 'oc_'; |
|
| 84 | + public function initialize($config) { |
|
| 85 | + $dbUser = $config['dbuser']; |
|
| 86 | + $dbPass = $config['dbpass']; |
|
| 87 | + $dbName = $config['dbname']; |
|
| 88 | + $dbHost = !empty($config['dbhost']) ? $config['dbhost'] : 'localhost'; |
|
| 89 | + $dbPort = !empty($config['dbport']) ? $config['dbport'] : ''; |
|
| 90 | + $dbTablePrefix = isset($config['dbtableprefix']) ? $config['dbtableprefix'] : 'oc_'; |
|
| 91 | 91 | |
| 92 | - $this->config->setValues([ |
|
| 93 | - 'dbname' => $dbName, |
|
| 94 | - 'dbhost' => $dbHost, |
|
| 95 | - 'dbport' => $dbPort, |
|
| 96 | - 'dbtableprefix' => $dbTablePrefix, |
|
| 97 | - ]); |
|
| 92 | + $this->config->setValues([ |
|
| 93 | + 'dbname' => $dbName, |
|
| 94 | + 'dbhost' => $dbHost, |
|
| 95 | + 'dbport' => $dbPort, |
|
| 96 | + 'dbtableprefix' => $dbTablePrefix, |
|
| 97 | + ]); |
|
| 98 | 98 | |
| 99 | - $this->dbUser = $dbUser; |
|
| 100 | - $this->dbPassword = $dbPass; |
|
| 101 | - $this->dbName = $dbName; |
|
| 102 | - $this->dbHost = $dbHost; |
|
| 103 | - $this->dbPort = $dbPort; |
|
| 104 | - $this->tablePrefix = $dbTablePrefix; |
|
| 105 | - } |
|
| 99 | + $this->dbUser = $dbUser; |
|
| 100 | + $this->dbPassword = $dbPass; |
|
| 101 | + $this->dbName = $dbName; |
|
| 102 | + $this->dbHost = $dbHost; |
|
| 103 | + $this->dbPort = $dbPort; |
|
| 104 | + $this->tablePrefix = $dbTablePrefix; |
|
| 105 | + } |
|
| 106 | 106 | |
| 107 | - /** |
|
| 108 | - * @param array $configOverwrite |
|
| 109 | - * @return \OC\DB\Connection |
|
| 110 | - */ |
|
| 111 | - protected function connect(array $configOverwrite = []): Connection { |
|
| 112 | - $connectionParams = [ |
|
| 113 | - 'host' => $this->dbHost, |
|
| 114 | - 'user' => $this->dbUser, |
|
| 115 | - 'password' => $this->dbPassword, |
|
| 116 | - 'tablePrefix' => $this->tablePrefix, |
|
| 117 | - 'dbname' => $this->dbName |
|
| 118 | - ]; |
|
| 107 | + /** |
|
| 108 | + * @param array $configOverwrite |
|
| 109 | + * @return \OC\DB\Connection |
|
| 110 | + */ |
|
| 111 | + protected function connect(array $configOverwrite = []): Connection { |
|
| 112 | + $connectionParams = [ |
|
| 113 | + 'host' => $this->dbHost, |
|
| 114 | + 'user' => $this->dbUser, |
|
| 115 | + 'password' => $this->dbPassword, |
|
| 116 | + 'tablePrefix' => $this->tablePrefix, |
|
| 117 | + 'dbname' => $this->dbName |
|
| 118 | + ]; |
|
| 119 | 119 | |
| 120 | - // adding port support through installer |
|
| 121 | - if (!empty($this->dbPort)) { |
|
| 122 | - if (ctype_digit($this->dbPort)) { |
|
| 123 | - $connectionParams['port'] = $this->dbPort; |
|
| 124 | - } else { |
|
| 125 | - $connectionParams['unix_socket'] = $this->dbPort; |
|
| 126 | - } |
|
| 127 | - } elseif (strpos($this->dbHost, ':')) { |
|
| 128 | - // Host variable may carry a port or socket. |
|
| 129 | - [$host, $portOrSocket] = explode(':', $this->dbHost, 2); |
|
| 130 | - if (ctype_digit($portOrSocket)) { |
|
| 131 | - $connectionParams['port'] = $portOrSocket; |
|
| 132 | - } else { |
|
| 133 | - $connectionParams['unix_socket'] = $portOrSocket; |
|
| 134 | - } |
|
| 135 | - $connectionParams['host'] = $host; |
|
| 136 | - } |
|
| 120 | + // adding port support through installer |
|
| 121 | + if (!empty($this->dbPort)) { |
|
| 122 | + if (ctype_digit($this->dbPort)) { |
|
| 123 | + $connectionParams['port'] = $this->dbPort; |
|
| 124 | + } else { |
|
| 125 | + $connectionParams['unix_socket'] = $this->dbPort; |
|
| 126 | + } |
|
| 127 | + } elseif (strpos($this->dbHost, ':')) { |
|
| 128 | + // Host variable may carry a port or socket. |
|
| 129 | + [$host, $portOrSocket] = explode(':', $this->dbHost, 2); |
|
| 130 | + if (ctype_digit($portOrSocket)) { |
|
| 131 | + $connectionParams['port'] = $portOrSocket; |
|
| 132 | + } else { |
|
| 133 | + $connectionParams['unix_socket'] = $portOrSocket; |
|
| 134 | + } |
|
| 135 | + $connectionParams['host'] = $host; |
|
| 136 | + } |
|
| 137 | 137 | |
| 138 | - $connectionParams = array_merge($connectionParams, $configOverwrite); |
|
| 139 | - $cf = new ConnectionFactory($this->config); |
|
| 140 | - return $cf->getConnection($this->config->getValue('dbtype', 'sqlite'), $connectionParams); |
|
| 141 | - } |
|
| 138 | + $connectionParams = array_merge($connectionParams, $configOverwrite); |
|
| 139 | + $cf = new ConnectionFactory($this->config); |
|
| 140 | + return $cf->getConnection($this->config->getValue('dbtype', 'sqlite'), $connectionParams); |
|
| 141 | + } |
|
| 142 | 142 | |
| 143 | - /** |
|
| 144 | - * @param string $username |
|
| 145 | - */ |
|
| 146 | - abstract public function setupDatabase($username); |
|
| 143 | + /** |
|
| 144 | + * @param string $username |
|
| 145 | + */ |
|
| 146 | + abstract public function setupDatabase($username); |
|
| 147 | 147 | |
| 148 | - public function runMigrations() { |
|
| 149 | - if (!is_dir(\OC::$SERVERROOT."/core/Migrations")) { |
|
| 150 | - return; |
|
| 151 | - } |
|
| 152 | - $ms = new MigrationService('core', \OC::$server->get(Connection::class)); |
|
| 153 | - $ms->migrate('latest', true); |
|
| 154 | - } |
|
| 148 | + public function runMigrations() { |
|
| 149 | + if (!is_dir(\OC::$SERVERROOT."/core/Migrations")) { |
|
| 150 | + return; |
|
| 151 | + } |
|
| 152 | + $ms = new MigrationService('core', \OC::$server->get(Connection::class)); |
|
| 153 | + $ms->migrate('latest', true); |
|
| 154 | + } |
|
| 155 | 155 | } |
@@ -30,81 +30,81 @@ |
||
| 30 | 30 | namespace OC\Setup; |
| 31 | 31 | |
| 32 | 32 | class OCI extends AbstractDatabase { |
| 33 | - public $dbprettyname = 'Oracle'; |
|
| 33 | + public $dbprettyname = 'Oracle'; |
|
| 34 | 34 | |
| 35 | - protected $dbtablespace; |
|
| 35 | + protected $dbtablespace; |
|
| 36 | 36 | |
| 37 | - public function initialize($config) { |
|
| 38 | - parent::initialize($config); |
|
| 39 | - if (array_key_exists('dbtablespace', $config)) { |
|
| 40 | - $this->dbtablespace = $config['dbtablespace']; |
|
| 41 | - } else { |
|
| 42 | - $this->dbtablespace = 'USERS'; |
|
| 43 | - } |
|
| 44 | - // allow empty hostname for oracle |
|
| 45 | - $this->dbHost = $config['dbhost']; |
|
| 37 | + public function initialize($config) { |
|
| 38 | + parent::initialize($config); |
|
| 39 | + if (array_key_exists('dbtablespace', $config)) { |
|
| 40 | + $this->dbtablespace = $config['dbtablespace']; |
|
| 41 | + } else { |
|
| 42 | + $this->dbtablespace = 'USERS'; |
|
| 43 | + } |
|
| 44 | + // allow empty hostname for oracle |
|
| 45 | + $this->dbHost = $config['dbhost']; |
|
| 46 | 46 | |
| 47 | - $this->config->setValues([ |
|
| 48 | - 'dbhost' => $this->dbHost, |
|
| 49 | - 'dbtablespace' => $this->dbtablespace, |
|
| 50 | - ]); |
|
| 51 | - } |
|
| 47 | + $this->config->setValues([ |
|
| 48 | + 'dbhost' => $this->dbHost, |
|
| 49 | + 'dbtablespace' => $this->dbtablespace, |
|
| 50 | + ]); |
|
| 51 | + } |
|
| 52 | 52 | |
| 53 | - public function validate($config) { |
|
| 54 | - $errors = []; |
|
| 55 | - if (empty($config['dbuser']) && empty($config['dbname'])) { |
|
| 56 | - $errors[] = $this->trans->t("Enter the database username and name for %s", [$this->dbprettyname]); |
|
| 57 | - } elseif (empty($config['dbuser'])) { |
|
| 58 | - $errors[] = $this->trans->t("Enter the database username for %s", [$this->dbprettyname]); |
|
| 59 | - } elseif (empty($config['dbname'])) { |
|
| 60 | - $errors[] = $this->trans->t("Enter the database name for %s", [$this->dbprettyname]); |
|
| 61 | - } |
|
| 62 | - return $errors; |
|
| 63 | - } |
|
| 53 | + public function validate($config) { |
|
| 54 | + $errors = []; |
|
| 55 | + if (empty($config['dbuser']) && empty($config['dbname'])) { |
|
| 56 | + $errors[] = $this->trans->t("Enter the database username and name for %s", [$this->dbprettyname]); |
|
| 57 | + } elseif (empty($config['dbuser'])) { |
|
| 58 | + $errors[] = $this->trans->t("Enter the database username for %s", [$this->dbprettyname]); |
|
| 59 | + } elseif (empty($config['dbname'])) { |
|
| 60 | + $errors[] = $this->trans->t("Enter the database name for %s", [$this->dbprettyname]); |
|
| 61 | + } |
|
| 62 | + return $errors; |
|
| 63 | + } |
|
| 64 | 64 | |
| 65 | - public function setupDatabase($username) { |
|
| 66 | - try { |
|
| 67 | - $this->connect(); |
|
| 68 | - } catch (\Exception $e) { |
|
| 69 | - $errorMessage = $this->getLastError(); |
|
| 70 | - if ($errorMessage) { |
|
| 71 | - throw new \OC\DatabaseSetupException($this->trans->t('Oracle connection could not be established'), |
|
| 72 | - $errorMessage . ' Check environment: ORACLE_HOME=' . getenv('ORACLE_HOME') |
|
| 73 | - . ' ORACLE_SID=' . getenv('ORACLE_SID') |
|
| 74 | - . ' LD_LIBRARY_PATH=' . getenv('LD_LIBRARY_PATH') |
|
| 75 | - . ' NLS_LANG=' . getenv('NLS_LANG') |
|
| 76 | - . ' tnsnames.ora is ' . (is_readable(getenv('ORACLE_HOME') . '/network/admin/tnsnames.ora') ? '' : 'not ') . 'readable', 0, $e); |
|
| 77 | - } |
|
| 78 | - throw new \OC\DatabaseSetupException($this->trans->t('Oracle username and/or password not valid'), |
|
| 79 | - 'Check environment: ORACLE_HOME=' . getenv('ORACLE_HOME') |
|
| 80 | - . ' ORACLE_SID=' . getenv('ORACLE_SID') |
|
| 81 | - . ' LD_LIBRARY_PATH=' . getenv('LD_LIBRARY_PATH') |
|
| 82 | - . ' NLS_LANG=' . getenv('NLS_LANG') |
|
| 83 | - . ' tnsnames.ora is ' . (is_readable(getenv('ORACLE_HOME') . '/network/admin/tnsnames.ora') ? '' : 'not ') . 'readable', 0, $e); |
|
| 84 | - } |
|
| 65 | + public function setupDatabase($username) { |
|
| 66 | + try { |
|
| 67 | + $this->connect(); |
|
| 68 | + } catch (\Exception $e) { |
|
| 69 | + $errorMessage = $this->getLastError(); |
|
| 70 | + if ($errorMessage) { |
|
| 71 | + throw new \OC\DatabaseSetupException($this->trans->t('Oracle connection could not be established'), |
|
| 72 | + $errorMessage . ' Check environment: ORACLE_HOME=' . getenv('ORACLE_HOME') |
|
| 73 | + . ' ORACLE_SID=' . getenv('ORACLE_SID') |
|
| 74 | + . ' LD_LIBRARY_PATH=' . getenv('LD_LIBRARY_PATH') |
|
| 75 | + . ' NLS_LANG=' . getenv('NLS_LANG') |
|
| 76 | + . ' tnsnames.ora is ' . (is_readable(getenv('ORACLE_HOME') . '/network/admin/tnsnames.ora') ? '' : 'not ') . 'readable', 0, $e); |
|
| 77 | + } |
|
| 78 | + throw new \OC\DatabaseSetupException($this->trans->t('Oracle username and/or password not valid'), |
|
| 79 | + 'Check environment: ORACLE_HOME=' . getenv('ORACLE_HOME') |
|
| 80 | + . ' ORACLE_SID=' . getenv('ORACLE_SID') |
|
| 81 | + . ' LD_LIBRARY_PATH=' . getenv('LD_LIBRARY_PATH') |
|
| 82 | + . ' NLS_LANG=' . getenv('NLS_LANG') |
|
| 83 | + . ' tnsnames.ora is ' . (is_readable(getenv('ORACLE_HOME') . '/network/admin/tnsnames.ora') ? '' : 'not ') . 'readable', 0, $e); |
|
| 84 | + } |
|
| 85 | 85 | |
| 86 | - $this->config->setValues([ |
|
| 87 | - 'dbuser' => $this->dbUser, |
|
| 88 | - 'dbname' => $this->dbName, |
|
| 89 | - 'dbpassword' => $this->dbPassword, |
|
| 90 | - ]); |
|
| 91 | - } |
|
| 86 | + $this->config->setValues([ |
|
| 87 | + 'dbuser' => $this->dbUser, |
|
| 88 | + 'dbname' => $this->dbName, |
|
| 89 | + 'dbpassword' => $this->dbPassword, |
|
| 90 | + ]); |
|
| 91 | + } |
|
| 92 | 92 | |
| 93 | - /** |
|
| 94 | - * @param resource $connection |
|
| 95 | - * @return string |
|
| 96 | - */ |
|
| 97 | - protected function getLastError($connection = null) { |
|
| 98 | - if ($connection) { |
|
| 99 | - $error = oci_error($connection); |
|
| 100 | - } else { |
|
| 101 | - $error = oci_error(); |
|
| 102 | - } |
|
| 103 | - foreach (['message', 'code'] as $key) { |
|
| 104 | - if (isset($error[$key])) { |
|
| 105 | - return $error[$key]; |
|
| 106 | - } |
|
| 107 | - } |
|
| 108 | - return ''; |
|
| 109 | - } |
|
| 93 | + /** |
|
| 94 | + * @param resource $connection |
|
| 95 | + * @return string |
|
| 96 | + */ |
|
| 97 | + protected function getLastError($connection = null) { |
|
| 98 | + if ($connection) { |
|
| 99 | + $error = oci_error($connection); |
|
| 100 | + } else { |
|
| 101 | + $error = oci_error(); |
|
| 102 | + } |
|
| 103 | + foreach (['message', 'code'] as $key) { |
|
| 104 | + if (isset($error[$key])) { |
|
| 105 | + return $error[$key]; |
|
| 106 | + } |
|
| 107 | + } |
|
| 108 | + return ''; |
|
| 109 | + } |
|
| 110 | 110 | } |