]> git.mxchange.org Git - friendica.git/blobdiff - mod/network.php
use config var in settings form
[friendica.git] / mod / network.php
index ba02ee5cc326dbf8dd88cf0a3b2a312f34f21b77..64f5cf505f64f3530f790713c54c79cd25e5f312 100644 (file)
@@ -5,6 +5,7 @@
  */
 
 use Friendica\App;
+use Friendica\BaseObject;
 use Friendica\Content\Feature;
 use Friendica\Content\ForumManager;
 use Friendica\Content\Nav;
@@ -12,7 +13,6 @@ use Friendica\Content\Pager;
 use Friendica\Content\Widget;
 use Friendica\Content\Text\HTML;
 use Friendica\Core\ACL;
-use Friendica\Core\Addon;
 use Friendica\Core\Config;
 use Friendica\Core\Hook;
 use Friendica\Core\L10n;
@@ -20,19 +20,18 @@ use Friendica\Core\Logger;
 use Friendica\Core\PConfig;
 use Friendica\Core\Protocol;
 use Friendica\Core\Renderer;
+use Friendica\Core\Session;
 use Friendica\Database\DBA;
 use Friendica\Model\Contact;
 use Friendica\Model\Group;
 use Friendica\Model\Item;
 use Friendica\Model\Profile;
+use Friendica\Model\Term;
 use Friendica\Module\Login;
 use Friendica\Util\DateTimeFormat;
 use Friendica\Util\Proxy as ProxyUtils;
 use Friendica\Util\Strings;
 
-require_once 'include/conversation.php';
-require_once 'include/items.php';
-
 function network_init(App $a)
 {
        if (!local_user()) {
@@ -42,36 +41,23 @@ function network_init(App $a)
 
        Hook::add('head', __FILE__, 'network_infinite_scroll_head');
 
-       $search = (x($_GET, 'search') ? Strings::escapeHtml($_GET['search']) : '');
-
-       if (($search != '') && !empty($_GET['submit'])) {
-               $a->internalRedirect('search?search=' . urlencode($search));
-       }
-
-       if (x($_GET, 'save')) {
-               $exists = DBA::exists('search', ['uid' => local_user(), 'term' => $search]);
-               if (!$exists) {
-                       DBA::insert('search', ['uid' => local_user(), 'term' => $search]);
-               }
-       }
-       if (x($_GET, 'remove')) {
-               DBA::delete('search', ['uid' => local_user(), 'term' => $search]);
-       }
-
        $is_a_date_query = false;
 
        $group_id = (($a->argc > 1 && is_numeric($a->argv[1])) ? intval($a->argv[1]) : 0);
 
        $cid = 0;
-       if (x($_GET, 'cid') && intval($_GET['cid']) != 0) {
+       if (!empty($_GET['cid'])) {
                $cid = $_GET['cid'];
-               $_GET['nets'] = 'all';
+               $_GET['nets'] = '';
                $group_id = 0;
        }
 
+       /** @var DateTimeFormat $dtFormat */
+       $dtFormat = BaseObject::getClass(DateTimeFormat::class);
+
        if ($a->argc > 1) {
                for ($x = 1; $x < $a->argc; $x ++) {
-                       if (is_a_date_arg($a->argv[$x])) {
+                       if ($dtFormat->isYearMonth($a->argv[$x])) {
                                $is_a_date_query = true;
                                break;
                        }
@@ -80,13 +66,11 @@ function network_init(App $a)
 
        // convert query string to array. remove friendica args
        $query_array = [];
-       $query_string = str_replace($a->cmd . '?', '', $a->query_string);
-       parse_str($query_string, $query_array);
-       array_shift($query_array);
+       parse_str(parse_url($a->query_string, PHP_URL_QUERY), $query_array);
 
        // fetch last used network view and redirect if needed
        if (!$is_a_date_query) {
-               $sel_nets = defaults($_GET, 'nets', false);
+               $sel_nets = $_GET['nets'] ?? '';
                $sel_tabs = network_query_get_sel_tab($a);
                $sel_groups = network_query_get_sel_group($a);
                $last_sel_tabs = PConfig::get(local_user(), 'network.view', 'tab.selected');
@@ -102,7 +86,7 @@ function network_init(App $a)
 
                if ($remember_tab) {
                        // redirect if current selected tab is '/network' and
-                       // last selected tab is _not_ '/network?f=&order=comment'.
+                       // last selected tab is _not_ '/network?order=comment'.
                        // and this isn't a date query
 
                        $tab_baseurls = [
@@ -114,12 +98,12 @@ function network_init(App $a)
                                '',     //bookmarked
                        ];
                        $tab_args = [
-                               'f=&order=comment', //all
-                               'f=&order=post',    //postord
-                               'f=&conv=1',        //conv
+                               'order=comment', //all
+                               'order=post',    //postord
+                               'conv=1',        //conv
                                '',                 //new
-                               'f=&star=1',        //starred
-                               'f=&bmark=1',       //bookmarked
+                               'star=1',        //starred
+                               'bmark=1',       //bookmarked
                        ];
 
                        $k = array_search('active', $last_sel_tabs);
@@ -137,13 +121,13 @@ function network_init(App $a)
                        }
                }
 
-               if ($sel_nets !== false) {
+               if ($sel_nets) {
                        $net_args['nets'] = $sel_nets;
                }
 
                if ($remember_tab) {
                        $net_args = array_merge($query_array, $net_args);
-                       $net_queries = build_querystring($net_args);
+                       $net_queries = http_build_query($net_args);
 
                        $redir_url = ($net_queries ? $net_baseurl . '?' . $net_queries : $net_baseurl);
 
@@ -151,82 +135,32 @@ function network_init(App $a)
                }
        }
 
-       // If nets is set to all, unset it
-       if (x($_GET, 'nets') && $_GET['nets'] === 'all') {
-               unset($_GET['nets']);
-       }
-
-       if (!x($a->page, 'aside')) {
+       if (empty($a->page['aside'])) {
                $a->page['aside'] = '';
        }
 
-       $a->page['aside'] .= (Feature::isEnabled(local_user(), 'groups') ?
-               Group::sidebarWidget('network/0', 'network', 'standard', $group_id) : '');
-       $a->page['aside'] .= (Feature::isEnabled(local_user(), 'forumlist_widget') ? ForumManager::widget(local_user(), $cid) : '');
-       $a->page['aside'] .= posted_date_widget('network', local_user(), false);
-       $a->page['aside'] .= Widget::networks('network', (x($_GET, 'nets') ? $_GET['nets'] : ''));
-       $a->page['aside'] .= saved_searches($search);
-       $a->page['aside'] .= Widget::fileAs('network', (x($_GET, 'file') ? $_GET['file'] : ''));
-}
-
-function saved_searches($search)
-{
-       if (!Feature::isEnabled(local_user(), 'savedsearch')) {
-               return '';
-       }
-
-       $a = get_app();
-
-       $srchurl = '/network?f='
-               . ((x($_GET, 'cid'))   ? '&cid='   . $_GET['cid']   : '')
-               . ((x($_GET, 'star'))  ? '&star='  . $_GET['star']  : '')
-               . ((x($_GET, 'bmark')) ? '&bmark=' . $_GET['bmark'] : '')
-               . ((x($_GET, 'conv'))  ? '&conv='  . $_GET['conv']  : '')
-               . ((x($_GET, 'nets'))  ? '&nets='  . $_GET['nets']  : '')
-               . ((x($_GET, 'cmin'))  ? '&cmin='  . $_GET['cmin']  : '')
-               . ((x($_GET, 'cmax'))  ? '&cmax='  . $_GET['cmax']  : '')
-               . ((x($_GET, 'file'))  ? '&file='  . $_GET['file']  : '');
-       ;
-
-       $o = '';
-
-       $terms = DBA::select('search', ['id', 'term'], ['uid' => local_user()]);
-       $saved = [];
-
-       while ($rr = DBA::fetch($terms)) {
-               $saved[] = [
-                       'id'          => $rr['id'],
-                       'term'        => $rr['term'],
-                       'encodedterm' => urlencode($rr['term']),
-                       'delete'      => L10n::t('Remove term'),
-                       'selected'    => ($search == $rr['term']),
-               ];
-       }
-
-       $tpl = Renderer::getMarkupTemplate('saved_searches_aside.tpl');
-       $o = Renderer::replaceMacros($tpl, [
-               '$title'     => L10n::t('Saved Searches'),
-               '$add'       => L10n::t('add'),
-               '$searchbox' => HTML::search($search, 'netsearch-box', $srchurl, true),
-               '$saved'     => $saved,
-       ]);
-
-       return $o;
+       $a->page['aside'] .= Group::sidebarWidget('network/0', 'network', 'standard', $group_id);
+       $a->page['aside'] .= ForumManager::widget(local_user(), $cid);
+       $a->page['aside'] .= Widget::postedByYear('network', local_user(), false);
+       $a->page['aside'] .= Widget::networks('network', $_GET['nets'] ?? '');
+       $a->page['aside'] .= Widget\SavedSearches::getHTML($a->query_string);
+       $a->page['aside'] .= Widget::fileAs('network', $_GET['file'] ?? '');
 }
 
 /**
  * Return selected tab from query
  *
  * urls -> returns
- *             '/network'                                      => $no_active = 'active'
- *             '/network?f=&order=comment'     => $comment_active = 'active'
- *             '/network?f=&order=post'        => $postord_active = 'active'
- *             '/network?f=&conv=1',           => $conv_active = 'active'
- *             '/network/new',                         => $new_active = 'active'
- *             '/network?f=&star=1',           => $starred_active = 'active'
- *             '/network?f=&bmark=1',          => $bookmarked_active = 'active'
+ *        '/network'                    => $no_active = 'active'
+ *        '/network?order=comment'    => $comment_active = 'active'
+ *        '/network?order=post'    => $postord_active = 'active'
+ *        '/network?conv=1',        => $conv_active = 'active'
+ *        '/network/new',                => $new_active = 'active'
+ *        '/network?star=1',        => $starred_active = 'active'
+ *        '/network?bmark=1',        => $bookmarked_active = 'active'
  *
- * @return Array ($no_active, $comment_active, $postord_active, $conv_active, $new_active, $starred_active, $bookmarked_active);
+ * @param App $a
+ * @return array ($no_active, $comment_active, $postord_active, $conv_active, $new_active, $starred_active, $bookmarked_active);
  */
 function network_query_get_sel_tab(App $a)
 {
@@ -242,15 +176,15 @@ function network_query_get_sel_tab(App $a)
                $new_active = 'active';
        }
 
-       if (x($_GET, 'star')) {
+       if (!empty($_GET['star'])) {
                $starred_active = 'active';
        }
 
-       if (x($_GET, 'bmark')) {
+       if (!empty($_GET['bmark'])) {
                $bookmarked_active = 'active';
        }
 
-       if (x($_GET, 'conv')) {
+       if (!empty($_GET['conv'])) {
                $conv_active = 'active';
        }
 
@@ -258,7 +192,7 @@ function network_query_get_sel_tab(App $a)
                $no_active = 'active';
        }
 
-       if ($no_active == 'active' && x($_GET, 'order')) {
+       if ($no_active == 'active' && !empty($_GET['order'])) {
                switch($_GET['order']) {
                        case 'post'    : $postord_active = 'active'; $no_active=''; break;
                        case 'comment' : $all_active     = 'active'; $no_active=''; break;
@@ -282,9 +216,11 @@ function network_query_get_sel_group(App $a)
 /**
  * @brief Sets the pager data and returns SQL
  *
- * @param App $a The global App
+ * @param App     $a      The global App
+ * @param Pager   $pager
  * @param integer $update Used for the automatic reloading
  * @return string SQL with the appropriate LIMIT clause
+ * @throws \Friendica\Network\HTTPException\InternalServerErrorException
  */
 function networkPager(App $a, Pager $pager, $update)
 {
@@ -318,6 +254,7 @@ function networkPager(App $a, Pager $pager, $update)
  * @brief Sets items as seen
  *
  * @param array $condition The array with the SQL condition
+ * @throws \Friendica\Network\HTTPException\InternalServerErrorException
  */
 function networkSetSeen($condition)
 {
@@ -328,7 +265,7 @@ function networkSetSeen($condition)
        $unseen = Item::exists($condition);
 
        if ($unseen) {
-               $r = Item::update(['unseen' => false], $condition);
+               Item::update(['unseen' => false], $condition);
        }
 }
 
@@ -337,9 +274,13 @@ function networkSetSeen($condition)
  *
  * @param App     $a      The global App
  * @param array   $items  Items of the conversation
+ * @param Pager   $pager
  * @param string  $mode   Display mode for the conversation
  * @param integer $update Used for the automatic reloading
+ * @param string  $ordering
  * @return string HTML of the conversation
+ * @throws ImagickException
+ * @throws \Friendica\Network\HTTPException\InternalServerErrorException
  */
 function networkConversation(App $a, $items, Pager $pager, $mode, $update, $ordering = '')
 {
@@ -372,7 +313,7 @@ function network_content(App $a, $update = 0, $parent = 0)
 
        /// @TODO Is this really necessary? $a is already available to hooks
        $arr = ['query' => $a->query_string];
-       Addon::callHooks('network_content_init', $arr);
+       Hook::callAll('network_content_init', $arr);
 
        $flat_mode = false;
 
@@ -404,10 +345,12 @@ function network_content(App $a, $update = 0, $parent = 0)
 /**
  * @brief Get the network content in flat view
  *
- * @param Pager   $pager
  * @param App     $a      The global App
  * @param integer $update Used for the automatic reloading
  * @return string HTML of the network content in flat view
+ * @throws ImagickException
+ * @throws \Friendica\Network\HTTPException\InternalServerErrorException
+ * @global Pager  $pager
  */
 function networkFlatView(App $a, $update = 0)
 {
@@ -415,15 +358,9 @@ function networkFlatView(App $a, $update = 0)
        // Rawmode is used for fetching new content at the end of the page
        $rawmode = (isset($_GET['mode']) && ($_GET['mode'] == 'raw'));
 
-       if (isset($_GET['last_id'])) {
-               $last_id = intval($_GET['last_id']);
-       } else {
-               $last_id = 0;
-       }
-
        $o = '';
 
-       $file = defaults($_GET, 'file', '');
+       $file = $_GET['file'] ?? '';
 
        if (!$update && !$rawmode) {
                $tabs = network_tabs($a);
@@ -457,14 +394,15 @@ function networkFlatView(App $a, $update = 0)
 
        $pager = new Pager($a->query_string);
 
-       /// @TODO Figure out why this variable is unused
-       $pager_sql = networkPager($a, $pager, $update);
+       networkPager($a, $pager, $update);
+
+       $item_params = ['order' => ['id' => true]];
 
        if (strlen($file)) {
-               $condition = ["`term` = ? AND `otype` = ? AND `type` = ? AND `uid` = ?",
-                       $file, TERM_OBJ_POST, TERM_FILE, local_user()];
-               $params = ['order' => ['tid' => true], 'limit' => [$pager->getStart(), $pager->getItemsPerPage()]];
-               $result = DBA::select('term', ['oid'], $condition);
+               $term_condition = ["`term` = ? AND `otype` = ? AND `type` = ? AND `uid` = ?",
+                       $file, Term::OBJECT_TYPE_POST, Term::FILE, local_user()];
+               $term_params = ['order' => ['tid' => true], 'limit' => [$pager->getStart(), $pager->getItemsPerPage()]];
+               $result = DBA::select('term', ['oid'], $term_condition, $term_params);
 
                $posts = [];
                while ($term = DBA::fetch($result)) {
@@ -475,18 +413,16 @@ function networkFlatView(App $a, $update = 0)
                if (count($posts) == 0) {
                        return '';
                }
-               $condition = ['uid' => local_user(), 'id' => $posts];
+               $item_condition = ['uid' => local_user(), 'id' => $posts];
        } else {
-               $condition = ['uid' => local_user()];
+               $item_condition = ['uid' => local_user()];
+               $item_params['limit'] = [$pager->getStart(), $pager->getItemsPerPage()];
+
+               networkSetSeen(['unseen' => true, 'uid' => local_user()]);
        }
 
-       $params = ['order' => ['id' => true], 'limit' => [$pager->getStart(), $pager->getItemsPerPage()]];
-       $result = Item::selectForUser(local_user(), [], $condition, $params);
+       $result = Item::selectForUser(local_user(), [], $item_condition, $item_params);
        $items = Item::inArray($result);
-
-       $condition = ['unseen' => true, 'uid' => local_user()];
-       networkSetSeen($condition);
-
        $o .= networkConversation($a, $items, $pager, 'network-new', $update);
 
        return $o;
@@ -495,11 +431,13 @@ function networkFlatView(App $a, $update = 0)
 /**
  * @brief Get the network content in threaded view
  *
- * @global Pager   $pager
  * @param  App     $a      The global App
  * @param  integer $update Used for the automatic reloading
  * @param  integer $parent
  * @return string HTML of the network content in flat view
+ * @throws ImagickException
+ * @throws \Friendica\Network\HTTPException\InternalServerErrorException
+ * @global Pager   $pager
  */
 function networkThreadedView(App $a, $update, $parent)
 {
@@ -527,9 +465,12 @@ function networkThreadedView(App $a, $update, $parent)
 
        $default_permissions = [];
 
+       /** @var DateTimeFormat $dtFormat */
+       $dtFormat = BaseObject::getClass(DateTimeFormat::class);
+
        if ($a->argc > 1) {
                for ($x = 1; $x < $a->argc; $x ++) {
-                       if (is_a_date_arg($a->argv[$x])) {
+                       if ($dtFormat->isYearMonth($a->argv[$x])) {
                                if ($datequery) {
                                        $datequery2 = Strings::escapeHtml($a->argv[$x]);
                                } else {
@@ -538,34 +479,42 @@ function networkThreadedView(App $a, $update, $parent)
                                }
                        } elseif (intval($a->argv[$x])) {
                                $gid = intval($a->argv[$x]);
-                               $default_permissions = ['allow_gid' => '<' . $gid . '>'];
+                               $default_permissions['allow_gid'] = [$gid];
                        }
                }
        }
 
        $o = '';
 
-       $cid   = intval(defaults($_GET, 'cid'  , 0));
-       $star  = intval(defaults($_GET, 'star' , 0));
-       $bmark = intval(defaults($_GET, 'bmark', 0));
-       $conv  = intval(defaults($_GET, 'conv' , 0));
-       $order = Strings::escapeTags(defaults($_GET, 'order', 'comment'));
-       $nets  =        defaults($_GET, 'nets' , '');
+       $cid   = intval($_GET['cid']   ?? 0);
+       $star  = intval($_GET['star']  ?? 0);
+       $bmark = intval($_GET['bmark'] ?? 0);
+       $conv  = intval($_GET['conv']  ?? 0);
+       $order = Strings::escapeTags(($_GET['order'] ?? '') ?: 'comment');
+       $nets  =        $_GET['nets']  ?? '';
 
+       $allowedCids = [];
        if ($cid) {
-               $default_permissions = ['allow_cid' => '<' . intval($cid) . '>'];
+               $allowedCids[] = (int) $cid;
+       } elseif ($nets) {
+               $condition = [
+                       'uid'     => local_user(),
+                       'network' => $nets,
+                       'self'    => false,
+                       'blocked' => false,
+                       'pending' => false,
+                       'archive' => false,
+                       'rel'     => [Contact::SHARING, Contact::FRIEND],
+               ];
+               $contactStmt = DBA::select('contact', ['id'], $condition);
+               while ($contact = DBA::fetch($contactStmt)) {
+                       $allowedCids[] = (int) $contact['id'];
+               }
+               DBA::close($contactStmt);
        }
 
-       if ($nets) {
-               $r = DBA::select('contact', ['id'], ['uid' => local_user(), 'network' => $nets], ['self' => false]);
-
-               $str = '';
-               while ($rr = DBA::fetch($r)) {
-                       $str .= '<' . $rr['id'] . '>';
-               }
-               if (strlen($str)) {
-                       $default_permissions = ['allow_cid' => $str];
-               }
+       if (count($allowedCids)) {
+               $default_permissions['allow_cid'] = $allowedCids;
        }
 
        if (!$update && !$rawmode) {
@@ -627,7 +576,6 @@ function networkThreadedView(App $a, $update, $parent)
        $sql_extra3 = '';
        $sql_table = '`thread`';
        $sql_parent = '`iid`';
-       $sql_order = '';
 
        if ($update) {
                $sql_table = '`item`';
@@ -642,14 +590,14 @@ function networkThreadedView(App $a, $update, $parent)
                $group = DBA::selectFirst('group', ['name'], ['id' => $gid, 'uid' => local_user()]);
                if (!DBA::isResult($group)) {
                        if ($update) {
-                               killme();
+                               exit();
                        }
                        notice(L10n::t('No such group') . EOL);
                        $a->internalRedirect('network/0');
                        // NOTREACHED
                }
 
-               $contacts = Group::expand([$gid]);
+               $contacts = Group::expand(local_user(), [$gid]);
 
                if ((is_array($contacts)) && count($contacts)) {
                        $contact_str_self = '';
@@ -681,8 +629,8 @@ function networkThreadedView(App $a, $update, $parent)
 
                        $entries[0] = [
                                'id' => 'network',
-                               'name' => htmlentities($contact['name']),
-                               'itemurl' => defaults($contact, 'addr', $contact['nurl']),
+                               'name' => $contact['name'],
+                               'itemurl' => ($contact['addr'] ?? '') ?: $contact['nurl'],
                                'thumb' => ProxyUtils::proxifyUrl($contact['thumb'], false, ProxyUtils::SIZE_THUMB),
                                'details' => $contact['location'],
                        ];
@@ -710,11 +658,11 @@ function networkThreadedView(App $a, $update, $parent)
        }
 
        if ($datequery) {
-               $sql_extra3 .= Strings::protectSprintf(sprintf(" AND $sql_table.created <= '%s' ",
+               $sql_extra3 .= Strings::protectSprintf(sprintf(" AND $sql_table.received <= '%s' ",
                                DBA::escape(DateTimeFormat::convert($datequery, 'UTC', date_default_timezone_get()))));
        }
        if ($datequery2) {
-               $sql_extra3 .= Strings::protectSprintf(sprintf(" AND $sql_table.created >= '%s' ",
+               $sql_extra3 .= Strings::protectSprintf(sprintf(" AND $sql_table.received >= '%s' ",
                                DBA::escape(DateTimeFormat::convert($datequery2, 'UTC', date_default_timezone_get()))));
        }
 
@@ -724,8 +672,8 @@ function networkThreadedView(App $a, $update, $parent)
 
        // Normal conversation view
        if ($order === 'post') {
-               $ordering = '`created`';
-               $order_mode = 'created';
+               $ordering = '`received`';
+               $order_mode = 'received';
        } else {
                $ordering = '`commented`';
                $order_mode = 'commented';
@@ -859,7 +807,7 @@ function networkThreadedView(App $a, $update, $parent)
                        ((time() - $_SESSION['network_last_date_timestamp']) < ($browser_update * 10))) {
                        $bottom_limit = $_SESSION['network_last_date'];
                }
-               $_SESSION['network_last_date'] = defaults($_SESSION, 'network_last_top_limit', $top_limit);
+               $_SESSION['network_last_date'] = Session::get('network_last_top_limit', $top_limit);
                $_SESSION['network_last_date_timestamp'] = time();
 
                if ($last_date > $top_limit) {
@@ -874,10 +822,10 @@ function networkThreadedView(App $a, $update, $parent)
                                (SELECT SUBSTR(`term`, 2) FROM `search` WHERE `uid` = ? AND `term` LIKE '#%') AND `otype` = ? AND `type` = ? AND `uid` = 0) AS `term`
                        ON `item`.`id` = `term`.`oid`
                        STRAIGHT_JOIN `contact` AS `author` ON `author`.`id` = `item`.`author-id`
-                       WHERE `item`.`uid` = 0 AND `item`.$ordering < ? AND `item`.$ordering > ?
+                       WHERE `item`.`uid` = 0 AND `item`.$ordering < ? AND `item`.$ordering > ? AND `item`.`gravity` = ?
                                AND NOT `author`.`hidden` AND NOT `author`.`blocked`" . $sql_tag_nets,
                        local_user(), TERM_OBJ_POST, TERM_HASHTAG,
-                       $top_limit, $bottom_limit);
+                       $top_limit, $bottom_limit, GRAVITY_PARENT);
 
                $data = DBA::toArray($items);
 
@@ -923,7 +871,7 @@ function networkThreadedView(App $a, $update, $parent)
                $parents_str = implode(', ', $parents_arr);
        }
 
-       if (x($_GET, 'offset')) {
+       if (!empty($_GET['offset'])) {
                $date_offset = $_GET['offset'];
        }
 
@@ -958,6 +906,7 @@ function networkThreadedView(App $a, $update, $parent)
  *
  * @param App $a The global App
  * @return string Html of the networktab
+ * @throws \Friendica\Network\HTTPException\InternalServerErrorException
  */
 function network_tabs(App $a)
 {
@@ -977,7 +926,7 @@ function network_tabs(App $a)
        $tabs = [
                [
                        'label' => L10n::t('Commented Order'),
-                       'url'   => str_replace('/new', '', $cmd) . '?f=&order=comment' . ((x($_GET,'cid')) ? '&cid=' . $_GET['cid'] : ''),
+                       'url'   => str_replace('/new', '', $cmd) . '?order=comment' . (!empty($_GET['cid']) ? '&cid=' . $_GET['cid'] : ''),
                        'sel'   => $all_active,
                        'title' => L10n::t('Sort by Comment Date'),
                        'id'    => 'commented-order-tab',
@@ -985,7 +934,7 @@ function network_tabs(App $a)
                ],
                [
                        'label' => L10n::t('Posted Order'),
-                       'url'   => str_replace('/new', '', $cmd) . '?f=&order=post' . ((x($_GET,'cid')) ? '&cid=' . $_GET['cid'] : ''),
+                       'url'   => str_replace('/new', '', $cmd) . '?order=post' . (!empty($_GET['cid']) ? '&cid=' . $_GET['cid'] : ''),
                        'sel'   => $postord_active,
                        'title' => L10n::t('Sort by Post Date'),
                        'id'    => 'posted-order-tab',
@@ -993,21 +942,19 @@ function network_tabs(App $a)
                ],
        ];
 
-       if (Feature::isEnabled(local_user(), 'personal_tab')) {
-               $tabs[] = [
-                       'label' => L10n::t('Personal'),
-                       'url'   => str_replace('/new', '', $cmd) . ((x($_GET,'cid')) ? '/?f=&cid=' . $_GET['cid'] : '/?f=') . '&conv=1',
-                       'sel'   => $conv_active,
-                       'title' => L10n::t('Posts that mention or involve you'),
-                       'id'    => 'personal-tab',
-                       'accesskey' => 'r',
-               ];
-       }
+       $tabs[] = [
+               'label' => L10n::t('Personal'),
+               'url'   => str_replace('/new', '', $cmd) . (!empty($_GET['cid']) ? '/?cid=' . $_GET['cid'] : '/?f=') . '&conv=1',
+               'sel'   => $conv_active,
+               'title' => L10n::t('Posts that mention or involve you'),
+               'id'    => 'personal-tab',
+               'accesskey' => 'r',
+       ];
 
        if (Feature::isEnabled(local_user(), 'new_tab')) {
                $tabs[] = [
                        'label' => L10n::t('New'),
-                       'url'   => 'network/new' . ((x($_GET,'cid')) ? '/?f=&cid=' . $_GET['cid'] : ''),
+                       'url'   => 'network/new' . (!empty($_GET['cid']) ? '/?cid=' . $_GET['cid'] : ''),
                        'sel'   => $new_active,
                        'title' => L10n::t('Activity Stream - by date'),
                        'id'    => 'activitiy-by-date-tab',
@@ -1018,7 +965,7 @@ function network_tabs(App $a)
        if (Feature::isEnabled(local_user(), 'link_tab')) {
                $tabs[] = [
                        'label' => L10n::t('Shared Links'),
-                       'url'   => str_replace('/new', '', $cmd) . ((x($_GET,'cid')) ? '/?f=&cid=' . $_GET['cid'] : '/?f=') . '&bmark=1',
+                       'url'   => str_replace('/new', '', $cmd) . (!empty($_GET['cid']) ? '/?cid=' . $_GET['cid'] : '/?f=') . '&bmark=1',
                        'sel'   => $bookmarked_active,
                        'title' => L10n::t('Interesting Links'),
                        'id'    => 'shared-links-tab',
@@ -1026,26 +973,24 @@ function network_tabs(App $a)
                ];
        }
 
-       if (Feature::isEnabled(local_user(), 'star_posts')) {
-               $tabs[] = [
-                       'label' => L10n::t('Starred'),
-                       'url'   => str_replace('/new', '', $cmd) . ((x($_GET,'cid')) ? '/?f=&cid=' . $_GET['cid'] : '/?f=') . '&star=1',
-                       'sel'   => $starred_active,
-                       'title' => L10n::t('Favourite Posts'),
-                       'id'    => 'starred-posts-tab',
-                       'accesskey' => 'm',
-               ];
-       }
+       $tabs[] = [
+               'label' => L10n::t('Starred'),
+               'url'   => str_replace('/new', '', $cmd) . (!empty($_GET['cid']) ? '/?cid=' . $_GET['cid'] : '/?f=') . '&star=1',
+               'sel'   => $starred_active,
+               'title' => L10n::t('Favourite Posts'),
+               'id'    => 'starred-posts-tab',
+               'accesskey' => 'm',
+       ];
 
        // save selected tab, but only if not in file mode
-       if (!x($_GET, 'file')) {
+       if (empty($_GET['file'])) {
                PConfig::set(local_user(), 'network.view', 'tab.selected', [
                        $all_active, $postord_active, $conv_active, $new_active, $starred_active, $bookmarked_active
                ]);
        }
 
        $arr = ['tabs' => $tabs];
-       Addon::callHooks('network_tabs', $arr);
+       Hook::callAll('network_tabs', $arr);
 
        $tpl = Renderer::getMarkupTemplate('common_tabs.tpl');
 
@@ -1061,17 +1006,21 @@ function network_tabs(App $a)
  * of the page to make the correct asynchronous call. This is obtained through the Pager that was instantiated in
  * networkThreadedView or networkFlatView.
  *
- * @global Pager  $pager
- * @param  App    $a
+ * @param App     $a
  * @param  string $htmlhead The head tag HTML string
+ * @throws \Friendica\Network\HTTPException\InternalServerErrorException
+ * @global Pager  $pager
  */
 function network_infinite_scroll_head(App $a, &$htmlhead)
 {
        /// @TODO this will have to be converted to a static property of the converted Module\Network class
+       /**
+        * @var $pager Pager
+        */
        global $pager;
 
        if (PConfig::get(local_user(), 'system', 'infinite_scroll')
-               && defaults($_GET, 'mode', '') != 'minimal'
+               && ($_GET['mode'] ?? '') != 'minimal'
        ) {
                $tpl = Renderer::getMarkupTemplate('infinite_scroll_head.tpl');
                $htmlhead .= Renderer::replaceMacros($tpl, [
@@ -1079,4 +1028,4 @@ function network_infinite_scroll_head(App $a, &$htmlhead)
                        '$reload_uri' => $pager->getBaseQueryString()
                ]);
        }
-}
\ No newline at end of file
+}