]> git.mxchange.org Git - friendica.git/blobdiff - mod/network.php
Fixes:
[friendica.git] / mod / network.php
index f7cc167642d259375f6dea764dcf4a909bf675f6..79910152b3937e3f8cb05a379a0efcf26642ff1a 100644 (file)
@@ -9,12 +9,13 @@ use Friendica\Content\Feature;
 use Friendica\Content\ForumManager;
 use Friendica\Content\Nav;
 use Friendica\Content\Widget;
-use Friendica\Core\Acl;
+use Friendica\Core\ACL;
 use Friendica\Core\Addon;
 use Friendica\Core\Config;
 use Friendica\Core\L10n;
 use Friendica\Core\PConfig;
 use Friendica\Core\System;
+use Friendica\Database\DBA;
 use Friendica\Database\DBM;
 use Friendica\Model\Contact;
 use Friendica\Model\Group;
@@ -25,7 +26,6 @@ use Friendica\Util\DateTimeFormat;
 
 require_once 'include/conversation.php';
 require_once 'include/items.php';
-require_once 'include/acl_selectors.php';
 
 function network_init(App $a)
 {
@@ -41,13 +41,13 @@ function network_init(App $a)
        }
 
        if (x($_GET, 'save')) {
-               $exists = dba::exists('search', ['uid' => local_user(), 'term' => $search]);
+               $exists = DBA::exists('search', ['uid' => local_user(), 'term' => $search]);
                if (!$exists) {
-                       dba::insert('search', ['uid' => local_user(), 'term' => $search]);
+                       DBA::insert('search', ['uid' => local_user(), 'term' => $search]);
                }
        }
        if (x($_GET, 'remove')) {
-               dba::delete('search', ['uid' => local_user(), 'term' => $search]);
+               DBA::delete('search', ['uid' => local_user(), 'term' => $search]);
        }
 
        $is_a_date_query = false;
@@ -104,7 +104,6 @@ function network_init(App $a)
                                '/new', //new
                                '',     //starred
                                '',     //bookmarked
-                               '',     //spam
                        ];
                        $tab_args = [
                                'f=&order=comment', //all
@@ -113,7 +112,6 @@ function network_init(App $a)
                                '',                 //new
                                'f=&star=1',        //starred
                                'f=&bmark=1',       //bookmarked
-                               'f=&spam=1',        //spam
                        ];
 
                        $k = array_search('active', $last_sel_tabs);
@@ -184,10 +182,10 @@ function saved_searches($search)
 
        $o = '';
 
-       $terms = dba::select('search', ['id', 'term'], ['uid' => local_user()]);
+       $terms = DBA::select('search', ['id', 'term'], ['uid' => local_user()]);
        $saved = [];
 
-       while ($rr = dba::fetch($terms)) {
+       while ($rr = DBA::fetch($terms)) {
                $saved[] = [
                        'id'          => $rr['id'],
                        'term'        => $rr['term'],
@@ -219,9 +217,8 @@ function saved_searches($search)
  *             '/network/new',                         => $new_active = 'active'
  *             '/network?f=&star=1',           => $starred_active = 'active'
  *             '/network?f=&bmark=1',          => $bookmarked_active = 'active'
- *             '/network?f=&spam=1',           => $spam_active = 'active'
  *
- * @return Array ($no_active, $comment_active, $postord_active, $conv_active, $new_active, $starred_active, $bookmarked_active, $spam_active);
+ * @return Array ($no_active, $comment_active, $postord_active, $conv_active, $new_active, $starred_active, $bookmarked_active);
  */
 function network_query_get_sel_tab(App $a)
 {
@@ -231,7 +228,6 @@ function network_query_get_sel_tab(App $a)
        $bookmarked_active = '';
        $all_active = '';
        $conv_active = '';
-       $spam_active = '';
        $postord_active = '';
 
        if (($a->argc > 1 && $a->argv[1] === 'new') || ($a->argc > 2 && $a->argv[2] === 'new')) {
@@ -250,11 +246,7 @@ function network_query_get_sel_tab(App $a)
                $conv_active = 'active';
        }
 
-       if (x($_GET, 'spam')) {
-               $spam_active = 'active';
-       }
-
-       if (($new_active == '') && ($starred_active == '') && ($bookmarked_active == '') && ($conv_active == '') && ($spam_active == '')) {
+       if (($new_active == '') && ($starred_active == '') && ($bookmarked_active == '') && ($conv_active == '')) {
                $no_active = 'active';
        }
 
@@ -265,7 +257,7 @@ function network_query_get_sel_tab(App $a)
                }
        }
 
-       return [$no_active, $all_active, $postord_active, $conv_active, $new_active, $starred_active, $bookmarked_active, $spam_active];
+       return [$no_active, $all_active, $postord_active, $conv_active, $new_active, $starred_active, $bookmarked_active];
 }
 
 function network_query_get_sel_group(App $a)
@@ -325,7 +317,7 @@ function networkSetSeen($condition)
                return;
        }
 
-       $unseen = dba::exists('item', $condition);
+       $unseen = DBA::exists('item', $condition);
 
        if ($unseen) {
                $r = Item::update(['unseen' => false], $condition);
@@ -346,7 +338,7 @@ function networkConversation($a, $items, $mode, $update, $ordering = '')
        // Set this so that the conversation function can find out contact info for our wall-wall items
        $a->page_contact = $a->contact;
 
-       $o = conversation($a, $items, $mode, $update, false, $ordering);
+       $o = conversation($a, $items, $mode, $update, false, $ordering, local_user());
 
        if (!$update) {
                if (PConfig::get(local_user(), 'system', 'infinite_scroll')) {
@@ -369,21 +361,21 @@ function network_content(App $a, $update = 0, $parent = 0)
        $arr = ['query' => $a->query_string];
        Addon::callHooks('network_content_init', $arr);
 
-       $nouveau = false;
+       $flat_mode = false;
 
        if ($a->argc > 1) {
                for ($x = 1; $x < $a->argc; $x ++) {
                        if ($a->argv[$x] === 'new') {
-                               $nouveau = true;
+                               $flat_mode = true;
                        }
                }
        }
 
        if (x($_GET, 'file')) {
-               $nouveau = true;
+               $flat_mode = true;
        }
 
-       if ($nouveau) {
+       if ($flat_mode) {
                $o = networkFlatView($a, $update);
        } else {
                $o = networkThreadedView($a, $update, $parent);
@@ -402,7 +394,7 @@ function network_content(App $a, $update = 0, $parent = 0)
 function networkFlatView(App $a, $update = 0)
 {
        // Rawmode is used for fetching new content at the end of the page
-       $rawmode = (isset($_GET['mode']) AND ( $_GET['mode'] == 'raw'));
+       $rawmode = (isset($_GET['mode']) && ($_GET['mode'] == 'raw'));
 
        if (isset($_GET['last_id'])) {
                $last_id = intval($_GET['last_id']);
@@ -428,8 +420,8 @@ function networkFlatView(App $a, $update = 0)
                        'lockstate' => (((is_array($a->user) &&
                        ((strlen($a->user['allow_cid'])) || (strlen($a->user['allow_gid'])) ||
                        (strlen($a->user['deny_cid'])) || (strlen($a->user['deny_gid']))))) ? 'lock' : 'unlock'),
-                       'default_perms' => Acl::getDefaultUserPermissions($a->user),
-                       'acl' => Acl::getFullSelectorHTML($a->user, true),
+                       'default_perms' => ACL::getDefaultUserPermissions($a->user),
+                       'acl' => ACL::getFullSelectorHTML($a->user, true),
                        'bang' => '',
                        'visitor' => 'block',
                        'profile_uid' => local_user(),
@@ -444,28 +436,33 @@ function networkFlatView(App $a, $update = 0)
                }
        }
 
+       $pager_sql = networkPager($a, $update);
+
        if (strlen($file)) {
-               $sql_post_table = sprintf("INNER JOIN (SELECT `oid` FROM `term` WHERE `term` = '%s' AND `otype` = %d AND `type` = %d AND `uid` = %d ORDER BY `tid` DESC) AS `term` ON `item`.`id` = `term`.`oid` ",
-                       dbesc(protect_sprintf($file)), intval(TERM_OBJ_POST), intval(TERM_FILE), intval(local_user()));
+               $condition = ["`term` = ? AND `otype` = ? AND `type` = ? AND `uid` = ?",
+                       $file, TERM_OBJ_POST, TERM_FILE, local_user()];
+               $params = ['order' => ['tid' => true], 'limit' => [$a->pager['start'], $a->pager['itemspage']]];
+               $result = DBA::select('term', ['oid'], $condition);
+
+               $posts = [];
+               while ($term = DBA::fetch($result)) {
+                       $posts[] = $term['oid'];
+               }
+               DBA::close($terms);
+
+               $condition = ['uid' => local_user(), 'id' => $posts];
        } else {
-               $sql_post_table = " INNER JOIN `thread` ON `thread`.`iid` = `item`.`parent`";
+               $condition = ['uid' => local_user()];
        }
 
-       $pager_sql = networkPager($a, $update);
-
-       // show all items unthreaded in reverse created date order
-       $items = q("SELECT %s FROM `item` $sql_post_table %s
-               WHERE %s AND `item`.`uid` = %d
-               ORDER BY `item`.`id` DESC $pager_sql ",
-               item_fieldlists(), item_joins(), item_condition(),
-               intval($_SESSION['uid'])
-       );
+       $params = ['order' => ['id' => true], 'limit' => [$a->pager['start'], $a->pager['itemspage']]];
+       $result = Item::selectForUser(local_user(), [], $condition, $params);
+       $items = Item::inArray($result);
 
        $condition = ['unseen' => true, 'uid' => local_user()];
        networkSetSeen($condition);
 
-       $mode = 'network-new';
-       $o .= networkConversation($a, $items, $mode, $update);
+       $o .= networkConversation($a, $items, 'network-new', $update);
 
        return $o;
 }
@@ -528,10 +525,10 @@ function networkThreadedView(App $a, $update, $parent)
        }
 
        if ($nets) {
-               $r = dba::select('contact', ['id'], ['uid' => local_user(), 'network' => $nets], ['self' => false]);
+               $r = DBA::select('contact', ['id'], ['uid' => local_user(), 'network' => $nets], ['self' => false]);
 
                $str = '';
-               while ($rr = dba::fetch($r)) {
+               while ($rr = DBA::fetch($r)) {
                        $str .= '<' . $rr['id'] . '>';
                }
                if (strlen($str)) {
@@ -559,7 +556,7 @@ function networkThreadedView(App $a, $update, $parent)
                if ($cid) {
                        // If $cid belongs to a communitity forum or a privat goup,.add a mention to the status editor
                        $condition = ["`id` = ? AND (`forum` OR `prv`)", $cid];
-                       $contact = dba::selectFirst('contact', ['addr', 'nick'], $condition);
+                       $contact = DBA::selectFirst('contact', ['addr', 'nick'], $condition);
                        if (DBM::is_result($contact)) {
                                if ($contact['addr'] != '') {
                                        $content = '!' . $contact['addr'];
@@ -577,8 +574,8 @@ function networkThreadedView(App $a, $update, $parent)
                        'lockstate' => ((($gid) || ($cid) || ($nets) || (is_array($a->user) &&
                        ((strlen($a->user['allow_cid'])) || (strlen($a->user['allow_gid'])) ||
                        (strlen($a->user['deny_cid'])) || (strlen($a->user['deny_gid']))))) ? 'lock' : 'unlock'),
-                       'default_perms' => Acl::getDefaultUserPermissions($a->user),
-                       'acl' => Acl::getFullSelectorHTML((($gid || $cid || $nets) ? $def_acl : $a->user), true),
+                       'default_perms' => ACL::getDefaultUserPermissions($a->user),
+                       'acl' => ACL::getFullSelectorHTML((($gid || $cid || $nets) ? $def_acl : $a->user), true),
                        'bang' => (($gid || $cid || $nets) ? '!' : ''),
                        'visitor' => 'block',
                        'profile_uid' => local_user(),
@@ -593,13 +590,14 @@ function networkThreadedView(App $a, $update, $parent)
        // desired.
 
        $sql_post_table = '';
-       $sql_options = (($star) ? " AND `thread`.`starred` " : '');
-       $sql_options .= (($bmark) ? " AND `thread`.`bookmark` " : '');
+       $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 = '';
        $sql_table = '`thread`';
        $sql_parent = '`iid`';
+       $sql_order = '';
 
        if ($update) {
                $sql_table = '`item`';
@@ -611,7 +609,7 @@ function networkThreadedView(App $a, $update, $parent)
        $sql_tag_nets = (($nets) ? sprintf(" AND `item`.`network` = '%s' ", dbesc($nets)) : '');
 
        if ($gid) {
-               $group = dba::selectFirst('group', ['name'], ['id' => $gid, 'uid' => $_SESSION['uid']]);
+               $group = DBA::selectFirst('group', ['name'], ['id' => $gid, 'uid' => local_user()]);
                if (!DBM::is_result($group)) {
                        if ($update) {
                                killme();
@@ -627,7 +625,7 @@ function networkThreadedView(App $a, $update, $parent)
                        $contact_str_self = '';
 
                        $contact_str = implode(',', $contacts);
-                       $self = dba::selectFirst('contact', ['id'], ['uid' => $_SESSION['uid'], 'self' => true]);
+                       $self = DBA::selectFirst('contact', ['id'], ['uid' => local_user(), 'self' => true]);
                        if (DBM::is_result($self)) {
                                $contact_str_self = $self['id'];
                        }
@@ -647,7 +645,7 @@ function networkThreadedView(App $a, $update, $parent)
                $fields = ['id', 'name', 'network', 'writable', 'nurl',
                        'forum', 'prv', 'contact-type', 'addr', 'thumb', 'location'];
                $condition = ["`id` = ? AND (NOT `blocked` OR `pending`)", $cid];
-               $contact = dba::selectFirst('contact', $fields, $condition);
+               $contact = DBA::selectFirst('contact', $fields, $condition);
                if (DBM::is_result($contact)) {
                        $sql_extra = " AND " . $sql_table . ".`contact-id` = " . intval($cid);
 
@@ -703,9 +701,7 @@ function networkThreadedView(App $a, $update, $parent)
                $order_mode = 'commented';
        }
 
-       if ($sql_order == '') {
-               $sql_order = "$sql_table.$ordering";
-       }
+       $sql_order = "$sql_table.$ordering";
 
        if (x($_GET, 'offset')) {
                $sql_range = sprintf(" AND $sql_order <= '%s'", dbesc($_GET['offset']));
@@ -760,7 +756,7 @@ function networkThreadedView(App $a, $update, $parent)
                        // Load all unseen items
                        $sql_extra4 = "`item`.`unseen`";
                        if (Config::get("system", "like_no_comment")) {
-                               $sql_extra4 .= " AND `item`.`verb` = '".ACTIVITY_POST."'";
+                               $sql_extra4 .= " AND `item`.`gravity` IN (" . GRAVITY_PARENT . "," . GRAVITY_COMMENT . ")";
                        }
                        if ($order === 'post') {
                                // Only show toplevel posts when updating posts in this order mode
@@ -772,10 +768,18 @@ function networkThreadedView(App $a, $update, $parent)
                        FROM `item` $sql_post_table
                        STRAIGHT_JOIN `contact` ON `contact`.`id` = `item`.`contact-id`
                                AND (NOT `contact`.`blocked` OR `contact`.`pending`)
+                               AND (`item`.`parent-uri` != `item`.`uri`
+                                       OR `contact`.`uid` = `item`.`uid` AND `contact`.`self`
+                                       OR `contact`.`rel` IN (%d, %d) AND NOT `contact`.`readonly`)
+                       LEFT JOIN `user-item` ON `user-item`.`iid` = `item`.`id` AND `user-item`.`uid` = %d
                        WHERE `item`.`uid` = %d AND `item`.`visible` AND NOT `item`.`deleted`
+                       AND (`user-item`.`hidden` IS NULL OR NOT `user-item`.`hidden`)
                        AND NOT `item`.`moderated` AND $sql_extra4
                        $sql_extra3 $sql_extra $sql_range $sql_nets
                        ORDER BY `order_date` DESC LIMIT 100",
+                       intval(CONTACT_IS_SHARING),
+                       intval(CONTACT_IS_FRIEND),
+                       intval(local_user()),
                        intval(local_user())
                );
        } else {
@@ -784,10 +788,18 @@ function networkThreadedView(App $a, $update, $parent)
                        STRAIGHT_JOIN `contact` ON `contact`.`id` = `thread`.`contact-id`
                                AND (NOT `contact`.`blocked` OR `contact`.`pending`)
                        STRAIGHT_JOIN `item` ON `item`.`id` = `thread`.`iid`
+                               AND (`item`.`parent-uri` != `item`.`uri`
+                                       OR `contact`.`uid` = `item`.`uid` AND `contact`.`self`
+                                       OR `contact`.`rel` IN (%d, %d) AND NOT `contact`.`readonly`)
+                       LEFT JOIN `user-item` ON `user-item`.`iid` = `item`.`id` AND `user-item`.`uid` = %d
                        WHERE `thread`.`uid` = %d AND `thread`.`visible` AND NOT `thread`.`deleted`
                        AND NOT `thread`.`moderated`
+                       AND (`user-item`.`hidden` IS NULL OR NOT `user-item`.`hidden`)
                        $sql_extra2 $sql_extra3 $sql_range $sql_extra $sql_nets
                        ORDER BY `order_date` DESC $pager_sql",
+                       intval(CONTACT_IS_SHARING),
+                       intval(CONTACT_IS_FRIEND),
+                       intval(local_user()),
                        intval(local_user())
                );
        }
@@ -823,16 +835,17 @@ function networkThreadedView(App $a, $update, $parent)
                        $top_limit = DateTimeFormat::utcNow();
                }
 
-               $items = dba::p("SELECT `item`.`parent-uri` AS `uri`, 0 AS `item_id`, `item`.$ordering AS `order_date` FROM `item`
+               $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` ON `contact`.`id` = `item`.`author-id`
+                       STRAIGHT_JOIN `contact` AS `author` ON `author`.`id` = `item`.`author-id`
                        WHERE `item`.`uid` = 0 AND `item`.$ordering < ? AND `item`.$ordering > ?
-                               AND NOT `contact`.`hidden` AND NOT `contact`.`blocked`" . $sql_tag_nets,
-                       local_user(), TERM_OBJ_POST, TERM_HASHTAG, $top_limit, $bottom_limit);
+                               AND NOT `author`.`hidden` AND NOT `author`.`blocked`" . $sql_tag_nets,
+                       local_user(), TERM_OBJ_POST, TERM_HASHTAG,
+                       $top_limit, $bottom_limit);
 
-               $data = dba::inArray($items);
+               $data = DBA::inArray($items);
 
                if (count($data) > 0) {
                        $tag_top_limit = current($data)['order_date'];
@@ -846,7 +859,12 @@ function networkThreadedView(App $a, $update, $parent)
                                $s[$item['uri']] = $item;
                        }
                        foreach ($data as $item) {
-                               $s[$item['uri']] = $item;
+                               // Don't show hash tag posts from blocked or ignored contacts
+                               $condition = ["`nurl` = ? AND `uid` = ? AND (`blocked` OR `readonly`)",
+                                       normalise_link($item['author-link']), local_user()];
+                               if (!DBA::exists('contact', $condition)) {
+                                       $s[$item['uri']] = $item;
+                               }
                        }
                        $r = $s;
                }
@@ -907,7 +925,7 @@ 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, $postord_active, $conv_active, $new_active, $starred_active, $bookmarked_active, $spam_active) = network_query_get_sel_tab($a);
+       list($no_active, $all_active, $postord_active, $conv_active, $new_active, $starred_active, $bookmarked_active) = network_query_get_sel_tab($a);
 
        // if no tabs are selected, defaults to comments
        if ($no_active == 'active') {
@@ -983,7 +1001,7 @@ function network_tabs(App $a)
        // save selected tab, but only if not in file mode
        if (!x($_GET, 'file')) {
                PConfig::set(local_user(), 'network.view', 'tab.selected', [
-                       $all_active, $postord_active, $conv_active, $new_active, $starred_active, $bookmarked_active, $spam_active
+                       $all_active, $postord_active, $conv_active, $new_active, $starred_active, $bookmarked_active
                ]);
        }