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

Remove unused 'Prefs::_delete()'. 

Related to dabb85c7dd.
This commit is contained in:
wn_ 2023-03-05 15:30:09 +00:00
parent dabb85c7dd
commit 42b287e964
1 changed files with 0 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
classes/prefs.php
View File

@ -325,14 +325,6 @@ class Prefs {
return self::get_instance()->_set($pref_name, $value, $owner_uid, $profile_id); 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 * @param bool|int|string $value
*/ */
@ -350,10 +342,6 @@ class Prefs {
$value = Config::cast_to($value, $type_hint); $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)) if ($value == $this->_get($pref_name, $owner_uid, $profile_id))
return false; return false;