]> git.mxchange.org Git - friendica.git/blobdiff - mod/network.php
Even more SQL improvements.
[friendica.git] / mod / network.php
index e23fd0de5ad551b0b0b379f1c0dee4ff56371859..5b2908f0e967b144afbb8fae0db3b00a03f03b9a 100644 (file)
@@ -1,5 +1,8 @@
 <?php
-
+/*
+To-Do:
+- Update berücksichtigt keine Konversationen
+*/
 
 function network_init(&$a) {
        if(! local_user()) {
@@ -48,7 +51,7 @@ function network_init(&$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?f=&order=comment'.
                        // and this isn't a date query
 
                        $tab_baseurls = array(
@@ -90,55 +93,14 @@ function network_init(&$a) {
                }
 
                if($remember_tab || $remember_net || $remember_group) {
-            $net_args = array_merge($query_array, $net_args);
-            $net_queries = build_querystring($net_args);
-
-            // groups filter is in form of "network/nnn". Add it to $dest_url, if it's possible
-            //if ($a->argc==2 && is_numeric($a->argv[1]) && strpos($net_baseurl, "/",1)===false){
-            //    $net_baseurl .= "/".$a->argv[1];
-            //}
+                       $net_args = array_merge($query_array, $net_args);
+                       $net_queries = build_querystring($net_args);
 
                        $redir_url = ($net_queries ? $net_baseurl."?".$net_queries : $net_baseurl);
                        goaway($a->get_baseurl() . $redir_url);
                }
        }
 
-/*     $sel_tabs = network_query_get_sel_tab($a);
-       $last_sel_tabs = get_pconfig(local_user(), 'network.view','tab.selected');
-       if (is_array($last_sel_tabs)){
-               $tab_urls = array(
-                       '/network?f=&order=comment',//all
-                       '/network?f=&order=post',               //postord
-                       '/network?f=&conv=1',                   //conv
-                       '/network/new',                                 //new
-                       '/network?f=&star=1',                   //starred
-                       '/network?f=&bmark=1',                  //bookmarked
-                       '/network?f=&spam=1',                   //spam
-               );
-
-               // redirect if current selected tab is 'no_active' and
-               // last selected tab is _not_ 'all_active'.
-               // and this isn't a date query
-
-               if ($sel_tabs[0] == 'active' && $last_sel_tabs[0]!='active' && (! $is_a_date_query)) {
-                       $k = array_search('active', $last_sel_tabs);
-
-            // merge tab querystring with request querystring
-            $dest_qa = array();
-            list($dest_url,$dest_qs) = explode("?", $tab_urls[$k]);
-            parse_str( $dest_qs, $dest_qa);
-            $dest_qa = array_merge($query_array, $dest_qa);
-            $dest_qs = build_querystring($dest_qa);
-
-            // groups filter is in form of "network/nnn". Add it to $dest_url, if it's possible
-            if ($a->argc==2 && is_numeric($a->argv[1]) && strpos($dest_url, "/",1)===false){
-                $dest_url .= "/".$a->argv[1];
-            }
-
-                       goaway($a->get_baseurl() . $dest_url."?".$dest_qs);
-               }
-       }*/
-
        if(x($_GET['nets']) && $_GET['nets'] === 'all')
                unset($_GET['nets']);
 
@@ -180,7 +142,7 @@ function network_init(&$a) {
        }
 
        $a->page['aside'] .= (feature_enabled(local_user(),'groups') ? group_side('network/0','network',true,$group_id) : '');
-       $a->page['aside'] .= posted_date_widget($a->get_baseurl() . '/network',local_user(),false);     
+       $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'] .= saved_searches($search);
        $a->page['aside'] .= fileas_widget($a->get_baseurl(true) . '/network',(x($_GET, 'file') ? $_GET['file'] : ''));
@@ -240,7 +202,7 @@ function saved_searches($search) {
 
 /**
  * Return selected tab from query
- * 
+ *
  * urls -> returns
  *             '/network'                                      => $no_active = 'active'
  *             '/network?f=&order=comment'     => $comment_active = 'active'
@@ -250,7 +212,7 @@ function saved_searches($search) {
  *             '/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 );
  */
 function network_query_get_sel_tab($a) {
@@ -383,7 +345,6 @@ function network_content(&$a, $update = 0) {
        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';
-       //echo "<pre>"; var_dump($no_active, $all_active, $postord_active, $conv_active, $new_active, $starred_active, $bookmarked_active, $spam_active); killme();
 
        $cmd = (($datequery) ? '' : $a->cmd);
        $len_naked_cmd = strlen(str_replace('/new','',$cmd));
@@ -392,48 +353,16 @@ function network_content(&$a, $update = 0) {
        $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'] : ''), 
+                       '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'),
                ),
                array(
                        'label' => t('Posted Order'),
-                       'url'=>$a->get_baseurl(true) . '/' . str_replace('/new', '', $cmd) . '?f=&order=post' . ((x($_GET,'cid')) ? '&cid=' . $_GET['cid'] : ''), 
+                       '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'),
                ),
-
-/*             array(
-                       'label' => t('Personal'),
-                       'url' => $a->get_baseurl(true) . '/' . str_replace('/new', '', $cmd) . ((x($_GET,'cid')) ? '/?f=&cid=' . $_GET['cid'] : '') . '&conv=1',
-                       'sel' => $conv_active,
-                       'title' => t('Posts that mention or involve you'),
-               ),*/
-/*             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'),
-               ),*/
-/*             array(
-                       'label' => t('Starred'),
-                       'url'=>$a->get_baseurl(true) . '/' . str_replace('/new', '', $cmd) . ((x($_GET,'cid')) ? '/?f=&cid=' . $_GET['cid'] : '') . '&star=1',
-                       'sel'=>$starred_active,
-                       'title' => t('Favourite Posts'),
-               ),*/
-/*             array(
-                       'label' => t('Shared Links'),
-                       'url'=>$a->get_baseurl(true) . '/' . str_replace('/new', '', $cmd) . ((x($_GET,'cid')) ? '/?f=&cid=' . $_GET['cid'] : '') . '&bmark=1',
-                       'sel'=>$bookmarked_active,
-                       'title'=> t('Interesting Links'),
-               ),      */
-//             array(
-//                     'label' => t('Spam'),
-//                     'url'=>$a->get_baseurl(true) . '/network?f=&spam=1'
-//                     'sel'=> $spam_active,
-//                     'title' => t('Posts flagged as SPAM'),
-//             ),
-
        );
 
        if(feature_enabled(local_user(),'personal_tab')) {
@@ -472,17 +401,6 @@ function network_content(&$a, $update = 0) {
                );
        }
 
-       // Not yet implemented
-
-/*     if(feature_enabled(local_user(),'spam_filter'))  {
-               $tabs[] = array(
-                       'label' => t('Spam'),
-                       'url'=>$a->get_baseurl(true) . '/network?f=&spam=1',
-                       'sel'=> $spam_active,
-                       'title' => t('Posts flagged as SPAM'),
-               );
-       }*/
-
        // 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) );
@@ -533,6 +451,11 @@ function network_content(&$a, $update = 0) {
        }
        set_pconfig(local_user(), 'network.view', 'net.selected', ($nets ? $nets : 'all'));
 
+/*if ($update) {
+print_r($_GET);
+die("ss");
+}*/
+
        if(! $update) {
                if($group) {
                        if(($t = group_public_members($group)) && (! get_pconfig(local_user(),'system','nowarn_insecure'))) {
@@ -567,19 +490,23 @@ function network_content(&$a, $update = 0) {
 
        // We don't have to deal with ACLs on this page. You're looking at everything
        // that belongs to you, hence you can see all of it. We will filter by group if
-       // desired. 
+       // desired.
 
+       $sql_post_table = "";
        $sql_options  = (($star) ? " and starred = 1 " : '');
        $sql_options .= (($bmark) ? " and bookmark = 1 " : '');
+       $sql_extra = $sql_options;
+       $sql_extra2 = "";
+       $sql_extra3 = "";
+       $sql_table = "`thread`";
+       $sql_parent = "`iid`";
 
-       //$sql_nets = (($nets) ? sprintf(" and `contact`.`network` = '%s' ", dbesc($nets)) : '');
-       $sql_nets = (($nets) ? sprintf(" and `item`.`network` = '%s' ", dbesc($nets)) : '');
+       if ($nouveau OR strlen($file) OR $update) {
+               $sql_table = "`item`";
+               $sql_parent = "`parent`";
+       }
 
-       if ($star OR $bmark) {
-               $sql_table = "`item` INNER JOIN (SELECT DISTINCT(`parent`) FROM `item` WHERE 1 $sql_options and deleted = 0 ORDER BY `commented` DESC) AS `temp1` ON item.parent = `temp1`.parent ";
-               $sql_extra = "";
-       } else
-               $sql_extra = " AND `item`.`parent` IN ( SELECT `parent` FROM `item` WHERE `id` = `parent` $sql_options ) ";
+       $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",
@@ -595,29 +522,33 @@ function network_content(&$a, $update = 0) {
                }
 
                $contacts = expand_groups(array($group));
+
+               $contact_str_self = "";
+
                if((is_array($contacts)) && count($contacts)) {
                        $contact_str = implode(',',$contacts);
+                       $self = q("SELECT `id` FROM `contact` WHERE `uid` = %d AND `self`", intval($_SESSION['uid']));
+                       if (count($self))
+                               $contact_str_self = ",".$self[0]["id"];
                }
                else {
                                $contact_str = ' 0 ';
                                info( t('Group is empty'));
                }
 
-               $sql_table = "`item` INNER JOIN (SELECT DISTINCT(`parent`) FROM `item` WHERE 1 $sql_options AND (`contact-id` IN ($contact_str) OR `allow_gid` like '".protect_sprintf('%<'.intval($group).'>%')."') and deleted = 0 ORDER BY `created` DESC) AS `temp1` ON item.parent = `temp1`.parent ";
-               $sql_extra = "";
-               //$sql_extra = " AND `item`.`parent` IN ( SELECT DISTINCT(`parent`) FROM `item` WHERE 1 $sql_options AND ( `contact-id` IN ( $contact_str ) OR `allow_gid` like '" . protect_sprintf('%<' . intval($group) . '>%') . "' ) and deleted = 0 ) ";
+               $sql_post_table = " INNER JOIN (SELECT DISTINCT(`parent`) FROM `item` WHERE (`contact-id` IN ($contact_str) OR `allow_gid` like '".protect_sprintf('%<'.intval($group).'>%')."') and deleted = 0 ORDER BY `created` DESC) AS `temp1` ON $sql_table.$sql_parent = `temp1`.`parent` ";
+
+               $sql_extra3 .= " AND `contact-id` IN ($contact_str.$contact_str_self) ";
                $o = '<h2>' . t('Group: ') . $r[0]['name'] . '</h2>' . $o;
-       }
-       elseif($cid) {
+       } elseif($cid) {
 
-               $r = q("SELECT `id`,`name`,`network`,`writable`,`nurl` FROM `contact` WHERE `id` = %d 
+               $r = q("SELECT `id`,`name`,`network`,`writable`,`nurl` FROM `contact` WHERE `id` = %d
                                AND `blocked` = 0 AND `pending` = 0 LIMIT 1",
                        intval($cid)
                );
                if(count($r)) {
-                       $sql_table = "`item` INNER JOIN (SELECT DISTINCT(`parent`) FROM `item` WHERE 1 $sql_options AND `contact-id` = ".intval($cid)." and deleted = 0 ORDER BY `item`.`received` DESC) AS `temp1` ON item.parent = `temp1`.parent ";
+                       $sql_post_table = " INNER JOIN (SELECT DISTINCT(`parent`) FROM `item` WHERE 1 $sql_options AND `contact-id` = ".intval($cid)." and deleted = 0 ORDER BY `item`.`received` DESC) AS `temp1` ON $sql_table.$sql_parent = `temp1`.`parent` ";
                        $sql_extra = "";
-                       //$sql_extra = " AND `item`.`parent` IN ( SELECT DISTINCT(`parent`) FROM `item` WHERE 1 $sql_options AND `contact-id` = " . intval($cid) . " and deleted = 0 ) ";
                        $o = '<h2>' . t('Contact: ') . $r[0]['name'] . '</h2>' . $o;
                        if($r[0]['network'] === NETWORK_OSTATUS && $r[0]['writable'] && (! get_pconfig(local_user(),'system','nowarn_insecure'))) {
                                notice( t('Private messages to this person are at risk of public disclosure.') . EOL);
@@ -636,23 +567,19 @@ function network_content(&$a, $update = 0) {
                $o .= get_events();
        }
 
-       $sql_extra3 = '';
-
        if($datequery) {
-               $sql_extra3 .= protect_sprintf(sprintf(" AND item.created <= '%s' ", dbesc(datetime_convert(date_default_timezone_get(),'',$datequery))));
+               $sql_extra3 .= protect_sprintf(sprintf(" AND $sql_table.created <= '%s' ", dbesc(datetime_convert(date_default_timezone_get(),'',$datequery))));
        }
        if($datequery2) {
-               $sql_extra3 .= protect_sprintf(sprintf(" AND item.created >= '%s' ", dbesc(datetime_convert(date_default_timezone_get(),'',$datequery2))));
+               $sql_extra3 .= protect_sprintf(sprintf(" AND $sql_table.created >= '%s' ", dbesc(datetime_convert(date_default_timezone_get(),'',$datequery2))));
        }
 
-       $sql_extra2 = (($nouveau) ? '' : " AND `item`.`parent` = `item`.`id` ");
+       //$sql_extra2 = (($nouveau) ? '' : " AND `item`.`parent` = `item`.`id` ");
+       $sql_extra2 = (($nouveau) ? '' : $sql_extra2);
        $sql_extra3 = (($nouveau) ? '' : $sql_extra3);
-       //$sql_order = "`item`.`received`";
        $sql_order = "";
        $order_mode = "received";
-
-       if ($sql_table == "")
-               $sql_table = "`item`";
+       $tag = false;
 
        if(x($_GET,'search')) {
                $search = escape_tags($_GET['search']);
@@ -666,44 +593,30 @@ function network_content(&$a, $update = 0) {
                        $tag = true;
 
                if($tag) {
-                       //$sql_extra = sprintf(" AND `term`.`term` = '%s' AND `term`.`otype` = %d AND `term`.`type` = %d ",
-                       //              dbesc(protect_sprintf($search)), intval(TERM_OBJ_POST), intval(TERM_HASHTAG));
-                       //$sql_table = "`term` LEFT JOIN `item` ON `item`.`id` = `term`.`oid` AND `item`.`uid` = `term`.`uid` ";
-
-                       //$sql_order = "`term`.`tid`";
-
                        $sql_extra = "";
 
-                       $sql_table = sprintf("`item` 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`.`received`";
-                       $order_mode = "received";
+                       $sql_order = "`item`.`id`";
+                       $order_mode = "id";
                } else {
                        if (get_config('system','use_fulltext_engine'))
                                $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";
                }
        }
        if(strlen($file)) {
-               $sql_extra .= file_tag_file_query('item',unxmlify($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()));
+               $sql_order = "`item`.`id`";
+               $order_mode = "id";
        }
 
-       if($conv) {
-               $myurl = $a->get_baseurl() . '/profile/'. $a->user['nickname'];
-               $myurl = substr($myurl,strpos($myurl,'://')+3);
-               $myurl = str_replace('www.','',$myurl);
-               $diasp_url = str_replace('/profile/','/u/',$myurl);
-
-               $sql_extra .= sprintf(" AND `item`.`parent` IN (SELECT distinct(`parent`) from item where `author-link` IN ('https://%s', 'http://%s') OR `mention`)",
-                       dbesc(protect_sprintf($myurl)),
-                       dbesc(protect_sprintf($myurl))
-               );
-       }
+       if($conv)
+               $sql_extra3 .= " AND `mention`";
 
        if($update) {
 
@@ -714,9 +627,9 @@ function network_content(&$a, $update = 0) {
        else {
                if( (! get_config('alt_pager', 'global')) && (! get_pconfig(local_user(),'system','alt_pager')) ) {
                        $r = q("SELECT COUNT(*) AS `total`
-                               FROM $sql_table LEFT JOIN `contact` ON `contact`.`id` = `item`.`contact-id`
-                               WHERE `item`.`uid` = %d AND `item`.`visible` = 1 AND `item`.`deleted` = 0
+                               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
                                $sql_extra2 $sql_extra3
                                $sql_extra $sql_nets ",
                                intval($_SESSION['uid'])
@@ -727,16 +640,16 @@ function network_content(&$a, $update = 0) {
                        }
                }
 
-               //  check if we serve a mobile device and get the user settings 
+               //  check if we serve a mobile device and get the user settings
                //  accordingly
-               if ($a->is_mobile) { 
+               if ($a->is_mobile) {
                    $itemspage_network = get_pconfig(local_user(),'system','itemspage_mobile_network');
                    $itemspage_network = ((intval($itemspage_network)) ? $itemspage_network : 20);
-               } else { 
+               } else {
                    $itemspage_network = get_pconfig(local_user(),'system','itemspage_network');
                    $itemspage_network = ((intval($itemspage_network)) ? $itemspage_network : 40);
                }
-               //  now that we have the user settings, see if the theme forces 
+               //  now that we have the user settings, see if the theme forces
                //  a maximum item number which is lower then the user choice
                if(($a->force_max_items > 0) && ($a->force_max_items < $itemspage_network))
                        $itemspage_network = $a->force_max_items;
@@ -745,28 +658,29 @@ function network_content(&$a, $update = 0) {
                $pager_sql = sprintf(" LIMIT %d, %d ",intval($a->pager['start']), intval($a->pager['itemspage']));
        }
 
-       $simple_update = (($update) ? " and `item`.`unseen` = 1 " : '');
-
        if($nouveau) {
-               // "New Item View" - show all items unthreaded in reverse created date order
+               $simple_update = (($update) ? " and `item`.`unseen` = 1 " : '');
+
+               if ($sql_order == "")
+                       $sql_order = "`item`.`received`";
 
+               // "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 LEFT JOIN `contact` ON `contact`.`id` = `item`.`contact-id`
-                       WHERE `item`.`uid` = %d AND `item`.`visible` = 1 
+                       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
                        $simple_update
-                       AND `contact`.`blocked` = 0 AND `contact`.`pending` = 0
                        $sql_extra $sql_nets
-                       ORDER BY `item`.`received` DESC $pager_sql ",
+                       ORDER BY $sql_order DESC $pager_sql ",
                        intval($_SESSION['uid'])
                );
 
                $update_unseen = ' WHERE uid = ' . intval($_SESSION['uid']) . " AND unseen = 1 $sql_extra $sql_nets";
-       }
-       else {
+       } else {
 
                // Normal conversation view
 
@@ -782,31 +696,29 @@ function network_content(&$a, $update = 0) {
                }
 
                if ($sql_order == "")
-                       $sql_order = "`item`.$ordering";
+                       $sql_order = "$sql_table.$ordering";
 
                if (($_GET["offset"] != ""))
                        $sql_extra3 .= sprintf(" AND $sql_order <= '%s'", dbesc($_GET["offset"]));
 
                // Fetch a page full of parent items for this page
-
                if($update) {
                        $r = q("SELECT `item`.`parent` AS `item_id`, `item`.`network` AS `item_network`, `contact`.`uid` AS `contact_uid`
-                               FROM $sql_table LEFT JOIN `contact` ON `contact`.`id` = `item`.`contact-id`
+                               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 OR item.verb = '" . ACTIVITY_LIKE ."' OR item.verb = '" . ACTIVITY_DISLIKE . "')
+                               (`item`.`deleted` = 0 OR `item`.`verb` = '" . ACTIVITY_LIKE ."' OR `item`.`verb` = '" . ACTIVITY_DISLIKE . "')
                                and `item`.`moderated` = 0 and `item`.`unseen` = 1
-                               AND `contact`.`blocked` = 0 AND `contact`.`pending` = 0
-                               $sql_extra3 $sql_extra $sql_nets ",
+                               $sql_extra3 $sql_extra $sql_nets ORDER BY `item_id` DESC LIMIT 100",
                                intval(local_user())
                        );
-               }
-               else {
-                       $r = q("SELECT `item`.`id` AS `item_id`, `item`.`network` AS `item_network`, `contact`.`uid` AS `contact_uid`
-                               FROM $sql_table LEFT JOIN `contact` ON `contact`.`id` = `item`.`contact-id`
-                               WHERE `item`.`uid` = %d AND `item`.`visible` = 1 AND `item`.`deleted` = 0
-                               AND `item`.`moderated` = 0 AND `contact`.`blocked` = 0 AND `contact`.`pending` = 0
-                               AND `item`.`parent` = `item`.`id`
-                               $sql_extra3 $sql_extra $sql_nets
+               } else {
+                       $r = q("SELECT `thread`.`iid` AS `item_id`, `thread`.`network` AS `item_network`, `contact`.`uid` AS `contact_uid`
+                               FROM $sql_table $sql_post_table INNER 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
+                               $sql_extra2 $sql_extra3 $sql_extra $sql_nets
                                ORDER BY $sql_order DESC $pager_sql ",
                                intval(local_user())
                        );
@@ -823,7 +735,7 @@ function network_content(&$a, $update = 0) {
                                if(! in_array($rr['item_id'],$parents_arr))
                                        $parents_arr[] = $rr['item_id'];
 
-                       //$parents_str = implode(', ', $parents_arr);
+                       $parents_str = implode(", ", $parents_arr);
 
                        // splitted into separate queries to avoid the problem with very long threads
                        // so always the last X comments are loaded
@@ -834,26 +746,25 @@ function network_content(&$a, $update = 0) {
 
                        $items = array();
 
-                       foreach ($parents_arr AS $parents_str) {
-
+                       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 $sql_table LEFT JOIN `contact` ON `contact`.`id` = `item`.`contact-id`
+                                       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
-                                       AND `contact`.`blocked` = 0 AND `contact`.`pending` = 0
-                                       AND `item`.`parent` IN ( %s )
-                                       $sql_extra ORDER BY `item`.`commented` DESC LIMIT %d",
+                                       AND `item`.`parent` = %d
+                                       ORDER BY `item`.`commented` DESC LIMIT %d",
                                        intval(local_user()),
-                                       dbesc($parents_str),
+                                       intval($parents),
                                        intval($max_comments + 1)
                                );
                                $items = array_merge($items, $thread_items);
                        }
                        $items = conv_sort($items,$ordering);
-
                } else {
                        $items = array();
                }
@@ -871,17 +782,18 @@ function network_content(&$a, $update = 0) {
 
        // We aren't going to try and figure out at the item, group, and page
        // level which items you've seen and which you haven't. If you're looking
-       // at the top level network page just mark everything seen. 
+       // at the top level network page just mark everything seen.
 
 
-// The $update_unseen is a bit unreliable if you have stuff coming into your stream from a new contact - 
-// and other feeds that bring in stuff from the past. One can't find it all. 
+// The $update_unseen is a bit unreliable if you have stuff coming into your stream from a new contact -
+// and other feeds that bring in stuff from the past. One can't find it all.
 // I'm reviving this block to mark everything seen on page 1 of the network as a temporary measure.
 // The correct solution is to implement a network notifications box just like the system notifications popup
 // with the ability in the popup to "mark all seen".
 // Several people are complaining because there are unseen messages they can't find and as time goes
 // on they just get buried deeper. It has happened to me a couple of times also.
 
+
        if((! $group) && (! $cid) && (! $star)) {
                $r = q("UPDATE `item` SET `unseen` = 0
                        WHERE `unseen` = 1 AND `uid` = %d",