valvin
/
ttrss
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

add prefs caching

This commit is contained in:
Andrew Dolgov 2021-02-25 10:34:59 +03:00
parent 8026f3c3bd
commit 49a9afadce
1 changed files with 45 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

44
classes/prefs.php
View File

@ -157,39 +157,66 @@ class Prefs {
}*/ }*/
} }
private function _get($pref_name, int $owner_uid, int $profile_id = null) { private function _get(string $pref_name, int $owner_uid, int $profile_id = null) {
if (isset(self::_DEFAULTS[$pref_name])) { if (isset(self::_DEFAULTS[$pref_name])) {
if (!$profile_id || in_array($pref_name, self::_PROFILE_BLACKLIST)) $profile_id = null; if (!$profile_id || in_array($pref_name, self::_PROFILE_BLACKLIST)) $profile_id = null;
list ($def_val, $type_hint) = self::_DEFAULTS[$pref_name];
$cached_value = $this->_get_cache($pref_name, $owner_uid, $profile_id);
if (!empty($cached_value)) {
return Config::cast_to($cached_value, $type_hint);
} else {
$sth = $this->pdo->prepare("SELECT value FROM ttrss_user_prefs2 $sth = $this->pdo->prepare("SELECT value FROM ttrss_user_prefs2
WHERE pref_name = :name AND owner_uid = :uid AND WHERE pref_name = :name AND owner_uid = :uid AND
(profile = :profile OR (:profile IS NULL AND profile IS NULL))"); (profile = :profile OR (:profile IS NULL AND profile IS NULL))");
$sth->execute(["uid" => $owner_uid, "profile" => $profile_id, "name" => $pref_name ]); $sth->execute(["uid" => $owner_uid, "profile" => $profile_id, "name" => $pref_name ]);
list ($def_val, $type_hint) = self::_DEFAULTS[$pref_name];
if ($row = $sth->fetch(PDO::FETCH_ASSOC)) { if ($row = $sth->fetch(PDO::FETCH_ASSOC)) {
$this->_set_cache($pref_name, $row["value"], $owner_uid, $profile_id);
return Config::cast_to($row["value"], $type_hint); return Config::cast_to($row["value"], $type_hint);
} else { } else {
$this->_set_cache($pref_name, $def_val, $owner_uid, $profile_id);
return $def_val; return $def_val;
} }
} }
}
return null; return null;
} }
private function _set($pref_name, $value, int $owner_uid, int $profile_id = null) { private function _get_cache(string $pref_name, int $owner_uid, int $profile_id = null) {
$cache_key = sprintf("%d/%d/%s", $owner_uid, $profile_id, $pref_name);
if (isset($this->cache[$cache_key]))
return $this->cache[$cache_key];
return null;
}
private function _set_cache(string $pref_name, $value, int $owner_uid, int $profile_id = null) {
$cache_key = sprintf("%d/%d/%s", $owner_uid, $profile_id, $pref_name);
$this->cache[$cache_key] = $value;
}
private function _set(string $pref_name, $value, int $owner_uid, int $profile_id = null) {
if (!$profile_id) $profile_id = null; if (!$profile_id) $profile_id = null;
if ($profile_id && in_array($pref_name, self::_PROFILE_BLACKLIST)) if ($profile_id && in_array($pref_name, self::_PROFILE_BLACKLIST))
return false; return false;
if (isset(self::_DEFAULTS[$pref_name])) { if (isset(self::_DEFAULTS[$pref_name])) {
$this->_set_cache($pref_name, $value, $owner_uid, $profile_id);
$sth = $this->pdo->prepare("SELECT COUNT(pref_name) AS count FROM ttrss_user_prefs2 $sth = $this->pdo->prepare("SELECT COUNT(pref_name) AS count FROM ttrss_user_prefs2
WHERE pref_name = :name AND owner_uid = :uid AND WHERE pref_name = :name AND owner_uid = :uid AND
(profile = :profile OR (:profile IS NULL AND profile IS NULL))"); (profile = :profile OR (:profile IS NULL AND profile IS NULL))");
$sth->execute(["uid" => $owner_uid, "profile" => $profile_id, "name" => $pref_name ]); $sth->execute(["uid" => $owner_uid, "profile" => $profile_id, "name" => $pref_name ]);
if ($row = $sth->fetch()) { if ($row = $sth->fetch()) {
@ -216,6 +243,9 @@ class Prefs {
} }
function migrate(int $owner_uid, int $profile_id = null) { function migrate(int $owner_uid, int $profile_id = null) {
if (get_schema_version(true) < 141)
return;
if (!$profile_id) $profile_id = null; if (!$profile_id) $profile_id = null;
if (!$this->_get(Prefs::_PREFS_MIGRATED, $owner_uid, $profile_id)) { if (!$this->_get(Prefs::_PREFS_MIGRATED, $owner_uid, $profile_id)) {
@ -239,11 +269,9 @@ class Prefs {
$this->_set(Prefs::_PREFS_MIGRATED, "1", $owner_uid, $profile_id); $this->_set(Prefs::_PREFS_MIGRATED, "1", $owner_uid, $profile_id);
} }
die;
} }
static function initialize($owner_uid, int $profile_id = null) { static function initialize(int $owner_uid, int $profile_id = null) {
$instance = self::get_instance(); $instance = self::get_instance();
$instance->migrate($owner_uid, $profile_id); $instance->migrate($owner_uid, $profile_id);