]> git.mxchange.org Git - friendica.git/commitdiff
Replace deprecated defaults() calls by ?? and ?: operators in src/Module/
authorHypolite Petovan <hypolite@mrpetovan.com>
Tue, 15 Oct 2019 13:20:32 +0000 (09:20 -0400)
committerHypolite Petovan <hypolite@mrpetovan.com>
Tue, 15 Oct 2019 13:20:32 +0000 (09:20 -0400)
40 files changed:
src/Module/Acctlink.php
src/Module/Admin/Addons/Details.php
src/Module/Admin/Addons/Index.php
src/Module/Admin/Blocklist/Contact.php
src/Module/Admin/Logs/Settings.php
src/Module/Admin/Site.php
src/Module/Admin/Themes/Index.php
src/Module/Admin/Users.php
src/Module/AllFriends.php
src/Module/Bookmarklet.php
src/Module/Contact.php
src/Module/Debug/Babel.php
src/Module/Debug/Feed.php
src/Module/Debug/Localtime.php
src/Module/Debug/Probe.php
src/Module/Debug/WebFinger.php
src/Module/Feed.php
src/Module/Filer/RemoveTag.php
src/Module/Filer/SaveTag.php
src/Module/FollowConfirm.php
src/Module/Followers.php
src/Module/Following.php
src/Module/Install.php
src/Module/Like.php
src/Module/Login.php
src/Module/Magic.php
src/Module/Outbox.php
src/Module/Profile.php
src/Module/Profile/Contacts.php
src/Module/Proxy.php
src/Module/Register.php
src/Module/Search/Acl.php
src/Module/Settings/TwoFactor/Index.php
src/Module/Settings/TwoFactor/Verify.php
src/Module/Smilies.php
src/Module/Special/HTTPException.php
src/Module/Starred.php
src/Module/ThemeDetails.php
src/Module/TwoFactor/Recovery.php
src/Module/Xrd.php

index 692e905914472081306002f19ef6ac74fb258b06..1c2500a224e6cc6dd0e2fcffa0200650cb8c5694 100644 (file)
@@ -13,10 +13,10 @@ class Acctlink extends BaseModule
 {
        public static function content()
        {
-               $addr = defaults($_GET, 'addr', false);
+               $addr = trim($_GET['addr'] ?? '');
 
                if ($addr) {
-                       $url = defaults(Probe::uri(trim($addr)), 'url', false);
+                       $url = Probe::uri($addr)['url'] ?? '';
 
                        if ($url) {
                                System::externalRedirect($url);
index 1fadf6768bbdf670062215c96df925cc87c18e22..1965102f03ef5fe4d8abac2420baf47a96d5d0c8 100644 (file)
@@ -53,7 +53,7 @@ class Details extends BaseAdminModule
                                $a->internalRedirect('admin/addons');
                        }
 
-                       if (defaults($_GET, 'action', '') == 'toggle') {
+                       if (($_GET['action'] ?? '') == 'toggle') {
                                parent::checkFormSecurityTokenRedirectOnError('/admin/addons', 'admin_themes', 't');
 
                                // Toggle addon status
index 2fa39d8e406da598dca8141a80791bdcf95ea16c..eed47defb4a973fc43581f080de1930040072848 100644 (file)
@@ -26,7 +26,7 @@ class Index extends BaseAdminModule
                                        break;
 
                                case 'toggle' :
-                                       $addon = defaults($_GET, 'addon', '');
+                                       $addon = $_GET['addon'] ?? '';
                                        if (Addon::isEnabled($addon)) {
                                                Addon::uninstall($addon);
                                                info(L10n::t('Addon %s disabled.', $addon));
index de3c717e37d06e93ba5d00ea81087a00729521ed..bf1c7bc081d86cfec1dacb12a2c681dbbeb130bb 100644 (file)
@@ -15,9 +15,9 @@ class Contact extends BaseAdminModule
        {
                parent::post();
 
-               $contact_url  = defaults($_POST, 'contact_url', '');
-               $block_reason = defaults($_POST, 'contact_block_reason', '');
-               $contacts     = defaults($_POST, 'contacts', []);
+               $contact_url  = $_POST['contact_url'] ?? '';
+               $block_reason = $_POST['contact_block_reason'] ?? '';
+               $contacts     = $_POST['contacts'] ?? [];
 
                parent::checkFormSecurityTokenRedirectOnError('/admin/blocklist/contact', 'admin_contactblock');
 
index b9a5e3832d87b3d3dd6027a0c9cb1442a1eb723d..be060e05359baf2012ffa540e62ef1a41ad918fe 100644 (file)
@@ -20,7 +20,7 @@ class Settings extends BaseAdminModule
 
                        $logfile   = (!empty($_POST['logfile']) ? Strings::escapeTags(trim($_POST['logfile'])) : '');
                        $debugging = !empty($_POST['debugging']);
-                       $loglevel  = defaults($_POST, 'loglevel', LogLevel::ERROR);
+                       $loglevel  = ($_POST['loglevel'] ?? '') ?: LogLevel::ERROR;
 
                        if (is_file($logfile) &&
                        !is_writeable($logfile)) {
index dff869df20ae2a7912c25ea611bc60a64fe2cd44..18a1cbf2a835f5d214e1731a478945c52672e492 100644 (file)
@@ -200,7 +200,7 @@ class Site extends BaseAdminModule
                /**
                 * @var $storagebackend \Friendica\Model\Storage\IStorage
                 */
-               $storagebackend    = Strings::escapeTags(trim(defaults($_POST, 'storagebackend', '')));
+               $storagebackend    = Strings::escapeTags(trim($_POST['storagebackend'] ?? ''));
 
                // save storage backend form
                if (!is_null($storagebackend) && $storagebackend != "") {
@@ -216,7 +216,7 @@ class Site extends BaseAdminModule
                                                        $value = !empty($_POST[$fieldname]);
                                                        break;
                                                default:
-                                                       $value = defaults($_POST, $fieldname, '');
+                                                       $value = $_POST[$fieldname] ?? '';
                                        }
                                        $storage_opts_data[$name] = $value;
                                }
index d29b4c33caea64b438d477887dc7ccb198fda47b..af2d1f28e11e0ae91e6b90ab79d2afaf93251c8c 100644 (file)
@@ -36,7 +36,7 @@ class Index extends BaseAdminModule
                                        break;
 
                                case 'toggle' :
-                                       $theme = defaults($_GET, 'addon', '');
+                                       $theme = $_GET['addon'] ?? '';
                                        if ($theme) {
                                                $theme = Strings::sanitizeFilePathItem($theme);
                                                if (!is_dir("view/theme/$theme")) {
index 92dddd9f73b3e323ea6138af8ecd1c227ab8c69f..a949c9331b51c37df41e8196850dcce11f115767 100644 (file)
@@ -21,11 +21,11 @@ class Users extends BaseAdminModule
 
                $a = self::getApp();
 
-               $pending     = defaults($_POST, 'pending'          , []);
-               $users       = defaults($_POST, 'user'             , []);
-               $nu_name     = defaults($_POST, 'new_user_name'    , '');
-               $nu_nickname = defaults($_POST, 'new_user_nickname', '');
-               $nu_email    = defaults($_POST, 'new_user_email'   , '');
+               $pending     = $_POST['pending']           ?? [];
+               $users       = $_POST['user']              ?? [];
+               $nu_name     = $_POST['new_user_name']     ?? '';
+               $nu_nickname = $_POST['new_user_nickname'] ?? '';
+               $nu_email    = $_POST['new_user_email']    ?? '';
                $nu_language = Config::get('system', 'language');
 
                parent::checkFormSecurityTokenRedirectOnError('/admin/users', 'admin_users');
index b0e6ea1490bcf00bc519be5061ef80a39a73631c..e5fbe697128d576374fd6fed2d9b2e75da3ee3ab 100644 (file)
@@ -78,7 +78,7 @@ class AllFriends extends BaseModule
 
                        $entry = [
                                'url'          => Model\Contact::magicLinkbyId($friend['id'], $friend['url']),
-                               'itemurl'      => defaults($contactDetails, 'addr', $friend['url']),
+                               'itemurl'      => ($contactDetails['addr'] ?? '') ?: $friend['url'],
                                'name'         => $contactDetails['name'],
                                'thumb'        => ProxyUtils::proxifyUrl($contactDetails['thumb'], false, ProxyUtils::SIZE_THUMB),
                                'img_hover'    => $contactDetails['name'],
index 283ab0117b40437838c89a0b90d8ea7b30557393..92130eeff1510d7adc5c5707db9525aaf2379c54 100644 (file)
@@ -27,7 +27,7 @@ class Bookmarklet extends BaseModule
                        return $output;
                }
 
-               $referer = Strings::normaliseLink(defaults($_SERVER, 'HTTP_REFERER', ''));
+               $referer = Strings::normaliseLink($_SERVER['HTTP_REFERER'] ?? '');
                $page = Strings::normaliseLink($app->getBaseURL() . "/bookmarklet");
 
                if (!strstr($referer, $page)) {
@@ -48,7 +48,7 @@ class Bookmarklet extends BaseModule
                                'bang'             => '',
                                'visitor'          => 'block',
                                'profile_uid'      => local_user(),
-                               'title'            => trim(defaults($_REQUEST, 'title', ''), '*'),
+                               'title'            => trim($_REQUEST['title'] ?? '', '*'),
                                'content'          => $content
                        ];
                        $output = status_editor($app, $x, 0, false);
index 0b27f0e4a37d1a20a3d222174778743fbd5f3897..c8bbbfe2e69a1d45b9467670375fd235d14e682e 100644 (file)
@@ -103,7 +103,7 @@ class Contact extends BaseModule
 
                Hook::callAll('contact_edit_post', $_POST);
 
-               $profile_id = intval(defaults($_POST, 'profile-assign', 0));
+               $profile_id = intval($_POST['profile-assign'] ?? 0);
                if ($profile_id) {
                        if (!DBA::exists('profile', ['id' => $profile_id, 'uid' => local_user()])) {
                                notice(L10n::t('Could not locate selected profile.') . EOL);
@@ -115,16 +115,16 @@ class Contact extends BaseModule
 
                $notify = !empty($_POST['notify']);
 
-               $fetch_further_information = intval(defaults($_POST, 'fetch_further_information', 0));
+               $fetch_further_information = intval($_POST['fetch_further_information'] ?? 0);
 
-               $ffi_keyword_blacklist = Strings::escapeHtml(trim(defaults($_POST, 'ffi_keyword_blacklist', '')));
+               $ffi_keyword_blacklist = Strings::escapeHtml(trim($_POST['ffi_keyword_blacklist'] ?? ''));
 
-               $priority = intval(defaults($_POST, 'poll', 0));
+               $priority = intval($_POST['poll'] ?? 0);
                if ($priority > 5 || $priority < 0) {
                        $priority = 0;
                }
 
-               $info = Strings::escapeHtml(trim(defaults($_POST, 'info', '')));
+               $info = Strings::escapeHtml(trim($_POST['info'] ?? ''));
 
                $r = DBA::update('contact', [
                        'profile-id' => $profile_id,
@@ -188,21 +188,42 @@ class Contact extends BaseModule
                Model\GContact::updateFromProbe($contact['url']);
        }
 
+       /**
+        * Toggles the blocked status of a contact identified by id.
+        *
+        * @param $contact_id
+        * @throws \Exception
+        */
        private static function blockContact($contact_id)
        {
                $blocked = !Model\Contact::isBlockedByUser($contact_id, local_user());
                Model\Contact::setBlockedForUser($contact_id, local_user(), $blocked);
        }
 
+       /**
+        * Toggles the ignored status of a contact identified by id.
+        *
+        * @param $contact_id
+        * @throws \Exception
+        */
        private static function ignoreContact($contact_id)
        {
                $ignored = !Model\Contact::isIgnoredByUser($contact_id, local_user());
                Model\Contact::setIgnoredForUser($contact_id, local_user(), $ignored);
        }
 
+       /**
+        * Toggles the archived status of a contact identified by id.
+        * If the current status isn't provided, this will always archive the contact.
+        *
+        * @param $contact_id
+        * @param $orig_record
+        * @return bool
+        * @throws \Exception
+        */
        private static function archiveContact($contact_id, $orig_record)
        {
-               $archived = (defaults($orig_record, 'archive', '') ? 0 : 1);
+               $archived = empty($orig_record['archive']);
                $r = DBA::update('contact', ['archive' => $archived], ['id' => $contact_id, 'uid' => local_user()]);
 
                return DBA::isResult($r);
@@ -227,8 +248,8 @@ class Contact extends BaseModule
 
                $a = self::getApp();
 
-               $nets = defaults($_GET, 'nets', '');
-               $rel  = defaults($_GET, 'rel' , '');
+               $nets = $_GET['nets'] ?? '';
+               $rel  = $_GET['rel']  ?? '';
 
                if (empty($a->page['aside'])) {
                        $a->page['aside'] = '';
@@ -290,7 +311,7 @@ class Contact extends BaseModule
                                '$name'         => $contact['name'],
                                '$photo'        => $contact['photo'],
                                '$url'          => Model\Contact::magicLinkByContact($contact, $contact['url']),
-                               '$addr'         => defaults($contact, 'addr', ''),
+                               '$addr'         => $contact['addr'] ?? '',
                                '$network_link' => $network_link,
                                '$network'      => L10n::t('Network:'),
                                '$account_type' => Model\Contact::getAccountType($contact),
@@ -626,7 +647,7 @@ class Contact extends BaseModule
                }
 
                // @TODO: Replace with parameter from router
-               $type = defaults($a->argv, 1, '');
+               $type = $a->argv[1] ?? '';
 
                switch ($type) {
                        case 'blocked':
@@ -651,9 +672,9 @@ class Contact extends BaseModule
 
                $sql_extra .= sprintf(" AND `network` != '%s' ", Protocol::PHANTOM);
 
-               $search = Strings::escapeTags(trim(defaults($_GET, 'search', '')));
-               $nets   = Strings::escapeTags(trim(defaults($_GET, 'nets'  , '')));
-               $rel    = Strings::escapeTags(trim(defaults($_GET, 'rel'   , '')));
+               $search = Strings::escapeTags(trim($_GET['search'] ?? ''));
+               $nets   = Strings::escapeTags(trim($_GET['nets']   ?? ''));
+               $rel    = Strings::escapeTags(trim($_GET['rel']    ?? ''));
 
                $tabs = [
                        [
@@ -1016,7 +1037,7 @@ class Contact extends BaseModule
                        'username'  => $rr['name'],
                        'account_type' => Model\Contact::getAccountType($rr),
                        'sparkle'   => $sparkle,
-                       'itemurl'   => defaults($rr, 'addr', $rr['url']),
+                       'itemurl'   => ($rr['addr'] ?? '') ?: $rr['url'],
                        'url'       => $url,
                        'network'   => ContactSelector::networkToName($rr['network'], $rr['url']),
                        'nick'      => $rr['nick'],
index 19e3ec2131282248455f0676c5d7ae2b8bfb1f3c..cf1f869552e6116b0c3bb097c97120bdafdb5851 100644 (file)
@@ -25,7 +25,7 @@ class Babel extends BaseModule
 
                $results = [];
                if (!empty($_REQUEST['text'])) {
-                       switch (defaults($_REQUEST, 'type', 'bbcode')) {
+                       switch (($_REQUEST['type'] ?? '') ?: 'bbcode') {
                                case 'bbcode':
                                        $bbcode = trim($_REQUEST['text']);
                                        $results[] = [
@@ -176,10 +176,10 @@ class Babel extends BaseModule
 
                $tpl = Renderer::getMarkupTemplate('babel.tpl');
                $o = Renderer::replaceMacros($tpl, [
-                       '$text'          => ['text', L10n::t('Source text'), defaults($_REQUEST, 'text', ''), ''],
-                       '$type_bbcode'   => ['type', L10n::t('BBCode'), 'bbcode', '', defaults($_REQUEST, 'type', 'bbcode') == 'bbcode'],
-                       '$type_markdown' => ['type', L10n::t('Markdown'), 'markdown', '', defaults($_REQUEST, 'type', 'bbcode') == 'markdown'],
-                       '$type_html'     => ['type', L10n::t('HTML'), 'html', '', defaults($_REQUEST, 'type', 'bbcode') == 'html'],
+                       '$text'          => ['text', L10n::t('Source text'), $_REQUEST['text'] ?? '', ''],
+                       '$type_bbcode'   => ['type', L10n::t('BBCode'), 'bbcode', '', (($_REQUEST['type'] ?? '') ?: 'bbcode') == 'bbcode'],
+                       '$type_markdown' => ['type', L10n::t('Markdown'), 'markdown', '', (($_REQUEST['type'] ?? '') ?: 'bbcode') == 'markdown'],
+                       '$type_html'     => ['type', L10n::t('HTML'), 'html', '', (($_REQUEST['type'] ?? '') ?: 'bbcode') == 'html'],
                        '$results'       => $results
                ]);
 
index a11df59b79f32ecfd07bdce37911a567845b1f1c..cc0be643b3030b858230b3248d8c9ed1af9f7e1a 100644 (file)
@@ -46,7 +46,7 @@ class Feed extends BaseModule
 
                $tpl = Renderer::getMarkupTemplate('feedtest.tpl');
                return Renderer::replaceMacros($tpl, [
-                       '$url'    => ['url', L10n::t('Source URL'), defaults($_REQUEST, 'url', ''), ''],
+                       '$url'    => ['url', L10n::t('Source URL'), $_REQUEST['url'] ?? '', ''],
                        '$result' => $result
                ]);
        }
index 7af9cb8dad2d56970cc96cd0990b433b4c72f9d1..197149837ae5f161f2e4d6caaf1b271b669b3921 100644 (file)
@@ -12,7 +12,7 @@ class Localtime extends BaseModule
 {
        public static function post()
        {
-               $time = defaults($_REQUEST, 'time', 'now');
+               $time = ($_REQUEST['time'] ?? '') ?: 'now';
 
                $bd_format = L10n::t('l F d, Y \@ g:i A');
 
@@ -25,7 +25,7 @@ class Localtime extends BaseModule
        {
                $app = self::getApp();
 
-               $time = defaults($_REQUEST, 'time', 'now');
+               $time = ($_REQUEST['time'] ?? '') ?: 'now';
 
                $output  = '<h3>' . L10n::t('Time Conversion') . '</h3>';
                $output .= '<p>' . L10n::t('Friendica provides this service for sharing events with other networks and friends in unknown timezones.') . '</p>';
@@ -41,7 +41,7 @@ class Localtime extends BaseModule
 
                $output .= '<form action ="' . $app->getBaseURL() . '/localtime?time=' . $time . '" method="post" >';
                $output .= '<p>' . L10n::t('Please select your timezone:') . '</p>';
-               $output .= Temporal::getTimezoneSelect(defaults($_REQUEST, 'timezone', Installer::DEFAULT_TZ));
+               $output .= Temporal::getTimezoneSelect(($_REQUEST['timezone'] ?? '') ?: Installer::DEFAULT_TZ);
                $output .= '<input type="submit" name="submit" value="' . L10n::t('Submit') . '" /></form>';
 
                return $output;
index f29f3b3bb79fe5ad24ffd1c72aed4f7c700dcadd..6762c5b82c99bc87bba154da10d283338ad5334e 100644 (file)
@@ -21,7 +21,7 @@ class Probe extends BaseModule
                        throw $e;
                }
 
-               $addr = defaults($_GET, 'addr', '');
+               $addr = $_GET['addr'] ?? '';
                $res  = '';
 
                if (!empty($addr)) {
index 2b0b9c53b494c54335627dcbad770b1f0cf11598..18cf4bb2a7386cbc86df382b8265a5097bf80d09 100644 (file)
@@ -20,7 +20,7 @@ class WebFinger extends BaseModule
                        throw $e;
                }
 
-               $addr = defaults($_GET, 'addr', '');
+               $addr = $_GET['addr'] ?? '';
                $res  = '';
 
                if (!empty($addr)) {
index 15b2026e2c246559668a4e30df495e8a811280d0..49ecfed96c166cd6d6676f77816ec1a100e77e43 100644 (file)
@@ -27,7 +27,7 @@ class Feed extends BaseModule
        {
                $a = self::getApp();
 
-               $last_update = defaults($_GET, 'last_update', '');
+               $last_update = $_GET['last_update'] ?? '';
                $nocache     = !empty($_GET['nocache']) && local_user();
 
                // @TODO: Replace with parameter from router
index 7e88b2e72eece491bc57e4ee2c1772f778727216..1dcc2e41e20a0f3a2a2b67bcc4d54c0e1f64bcb2 100644 (file)
@@ -23,8 +23,8 @@ class RemoveTag extends BaseModule
 
                $item_id = (($app->argc > 1) ? intval($app->argv[1]) : 0);
 
-               $term = XML::unescape(trim(defaults($_GET, 'term', '')));
-               $cat = XML::unescape(trim(defaults($_GET, 'cat', '')));
+               $term = XML::unescape(trim($_GET['term'] ?? ''));
+               $cat = XML::unescape(trim($_GET['cat'] ?? ''));
 
                $category = (($cat) ? true : false);
 
index 08ff1a32d04545c763f060aaef8f495e118b19f4..e8e311268149575843312fe9ed6e6bd88d22e386 100644 (file)
@@ -27,7 +27,7 @@ class SaveTag extends BaseModule
                $a = self::getApp();
                $logger = $a->getLogger();
 
-               $term = XML::unescape(trim(defaults($_GET, 'term', '')));
+               $term = XML::unescape(trim($_GET['term'] ?? ''));
                // @TODO: Replace with parameter from router
                $item_id = (($a->argc > 1) ? intval($a->argv[1]) : 0);
 
index 1921e748a8161dddb51ff33edb702935506ff5f8..f4f2a877c4f55fb1833f443ead4ebf0b304b3f44 100644 (file)
@@ -28,10 +28,10 @@ class FollowConfirm extends BaseModule
                        return;
                }
 
-               $intro_id = intval(defaults($_POST, 'intro_id'  , 0));
-               $duplex   = intval(defaults($_POST, 'duplex'    , 0));
-               $cid      = intval(defaults($_POST, 'contact_id', 0));
-               $hidden   = intval(defaults($_POST, 'hidden'    , 0));
+               $intro_id = intval($_POST['intro_id']   ?? 0);
+               $duplex   = intval($_POST['duplex']     ?? 0);
+               $cid      = intval($_POST['contact_id'] ?? 0);
+               $hidden   = intval($_POST['hidden']     ?? 0);
 
                if (empty($cid)) {
                        notice(L10n::t('No given contact.') . EOL);
index 79f34021b13243ebc883b01106cbf31b92d46a1c..5bd3fe0ce2ef238b165fa52999c491400ecf4bc8 100644 (file)
@@ -29,7 +29,7 @@ class Followers extends BaseModule
                        throw new \Friendica\Network\HTTPException\NotFoundException();
                }
 
-               $page = defaults($_REQUEST, 'page', null);
+               $page = $_REQUEST['page'] ?? null;
 
                $followers = ActivityPub\Transmitter::getFollowers($owner, $page);
 
index 3a68e7e0afbd9bb21852e878e2decbd718e6e0ad..5b5f4dc986d491d5c086f1f459ffef8bf5645e1e 100644 (file)
@@ -29,7 +29,7 @@ class Following extends BaseModule
                        throw new \Friendica\Network\HTTPException\NotFoundException();
                }
 
-               $page = defaults($_REQUEST, 'page', null);
+               $page = $_REQUEST['page'] ?? null;
 
                $Following = ActivityPub\Transmitter::getFollowing($owner, $page);
 
index 7ba40396659927f15df9f6b28b3759c33c2d4e13..39d6a062afd438cb47437da531660251e4117f14 100644 (file)
@@ -73,7 +73,7 @@ class Install extends BaseModule
                // so we may not have a css at all. Here we set a static css file for the install procedure pages
                Renderer::$theme['stylesheet'] = $a->getBaseURL() . '/view/install/style.css';
 
-               self::$currentWizardStep = defaults($_POST, 'pass', self::SYSTEM_CHECK);
+               self::$currentWizardStep = ($_POST['pass'] ?? '') ?: self::SYSTEM_CHECK;
        }
 
        public static function post()
@@ -345,8 +345,8 @@ class Install extends BaseModule
        {
                $configCache->set($cat, $key,
                        Strings::escapeTags(
-                               trim(defaults($post, sprintf('%s-%s', $cat, $key),
-                                               (!isset($default) ? $configCache->get($cat, $key) : $default))
+                               trim(($post[sprintf('%s-%s', $cat, $key)] ?? '') ?:
+                                               ($default ?? $configCache->get($cat, $key))
                                )
                        )
                );
index a43e38045c8c611213a81ba7b240ebd2a98603c9..cc450dd9d0d381e2f7c2df211eb04ff1cc727135 100644 (file)
@@ -36,7 +36,7 @@ class Like extends BaseModule
 
                // Decide how to return. If we were called with a 'return' argument,
                // then redirect back to the calling page. If not, just quietly end
-               $returnPath = defaults($_REQUEST, 'return', '');
+               $returnPath = $_REQUEST['return'] ?? '';
 
                if (!empty($returnPath)) {
                        $rand = '_=' . time();
index 8affd77557a3e9fe557024801be30330ee798d91..b67f48fb95daca5be645f7f3104ef6a12c0b277e 100644 (file)
@@ -53,7 +53,7 @@ class Login extends BaseModule
                        && (!empty($_POST['openid_url'])
                                || !empty($_POST['username']))
                ) {
-                       $openid_url = trim(defaults($_POST, 'openid_url', $_POST['username']));
+                       $openid_url = trim(($_POST['openid_url'] ?? '') ?: $_POST['username']);
 
                        self::openIdAuthentication($openid_url, !empty($_POST['remember']));
                }
index 4cb3dc7328052df1f9fe89da9cb603b24c753267..b04ea80c04bdd73f84ae90a4b515fa7f02c05dd5 100644 (file)
@@ -28,8 +28,8 @@ class Magic extends BaseModule
 
                Logger::log('args: ' . print_r($_REQUEST, true), Logger::DATA);
 
-               $addr = defaults($_REQUEST, 'addr', '');
-               $dest = defaults($_REQUEST, 'dest', '');
+               $addr = $_REQUEST['addr'] ?? '';
+               $dest = $_REQUEST['dest'] ?? '';
                $test = (!empty($_REQUEST['test']) ? intval($_REQUEST['test']) : 0);
                $owa  = (!empty($_REQUEST['owa'])  ? intval($_REQUEST['owa'])  : 0);
                $cid  = 0;
index 14825677910919e92a50833d6866bbeb1b16f5c9..4fc05076310ad2593fb6dff4d469b9c243eb6da1 100644 (file)
@@ -28,7 +28,7 @@ class Outbox extends BaseModule
                        throw new \Friendica\Network\HTTPException\NotFoundException();
                }
 
-               $page = defaults($_REQUEST, 'page', null);
+               $page = $_REQUEST['page'] ?? null;
 
                /// @todo Add Authentication to enable fetching of non public content
                // $requester = HTTPSignature::getSigner('', $_SERVER);
index cb710b10ba97bee752cc13cd3dcf7aab7156603c..ed37540753582fea049838a2139b5f285f0c88ad 100644 (file)
@@ -102,7 +102,7 @@ class Profile extends BaseModule
 
                        // site block
                        if (!$blocked && !$userblock) {
-                               $keywords = str_replace(['#', ',', ' ', ',,'], ['', ' ', ',', ','], defaults($a->profile, 'pub_keywords', ''));
+                               $keywords = str_replace(['#', ',', ' ', ',,'], ['', ' ', ',', ','], $a->profile['pub_keywords'] ?? '');
                                if (strlen($keywords)) {
                                        $a->page['htmlhead'] .= '<meta name="keywords" content="' . $keywords . '" />' . "\n";
                                }
@@ -146,10 +146,10 @@ class Profile extends BaseModule
                }
 
                if (empty($category)) {
-                       $category = defaults($_GET, 'category', '');
+                       $category = $_GET['category'] ?? '';
                }
 
-               $hashtags = defaults($_GET, 'tag', '');
+               $hashtags = $_GET['tag'] ?? '';
 
                if (Config::get('system', 'block_public') && !local_user() && !Session::getRemoteContactID($a->profile['profile_uid'])) {
                        return Login::form();
@@ -174,7 +174,7 @@ class Profile extends BaseModule
                }
 
                if (!$update) {
-            $tab = Strings::escapeTags(trim(defaults($_GET, 'tab', '')));
+            $tab = Strings::escapeTags(trim($_GET['tab'] ?? ''));
 
                        $o .= ProfileModel::getTabs($a, $tab, $is_owner, $a->profile['nickname']);
 
index ed41f421b12a8dec7c81de7d4458bbd08e9e5f04..1bf88d7c5b820b7da0741f0674d7d5740cc057e7 100644 (file)
@@ -28,7 +28,7 @@ class Contacts extends BaseModule
 
                //@TODO: Get value from router parameters
                $nickname = $a->argv[1];
-               $type = defaults($a->argv, 3, 'all');
+               $type = ($a->argv[3] ?? '') ?: 'all';
 
                Nav::setSelected('home');
 
index 38a9b17e014b631b718f0d29a7db4a5f30a931c4..2b8ad01ddadfd9cd89346ceec962350850845ff5 100644 (file)
@@ -210,7 +210,7 @@ class Proxy extends BaseModule
                        $url = base64_decode(strtr($url, '-_', '+/'), true);
 
                } else {
-                       $url = defaults($_REQUEST, 'url', '');
+                       $url = $_REQUEST['url'] ?? '';
                }
 
                return [
index 367eeb5adee31f84fa039afe5e65efb8055719d8..03f9dbb69871dd44fb666b778fabede9aa6a2cfd 100644 (file)
@@ -62,12 +62,12 @@ class Register extends BaseModule
                        }
                }
 
-               $username   = defaults($_REQUEST, 'username'  , '');
-               $email      = defaults($_REQUEST, 'email'     , '');
-               $openid_url = defaults($_REQUEST, 'openid_url', '');
-               $nickname   = defaults($_REQUEST, 'nickname'  , '');
-               $photo      = defaults($_REQUEST, 'photo'     , '');
-               $invite_id  = defaults($_REQUEST, 'invite_id' , '');
+               $username   = $_REQUEST['username']   ?? '';
+               $email      = $_REQUEST['email']      ?? '';
+               $openid_url = $_REQUEST['openid_url'] ?? '';
+               $nickname   = $_REQUEST['nickname']   ?? '';
+               $photo      = $_REQUEST['photo']      ?? '';
+               $invite_id  = $_REQUEST['invite_id']  ?? '';
 
                if (Config::get('system', 'no_openid')) {
                        $fillwith = '';
@@ -290,7 +290,7 @@ class Register extends BaseModule
                                        'source_photo' => $base_url . '/photo/avatar/' . $user['uid'] . '.jpg',
                                        'to_email'     => $admin['email'],
                                        'uid'          => $admin['uid'],
-                                       'language'     => defaults($admin, 'language', 'en'),
+                                       'language'     => ($admin['language'] ?? '') ?: 'en',
                                        'show_in_notification_page' => false
                                ]);
                        }
index 95ebd1cf74025c1bb31c75f617baa59b49fc2959..1016756008973291dca02bba042ac73454f950fe 100644 (file)
@@ -275,8 +275,8 @@ class Acl extends BaseModule
                                        'id'      => intval($g['id']),
                                        'network' => $g['network'],
                                        'link'    => $g['url'],
-                                       'nick'    => htmlentities(defaults($g, 'attag', $g['nick'])),
-                                       'addr'    => htmlentities(defaults($g, 'addr', $g['url'])),
+                                       'nick'    => htmlentities(($g['attag'] ?? '') ?: $g['nick']),
+                                       'addr'    => htmlentities(($g['addr'] ?? '') ?: $g['url']),
                                        'forum'   => !empty($g['forum']) || !empty($g['prv']) ? 1 : 0,
                                ];
                                if ($entry['forum']) {
@@ -336,8 +336,8 @@ class Acl extends BaseModule
                                                'id'      => intval($contact['cid']),
                                                'network' => $contact['network'],
                                                'link'    => $contact['url'],
-                                               'nick'    => htmlentities(defaults($contact, 'nick', $contact['addr'])),
-                                               'addr'    => htmlentities(defaults($contact, 'addr', $contact['url'])),
+                                               'nick'    => htmlentities(($contact['nick'] ?? '') ?: $contact['addr']),
+                                               'addr'    => htmlentities(($contact['addr'] ?? '') ?: $contact['url']),
                                                'forum'   => $contact['forum']
                                        ];
                                }
index 79b92f159291c2e94d3d434aaf5a33009fa460c2..e7694225c43f9bb3ef607e42bc7bf50d838c1a80 100644 (file)
@@ -26,12 +26,12 @@ class Index extends BaseSettingsModule
                self::checkFormSecurityTokenRedirectOnError('settings/2fa', 'settings_2fa');
 
                try {
-                       User::getIdFromPasswordAuthentication(local_user(), defaults($_POST, 'password', ''));
+                       User::getIdFromPasswordAuthentication(local_user(), $_POST['password'] ?? '');
 
                        $has_secret = (bool) PConfig::get(local_user(), '2fa', 'secret');
                        $verified = PConfig::get(local_user(), '2fa', 'verified');
 
-                       switch (defaults($_POST, 'action', '')) {
+                       switch ($_POST['action'] ?? '') {
                                case 'enable':
                                        if (!$has_secret && !$verified) {
                                                $Google2FA = new Google2FA();
index caed464ecea8200fd10cd3a9c2642961190f3b52..b9205852d835199170a6ac79c2a14d2c5d91ebcd 100644 (file)
@@ -49,12 +49,12 @@ class Verify extends BaseSettingsModule
                        return;
                }
 
-               if (defaults($_POST, 'action', null) == 'verify') {
+               if (($_POST['action'] ?? '') == 'verify') {
                        self::checkFormSecurityTokenRedirectOnError('settings/2fa/verify', 'settings_2fa_verify');
 
                        $google2fa = new Google2FA();
 
-                       $valid = $google2fa->verifyKey(PConfig::get(local_user(), '2fa', 'secret'), defaults($_POST, 'verify_code', ''));
+                       $valid = $google2fa->verifyKey(PConfig::get(local_user(), '2fa', 'secret'), $_POST['verify_code'] ?? '');
 
                        if ($valid) {
                                PConfig::set(local_user(), '2fa', 'verified', true);
index 422f37d3e54bbb40df037546c347a954120d60a5..ded58768fe347cf7f6913eb4a00c3c8e66637abf 100644 (file)
@@ -29,7 +29,7 @@ class Smilies extends BaseModule
        public static function content()
        {
                $smilies = Content\Smilies::getList();
-               $count = count(defaults($smilies, 'texts', []));
+               $count = count($smilies['texts'] ?? []);
 
                $tpl = Renderer::getMarkupTemplate('smilies.tpl');
                return Renderer::replaceMacros($tpl, [
index 6446ec38cb3bc764cbba57504204e787dd3a1486..36c770fba78d356493cdd809f2f4230d02c8c1c4 100644 (file)
@@ -36,7 +36,7 @@ class HTTPException
                        500 => L10n::t('Internal Server Error'),
                        503 => L10n::t('Service Unavailable'),
                ];
-               $title = defaults($titles, $e->getCode(), 'Error ' . $e->getCode());
+               $title = ($titles[$e->getCode()] ?? '') ?: 'Error ' . $e->getCode();
 
                if (empty($message)) {
                        // Explanations are taken from https://en.wikipedia.org/wiki/List_of_HTTP_status_codes
@@ -49,7 +49,7 @@ class HTTPException
                                503 => L10n::t('The server is currently unavailable (because it is overloaded or down for maintenance). Please try again later.'),
                        ];
 
-                       $message = defaults($explanation, $e->getCode(), '');
+                       $message = $explanation[$e->getCode()] ?? '';
                }
 
                return ['$title' => $title, '$message' => $message, '$back' => L10n::t('Go back')];
index df7da40992ddd88ee6bc989a1590a308298d922c..70cd3973518a81584604b2ef68439701a5fcfb7e 100644 (file)
@@ -41,7 +41,7 @@ class Starred extends BaseModule
                Item::update(['starred' => $starred], ['id' => $itemId]);
 
                // See if we've been passed a return path to redirect to
-               $returnPath = defaults($_REQUEST, 'return', '');
+               $returnPath = $_REQUEST['return'] ?? '';
                if ($returnPath) {
                        $rand = '_=' . time();
                        if (strpos($returnPath, '?')) {
index 7b53d1cfde928f89cbf9fa92b536af59d455db7b..9a2e913bcad00e7cd58144b226eb65ddbcc95f89 100644 (file)
@@ -17,9 +17,9 @@ class ThemeDetails extends BaseModule
                        $info = Theme::getInfo($theme);
 
                        // Unfortunately there will be no translation for this string
-                       $description = defaults($info, 'description', '');
-                       $version     = defaults($info, 'version'    , '');
-                       $credits     = defaults($info, 'credits'    , '');
+                       $description = $info['description'] ?? '';
+                       $version     = $info['version']     ?? '';
+                       $credits     = $info['credits']     ?? '';
 
                        echo json_encode([
                                'img'     => Theme::getScreenshot($theme),
index 60f443c35ffe88bc54cf0658c52625c8b394f40d..7c17fdace028893003c69bfd61057522b810d38e 100644 (file)
@@ -28,12 +28,12 @@ class Recovery extends BaseModule
                        return;
                }
 
-               if (defaults($_POST, 'action', null) == 'recover') {
+               if (($_POST['action'] ?? '') == 'recover') {
                        self::checkFormSecurityTokenRedirectOnError('2fa', 'twofactor_recovery');
 
                        $a = self::getApp();
 
-                       $recovery_code = defaults($_POST, 'recovery_code', '');
+                       $recovery_code = $_POST['recovery_code'] ?? '';
 
                        if (RecoveryCode::existsForUser(local_user(), $recovery_code)) {
                                RecoveryCode::markUsedForUser(local_user(), $recovery_code);
index ae020d35546add6ee2b72c823960c76dbea50b56..5e108c3b53bbd3ead765995e3ab6c5afcf2297e0 100644 (file)
@@ -28,7 +28,7 @@ class Xrd extends BaseModule
                        }
 
                        $uri = urldecode(Strings::escapeTags(trim($_GET['uri'])));
-                       if (defaults($_SERVER, 'HTTP_ACCEPT', '') == 'application/jrd+json') {
+                       if (($_SERVER['HTTP_ACCEPT'] ?? '') == 'application/jrd+json') {
                                $mode = 'json';
                        } else {
                                $mode = 'xml';
@@ -39,7 +39,7 @@ class Xrd extends BaseModule
                        }
 
                        $uri = urldecode(Strings::escapeTags(trim($_GET['resource'])));
-                       if (defaults($_SERVER, 'HTTP_ACCEPT', '') == 'application/xrd+xml') {
+                       if (($_SERVER['HTTP_ACCEPT'] ?? '') == 'application/xrd+xml') {
                                $mode = 'xml';
                        } else {
                                $mode = 'json';