diff options
author | fox <[email protected]> | 2023-03-07 16:50:13 +0300 |
---|---|---|
committer | fox <[email protected]> | 2023-03-07 16:50:13 +0300 |
commit | c4026b408b824769de6152d0c3bf0eb504a1e935 (patch) | |
tree | c81e07021256dba549c0771c230535192bac0004 /classes/prefs.php | |
parent | d210ae50ad14ded6cf204242bc072c82b9e8e70c (diff) | |
parent | 029cb8f4420a99aef225413f3c32ec1db35717a8 (diff) |
Merge pull request 'Bump PHPStan to 1.10.3, address some new warnings' (#103) from wn/tt-rss:feature/bump-phpstan into master
Reviewed-on: https://dev.tt-rss.org/tt-rss/tt-rss/pulls/103
Diffstat (limited to 'classes/prefs.php')
-rw-r--r-- | classes/prefs.php | 12 |
1 files changed, 0 insertions, 12 deletions
diff --git a/classes/prefs.php b/classes/prefs.php index 378fea293..9d68a14b7 100644 --- a/classes/prefs.php +++ b/classes/prefs.php @@ -325,14 +325,6 @@ class Prefs { return self::get_instance()->_set($pref_name, $value, $owner_uid, $profile_id); } - private function _delete(string $pref_name, int $owner_uid, ?int $profile_id): bool { - $sth = $this->pdo->prepare("DELETE FROM ttrss_user_prefs2 - WHERE pref_name = :name AND owner_uid = :uid AND - (profile = :profile OR (:profile IS NULL AND profile IS NULL))"); - - return $sth->execute(["uid" => $owner_uid, "profile" => $profile_id, "name" => $pref_name ]); - } - /** * @param bool|int|string $value */ @@ -350,10 +342,6 @@ class Prefs { $value = Config::cast_to($value, $type_hint); - // is this a good idea or not? probably not (user-set value remains user-set even if its at default) - //if ($value == $def_val) - // return $this->_delete($pref_name, $owner_uid, $profile_id); - if ($value == $this->_get($pref_name, $owner_uid, $profile_id)) return false; |