]> git.mxchange.org Git - friendica.git/blobdiff - mod/network.php
Merge branch 'rewrites/dbm_is_result' of github.com:Quix0r/friendica into rewrites...
[friendica.git] / mod / network.php
index 0010a3d8246b9fb2d681c06e394df02638834d45..5281a2ede9aaf3ea9b7d9558fa2fd4721171e163 100644 (file)
@@ -122,11 +122,11 @@ function network_init(&$a) {
        $search = ((x($_GET,'search')) ? escape_tags($_GET['search']) : '');
 
        if(x($_GET,'save')) {
-               $r = q("SELECT * FROM `search` WHERE `uid` = %d AND `term` = '%s' LIMIT 1",
+               $r = qu("SELECT * FROM `search` WHERE `uid` = %d AND `term` = '%s' LIMIT 1",
                        intval(local_user()),
                        dbesc($search)
                );
-               if(! count($r)) {
+               if(! dbm::is_result($r)) {
                        q("INSERT INTO `search` ( `uid`,`term` ) VALUES ( %d, '%s') ",
                                intval(local_user()),
                                dbesc($search)
@@ -143,16 +143,16 @@ function network_init(&$a) {
        // search terms header
        if(x($_GET,'search')) {
                $a->page['content'] .= replace_macros(get_markup_template("section_title.tpl"),array(
-                       '$title' => sprintf( t('Search Results For: %s'), $search)
+                       '$title' => sprintf( t('Results for: %s'), $search)
                ));
        }
 
        $a->page['aside'] .= (feature_enabled(local_user(),'groups') ? group_side('network/0','network','standard',$group_id) : '');
        $a->page['aside'] .= (feature_enabled(local_user(),'forumlist_widget') ? ForumManager::widget(local_user(),$cid) : '');
-       $a->page['aside'] .= posted_date_widget($a->get_baseurl() . '/network',local_user(),false);
-       $a->page['aside'] .= networks_widget($a->get_baseurl(true) . '/network',(x($_GET, 'nets') ? $_GET['nets'] : ''));
+       $a->page['aside'] .= posted_date_widget('network',local_user(),false);
+       $a->page['aside'] .= networks_widget('network',(x($_GET, 'nets') ? $_GET['nets'] : ''));
        $a->page['aside'] .= saved_searches($search);
-       $a->page['aside'] .= fileas_widget($a->get_baseurl(true) . '/network',(x($_GET, 'file') ? $_GET['file'] : ''));
+       $a->page['aside'] .= fileas_widget('network',(x($_GET, 'file') ? $_GET['file'] : ''));
 
 }
 
@@ -176,13 +176,13 @@ function saved_searches($search) {
 
        $o = '';
 
-       $r = q("SELECT `id`,`term` FROM `search` WHERE `uid` = %d",
+       $r = qu("SELECT `id`,`term` FROM `search` WHERE `uid` = %d",
                intval(local_user())
        );
 
        $saved = array();
 
-       if(count($r)) {
+       if(dbm::is_result($r)) {
                foreach($r as $rr) {
                        $saved[] = array(
                                'id'            => $rr['id'],
@@ -280,7 +280,8 @@ function network_query_get_sel_tab($a) {
 }
 
 /**
- * Return selected network from query
+ * @brief Return selected network from query
+ * @return string Name of the selected network
  */
 function network_query_get_sel_net() {
        $network = false;
@@ -348,92 +349,7 @@ function network_content(&$a, $update = 0) {
 
        $o = '';
 
-       // 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);
-       // if no tabs are selected, defaults to comments
-       if ($no_active=='active') $all_active='active';
 
-       $cmd = (($datequery) ? '' : $a->cmd);
-       $len_naked_cmd = strlen(str_replace('/new','',$cmd));
-
-       // tabs
-       $tabs = array(
-               array(
-                       'label' => t('Commented Order'),
-                       'url'   => $a->get_baseurl(true) . '/' . str_replace('/new', '', $cmd) . '?f=&order=comment' . ((x($_GET,'cid')) ? '&cid=' . $_GET['cid'] : ''),
-                       'sel'   => $all_active,
-                       'title' => t('Sort by Comment Date'),
-                       'id'    => 'commented-order-tab',
-                       'accesskey' => "e",
-               ),
-               array(
-                       'label' => t('Posted Order'),
-                       'url'   => $a->get_baseurl(true) . '/' . str_replace('/new', '', $cmd) . '?f=&order=post' . ((x($_GET,'cid')) ? '&cid=' . $_GET['cid'] : ''),
-                       'sel'   => $postord_active,
-                       'title' => t('Sort by Post Date'),
-                       'id'    => 'posted-order-tab',
-                       'accesskey' => "t",
-               ),
-       );
-
-       if(feature_enabled(local_user(),'personal_tab')) {
-               $tabs[] = array(
-                       'label' => t('Personal'),
-                       'url'   => $a->get_baseurl(true) . '/' . str_replace('/new', '', $cmd) . ((x($_GET,'cid')) ? '/?f=&cid=' . $_GET['cid'] : '/?f=') . '&conv=1',
-                       'sel'   => $conv_active,
-                       'title' => t('Posts that mention or involve you'),
-                       'id'    => 'personal-tab',
-                       'accesskey' => "r",
-               );
-       }
-
-       if(feature_enabled(local_user(),'new_tab')) {
-               $tabs[] = array(
-                       'label' => t('New'),
-                       'url'   => $a->get_baseurl(true) . '/' . str_replace('/new', '', $cmd) . ($len_naked_cmd ? '/' : '') . 'new' . ((x($_GET,'cid')) ? '/?f=&cid=' . $_GET['cid'] : ''),
-                       'sel'   => $new_active,
-                       'title' => t('Activity Stream - by date'),
-                       'id'    => 'activitiy-by-date-tab',
-                       'accesskey' => "w",
-               );
-       }
-
-       if(feature_enabled(local_user(),'link_tab')) {
-               $tabs[] = array(
-                       'label' => t('Shared Links'),
-                       'url'   => $a->get_baseurl(true) . '/' . str_replace('/new', '', $cmd) . ((x($_GET,'cid')) ? '/?f=&cid=' . $_GET['cid'] : '/?f=') . '&bmark=1',
-                       'sel'   => $bookmarked_active,
-                       'title' => t('Interesting Links'),
-                       'id'    => 'shared-links-tab',
-                       'accesskey' => "b",
-               );
-       }
-
-       if(feature_enabled(local_user(),'star_posts')) {
-               $tabs[] = array(
-                       'label' => t('Starred'),
-                       'url'   => $a->get_baseurl(true) . '/' . str_replace('/new', '', $cmd) . ((x($_GET,'cid')) ? '/?f=&cid=' . $_GET['cid'] : '/?f=') . '&star=1',
-                       'sel'   => $starred_active,
-                       'title' => t('Favourite Posts'),
-                       'id'    => 'starred-posts-tab',
-                       'accesskey' => "m",
-               );
-       }
-
-       // save selected tab, but only if not in search or file mode
-       if(!x($_GET,'search') && !x($_GET,'file')) {
-               set_pconfig( local_user(), 'network.view','tab.selected',array($all_active, $postord_active, $conv_active, $new_active, $starred_active, $bookmarked_active, $spam_active) );
-       }
-
-       $arr = array('tabs' => $tabs);
-       call_hooks('network_tabs', $arr);
-
-       $o .= replace_macros(get_markup_template('common_tabs.tpl'), array('$tabs'=> $arr['tabs']));
-
-       // --- end item filter tabs
 
        $contact_id = $a->cid;
 
@@ -459,13 +375,13 @@ function network_content(&$a, $update = 0) {
                $def_acl = array('allow_cid' => '<' . intval($cid) . '>');
 
        if($nets) {
-               $r = q("SELECT `id` FROM `contact` WHERE `uid` = %d AND network = '%s' AND `self` = 0",
+               $r = qu("SELECT `id` FROM `contact` WHERE `uid` = %d AND network = '%s' AND `self` = 0",
                        intval(local_user()),
                        dbesc($nets)
                );
 
                $str = '';
-               if(count($r))
+               if(dbm::is_result($r))
                        foreach($r as $rr)
                                $str .= '<' . $rr['id'] . '>';
                if(strlen($str))
@@ -474,12 +390,15 @@ function network_content(&$a, $update = 0) {
        set_pconfig(local_user(), 'network.view', 'net.selected', ($nets ? $nets : 'all'));
 
        if(!$update AND !$rawmode) {
+               $tabs = network_tabs($a);
+               $o .= $tabs;
+
                if($group) {
                        if(($t = group_public_members($group)) && (! get_pconfig(local_user(),'system','nowarn_insecure'))) {
-                               notice( sprintf( tt('Warning: This group contains %s member from an insecure network.',
-                                                                       'Warning: This group contains %s members from an insecure network.',
-                                                                       $t), $t ) . EOL);
-                               notice( t('Private messages to this group are at risk of public disclosure.') . EOL);
+                               notice(sprintf(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), $t).EOL);
+                               notice(t("Messages in this group won't be send to these receivers.").EOL);
                        }
                }
 
@@ -489,7 +408,7 @@ function network_content(&$a, $update = 0) {
 
                if ($cid) {
                        // If $cid belongs to a communitity forum or a privat goup,.add a mention to the status editor
-                       $contact = q("SELECT `nick` FROM `contact` WHERE `id` = %d AND `uid` = %d AND (`forum` OR `prv`) ",
+                       $contact = qu("SELECT `nick` FROM `contact` WHERE `id` = %d AND `uid` = %d AND (`forum` OR `prv`) ",
                                intval($cid),
                                intval(local_user())
                        );
@@ -523,8 +442,8 @@ function network_content(&$a, $update = 0) {
        // desired.
 
        $sql_post_table = "";
-       $sql_options  = (($star) ? " and starred = 1 " : '');
-       $sql_options .= (($bmark) ? " and bookmark = 1 " : '');
+       $sql_options  = (($star) ? " AND `thread`.`starred` " : '');
+       $sql_options .= (($bmark) ? " AND `thread`.`bookmark` " : '');
        $sql_extra = $sql_options;
        $sql_extra2 = "";
        $sql_extra3 = "";
@@ -534,20 +453,21 @@ function network_content(&$a, $update = 0) {
        if ($nouveau OR strlen($file) OR $update) {
                $sql_table = "`item`";
                $sql_parent = "`parent`";
+               $sql_post_table = " INNER JOIN `thread` ON `thread`.`iid` = `item`.`parent`";
        }
 
        $sql_nets = (($nets) ? sprintf(" and $sql_table.`network` = '%s' ", dbesc($nets)) : '');
 
        if($group) {
-               $r = q("SELECT `name`, `id` FROM `group` WHERE `id` = %d AND `uid` = %d LIMIT 1",
+               $r = qu("SELECT `name`, `id` FROM `group` WHERE `id` = %d AND `uid` = %d LIMIT 1",
                        intval($group),
                        intval($_SESSION['uid'])
                );
-               if(! count($r)) {
+               if(! dbm::is_result($r)) {
                        if($update)
                                killme();
                        notice( t('No such group') . EOL );
-                       goaway($a->get_baseurl(true) . '/network/0');
+                       goaway('network/0');
                        // NOTREACHED
                }
 
@@ -560,7 +480,7 @@ function network_content(&$a, $update = 0) {
 
                        $contact_str = implode(',',$contacts);
                        $gcontact_str = implode(',',$gcontacts);
-                       $self = q("SELECT `contact`.`id`, `gcontact`.`id` AS `gid` FROM `contact`
+                       $self = qu("SELECT `contact`.`id`, `gcontact`.`id` AS `gid` FROM `contact`
                                        INNER JOIN `gcontact` ON `gcontact`.`nurl` = `contact`.`nurl`
                                        WHERE `uid` = %d AND `self`", intval($_SESSION['uid']));
                        if (count($self)) {
@@ -568,9 +488,9 @@ function network_content(&$a, $update = 0) {
                                $gcontact_str_self = $self[0]["gid"];
                        }
 
-                       $sql_post_table = " INNER JOIN `item` AS `temp1` ON `temp1`.`id` = ".$sql_table.".".$sql_parent;
-                       $sql_extra3 .= " AND ($sql_table.`contact-id` IN ($contact_str) ";
-                       $sql_extra3 .= " OR ($sql_table.`contact-id` = '$contact_str_self' AND `temp1`.`allow_gid` LIKE '".protect_sprintf('%<'.intval($group).'>%')."' AND `temp1`.`private`))";
+                       $sql_post_table .= " INNER JOIN `item` AS `temp1` ON `temp1`.`id` = ".$sql_table.".".$sql_parent;
+                       $sql_extra3 .= " AND (`thread`.`contact-id` IN ($contact_str) ";
+                       $sql_extra3 .= " OR (`thread`.`contact-id` = '$contact_str_self' AND `temp1`.`allow_gid` LIKE '".protect_sprintf('%<'.intval($group).'>%')."' AND `temp1`.`private`))";
                } else {
                        $sql_extra3 .= " AND false ";
                        info( t('Group is empty'));
@@ -583,11 +503,11 @@ function network_content(&$a, $update = 0) {
        }
        elseif($cid) {
 
-               $r = q("SELECT `id`,`name`,`network`,`writable`,`nurl`, `forum`, `prv`, `addr`, `thumb`, `location` FROM `contact` WHERE `id` = %d
-                               AND `blocked` = 0 AND `pending` = 0 LIMIT 1",
+               $r = qu("SELECT `id`,`name`,`network`,`writable`,`nurl`, `forum`, `prv`, `contact-type`, `addr`, `thumb`, `location` FROM `contact` WHERE `id` = %d
+                               AND (NOT `blocked` OR `pending`) LIMIT 1",
                        intval($cid)
                );
-               if(count($r)) {
+               if(dbm::is_result($r)) {
                        $sql_extra = " AND ".$sql_table.".`contact-id` = ".intval($cid);
 
                        $entries[0] = array(
@@ -595,10 +515,11 @@ function network_content(&$a, $update = 0) {
                                'name' => htmlentities($r[0]['name']),
                                'itemurl' => (($r[0]['addr']) ? ($r[0]['addr']) : ($r[0]['nurl'])),
                                'thumb' => proxy_url($r[0]['thumb'], false, PROXY_SIZE_THUMB),
-                               'account_type' => (($r[0]['forum']) || ($r[0]['prv']) ? t('Forum') : ''),
                                'details' => $r[0]['location'],
                        );
 
+                       $entries[0]["account_type"] = account_type($r[0]);
+
                        $o = replace_macros(get_markup_template("viewcontact_template.tpl"),array(
                                'contacts' => $entries,
                                'id' => 'network',
@@ -611,7 +532,7 @@ function network_content(&$a, $update = 0) {
                }
                else {
                        notice( t('Invalid contact.') . EOL);
-                       goaway($a->get_baseurl(true) . '/network');
+                       goaway('network');
                        // NOTREACHED
                }
        }
@@ -639,7 +560,7 @@ function network_content(&$a, $update = 0) {
                $search = escape_tags($_GET['search']);
 
                if(strpos($search,'#') === 0) {
-                       $tag = true;
+                       $tag = true;
                        $search = substr($search,1);
                }
 
@@ -649,7 +570,7 @@ function network_content(&$a, $update = 0) {
                if($tag) {
                        $sql_extra = "";
 
-                       $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` ",
+                       $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($search)), intval(TERM_OBJ_POST), intval(TERM_HASHTAG), intval(local_user()));
                        $sql_order = "`item`.`id`";
                        $order_mode = "id";
@@ -658,12 +579,12 @@ function network_content(&$a, $update = 0) {
                                $sql_extra = sprintf(" AND MATCH (`item`.`body`, `item`.`title`) AGAINST ('%s' in boolean mode) ", dbesc(protect_sprintf($search)));
                        else
                                $sql_extra = sprintf(" AND `item`.`body` REGEXP '%s' ", dbesc(protect_sprintf(preg_quote($search))));
-                       $sql_order = "`item`.`received`";
-                       $order_mode = "received";
+                       $sql_order = "`item`.`id`";
+                       $order_mode = "id";
                }
        }
        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` ",
+               $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()));
                $sql_order = "`item`.`id`";
                $order_mode = "id";
@@ -677,19 +598,18 @@ function network_content(&$a, $update = 0) {
                // only setup pagination on initial page view
                $pager_sql = '';
 
-       }
-       else {
+       } else {
                if(get_config('system', 'old_pager')) {
-                       $r = q("SELECT COUNT(*) AS `total`
+                       $r = qu("SELECT COUNT(*) AS `total`
                                FROM $sql_table $sql_post_table INNER JOIN `contact` ON `contact`.`id` = $sql_table.`contact-id`
-                               AND `contact`.`blocked` = 0 AND `contact`.`pending` = 0
-                               WHERE $sql_table.`uid` = %d AND $sql_table.`visible` = 1 AND $sql_table.`deleted` = 0
+                               AND (NOT `contact`.`blocked` OR `contact`.`pending`)
+                               WHERE $sql_table.`uid` = %d AND $sql_table.`visible` AND NOT $sql_table.`deleted`
                                $sql_extra2 $sql_extra3
                                $sql_extra $sql_nets ",
                                intval($_SESSION['uid'])
                        );
 
-                       if(count($r)) {
+                       if(dbm::is_result($r)) {
                                $a->set_pager_total($r[0]['total']);
                        }
                }
@@ -714,23 +634,18 @@ function network_content(&$a, $update = 0) {
        }
 
        if($nouveau) {
-               $simple_update = (($update) ? " AND `item`.`unseen` = 1 " : '');
+               $simple_update = (($update) ? " AND `item`.`unseen` " : '');
 
                if ($sql_order == "")
-                       $sql_order = "`item`.`received`";
+                       $sql_order = "`item`.`id`";
 
                // "New Item View" - show all items unthreaded in reverse created date order
-               $items = q("SELECT `item`.*, `item`.`id` AS `item_id`, `item`.`network` AS `item_network`,
-                       `contact`.`name`, `contact`.`photo`, `contact`.`url`, `contact`.`rel`, `contact`.`writable`,
-                       `contact`.`network`, `contact`.`thumb`, `contact`.`dfrn-id`, `contact`.`self`,
-                       `contact`.`id` AS `cid`, `contact`.`uid` AS `contact-uid`
-                       FROM $sql_table $sql_post_table INNER JOIN `contact` ON `contact`.`id` = `item`.`contact-id`
-                       AND `contact`.`blocked` = 0 AND `contact`.`pending` = 0
-                       WHERE `item`.`uid` = %d AND `item`.`visible` = 1
-                       AND `item`.`deleted` = 0 AND `item`.`moderated` = 0
+               $items = qu("SELECT %s FROM $sql_table $sql_post_table %s
+                       WHERE %s AND `item`.`uid` = %d
                        $simple_update
                        $sql_extra $sql_nets
                        ORDER BY $sql_order DESC $pager_sql ",
+                       item_fieldlists(), item_joins(), item_condition(),
                        intval($_SESSION['uid'])
                );
 
@@ -758,28 +673,26 @@ function network_content(&$a, $update = 0) {
 
                // Fetch a page full of parent items for this page
                if($update) {
-                       if (!get_config("system", "like_no_comment"))
-                               $sql_extra4 = "(`item`.`deleted` = 0
-                                               OR `item`.`verb` = '".ACTIVITY_LIKE."' OR `item`.`verb` = '".ACTIVITY_DISLIKE."'
-                                               OR `item`.`verb` = '".ACTIVITY_ATTEND."' OR `item`.`verb` = '".ACTIVITY_ATTENDNO."'
-                                               OR `item`.`verb` = '".ACTIVITY_ATTENDMAYBE."')";
+                       if (get_config("system", "like_no_comment"))
+                               $sql_extra4 = " AND `item`.`verb` = '".ACTIVITY_POST."'";
                        else
-                               $sql_extra4 = "`item`.`deleted` = 0 AND `item`.`verb` = '".ACTIVITY_POST."'";
+                               $sql_extra4 = "";
 
-                       $r = q("SELECT `item`.`parent` AS `item_id`, `item`.`network` AS `item_network`, `contact`.`uid` AS `contact_uid`
+                       $r = qu("SELECT `item`.`parent` AS `item_id`, `item`.`network` AS `item_network`, `contact`.`uid` AS `contact_uid`
                                FROM $sql_table $sql_post_table INNER JOIN `contact` ON `contact`.`id` = `item`.`contact-id`
-                               AND `contact`.`blocked` = 0 AND `contact`.`pending` = 0
-                               WHERE `item`.`uid` = %d AND `item`.`visible` = 1 AND $sql_extra4
-                               AND `item`.`moderated` = 0 AND `item`.`unseen` = 1
-                               $sql_extra3 $sql_extra $sql_nets ORDER BY `item_id` DESC LIMIT 100",
+                               AND (NOT `contact`.`blocked` OR `contact`.`pending`)
+                               WHERE `item`.`uid` = %d AND `item`.`visible` AND NOT `item`.`deleted` $sql_extra4
+                               AND NOT `item`.`moderated` AND `item`.`unseen`
+                               $sql_extra3 $sql_extra $sql_nets
+                               ORDER BY `item_id` DESC LIMIT 100",
                                intval(local_user())
                        );
                } else {
-                       $r = q("SELECT `thread`.`iid` AS `item_id`, `thread`.`network` AS `item_network`, `contact`.`uid` AS `contact_uid`
+                       $r = qu("SELECT `thread`.`iid` AS `item_id`, `thread`.`network` AS `item_network`, `contact`.`uid` AS `contact_uid`
                                FROM $sql_table $sql_post_table STRAIGHT_JOIN `contact` ON `contact`.`id` = `thread`.`contact-id`
-                               AND `contact`.`blocked` = 0 AND `contact`.`pending` = 0
-                               WHERE `thread`.`uid` = %d AND `thread`.`visible` = 1 AND `thread`.`deleted` = 0
-                               AND `thread`.`moderated` = 0
+                               AND (NOT `contact`.`blocked` OR `contact`.`pending`)
+                               WHERE `thread`.`uid` = %d AND `thread`.`visible` AND NOT `thread`.`deleted`
+                               AND NOT `thread`.`moderated`
                                $sql_extra2 $sql_extra3 $sql_extra $sql_nets
                                ORDER BY $sql_order DESC $pager_sql ",
                                intval(local_user())
@@ -792,7 +705,7 @@ function network_content(&$a, $update = 0) {
                $parents_str = '';
                $date_offset = "";
 
-               if(count($r)) {
+               if(dbm::is_result($r)) {
                        foreach($r as $rr)
                                if(! in_array($rr['item_id'],$parents_arr))
                                        $parents_arr[] = $rr['item_id'];
@@ -809,22 +722,16 @@ function network_content(&$a, $update = 0) {
                        $items = array();
 
                        foreach ($parents_arr AS $parents) {
-//                                     $sql_extra ORDER BY `item`.`commented` DESC LIMIT %d",
-                               $thread_items = q("SELECT `item`.*, `item`.`id` AS `item_id`, `item`.`network` AS `item_network`,
-                                       `contact`.`name`, `contact`.`photo`, `contact`.`url`, `contact`.`alias`, `contact`.`rel`, `contact`.`writable`,
-                                       `contact`.`network`, `contact`.`thumb`, `contact`.`dfrn-id`, `contact`.`self`,
-                                       `contact`.`id` AS `cid`, `contact`.`uid` AS `contact-uid`
-                                       FROM `item` INNER JOIN `contact` ON `contact`.`id` = `item`.`contact-id`
-                                       AND `contact`.`blocked` = 0 AND `contact`.`pending` = 0
-                                       WHERE `item`.`uid` = %d AND `item`.`visible` = 1 AND `item`.`deleted` = 0
-                                       AND `item`.`moderated` = 0
+                               $thread_items = qu(item_query()." AND `item`.`uid` = %d
                                        AND `item`.`parent` = %d
                                        ORDER BY `item`.`commented` DESC LIMIT %d",
                                        intval(local_user()),
                                        intval($parents),
                                        intval($max_comments + 1)
                                );
-                               $items = array_merge($items, $thread_items);
+       
+                               if (dbm::is_result($thread_items))
+                                       $items = array_merge($items, $thread_items);
                        }
                        $items = conv_sort($items,$ordering);
                } else {
@@ -857,14 +764,24 @@ function network_content(&$a, $update = 0) {
 
 
        if((! $group) && (! $cid) && (! $star)) {
-               $r = q("UPDATE `item` SET `unseen` = 0
-                       WHERE `unseen` = 1 AND `uid` = %d",
-                       intval(local_user())
-               );
+
+               $unseen = q("SELECT `id` FROM `item` WHERE `unseen` AND `uid` = %d",
+                               intval(local_user()));
+
+               if ($unseen)
+                       $r = q("UPDATE `item` SET `unseen` = 0
+                               WHERE `unseen` = 1 AND `uid` = %d",
+                               intval(local_user())
+                       );
        }
        else {
-               if($update_unseen)
-                       $r = q("UPDATE `item` SET `unseen` = 0 $update_unseen");
+               if($update_unseen) {
+
+                       $unseen = q("SELECT `id` FROM `item` ".$update_unseen);
+
+                       if ($unseen)
+                               $r = q("UPDATE `item` SET `unseen` = 0 $update_unseen");
+               }
        }
 
        // Set this so that the conversation function can find out contact info for our wall-wall items
@@ -876,14 +793,110 @@ function network_content(&$a, $update = 0) {
 
        if(!$update) {
                if(get_pconfig(local_user(),'system','infinite_scroll')) {
-                               $o .= scroll_loader();
+                       $o .= scroll_loader();
                } elseif(!get_config('system', 'old_pager')) {
-                       $o .= alt_pager($a,count($items));
+                       $o .= alt_pager($a,count($items));
                } else {
-                       $o .= paginate($a);
+                       $o .= paginate($a);
                }
        }
 
        return $o;
 }
 
+/**
+ * @brief Get the network tabs menu
+ * 
+ * @param app $a The global App
+ * @return string Html of the networktab
+ */
+function network_tabs($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);
+       // if no tabs are selected, defaults to comments
+       if ($no_active=='active') $all_active='active';
+
+       $cmd = (($datequery) ? '' : $a->cmd);
+       $len_naked_cmd = strlen(str_replace('/new','',$cmd));
+
+       // tabs
+       $tabs = array(
+               array(
+                       'label' => t('Commented Order'),
+                       'url'   => str_replace('/new', '', $cmd) . '?f=&order=comment' . ((x($_GET,'cid')) ? '&cid=' . $_GET['cid'] : ''),
+                       'sel'   => $all_active,
+                       'title' => t('Sort by Comment Date'),
+                       'id'    => 'commented-order-tab',
+                       'accesskey' => "e",
+               ),
+               array(
+                       'label' => t('Posted Order'),
+                       'url'   => str_replace('/new', '', $cmd) . '?f=&order=post' . ((x($_GET,'cid')) ? '&cid=' . $_GET['cid'] : ''),
+                       'sel'   => $postord_active,
+                       'title' => t('Sort by Post Date'),
+                       'id'    => 'posted-order-tab',
+                       'accesskey' => "t",
+               ),
+       );
+
+       if(feature_enabled(local_user(),'personal_tab')) {
+               $tabs[] = array(
+                       'label' => t('Personal'),
+                       'url'   => str_replace('/new', '', $cmd) . ((x($_GET,'cid')) ? '/?f=&cid=' . $_GET['cid'] : '/?f=') . '&conv=1',
+                       'sel'   => $conv_active,
+                       'title' => t('Posts that mention or involve you'),
+                       'id'    => 'personal-tab',
+                       'accesskey' => "r",
+               );
+       }
+
+       if(feature_enabled(local_user(),'new_tab')) {
+               $tabs[] = array(
+                       'label' => t('New'),
+                       'url'   => str_replace('/new', '', $cmd) . ($len_naked_cmd ? '/' : '') . 'new' . ((x($_GET,'cid')) ? '/?f=&cid=' . $_GET['cid'] : ''),
+                       'sel'   => $new_active,
+                       'title' => t('Activity Stream - by date'),
+                       'id'    => 'activitiy-by-date-tab',
+                       'accesskey' => "w",
+               );
+       }
+
+       if(feature_enabled(local_user(),'link_tab')) {
+               $tabs[] = array(
+                       'label' => t('Shared Links'),
+                       'url'   => str_replace('/new', '', $cmd) . ((x($_GET,'cid')) ? '/?f=&cid=' . $_GET['cid'] : '/?f=') . '&bmark=1',
+                       'sel'   => $bookmarked_active,
+                       'title' => t('Interesting Links'),
+                       'id'    => 'shared-links-tab',
+                       'accesskey' => "b",
+               );
+       }
+
+       if(feature_enabled(local_user(),'star_posts')) {
+               $tabs[] = array(
+                       'label' => t('Starred'),
+                       'url'   => str_replace('/new', '', $cmd) . ((x($_GET,'cid')) ? '/?f=&cid=' . $_GET['cid'] : '/?f=') . '&star=1',
+                       'sel'   => $starred_active,
+                       'title' => t('Favourite Posts'),
+                       'id'    => 'starred-posts-tab',
+                       'accesskey' => "m",
+               );
+       }
+
+       // save selected tab, but only if not in search or file mode
+       if(!x($_GET,'search') && !x($_GET,'file')) {
+               set_pconfig( local_user(), 'network.view','tab.selected',array($all_active, $postord_active, $conv_active, $new_active, $starred_active, $bookmarked_active, $spam_active) );
+       }
+
+       $arr = array('tabs' => $tabs);
+       call_hooks('network_tabs', $arr);
+       
+       $tpl = get_markup_template('common_tabs.tpl');
+
+       return replace_macros($tpl, array('$tabs' => $arr['tabs']));
+
+       // --- end item filter tabs
+}