From b5c0df439fb13f110327c13e3f822244db9da855 Mon Sep 17 00:00:00 2001 From: nupplaPhil Date: Sat, 18 Jan 2020 16:56:46 +0100 Subject: [PATCH] Move PConfig::delete() to DI::pConfig()->delete() --- mod/ostatus_subscribe.php | 10 +++++----- src/Core/PConfig.php | 13 ------------- src/Module/Settings/TwoFactor/Index.php | 4 ++-- update.php | 6 +++--- 4 files changed, 10 insertions(+), 23 deletions(-) diff --git a/mod/ostatus_subscribe.php b/mod/ostatus_subscribe.php index c22572d974..0ab39e0c89 100644 --- a/mod/ostatus_subscribe.php +++ b/mod/ostatus_subscribe.php @@ -29,14 +29,14 @@ function ostatus_subscribe_content(App $a) if (DI::pConfig()->get($uid, 'ostatus', 'legacy_friends') == '') { if ($_REQUEST['url'] == '') { - PConfig::delete($uid, 'ostatus', 'legacy_contact'); + DI::pConfig()->delete($uid, 'ostatus', 'legacy_contact'); return $o . L10n::t('No contact provided.'); } $contact = Probe::uri($_REQUEST['url']); if (!$contact) { - PConfig::delete($uid, 'ostatus', 'legacy_contact'); + DI::pConfig()->delete($uid, 'ostatus', 'legacy_contact'); return $o . L10n::t('Couldn\'t fetch information for contact.'); } @@ -46,7 +46,7 @@ function ostatus_subscribe_content(App $a) $curlResult = Network::curl($api . 'statuses/friends.json?screen_name=' . $contact['nick']); if (!$curlResult->isSuccess()) { - PConfig::delete($uid, 'ostatus', 'legacy_contact'); + DI::pConfig()->delete($uid, 'ostatus', 'legacy_contact'); return $o . L10n::t('Couldn\'t fetch friends for contact.'); } @@ -63,8 +63,8 @@ function ostatus_subscribe_content(App $a) if ($counter >= $total) { DI::page()['htmlhead'] = ''; - PConfig::delete($uid, 'ostatus', 'legacy_friends'); - PConfig::delete($uid, 'ostatus', 'legacy_contact'); + DI::pConfig()->delete($uid, 'ostatus', 'legacy_friends'); + DI::pConfig()->delete($uid, 'ostatus', 'legacy_contact'); $o .= L10n::t('Done'); return $o; } diff --git a/src/Core/PConfig.php b/src/Core/PConfig.php index 39a262d43d..bb3334c4c9 100644 --- a/src/Core/PConfig.php +++ b/src/Core/PConfig.php @@ -19,17 +19,4 @@ use Friendica\DI; */ class PConfig { - /** - * @brief Deletes the given key from the users's configuration. - * - * @param int $uid The user_id - * @param string $cat The category of the configuration value - * @param string $key The configuration key to delete - * - * @return bool - */ - public static function delete(int $uid, string $cat, string $key) - { - return DI::pConfig()->delete($uid, $cat, $key); - } } diff --git a/src/Module/Settings/TwoFactor/Index.php b/src/Module/Settings/TwoFactor/Index.php index 187d6a15a9..26d40565f7 100644 --- a/src/Module/Settings/TwoFactor/Index.php +++ b/src/Module/Settings/TwoFactor/Index.php @@ -45,8 +45,8 @@ class Index extends BaseSettingsModule case 'disable': if ($has_secret) { RecoveryCode::deleteForUser(local_user()); - PConfig::delete(local_user(), '2fa', 'secret'); - PConfig::delete(local_user(), '2fa', 'verified'); + DI::pConfig()->delete(local_user(), '2fa', 'secret'); + DI::pConfig()->delete(local_user(), '2fa', 'verified'); Session::remove('2fa'); notice(L10n::t('Two-factor authentication successfully disabled.')); diff --git a/update.php b/update.php index b583a64b9f..0e69f90943 100644 --- a/update.php +++ b/update.php @@ -135,7 +135,7 @@ function update_1191() $value = $rr['v']; if ($key === 'randomise') { - PConfig::delete($uid, $family, $key); + DI::pConfig()->delete($uid, $family, $key); } if ($key === 'show_on_profile') { @@ -143,7 +143,7 @@ function update_1191() DI::pConfig()->set($uid, feature, forumlist_profile, $value); } - PConfig::delete($uid, $family, $key); + DI::pConfig()->delete($uid, $family, $key); } if ($key === 'show_on_network') { @@ -151,7 +151,7 @@ function update_1191() DI::pConfig()->set($uid, feature, forumlist_widget, $value); } - PConfig::delete($uid, $family, $key); + DI::pConfig()->delete($uid, $family, $key); } } } -- 2.39.5