From d78ac575140c57b6cc8682eedb0bd227d37fb36f Mon Sep 17 00:00:00 2001 From: Philipp Holzer Date: Mon, 4 Feb 2019 09:30:48 +0100 Subject: [PATCH] Updating type-hints and some naming conventions --- src/Core/Config.php | 4 +-- src/Core/Config/ConfigCache.php | 8 +++--- src/Core/Config/ConfigCacheLoader.php | 4 +-- src/Core/Config/JITConfigAdapter.php | 24 ++++++++--------- src/Core/Config/JITPConfigAdapter.php | 30 ++++++++++----------- src/Core/Config/PreloadConfigAdapter.php | 20 +++++++------- src/Core/Config/PreloadPConfigAdapter.php | 22 ++++++++-------- src/Database/DBA.php | 32 +++++++++++------------ src/Factory/ConfigFactory.php | 4 +-- 9 files changed, 74 insertions(+), 74 deletions(-) diff --git a/src/Core/Config.php b/src/Core/Config.php index 0db8f0b88..3eace0fbc 100644 --- a/src/Core/Config.php +++ b/src/Core/Config.php @@ -32,7 +32,7 @@ class Config * * @param Config\IConfigCache $config The configuration cache */ - public static function init($config) + public static function init(Config\IConfigCache $config) { self::$config = $config; } @@ -42,7 +42,7 @@ class Config * * @param Config\IConfigAdapter $adapter */ - public static function setAdapter($adapter) + public static function setAdapter(Config\IConfigAdapter $adapter) { self::$adapter = $adapter; } diff --git a/src/Core/Config/ConfigCache.php b/src/Core/Config/ConfigCache.php index 355b1df2f..dba2a9dd1 100644 --- a/src/Core/Config/ConfigCache.php +++ b/src/Core/Config/ConfigCache.php @@ -22,7 +22,7 @@ class ConfigCache implements IConfigCache, IPConfigCache /** * @param array $config A initial config array */ - public function __construct($config = []) + public function __construct(array $config = []) { $this->config = []; @@ -43,9 +43,9 @@ class ConfigCache implements IConfigCache, IPConfigCache foreach ($config as $category => $values) { foreach ($values as $key => $value) { if ($overwrite) { - self::set($category, $key, $value); + $this->set($category, $key, $value); } else { - self::setDefault($category, $key, $value); + $this->setDefault($category, $key, $value); } } } @@ -83,7 +83,7 @@ class ConfigCache implements IConfigCache, IPConfigCache private function setDefault($cat, $k, $v) { if (!isset($this->config[$cat][$k])) { - self::set($cat, $k, $v); + $this->set($cat, $k, $v); } } diff --git a/src/Core/Config/ConfigCacheLoader.php b/src/Core/Config/ConfigCacheLoader.php index bb6207f93..4f595bc1c 100644 --- a/src/Core/Config/ConfigCacheLoader.php +++ b/src/Core/Config/ConfigCacheLoader.php @@ -21,7 +21,7 @@ class ConfigCacheLoader * The addon sub-directory * @var string */ - const ADDONSDIRECTORY = '/addons/'; + const ADDONDIRECTORY = '/addon/'; private $baseDir; private $configDir; @@ -141,7 +141,7 @@ class ConfigCacheLoader public function loadConfigFile($filename, $addon = false) { if ($addon) { - $filepath = $this->baseDir . self::ADDONSDIRECTORY . $filename . self::SUBDIRECTORY . $filename . ".config.php"; + $filepath = $this->baseDir . self::ADDONDIRECTORY . $filename . self::SUBDIRECTORY . $filename . ".config.php"; } else { $filepath = $this->configDir . $filename . ".config.php"; } diff --git a/src/Core/Config/JITConfigAdapter.php b/src/Core/Config/JITConfigAdapter.php index 256a2a956..5e7e786f0 100644 --- a/src/Core/Config/JITConfigAdapter.php +++ b/src/Core/Config/JITConfigAdapter.php @@ -18,14 +18,14 @@ class JITConfigAdapter implements IConfigAdapter /** * @var IConfigCache The config cache of this driver */ - private $config; + private $configCache; /** - * @param IConfigCache $config The config cache of this driver + * @param IConfigCache $configCache The config cache of this driver */ - public function __construct($config) + public function __construct(IConfigCache $configCache) { - $this->config = $config; + $this->configCache = $configCache; } public function load($cat = "config") @@ -40,7 +40,7 @@ class JITConfigAdapter implements IConfigAdapter while ($config = DBA::fetch($configs)) { $k = $config['k']; - $this->config->set($cat, $k, $config['v']); + $this->configCache->set($cat, $k, $config['v']); if ($cat !== 'config') { $this->cache[$cat][$k] = $config['v']; @@ -72,18 +72,18 @@ class JITConfigAdapter implements IConfigAdapter $this->cache[$cat][$k] = $value; $this->in_db[$cat][$k] = true; return $value; - } elseif ($this->config->get($cat, $k) !== null) { + } elseif ($this->configCache->get($cat, $k) !== null) { // Assign the value (mostly) from config/local.config.php file to the cache - $this->cache[$cat][$k] = $this->config->get($cat, $k); + $this->cache[$cat][$k] = $this->configCache->get($cat, $k); $this->in_db[$cat][$k] = false; - return $this->config->get($cat, $k); - } elseif ($this->config->get('config', $k) !== null) { + return $this->configCache->get($cat, $k); + } elseif ($this->configCache->get('config', $k) !== null) { // Assign the value (mostly) from config/local.config.php file to the cache - $this->cache[$k] = $this->config->get('config', $k); + $this->cache[$k] = $this->configCache->get('config', $k); $this->in_db[$k] = false; - return $this->config->get('config', $k); + return $this->configCache->get('config', $k); } $this->cache[$cat][$k] = '!!'; @@ -112,7 +112,7 @@ class JITConfigAdapter implements IConfigAdapter return true; } - $this->config->set($cat, $k, $value); + $this->configCache->set($cat, $k, $value); // Assign the just added value to the cache $this->cache[$cat][$k] = $dbvalue; diff --git a/src/Core/Config/JITPConfigAdapter.php b/src/Core/Config/JITPConfigAdapter.php index 223cc542d..2fd770592 100644 --- a/src/Core/Config/JITPConfigAdapter.php +++ b/src/Core/Config/JITPConfigAdapter.php @@ -18,14 +18,14 @@ class JITPConfigAdapter implements IPConfigAdapter * The config cache of this adapter * @var IPConfigCache */ - private $config; + private $configCache; /** - * @param IPConfigCache $config The config cache of this adapter + * @param IPConfigCache $configCache The config cache of this adapter */ - public function __construct($config) + public function __construct(IPConfigCache $configCache) { - $this->config = $config; + $this->configCache = $configCache; } public function load($uid, $cat) @@ -35,13 +35,13 @@ class JITPConfigAdapter implements IPConfigAdapter while ($pconfig = DBA::fetch($pconfigs)) { $k = $pconfig['k']; - $this->config->setP($uid, $cat, $k, $pconfig['v']); + $this->configCache->setP($uid, $cat, $k, $pconfig['v']); $this->in_db[$uid][$cat][$k] = true; } } else if ($cat != 'config') { // Negative caching - $this->config->setP($uid, $cat, null, "!!"); + $this->configCache->setP($uid, $cat, null, "!!"); } DBA::close($pconfigs); } @@ -50,17 +50,17 @@ class JITPConfigAdapter implements IPConfigAdapter { if (!$refresh) { // Looking if the whole family isn't set - if ($this->config->getP($uid, $cat) !== null) { - if ($this->config->getP($uid, $cat) === '!!') { + if ($this->configCache->getP($uid, $cat) !== null) { + if ($this->configCache->getP($uid, $cat) === '!!') { return $default_value; } } - if ($this->config->getP($uid, $cat, $k) !== null) { - if ($this->config->getP($uid, $cat, $k) === '!!') { + if ($this->configCache->getP($uid, $cat, $k) !== null) { + if ($this->configCache->getP($uid, $cat, $k) === '!!') { return $default_value; } - return $this->config->getP($uid, $cat, $k); + return $this->configCache->getP($uid, $cat, $k); } } @@ -68,13 +68,13 @@ class JITPConfigAdapter implements IPConfigAdapter if (DBA::isResult($pconfig)) { $val = (preg_match("|^a:[0-9]+:{.*}$|s", $pconfig['v']) ? unserialize($pconfig['v']) : $pconfig['v']); - $this->config->setP($uid, $cat, $k, $val); + $this->configCache->setP($uid, $cat, $k, $val); $this->in_db[$uid][$cat][$k] = true; return $val; } else { - $this->config->setP($uid, $cat, $k, '!!'); + $this->configCache->setP($uid, $cat, $k, '!!'); $this->in_db[$uid][$cat][$k] = false; @@ -95,7 +95,7 @@ class JITPConfigAdapter implements IPConfigAdapter return true; } - $this->config->setP($uid, $cat, $k, $value); + $this->configCache->setP($uid, $cat, $k, $value); // manage array value $dbvalue = (is_array($value) ? serialize($value) : $dbvalue); @@ -111,7 +111,7 @@ class JITPConfigAdapter implements IPConfigAdapter public function delete($uid, $cat, $k) { - $this->config->deleteP($uid, $cat, $k); + $this->configCache->deleteP($uid, $cat, $k); if (!empty($this->in_db[$uid][$cat][$k])) { unset($this->in_db[$uid][$cat][$k]); diff --git a/src/Core/Config/PreloadConfigAdapter.php b/src/Core/Config/PreloadConfigAdapter.php index 4461105e1..bda9b28aa 100644 --- a/src/Core/Config/PreloadConfigAdapter.php +++ b/src/Core/Config/PreloadConfigAdapter.php @@ -19,14 +19,14 @@ class PreloadConfigAdapter implements IConfigAdapter /** * @var IConfigCache The config cache of this driver */ - private $config; + private $configCache; /** - * @param IConfigCache $config The config cache of this driver + * @param IConfigCache $configCache The config cache of this driver */ - public function __construct($config) + public function __construct(IConfigCache $configCache) { - $this->config = $config; + $this->configCache = $configCache; $this->load(); } @@ -38,7 +38,7 @@ class PreloadConfigAdapter implements IConfigAdapter $configs = DBA::select('config', ['cat', 'v', 'k']); while ($config = DBA::fetch($configs)) { - $this->config->set($config['cat'], $config['k'], $config['v']); + $this->configCache->set($config['cat'], $config['k'], $config['v']); } DBA::close($configs); @@ -50,11 +50,11 @@ class PreloadConfigAdapter implements IConfigAdapter if ($refresh) { $config = DBA::selectFirst('config', ['v'], ['cat' => $cat, 'k' => $k]); if (DBA::isResult($config)) { - $this->config->set($cat, $k, $config['v']); + $this->configCache->set($cat, $k, $config['v']); } } - $return = $this->config->get($cat, $k, $default_value); + $return = $this->configCache->get($cat, $k, $default_value); return $return; } @@ -66,11 +66,11 @@ class PreloadConfigAdapter implements IConfigAdapter // The exception are array values. $compare_value = !is_array($value) ? (string)$value : $value; - if ($this->config->get($cat, $k) === $compare_value) { + if ($this->configCache->get($cat, $k) === $compare_value) { return true; } - $this->config->set($cat, $k, $value); + $this->configCache->set($cat, $k, $value); // manage array value $dbvalue = is_array($value) ? serialize($value) : $value; @@ -85,7 +85,7 @@ class PreloadConfigAdapter implements IConfigAdapter public function delete($cat, $k) { - $this->config->delete($cat, $k); + $this->configCache->delete($cat, $k); $result = DBA::delete('config', ['cat' => $cat, 'k' => $k]); diff --git a/src/Core/Config/PreloadPConfigAdapter.php b/src/Core/Config/PreloadPConfigAdapter.php index ed18bdf2e..b49f39382 100644 --- a/src/Core/Config/PreloadPConfigAdapter.php +++ b/src/Core/Config/PreloadPConfigAdapter.php @@ -20,15 +20,15 @@ class PreloadPConfigAdapter implements IPConfigAdapter * The config cache of this adapter * @var IPConfigCache */ - private $config; + private $configCache; /** - * @param IPConfigCache $config The config cache of this adapter + * @param IPConfigCache $configCache The config cache of this adapter * @param int $uid The UID of the current user */ - public function __construct($config, $uid = null) + public function __construct(IPConfigCache $configCache, $uid = null) { - $this->config = $config; + $this->configCache = $configCache; if (isset($uid)) { $this->load($uid, 'config'); } @@ -46,7 +46,7 @@ class PreloadPConfigAdapter implements IPConfigAdapter $pconfigs = DBA::select('pconfig', ['cat', 'v', 'k'], ['uid' => $uid]); while ($pconfig = DBA::fetch($pconfigs)) { - $this->config->setP($uid, $pconfig['cat'], $pconfig['k'], $pconfig['v']); + $this->configCache->setP($uid, $pconfig['cat'], $pconfig['k'], $pconfig['v']); } DBA::close($pconfigs); @@ -62,13 +62,13 @@ class PreloadPConfigAdapter implements IPConfigAdapter if ($refresh) { $config = DBA::selectFirst('pconfig', ['v'], ['uid' => $uid, 'cat' => $cat, 'k' => $k]); if (DBA::isResult($config)) { - $this->config->setP($uid, $cat, $k, $config['v']); + $this->configCache->setP($uid, $cat, $k, $config['v']); } else { - $this->config->deleteP($uid, $cat, $k); + $this->configCache->deleteP($uid, $cat, $k); } } - return $this->config->getP($uid, $cat, $k, $default_value);; + return $this->configCache->getP($uid, $cat, $k, $default_value);; } public function set($uid, $cat, $k, $value) @@ -81,11 +81,11 @@ class PreloadPConfigAdapter implements IPConfigAdapter // The exception are array values. $compare_value = !is_array($value) ? (string)$value : $value; - if ($this->config->getP($uid, $cat, $k) === $compare_value) { + if ($this->configCache->getP($uid, $cat, $k) === $compare_value) { return true; } - $this->config->setP($uid, $cat, $k, $value); + $this->configCache->setP($uid, $cat, $k, $value); // manage array value $dbvalue = is_array($value) ? serialize($value) : $value; @@ -104,7 +104,7 @@ class PreloadPConfigAdapter implements IPConfigAdapter $this->load($uid, $cat); } - $this->config->deleteP($uid, $cat, $k); + $this->configCache->deleteP($uid, $cat, $k); $result = DBA::delete('pconfig', ['uid' => $uid, 'cat' => $cat, 'k' => $k]); diff --git a/src/Database/DBA.php b/src/Database/DBA.php index eaeeb9cbd..d2a739e93 100644 --- a/src/Database/DBA.php +++ b/src/Database/DBA.php @@ -34,7 +34,7 @@ class DBA /** * @var IConfigCache */ - private static $config; + private static $configCache; private static $server_info = ''; private static $connection; private static $driver; @@ -50,14 +50,14 @@ class DBA private static $db_name = ''; private static $db_charset = ''; - public static function connect($config, $serveraddr, $user, $pass, $db, $charset = null) + public static function connect($configCache, $serveraddr, $user, $pass, $db, $charset = null) { if (!is_null(self::$connection) && self::connected()) { return true; } // We are storing these values for being able to perform a reconnect - self::$config = $config; + self::$configCache = $configCache; self::$db_serveraddr = $serveraddr; self::$db_user = $user; self::$db_pass = $pass; @@ -158,7 +158,7 @@ class DBA public static function reconnect() { self::disconnect(); - $ret = self::connect(self::$config, self::$db_serveraddr, self::$db_user, self::$db_pass, self::$db_name, self::$db_charset); + $ret = self::connect(self::$configCache, self::$db_serveraddr, self::$db_user, self::$db_pass, self::$db_name, self::$db_charset); return $ret; } @@ -213,7 +213,7 @@ class DBA */ private static function logIndex($query) { - if (!self::$config->get('system', 'db_log_index')) { + if (!self::$configCache->get('system', 'db_log_index')) { return; } @@ -232,18 +232,18 @@ class DBA return; } - $watchlist = explode(',', self::$config->get('system', 'db_log_index_watch')); - $blacklist = explode(',', self::$config->get('system', 'db_log_index_blacklist')); + $watchlist = explode(',', self::$configCache->get('system', 'db_log_index_watch')); + $blacklist = explode(',', self::$configCache->get('system', 'db_log_index_blacklist')); while ($row = self::fetch($r)) { - if ((intval(self::$config->get('system', 'db_loglimit_index')) > 0)) { + if ((intval(self::$configCache->get('system', 'db_loglimit_index')) > 0)) { $log = (in_array($row['key'], $watchlist) && - ($row['rows'] >= intval(self::$config->get('system', 'db_loglimit_index')))); + ($row['rows'] >= intval(self::$configCache->get('system', 'db_loglimit_index')))); } else { $log = false; } - if ((intval(self::$config->get('system', 'db_loglimit_index_high')) > 0) && ($row['rows'] >= intval($Config::getConfigValue('system', 'db_loglimit_index_high')))) { + if ((intval(self::$configCache->get('system', 'db_loglimit_index_high')) > 0) && ($row['rows'] >= intval(self::$configCache->get('system', 'db_loglimit_index_high')))) { $log = true; } @@ -253,7 +253,7 @@ class DBA if ($log) { $backtrace = debug_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS); - @file_put_contents(self::$config->get('system', 'db_log_index'), DateTimeFormat::utcNow()."\t". + @file_put_contents(self::$configCache->get('system', 'db_log_index'), DateTimeFormat::utcNow()."\t". $row['key']."\t".$row['rows']."\t".$row['Extra']."\t". basename($backtrace[1]["file"])."\t". $backtrace[1]["line"]."\t".$backtrace[2]["function"]."\t". @@ -423,7 +423,7 @@ class DBA $orig_sql = $sql; - if (self::$config->get('system', 'db_callstack') !== null) { + if (self::$configCache->get('system', 'db_callstack') !== null) { $sql = "/*".System::callstack()." */ ".$sql; } @@ -584,15 +584,15 @@ class DBA $a->saveTimestamp($stamp1, 'database'); - if (self::$config->get('system', 'db_log')) { + if (self::$configCache->get('system', 'db_log')) { $stamp2 = microtime(true); $duration = (float)($stamp2 - $stamp1); - if (($duration > self::$config->get('system', 'db_loglimit'))) { + if (($duration > self::$configCache->get('system', 'db_loglimit'))) { $duration = round($duration, 3); $backtrace = debug_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS); - @file_put_contents(self::$config->get('system', 'db_log'), DateTimeFormat::utcNow()."\t".$duration."\t". + @file_put_contents(self::$configCache->get('system', 'db_log'), DateTimeFormat::utcNow()."\t".$duration."\t". basename($backtrace[1]["file"])."\t". $backtrace[1]["line"]."\t".$backtrace[2]["function"]."\t". substr(self::replaceParameters($sql, $args), 0, 2000)."\n", FILE_APPEND); @@ -1031,7 +1031,7 @@ class DBA * This process must only be started once, since the value is cached. */ private static function buildRelationData() { - $definition = DBStructure::definition(self::$config->get('system', 'basepath')); + $definition = DBStructure::definition(self::$configCache->get('system', 'basepath')); foreach ($definition AS $table => $structure) { foreach ($structure['fields'] AS $field => $field_struct) { diff --git a/src/Factory/ConfigFactory.php b/src/Factory/ConfigFactory.php index 46d55b30c..269daea8b 100644 --- a/src/Factory/ConfigFactory.php +++ b/src/Factory/ConfigFactory.php @@ -25,7 +25,7 @@ class ConfigFactory * * @return Config\IConfigAdapter */ - public static function createConfig($type, $config) + public static function createConfig($type, Config\IConfigCache $config) { if ($type == 'preload') { return new Config\PreloadConfigAdapter($config); @@ -41,7 +41,7 @@ class ConfigFactory * * @return Config\IPConfigAdapter */ - public static function createPConfig($type, $config, $uid = null) + public static function createPConfig($type, Config\IPConfigCache $config, $uid = null) { if ($type == 'preload') { return new Config\PreloadPConfigAdapter($config, $uid);