]> git.mxchange.org Git - friendica.git/commitdiff
Move PConfig::set() to DI::pConfig()->set()
authornupplaPhil <admin@philipp.info>
Sat, 18 Jan 2020 15:54:50 +0000 (16:54 +0100)
committernupplaPhil <admin@philipp.info>
Sat, 18 Jan 2020 15:54:50 +0000 (16:54 +0100)
17 files changed:
doc/themes.md
mod/network.php
mod/ostatus_subscribe.php
mod/profiles.php
mod/settings.php
src/Core/PConfig.php
src/Core/UserImport.php
src/Model/FileTag.php
src/Module/Invite.php
src/Module/Register.php
src/Module/Settings/TwoFactor/Index.php
src/Module/Settings/TwoFactor/Verify.php
update.php
view/theme/duepuntozero/config.php
view/theme/frio/config.php
view/theme/quattro/config.php
view/theme/vier/config.php

index 94e66d48d6e75cc9650df4539bcef403fe6d994b..577d6a5da9edbdedaabb07b14d9745a1baf4bf0a 100644 (file)
@@ -129,7 +129,7 @@ The selected 1st part will be saved in the database by the theme_post function.
         // if the one specific submit button was pressed then proceed
         if (isset($_POST['duepuntozero-settings-submit'])){
             // and save the selection key into the personal config of the user
-            PConfig::set(local_user(), 'duepuntozero', 'colorset', $_POST['duepuntozero_colorset']);
+            DI::pConfig()->set(local_user(), 'duepuntozero', 'colorset', $_POST['duepuntozero_colorset']);
         }
     }
 
index 51c3ccd9f4ad5cd2c7592e30c7222606b25fefe4..28d849e7a1720027170d5f6d95efbaf2135d957e 100644 (file)
@@ -959,7 +959,7 @@ function network_tabs(App $a)
 
        // save selected tab, but only if not in file mode
        if (empty($_GET['file'])) {
-               PConfig::set(local_user(), 'network.view', 'tab.selected', [
+               DI::pConfig()->set(local_user(), 'network.view', 'tab.selected', [
                        $all_active, $post_active, $conv_active, $new_active, $starred_active, $bookmarked_active
                ]);
        }
index 63b95da4b1a6d29f94dce2354aa29d2e3252824f..c22572d974b07be01e35cb8804c2087bb1758b17 100644 (file)
@@ -50,7 +50,7 @@ function ostatus_subscribe_content(App $a)
                        return $o . L10n::t('Couldn\'t fetch friends for contact.');
                }
 
-               PConfig::set($uid, 'ostatus', 'legacy_friends', $curlResult->getBody());
+               DI::pConfig()->set($uid, 'ostatus', 'legacy_friends', $curlResult->getBody());
        }
 
        $friends = json_decode(DI::pConfig()->get($uid, 'ostatus', 'legacy_friends'));
index 56552049dc462132db188a9acd222f77ebd788be..9c394b33bc610876bd22ec3028a731049f7ca15f 100644 (file)
@@ -338,7 +338,7 @@ function profiles_post(App $a) {
 
                $hide_friends = (($_POST['hide-friends'] == 1) ? 1: 0);
 
-               PConfig::set(local_user(), 'system', 'detailled_profile', !empty($_POST['detailed_profile']) ? 1: 0);
+               DI::pConfig()->set(local_user(), 'system', 'detailled_profile', !empty($_POST['detailed_profile']) ? 1: 0);
 
                $changes = [];
                if ($is_default) {
index 976d7fa6d62a984291beee4d45bd2488b30caed7..2713b35df87fc658fb3d11fb4c58e1a22cf83da6 100644 (file)
@@ -233,13 +233,13 @@ function settings_post(App $a)
                BaseModule::checkFormSecurityTokenRedirectOnError('/settings/connectors', 'settings_connectors');
 
                if (!empty($_POST['general-submit'])) {
-                       PConfig::set(local_user(), 'system', 'accept_only_sharer', intval($_POST['accept_only_sharer']));
-                       PConfig::set(local_user(), 'system', 'disable_cw', intval($_POST['disable_cw']));
-                       PConfig::set(local_user(), 'system', 'no_intelligent_shortening', intval($_POST['no_intelligent_shortening']));
-                       PConfig::set(local_user(), 'system', 'attach_link_title', intval($_POST['attach_link_title']));
-                       PConfig::set(local_user(), 'system', 'ostatus_autofriend', intval($_POST['snautofollow']));
-                       PConfig::set(local_user(), 'ostatus', 'default_group', $_POST['group-selection']);
-                       PConfig::set(local_user(), 'ostatus', 'legacy_contact', $_POST['legacy_contact']);
+                       DI::pConfig()->set(local_user(), 'system', 'accept_only_sharer', intval($_POST['accept_only_sharer']));
+                       DI::pConfig()->set(local_user(), 'system', 'disable_cw', intval($_POST['disable_cw']));
+                       DI::pConfig()->set(local_user(), 'system', 'no_intelligent_shortening', intval($_POST['no_intelligent_shortening']));
+                       DI::pConfig()->set(local_user(), 'system', 'attach_link_title', intval($_POST['attach_link_title']));
+                       DI::pConfig()->set(local_user(), 'system', 'ostatus_autofriend', intval($_POST['snautofollow']));
+                       DI::pConfig()->set(local_user(), 'ostatus', 'default_group', $_POST['group-selection']);
+                       DI::pConfig()->set(local_user(), 'ostatus', 'legacy_contact', $_POST['legacy_contact']);
                } elseif (!empty($_POST['imap-submit'])) {
                        $mail_server       =                 $_POST['mail_server']       ?? '';
                        $mail_port         =                 $_POST['mail_port']         ?? '';
@@ -314,7 +314,7 @@ function settings_post(App $a)
                BaseModule::checkFormSecurityTokenRedirectOnError('/settings/features', 'settings_features');
                foreach ($_POST as $k => $v) {
                        if (strpos($k, 'feature_') === 0) {
-                               PConfig::set(local_user(), 'feature', substr($k, 8), ((intval($v)) ? 1 : 0));
+                               DI::pConfig()->set(local_user(), 'feature', substr($k, 8), ((intval($v)) ? 1 : 0));
                        }
                }
                info(L10n::t('Features updated') . EOL);
@@ -352,20 +352,20 @@ function settings_post(App $a)
                }
 
                if ($mobile_theme !== '') {
-                       PConfig::set(local_user(), 'system', 'mobile_theme', $mobile_theme);
+                       DI::pConfig()->set(local_user(), 'system', 'mobile_theme', $mobile_theme);
                }
 
-               PConfig::set(local_user(), 'system', 'nowarn_insecure'         , $nowarn_insecure);
-               PConfig::set(local_user(), 'system', 'update_interval'         , $browser_update);
-               PConfig::set(local_user(), 'system', 'itemspage_network'       , $itemspage_network);
-               PConfig::set(local_user(), 'system', 'itemspage_mobile_network', $itemspage_mobile_network);
-               PConfig::set(local_user(), 'system', 'no_smilies'              , $nosmile);
-               PConfig::set(local_user(), 'system', 'first_day_of_week'       , $first_day_of_week);
-               PConfig::set(local_user(), 'system', 'ignore_info'             , $noinfo);
-               PConfig::set(local_user(), 'system', 'infinite_scroll'         , $infinite_scroll);
-               PConfig::set(local_user(), 'system', 'no_auto_update'          , $no_auto_update);
-               PConfig::set(local_user(), 'system', 'bandwidth_saver'         , $bandwidth_saver);
-               PConfig::set(local_user(), 'system', 'no_smart_threading'      , $no_smart_threading);
+               DI::pConfig()->set(local_user(), 'system', 'nowarn_insecure'         , $nowarn_insecure);
+               DI::pConfig()->set(local_user(), 'system', 'update_interval'         , $browser_update);
+               DI::pConfig()->set(local_user(), 'system', 'itemspage_network'       , $itemspage_network);
+               DI::pConfig()->set(local_user(), 'system', 'itemspage_mobile_network', $itemspage_mobile_network);
+               DI::pConfig()->set(local_user(), 'system', 'no_smilies'              , $nosmile);
+               DI::pConfig()->set(local_user(), 'system', 'first_day_of_week'       , $first_day_of_week);
+               DI::pConfig()->set(local_user(), 'system', 'ignore_info'             , $noinfo);
+               DI::pConfig()->set(local_user(), 'system', 'infinite_scroll'         , $infinite_scroll);
+               DI::pConfig()->set(local_user(), 'system', 'no_auto_update'          , $no_auto_update);
+               DI::pConfig()->set(local_user(), 'system', 'bandwidth_saver'         , $bandwidth_saver);
+               DI::pConfig()->set(local_user(), 'system', 'no_smart_threading'      , $no_smart_threading);
 
                if (in_array($theme, Theme::getAllowedList())) {
                        if ($theme == $a->user['theme']) {
@@ -568,16 +568,16 @@ function settings_post(App $a)
        $str_group_deny    = !empty($_POST['group_deny'])    ? $aclFormatter->toString($_POST['group_deny'])    : '';
        $str_contact_deny  = !empty($_POST['contact_deny'])  ? $aclFormatter->toString($_POST['contact_deny'])  : '';
 
-       PConfig::set(local_user(), 'expire', 'items', $expire_items);
-       PConfig::set(local_user(), 'expire', 'notes', $expire_notes);
-       PConfig::set(local_user(), 'expire', 'starred', $expire_starred);
-       PConfig::set(local_user(), 'expire', 'photos', $expire_photos);
-       PConfig::set(local_user(), 'expire', 'network_only', $expire_network_only);
+       DI::pConfig()->set(local_user(), 'expire', 'items', $expire_items);
+       DI::pConfig()->set(local_user(), 'expire', 'notes', $expire_notes);
+       DI::pConfig()->set(local_user(), 'expire', 'starred', $expire_starred);
+       DI::pConfig()->set(local_user(), 'expire', 'photos', $expire_photos);
+       DI::pConfig()->set(local_user(), 'expire', 'network_only', $expire_network_only);
 
-       PConfig::set(local_user(), 'system', 'suggestme', $suggestme);
+       DI::pConfig()->set(local_user(), 'system', 'suggestme', $suggestme);
 
-       PConfig::set(local_user(), 'system', 'email_textonly', $email_textonly);
-       PConfig::set(local_user(), 'system', 'detailed_notif', $detailed_notif);
+       DI::pConfig()->set(local_user(), 'system', 'email_textonly', $email_textonly);
+       DI::pConfig()->set(local_user(), 'system', 'detailed_notif', $detailed_notif);
 
        if ($page_flags == User::PAGE_FLAGS_PRVGROUP) {
                $hidewall = 1;
index c096c178fa9512abe843fb3707638316476617a7..39a262d43d92a86c7f2632d4841ac94cd7e4dfee 100644 (file)
@@ -19,21 +19,6 @@ use Friendica\DI;
  */
 class PConfig
 {
-       /**
-        * @brief Sets a configuration value for a user
-        *
-        * @param int    $uid    The user_id
-        * @param string $cat    The category of the configuration value
-        * @param string $key    The configuration key to set
-        * @param mixed  $value  The value to store
-        *
-        * @return bool Operation success
-        */
-       public static function set(int $uid, string $cat, string $key, $value)
-       {
-               return DI::pConfig()->set($uid, $cat, $key, $value);
-       }
-
        /**
         * @brief Deletes the given key from the users's configuration.
         *
index fdacd2b2bf484bb8bb52128d0f906e549ee7dcf7..ae9bd944cee6fca17333703d664e1fe3a54d5863 100644 (file)
@@ -159,7 +159,7 @@ class UserImport
                }
                $newuid = self::lastInsertId();
 
-               PConfig::set($newuid, 'system', 'previous_addr', $old_handle);
+               DI::pConfig()->set($newuid, 'system', 'previous_addr', $old_handle);
 
                foreach ($account['profile'] as &$profile) {
                        foreach ($profile as $k => &$v) {
index eecaf5063a28fb67ec3336801cd549b9b091cac3..f52bda0adb62c7b5f52ccc431c1a0d1501c0eafc 100644 (file)
@@ -223,12 +223,12 @@ class FileTag
                        }
 
                        if ($saved != $filetags_updated) {
-                               PConfig::set($uid, 'system', 'filetags', $filetags_updated);
+                               DI::pConfig()->set($uid, 'system', 'filetags', $filetags_updated);
                        }
 
                        return true;
                } elseif (strlen($file_new)) {
-                       PConfig::set($uid, 'system', 'filetags', $file_new);
+                       DI::pConfig()->set($uid, 'system', 'filetags', $file_new);
                }
 
                return true;
@@ -260,7 +260,7 @@ class FileTag
                        $saved = DI::pConfig()->get($uid, 'system', 'filetags');
 
                        if (!strlen($saved) || !stristr($saved, '[' . self::encode($file) . ']')) {
-                               PConfig::set($uid, 'system', 'filetags', $saved . '[' . self::encode($file) . ']');
+                               DI::pConfig()->set($uid, 'system', 'filetags', $saved . '[' . self::encode($file) . ']');
                        }
 
                        info(L10n::t('Item filed'));
@@ -313,7 +313,7 @@ class FileTag
 
                if (!DBA::isResult($r)) {
                        $saved = DI::pConfig()->get($uid, 'system', 'filetags');
-                       PConfig::set($uid, 'system', 'filetags', str_replace($pattern, '', $saved));
+                       DI::pConfig()->set($uid, 'system', 'filetags', str_replace($pattern, '', $saved));
                }
 
                return true;
index b56aaf5ae672843cc02f4d33998be5f644c0fc0d..c84795abb014c3cd785a10be231b2e56b1524473 100644 (file)
@@ -69,7 +69,7 @@ class Invite extends BaseModule
                                if (!is_site_admin()) {
                                        $invites_remaining--;
                                        if ($invites_remaining >= 0) {
-                                               PConfig::set(local_user(), 'system', 'invites_remaining', $invites_remaining);
+                                               DI::pConfig()->set(local_user(), 'system', 'invites_remaining', $invites_remaining);
                                        } else {
                                                return;
                                        }
@@ -92,7 +92,7 @@ class Invite extends BaseModule
                        if ($res) {
                                $total++;
                                $current_invites++;
-                               PConfig::set(local_user(), 'system', 'sent_invites', $current_invites);
+                               DI::pConfig()->set(local_user(), 'system', 'sent_invites', $current_invites);
                                if ($current_invites > $max_invites) {
                                        notice(L10n::t('Invitation limit exceeded. Please contact your site administrator.') . EOL);
                                        return;
index 2ea4373b9b2526ff7793e08ed0137bf9d82b886b..e791bf393a4ff5fa40449510e4853f12309b0bfe 100644 (file)
@@ -293,7 +293,7 @@ class Register extends BaseModule
                if (intval(Config::get('config', 'register_policy')) === self::OPEN) {
                        if ($using_invites && $invite_id) {
                                Model\Register::deleteByHash($invite_id);
-                               PConfig::set($user['uid'], 'system', 'invites_remaining', $num_invites);
+                               DI::pConfig()->set($user['uid'], 'system', 'invites_remaining', $num_invites);
                        }
 
                        // Only send a password mail when the password wasn't manually provided
@@ -339,7 +339,7 @@ class Register extends BaseModule
                        // invite system
                        if ($using_invites && $invite_id) {
                                Model\Register::deleteByHash($invite_id);
-                               PConfig::set($user['uid'], 'system', 'invites_remaining', $num_invites);
+                               DI::pConfig()->set($user['uid'], 'system', 'invites_remaining', $num_invites);
                        }
 
                        // send email to admins
index 81140c1ac55d0bd3666fb2eeb243cb492ddfb576..187d6a15a99ba5a4e3f359ca4d6b34be2874a383 100644 (file)
@@ -37,7 +37,7 @@ class Index extends BaseSettingsModule
                                        if (!$has_secret && !$verified) {
                                                $Google2FA = new Google2FA();
 
-                                               PConfig::set(local_user(), '2fa', 'secret', $Google2FA->generateSecretKey(32));
+                                               DI::pConfig()->set(local_user(), '2fa', 'secret', $Google2FA->generateSecretKey(32));
 
                                                DI::baseUrl()->redirect('settings/2fa/recovery?t=' . self::getFormSecurityToken('settings_2fa_password'));
                                        }
index 9489522540b1400c5a9b1dd56fe2dc28f92a7fe0..6b3abe9a7ee3f889c8f0adc003660592d32d4223 100644 (file)
@@ -58,7 +58,7 @@ class Verify extends BaseSettingsModule
                        $valid = $google2fa->verifyKey(DI::pConfig()->get(local_user(), '2fa', 'secret'), $_POST['verify_code'] ?? '');
 
                        if ($valid) {
-                               PConfig::set(local_user(), '2fa', 'verified', true);
+                               DI::pConfig()->set(local_user(), '2fa', 'verified', true);
                                Session::set('2fa', true);
 
                                notice(L10n::t('Two-factor authentication successfully activated.'));
index 55fd82b2ed685e3bfebec526d4a0f804b6430073..b583a64b9f2b4a11c8ebb783cda544d6fe970fe8 100644 (file)
@@ -8,6 +8,7 @@ use Friendica\Core\PConfig;
 use Friendica\Core\Update;
 use Friendica\Core\Worker;
 use Friendica\Database\DBA;
+use Friendica\DI;
 use Friendica\Model\Contact;
 use Friendica\Model\GContact;
 use Friendica\Model\Item;
@@ -139,7 +140,7 @@ function update_1191()
 
                        if ($key === 'show_on_profile') {
                                if ($value) {
-                                       PConfig::set($uid, feature, forumlist_profile, $value);
+                                       DI::pConfig()->set($uid, feature, forumlist_profile, $value);
                                }
 
                                PConfig::delete($uid, $family, $key);
@@ -147,7 +148,7 @@ function update_1191()
 
                        if ($key === 'show_on_network') {
                                if ($value) {
-                                       PConfig::set($uid, feature, forumlist_widget, $value);
+                                       DI::pConfig()->set($uid, feature, forumlist_widget, $value);
                                }
 
                                PConfig::delete($uid, $family, $key);
index a5054dab91da2be327f27ee248cec2e6b9f75e67..188cbe5f79cd4fc123395d936f31622f9b8c809d 100644 (file)
@@ -29,7 +29,7 @@ function theme_post(App $a)
        }
 
        if (isset($_POST['duepuntozero-settings-submit'])) {
-               PConfig::set(local_user(), 'duepuntozero', 'colorset', $_POST['duepuntozero_colorset']);
+               DI::pConfig()->set(local_user(), 'duepuntozero', 'colorset', $_POST['duepuntozero_colorset']);
        }
 }
 
index 1ea2d99733d10c8100a0e113c0d5b6813998b4f8..8dcdb84edcc77c5c41219720be774fdeaece7886 100644 (file)
@@ -17,16 +17,16 @@ function theme_post(App $a)
        }
 
        if (isset($_POST['frio-settings-submit'])) {
-               PConfig::set(local_user(), 'frio', 'scheme',           $_POST['frio_scheme']           ?? '');
-               PConfig::set(local_user(), 'frio', 'nav_bg',           $_POST['frio_nav_bg']           ?? '');
-               PConfig::set(local_user(), 'frio', 'nav_icon_color',   $_POST['frio_nav_icon_color']   ?? '');
-               PConfig::set(local_user(), 'frio', 'link_color',       $_POST['frio_link_color']       ?? '');
-               PConfig::set(local_user(), 'frio', 'background_color', $_POST['frio_background_color'] ?? '');
-               PConfig::set(local_user(), 'frio', 'contentbg_transp', $_POST['frio_contentbg_transp'] ?? '');
-               PConfig::set(local_user(), 'frio', 'background_image', $_POST['frio_background_image'] ?? '');
-               PConfig::set(local_user(), 'frio', 'bg_image_option',  $_POST['frio_bg_image_option']  ?? '');
-               PConfig::set(local_user(), 'frio', 'css_modified',     time());
-               PConfig::set(local_user(), 'frio', 'enable_compose',   $_POST['frio_enable_compose']   ?? 0);
+               DI::pConfig()->set(local_user(), 'frio', 'scheme',           $_POST['frio_scheme']           ?? '');
+               DI::pConfig()->set(local_user(), 'frio', 'nav_bg',           $_POST['frio_nav_bg']           ?? '');
+               DI::pConfig()->set(local_user(), 'frio', 'nav_icon_color',   $_POST['frio_nav_icon_color']   ?? '');
+               DI::pConfig()->set(local_user(), 'frio', 'link_color',       $_POST['frio_link_color']       ?? '');
+               DI::pConfig()->set(local_user(), 'frio', 'background_color', $_POST['frio_background_color'] ?? '');
+               DI::pConfig()->set(local_user(), 'frio', 'contentbg_transp', $_POST['frio_contentbg_transp'] ?? '');
+               DI::pConfig()->set(local_user(), 'frio', 'background_image', $_POST['frio_background_image'] ?? '');
+               DI::pConfig()->set(local_user(), 'frio', 'bg_image_option',  $_POST['frio_bg_image_option']  ?? '');
+               DI::pConfig()->set(local_user(), 'frio', 'css_modified',     time());
+               DI::pConfig()->set(local_user(), 'frio', 'enable_compose',   $_POST['frio_enable_compose']   ?? 0);
        }
 }
 
index de67074a9808c866c224cc6671b19366173dd848..ce9354481f115e487f3d07ec250be0bd22ac13ba 100644 (file)
@@ -30,10 +30,10 @@ function theme_post(App $a) {
        }
 
        if (isset($_POST['quattro-settings-submit'])){
-               PConfig::set(local_user(), 'quattro', 'align', $_POST['quattro_align']);
-               PConfig::set(local_user(), 'quattro', 'color', $_POST['quattro_color']);
-               PConfig::set(local_user(), 'quattro', 'tfs', $_POST['quattro_tfs']);
-               PConfig::set(local_user(), 'quattro', 'pfs', $_POST['quattro_pfs']);
+               DI::pConfig()->set(local_user(), 'quattro', 'align', $_POST['quattro_align']);
+               DI::pConfig()->set(local_user(), 'quattro', 'color', $_POST['quattro_color']);
+               DI::pConfig()->set(local_user(), 'quattro', 'tfs', $_POST['quattro_tfs']);
+               DI::pConfig()->set(local_user(), 'quattro', 'pfs', $_POST['quattro_pfs']);
        }
 }
 
index 193b7246e49b5517c28a7e6f49be46ab4bac21de..106303b921aa67f3e103ec64a75f58e7644774a2 100644 (file)
@@ -51,13 +51,13 @@ function theme_post(App $a)
        }
 
        if (isset($_POST['vier-settings-submit'])) {
-               PConfig::set(local_user(), 'vier', 'style', $_POST['vier_style']);
-               PConfig::set(local_user(), 'vier', 'show_pages', $_POST['vier_show_pages']);
-               PConfig::set(local_user(), 'vier', 'show_profiles', $_POST['vier_show_profiles']);
-               PConfig::set(local_user(), 'vier', 'show_helpers', $_POST['vier_show_helpers']);
-               PConfig::set(local_user(), 'vier', 'show_services', $_POST['vier_show_services']);
-               PConfig::set(local_user(), 'vier', 'show_friends', $_POST['vier_show_friends']);
-               PConfig::set(local_user(), 'vier', 'show_lastusers', $_POST['vier_show_lastusers']);
+               DI::pConfig()->set(local_user(), 'vier', 'style', $_POST['vier_style']);
+               DI::pConfig()->set(local_user(), 'vier', 'show_pages', $_POST['vier_show_pages']);
+               DI::pConfig()->set(local_user(), 'vier', 'show_profiles', $_POST['vier_show_profiles']);
+               DI::pConfig()->set(local_user(), 'vier', 'show_helpers', $_POST['vier_show_helpers']);
+               DI::pConfig()->set(local_user(), 'vier', 'show_services', $_POST['vier_show_services']);
+               DI::pConfig()->set(local_user(), 'vier', 'show_friends', $_POST['vier_show_friends']);
+               DI::pConfig()->set(local_user(), 'vier', 'show_lastusers', $_POST['vier_show_lastusers']);
        }
 }