]> git.mxchange.org Git - friendica.git/blobdiff - mod/network.php
Some more avatar function replacements
[friendica.git] / mod / network.php
index e71bbe910e6743362d9818d8edd7768386bfa31e..eb1dbdb0437ac8f52e3f26646adff4da980bddc6 100644 (file)
@@ -1,41 +1,49 @@
 <?php
-
 /**
- * @file mod/network.php
+ * @copyright Copyright (C) 2020, Friendica
+ *
+ * @license GNU AGPL version 3 or any later version
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Affero General Public License as
+ * published by the Free Software Foundation, either version 3 of the
+ * License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU Affero General Public License for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public License
+ * along with this program.  If not, see <https://www.gnu.org/licenses/>.
+ *
  */
 
 use Friendica\App;
-use Friendica\Content\Feature;
 use Friendica\Content\ForumManager;
 use Friendica\Content\Nav;
 use Friendica\Content\Pager;
 use Friendica\Content\Widget;
 use Friendica\Content\Text\HTML;
 use Friendica\Core\ACL;
-use Friendica\Core\Config;
 use Friendica\Core\Hook;
-use Friendica\Core\L10n;
 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\DI;
 use Friendica\Model\Contact;
 use Friendica\Model\Group;
 use Friendica\Model\Item;
+use Friendica\Model\Post\Category;
 use Friendica\Model\Profile;
-use Friendica\Model\Term;
 use Friendica\Module\Security\Login;
 use Friendica\Util\DateTimeFormat;
-use Friendica\Util\Proxy as ProxyUtils;
 use Friendica\Util\Strings;
 
 function network_init(App $a)
 {
        if (!local_user()) {
-               notice(L10n::t('Permission denied.') . EOL);
+               notice(DI::l10n()->t('Permission denied.'));
                return;
        }
 
@@ -46,15 +54,15 @@ function network_init(App $a)
        $group_id = (($a->argc > 1 && is_numeric($a->argv[1])) ? intval($a->argv[1]) : 0);
 
        $cid = 0;
-       if (!empty($_GET['cid'])) {
-               $cid = $_GET['cid'];
+       if (!empty($_GET['contactid'])) {
+               $cid = $_GET['contactid'];
                $_GET['nets'] = '';
                $group_id = 0;
        }
 
        if ($a->argc > 1) {
                for ($x = 1; $x < $a->argc; $x ++) {
-                       if (DI::dtFormat()->isYearMonth($a->argv[$x])) {
+                       if (DI::dtFormat()->isYearMonthDay($a->argv[$x])) {
                                $is_a_date_query = true;
                                break;
                        }
@@ -70,7 +78,7 @@ function network_init(App $a)
                $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');
+               $last_sel_tabs = DI::pConfig()->get(local_user(), 'network.view', 'tab.selected');
 
                $remember_tab = ($sel_tabs[0] === 'active' && is_array($last_sel_tabs) && $last_sel_tabs[0] !== 'active');
 
@@ -86,28 +94,16 @@ function network_init(App $a)
                        // last selected tab is _not_ '/network?order=activity'.
                        // and this isn't a date query
 
-                       $tab_baseurls = [
-                               '',     //all
-                               '',     //postord
-                               '',     //conv
-                               '/new', //new
-                               '',     //starred
-                               '',     //bookmarked
-                       ];
                        $tab_args = [
                                'order=activity', //all
                                'order=post',     //postord
                                'conv=1',         //conv
-                               '',               //new
                                'star=1',         //starred
-                               'bmark=1',        //bookmarked
                        ];
 
                        $k = array_search('active', $last_sel_tabs);
 
                        if ($k != 3) {
-                               $net_baseurl .= $tab_baseurls[$k];
-
                                // parse out tab queries
                                $dest_qa = [];
                                $dest_qs = $tab_args[$k];
@@ -152,40 +148,28 @@ function network_init(App $a)
  *        '/network?order=activity' => $activity_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'
  *
  * @param App $a
- * @return array ($no_active, $activity_active, $postord_active, $conv_active, $new_active, $starred_active, $bookmarked_active);
+ * @return array ($no_active, $activity_active, $postord_active, $conv_active, $starred_active);
  */
 function network_query_get_sel_tab(App $a)
 {
        $no_active = '';
        $starred_active = '';
-       $new_active = '';
-       $bookmarked_active = '';
        $all_active = '';
        $conv_active = '';
        $postord_active = '';
 
-       if (($a->argc > 1 && $a->argv[1] === 'new') || ($a->argc > 2 && $a->argv[2] === 'new')) {
-               $new_active = 'active';
-       }
-
        if (!empty($_GET['star'])) {
                $starred_active = 'active';
        }
 
-       if (!empty($_GET['bmark'])) {
-               $bookmarked_active = 'active';
-       }
-
        if (!empty($_GET['conv'])) {
                $conv_active = 'active';
        }
 
-       if (($new_active == '') && ($starred_active == '') && ($bookmarked_active == '') && ($conv_active == '')) {
+       if (($starred_active == '') && ($conv_active == '')) {
                $no_active = 'active';
        }
 
@@ -196,7 +180,7 @@ function network_query_get_sel_tab(App $a)
                }
        }
 
-       return [$no_active, $all_active, $postord_active, $conv_active, $new_active, $starred_active, $bookmarked_active];
+       return [$no_active, $all_active, $postord_active, $conv_active, $starred_active];
 }
 
 function network_query_get_sel_group(App $a)
@@ -211,7 +195,7 @@ function network_query_get_sel_group(App $a)
 }
 
 /**
- * @brief Sets the pager data and returns SQL
+ * Sets the pager data and returns SQL
  *
  * @param App     $a      The global App
  * @param Pager   $pager
@@ -226,14 +210,12 @@ function networkPager(App $a, Pager $pager, $update)
                return ' LIMIT 100';
        }
 
-       //  check if we serve a mobile device and get the user settings
-       //  accordingly
        if (DI::mode()->isMobile()) {
-               $itemspage_network = PConfig::get(local_user(), 'system', 'itemspage_mobile_network');
-               $itemspage_network = ((intval($itemspage_network)) ? $itemspage_network : 20);
+               $itemspage_network = DI::pConfig()->get(local_user(), 'system', 'itemspage_mobile_network',
+                       DI::config()->get('system', 'itemspage_network_mobile'));
        } else {
-               $itemspage_network = PConfig::get(local_user(), 'system', 'itemspage_network');
-               $itemspage_network = ((intval($itemspage_network)) ? $itemspage_network : 40);
+               $itemspage_network = DI::pConfig()->get(local_user(), 'system', 'itemspage_network',
+                       DI::config()->get('system', 'itemspage_network'));
        }
 
        //  now that we have the user settings, see if the theme forces
@@ -248,7 +230,7 @@ function networkPager(App $a, Pager $pager, $update)
 }
 
 /**
- * @brief Sets items as seen
+ * Sets items as seen
  *
  * @param array $condition The array with the SQL condition
  * @throws \Friendica\Network\HTTPException\InternalServerErrorException
@@ -267,7 +249,7 @@ function networkSetSeen($condition)
 }
 
 /**
- * @brief Create the conversation HTML
+ * Create the conversation HTML
  *
  * @param App     $a      The global App
  * @param array   $items  Items of the conversation
@@ -285,14 +267,14 @@ function networkConversation(App $a, $items, Pager $pager, $mode, $update, $orde
        $a->page_contact = $a->contact;
 
        if (!is_array($items)) {
-               Logger::log("Expecting items to be an array. Got " . print_r($items, true));
+               Logger::info('Expecting items to be an array.', ['items' => $items]);
                $items = [];
        }
 
-       $o = conversation($a, $items, $pager, $mode, $update, false, $ordering, local_user());
+       $o = conversation($a, $items, $mode, $update, false, $ordering, local_user());
 
        if (!$update) {
-               if (PConfig::get(local_user(), 'system', 'infinite_scroll')) {
+               if (DI::pConfig()->get(local_user(), 'system', 'infinite_scroll')) {
                        $o .= HTML::scrollLoader();
                } else {
                        $o .= $pager->renderMinimal(count($items));
@@ -312,35 +294,21 @@ function network_content(App $a, $update = 0, $parent = 0)
        $arr = ['query' => DI::args()->getQueryString()];
        Hook::callAll('network_content_init', $arr);
 
-       $flat_mode = false;
-
-       if ($a->argc > 1) {
-               for ($x = 1; $x < $a->argc; $x ++) {
-                       if ($a->argv[$x] === 'new') {
-                               $flat_mode = true;
-                       }
-               }
-       }
-
        if (!empty($_GET['file'])) {
-               $flat_mode = true;
-       }
-
-       if ($flat_mode) {
                $o = networkFlatView($a, $update);
        } else {
                $o = networkThreadedView($a, $update, $parent);
        }
 
        if ($o === '') {
-               info("No items found");
+               notice(DI::l10n()->t("No items found"));
        }
 
        return $o;
 }
 
 /**
- * @brief Get the network content in flat view
+ * Get the network content in flat view
  *
  * @param App     $a      The global App
  * @param integer $update Used for the automatic reloading
@@ -383,35 +351,35 @@ function networkFlatView(App $a, $update = 0)
 
                $o .= status_editor($a, $x);
 
-               if (!Config::get('theme', 'hide_eventlist')) {
+               if (!DI::config()->get('theme', 'hide_eventlist')) {
                        $o .= Profile::getBirthdays();
                        $o .= Profile::getEventsReminderHTML();
                }
        }
 
-       $pager = new Pager(DI::args()->getQueryString());
+       $pager = new Pager(DI::l10n(), DI::args()->getQueryString());
 
        networkPager($a, $pager, $update);
 
-       $item_params = ['order' => ['id' => true]];
 
        if (strlen($file)) {
-               $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);
+               $item_params = ['order' => ['uri-id' => true]];
+               $term_condition = ['name' => $file, 'type' => Category::FILE, 'uid' => local_user()];
+               $term_params = ['order' => ['uri-id' => true], 'limit' => [$pager->getStart(), $pager->getItemsPerPage()]];
+               $result = DBA::select('category-view', ['uri-id'], $term_condition, $term_params);
 
                $posts = [];
                while ($term = DBA::fetch($result)) {
-                       $posts[] = $term['oid'];
+                       $posts[] = $term['uri-id'];
                }
                DBA::close($result);
 
                if (count($posts) == 0) {
                        return '';
                }
-               $item_condition = ['uid' => local_user(), 'id' => $posts];
+               $item_condition = ['uid' => local_user(), 'uri-id' => $posts];
        } else {
+               $item_params = ['order' => ['id' => true]];
                $item_condition = ['uid' => local_user()];
                $item_params['limit'] = [$pager->getStart(), $pager->getItemsPerPage()];
 
@@ -426,7 +394,7 @@ function networkFlatView(App $a, $update = 0)
 }
 
 /**
- * @brief Get the network content in threaded view
+ * Get the network content in threaded view
  *
  * @param  App     $a      The global App
  * @param  integer $update Used for the automatic reloading
@@ -464,7 +432,7 @@ function networkThreadedView(App $a, $update, $parent)
 
        if ($a->argc > 1) {
                for ($x = 1; $x < $a->argc; $x ++) {
-                       if (DI::dtFormat()->isYearMonth($a->argv[$x])) {
+                       if (DI::dtFormat()->isYearMonthDay($a->argv[$x])) {
                                if ($datequery) {
                                        $datequery2 = Strings::escapeHtml($a->argv[$x]);
                                } else {
@@ -480,12 +448,11 @@ function networkThreadedView(App $a, $update, $parent)
 
        $o = '';
 
-       $cid   = intval($_GET['cid']   ?? 0);
-       $star  = intval($_GET['star']  ?? 0);
-       $bmark = intval($_GET['bmark'] ?? 0);
-       $conv  = intval($_GET['conv']  ?? 0);
+       $cid   = intval($_GET['contactid'] ?? 0);
+       $star  = intval($_GET['star']      ?? 0);
+       $conv  = intval($_GET['conv']      ?? 0);
        $order = Strings::escapeTags(($_GET['order'] ?? '') ?: 'activity');
-       $nets  =        $_GET['nets']  ?? '';
+       $nets  =        $_GET['nets']      ?? '';
 
        $allowedCids = [];
        if ($cid) {
@@ -515,13 +482,6 @@ function networkThreadedView(App $a, $update, $parent)
                $tabs = network_tabs($a);
                $o .= $tabs;
 
-               if ($gid && ($t = Contact::getOStatusCountByGroupId($gid)) && !PConfig::get(local_user(), 'system', 'nowarn_insecure')) {
-                       notice(L10n::tt("Warning: This group contains %s member from a network that doesn't allow non public messages.",
-                               "Warning: This group contains %s members from a network that doesn't allow non public messages.",
-                               $t) . EOL);
-                       notice(L10n::t("Messages in this group won't be send to these receivers.").EOL);
-               }
-
                Nav::setSelected('network');
 
                $content = '';
@@ -564,7 +524,6 @@ function networkThreadedView(App $a, $update, $parent)
 
        $sql_post_table = '';
        $sql_options = ($star ? " AND `thread`.`starred` " : '');
-       $sql_options .= ($bmark ? sprintf(" AND `thread`.`post-type` = %d ", Item::PT_PAGE) : '');
        $sql_extra = $sql_options;
        $sql_extra2 = '';
        $sql_extra3 = '';
@@ -578,7 +537,6 @@ function networkThreadedView(App $a, $update, $parent)
        }
 
        $sql_nets = (($nets) ? sprintf(" AND $sql_table.`network` = '%s' ", DBA::escape($nets)) : '');
-       $sql_tag_nets = (($nets) ? sprintf(" AND `item`.`network` = '%s' ", DBA::escape($nets)) : '');
 
        if ($gid) {
                $group = DBA::selectFirst('group', ['name'], ['id' => $gid, 'uid' => local_user()]);
@@ -586,7 +544,7 @@ function networkThreadedView(App $a, $update, $parent)
                        if ($update) {
                                exit();
                        }
-                       notice(L10n::t('No such group') . EOL);
+                       notice(DI::l10n()->t('No such group'));
                        DI::baseUrl()->redirect('network/0');
                        // NOTREACHED
                }
@@ -607,14 +565,14 @@ function networkThreadedView(App $a, $update, $parent)
                        $sql_extra3 .= " OR (`thread`.`contact-id` = '$contact_str_self' AND `temp1`.`allow_gid` LIKE '" . Strings::protectSprintf('%<' . intval($gid) . '>%') . "' AND `temp1`.`private`))";
                } else {
                        $sql_extra3 .= " AND false ";
-                       info(L10n::t('Group is empty'));
+                       notice(DI::l10n()->t('Group is empty'));
                }
 
                $o = Renderer::replaceMacros(Renderer::getMarkupTemplate('section_title.tpl'), [
-                       '$title' => L10n::t('Group: %s', $group['name'])
+                       '$title' => DI::l10n()->t('Group: %s', $group['name'])
                ]) . $o;
        } elseif ($cid) {
-               $fields = ['id', 'name', 'network', 'writable', 'nurl',
+               $fields = ['id', 'name', 'network', 'writable', 'nurl', 'avatar',
                        'forum', 'prv', 'contact-type', 'addr', 'thumb', 'location'];
                $condition = ["`id` = ? AND (NOT `blocked` OR `pending`)", $cid];
                $contact = DBA::selectFirst('contact', $fields, $condition);
@@ -625,7 +583,7 @@ function networkThreadedView(App $a, $update, $parent)
                                'id' => 'network',
                                'name' => $contact['name'],
                                'itemurl' => ($contact['addr'] ?? '') ?: $contact['nurl'],
-                               'thumb' => ProxyUtils::proxifyUrl($contact['thumb'], false, ProxyUtils::SIZE_THUMB),
+                               'thumb' => Contact::getThumb($contact),
                                'details' => $contact['location'],
                        ];
 
@@ -635,18 +593,14 @@ function networkThreadedView(App $a, $update, $parent)
                                'contacts' => $entries,
                                'id' => 'network',
                        ]) . $o;
-
-                       if ($contact['network'] === Protocol::OSTATUS && $contact['writable'] && !PConfig::get(local_user(),'system','nowarn_insecure')) {
-                               notice(L10n::t('Private messages to this person are at risk of public disclosure.') . EOL);
-                       }
                } else {
-                       notice(L10n::t('Invalid contact.') . EOL);
+                       notice(DI::l10n()->t('Invalid contact.'));
                        DI::baseUrl()->redirect('network');
                        // NOTREACHED
                }
        }
 
-       if (!$gid && !$cid && !$update && !Config::get('theme', 'hide_eventlist')) {
+       if (!$gid && !$cid && !$update && !DI::config()->get('theme', 'hide_eventlist')) {
                $o .= Profile::getBirthdays();
                $o .= Profile::getEventsReminderHTML();
        }
@@ -681,7 +635,7 @@ function networkThreadedView(App $a, $update, $parent)
                $sql_range = '';
        }
 
-       $pager = new Pager(DI::args()->getQueryString());
+       $pager = new Pager(DI::l10n(), DI::args()->getQueryString());
 
        $pager_sql = networkPager($a, $pager, $update);
 
@@ -729,12 +683,12 @@ function networkThreadedView(App $a, $update, $parent)
                } else {
                        // Load all unseen items
                        $sql_extra4 = "`item`.`unseen`";
-                       if (Config::get("system", "like_no_comment")) {
+                       if (DI::config()->get("system", "like_no_comment")) {
                                $sql_extra4 .= " AND `item`.`gravity` IN (" . GRAVITY_PARENT . "," . GRAVITY_COMMENT . ")";
                        }
                        if ($order === 'post') {
                                // Only show toplevel posts when updating posts in this order mode
-                               $sql_extra4 .= " AND `item`.`id` = `item`.`parent`";
+                               $sql_extra4 .= " AND `item`.`gravity` = " . GRAVITY_PARENT;
                        }
                }
 
@@ -780,72 +734,6 @@ function networkThreadedView(App $a, $update, $parent)
                );
        }
 
-       // Only show it when unfiltered (no groups, no networks, ...)
-       if (in_array($nets, ['', Protocol::ACTIVITYPUB, Protocol::DFRN, Protocol::DIASPORA, Protocol::OSTATUS]) && (strlen($sql_extra . $sql_extra2 . $sql_extra3) == 0)) {
-               if (DBA::isResult($r)) {
-                       $top_limit = current($r)['order_date'];
-                       $bottom_limit = end($r)['order_date'];
-                       if (empty($_SESSION['network_last_top_limit']) || ($_SESSION['network_last_top_limit'] < $top_limit)) {
-                               $_SESSION['network_last_top_limit'] = $top_limit;
-                       }
-               } else {
-                       $top_limit = $bottom_limit = DateTimeFormat::utcNow();
-               }
-
-               // When checking for updates we need to fetch from the newest date to the newest date before
-               // Only do this, when the last stored date isn't too long ago (10 times the update interval)
-               $browser_update = PConfig::get(local_user(), 'system', 'update_interval', 40000) / 1000;
-
-               if (($browser_update > 0) && $update && !empty($_SESSION['network_last_date']) &&
-                       (($bottom_limit < $_SESSION['network_last_date']) || ($top_limit == $bottom_limit)) &&
-                       ((time() - $_SESSION['network_last_date_timestamp']) < ($browser_update * 10))) {
-                       $bottom_limit = $_SESSION['network_last_date'];
-               }
-               $_SESSION['network_last_date'] = Session::get('network_last_top_limit', $top_limit);
-               $_SESSION['network_last_date_timestamp'] = time();
-
-               if ($last_date > $top_limit) {
-                       $top_limit = $last_date;
-               } elseif ($pager->getPage() == 1) {
-                       // Highest possible top limit when we are on the first page
-                       $top_limit = DateTimeFormat::utcNow();
-               }
-
-               $items = DBA::p("SELECT `item`.`parent-uri` AS `uri`, 0 AS `item_id`, `item`.$ordering AS `order_date`, `author`.`url` AS `author-link` FROM `item`
-                       STRAIGHT_JOIN (SELECT `oid` FROM `term` WHERE `term` IN
-                               (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 > ? 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, GRAVITY_PARENT);
-
-               $data = DBA::toArray($items);
-
-               if (count($data) > 0) {
-                       $tag_top_limit = current($data)['order_date'];
-                       if ($_SESSION['network_last_date'] < $tag_top_limit) {
-                               $_SESSION['network_last_date'] = $tag_top_limit;
-                       }
-
-                       Logger::log('Tagged items: ' . count($data) . ' - ' . $bottom_limit . ' - ' . $top_limit . ' - ' . local_user().' - '.(int)$update);
-                       $s = [];
-                       foreach ($r as $item) {
-                               $s[$item['uri']] = $item;
-                       }
-                       foreach ($data as $item) {
-                               // Don't show hash tag posts from blocked or ignored contacts
-                               $condition = ["`nurl` = ? AND `uid` = ? AND (`blocked` OR `readonly`)",
-                                       Strings::normaliseLink($item['author-link']), local_user()];
-                               if (!DBA::exists('contact', $condition)) {
-                                       $s[$item['uri']] = $item;
-                               }
-                       }
-                       $r = $s;
-               }
-       }
-
        $parents_str = '';
        $date_offset = '';
 
@@ -896,7 +784,7 @@ function networkThreadedView(App $a, $update, $parent)
 }
 
 /**
- * @brief Get the network tabs menu
+ * Get the network tabs menu
  *
  * @param App $a The global App
  * @return string Html of the networktab
@@ -906,8 +794,8 @@ function network_tabs(App $a)
 {
        // item filter tabs
        /// @TODO fix this logic, reduce duplication
-       /// $a->page['content'] .= '<div class="tabs-wrapper">';
-       list($no_active, $all_active, $post_active, $conv_active, $new_active, $starred_active, $bookmarked_active) = network_query_get_sel_tab($a);
+       /// DI::page()['content'] .= '<div class="tabs-wrapper">';
+       list($no_active, $all_active, $post_active, $conv_active, $starred_active) = network_query_get_sel_tab($a);
 
        // if no tabs are selected, defaults to activitys
        if ($no_active == 'active') {
@@ -916,94 +804,53 @@ function network_tabs(App $a)
 
        $cmd = DI::args()->getCommand();
 
+       $def_param = [];
+       if (!empty($_GET['contactid'])) {
+               $def_param['contactid'] = $_GET['contactid'];
+       }
+
        // tabs
        $tabs = [
                [
-                       'label' => L10n::t('Latest Activity'),
-                       'url'   => str_replace('/new', '', $cmd) . '?order=activity' . (!empty($_GET['cid']) ? '&cid=' . $_GET['cid'] : ''),
+                       'label' => DI::l10n()->t('Latest Activity'),
+                       'url'   => $cmd . '?' . http_build_query(array_merge($def_param, ['order' => 'activity'])),
                        'sel'   => $all_active,
-                       'title' => L10n::t('Sort by latest activity'),
+                       'title' => DI::l10n()->t('Sort by latest activity'),
                        'id'    => 'activity-order-tab',
                        'accesskey' => 'e',
                ],
                [
-                       'label' => L10n::t('Latest Posts'),
-                       'url'   => str_replace('/new', '', $cmd) . '?order=post' . (!empty($_GET['cid']) ? '&cid=' . $_GET['cid'] : ''),
+                       'label' => DI::l10n()->t('Latest Posts'),
+                       'url'   => $cmd . '?' . http_build_query(array_merge($def_param, ['order' => 'post'])),
                        'sel'   => $post_active,
-                       'title' => L10n::t('Sort by post received date'),
+                       'title' => DI::l10n()->t('Sort by post received date'),
                        'id'    => 'post-order-tab',
                        'accesskey' => 't',
                ],
        ];
 
-       $parameters = ['conv' => true];
-       if (!empty($_GET['cid'])) {
-               $parameters['cid'] = $_GET['cid'];
-       }
-       $query = '?' . http_build_query($parameters);
-
        $tabs[] = [
-               'label' => L10n::t('Personal'),
-               'url'   => str_replace('/new', '', $cmd) . $query,
+               'label' => DI::l10n()->t('Personal'),
+               'url'   => $cmd . '?' . http_build_query(array_merge($def_param, ['conv' => true])),
                'sel'   => $conv_active,
-               'title' => L10n::t('Posts that mention or involve you'),
+               'title' => DI::l10n()->t('Posts that mention or involve you'),
                'id'    => 'personal-tab',
                'accesskey' => 'r',
        ];
 
-       if (Feature::isEnabled(local_user(), 'new_tab')) {
-               if (!empty($_GET['cid'])) {
-                       $query = '?' . http_build_query(['cid' => $_GET['cid']]);
-               } else {
-                       $query = '';
-               }
-
-               $tabs[] = [
-                       'label' => L10n::t('New'),
-                       'url'   => 'network/new' . $query,
-                       'sel'   => $new_active,
-                       'title' => L10n::t('Activity Stream - by date'),
-                       'id'    => 'activitiy-by-date-tab',
-                       'accesskey' => 'w',
-               ];
-       }
-
-       if (Feature::isEnabled(local_user(), 'link_tab')) {
-               $parameters = ['bmark' => true];
-               if (!empty($_GET['cid'])) {
-                       $parameters['cid'] = $_GET['cid'];
-               }
-               $query = '?' . http_build_query($parameters);
-
-               $tabs[] = [
-                       'label' => L10n::t('Shared Links'),
-                       'url'   => str_replace('/new', '', $cmd) . $query,
-                       'sel'   => $bookmarked_active,
-                       'title' => L10n::t('Interesting Links'),
-                       'id'    => 'shared-links-tab',
-                       'accesskey' => 'b',
-               ];
-       }
-
-       $parameters = ['tar' => true];
-       if (!empty($_GET['cid'])) {
-               $parameters['cid'] = $_GET['cid'];
-       }
-       $query = '?' . http_build_query($parameters);
-
        $tabs[] = [
-               'label' => L10n::t('Starred'),
-               'url'   => str_replace('/new', '', $cmd) . $query,
+               'label' => DI::l10n()->t('Starred'),
+               'url'   => $cmd . '?' . http_build_query(array_merge($def_param, ['star' => true])),
                'sel'   => $starred_active,
-               'title' => L10n::t('Favourite Posts'),
+               'title' => DI::l10n()->t('Favourite Posts'),
                'id'    => 'starred-posts-tab',
                'accesskey' => 'm',
        ];
 
        // save selected tab, but only if not in file mode
        if (empty($_GET['file'])) {
-               PConfig::set(local_user(), 'network.view', 'tab.selected', [
-                       $all_active, $post_active, $conv_active, $new_active, $starred_active, $bookmarked_active
+               DI::pConfig()->set(local_user(), 'network.view', 'tab.selected', [
+                       $all_active, $post_active, $conv_active, $starred_active
                ]);
        }
 
@@ -1037,7 +884,7 @@ function network_infinite_scroll_head(App $a, &$htmlhead)
         */
        global $pager;
 
-       if (PConfig::get(local_user(), 'system', 'infinite_scroll')
+       if (DI::pConfig()->get(local_user(), 'system', 'infinite_scroll')
                && ($_GET['mode'] ?? '') != 'minimal'
        ) {
                $tpl = Renderer::getMarkupTemplate('infinite_scroll_head.tpl');