]> git.mxchange.org Git - friendica.git/blobdiff - mod/community.php
Don't process empty hash tags in Model\Item
[friendica.git] / mod / community.php
index 59be7487bdca55a8318410b80e4eeab16f2dc199..4d98f0c4fa24d6dabd74ad7097bb34e1b240f168 100644 (file)
@@ -4,26 +4,24 @@
  */
 
 use Friendica\App;
+use Friendica\Content\Feature;
 use Friendica\Content\Nav;
+use Friendica\Content\Pager;
+use Friendica\Content\Widget\TrendingTags;
 use Friendica\Core\ACL;
 use Friendica\Core\Config;
 use Friendica\Core\L10n;
 use Friendica\Core\PConfig;
-use Friendica\Database\DBM;
-
-function community_init(App $a)
-{
-       if (!local_user()) {
-               unset($_SESSION['theme']);
-               unset($_SESSION['mobile-theme']);
-       }
-}
+use Friendica\Core\Renderer;
+use Friendica\Database\DBA;
+use Friendica\Model\Item;
+use Friendica\Model\User;
 
 function community_content(App $a, $update = 0)
 {
        $o = '';
 
-       if (Config::get('system', 'block_public') && !local_user() && !remote_user()) {
+       if (Config::get('system', 'block_public') && !Session::isAuthenticated()) {
                notice(L10n::t('Public access denied.') . EOL);
                return;
        }
@@ -35,6 +33,25 @@ function community_content(App $a, $update = 0)
                return;
        }
 
+       $accounttype = null;
+
+       if ($a->argc > 2) {
+               switch ($a->argv[2]) {
+                       case 'person':
+                               $accounttype = User::ACCOUNT_TYPE_PERSON;
+                               break;
+                       case 'organisation':
+                               $accounttype = User::ACCOUNT_TYPE_ORGANISATION;
+                               break;
+                       case 'news':
+                               $accounttype = User::ACCOUNT_TYPE_NEWS;
+                               break;
+                       case 'community':
+                               $accounttype = User::ACCOUNT_TYPE_COMMUNITY;
+                               break;
+               }
+       }
+
        if ($a->argc > 1) {
                $content = $a->argv[1];
        } else {
@@ -70,9 +87,6 @@ function community_content(App $a, $update = 0)
                }
        }
 
-       require_once 'include/security.php';
-       require_once 'include/conversation.php';
-
        if (!$update) {
                $tabs = [];
 
@@ -98,8 +112,8 @@ function community_content(App $a, $update = 0)
                        ];
                }
 
-               $tab_tpl = get_markup_template('common_tabs.tpl');
-               $o .= replace_macros($tab_tpl, ['$tabs' => $tabs]);
+               $tab_tpl = Renderer::getMarkupTemplate('common_tabs.tpl');
+               $o .= Renderer::replaceMacros($tab_tpl, ['$tabs' => $tabs]);
 
                Nav::setSelected('community');
 
@@ -111,7 +125,7 @@ function community_content(App $a, $update = 0)
                                'default_location' => $a->user['default-location'],
                                'nickname' => $a->user['nickname'],
                                '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'),
-                               'acl' => ACL::getFullSelectorHTML($a->user, true),
+                               'acl' => ACL::getFullSelectorHTML($a->page, $a->user, true),
                                'bang' => '',
                                'visitor' => 'block',
                                'profile_uid' => local_user(),
@@ -133,11 +147,11 @@ function community_content(App $a, $update = 0)
                $itemspage_network = $a->force_max_items;
        }
 
-       $a->set_pager_itemspage($itemspage_network);
+       $pager = new Pager($a->query_string, $itemspage_network);
 
-       $r = community_getitems($a->pager['start'], $a->pager['itemspage'], $content);
+       $r = community_getitems($pager->getStart(), $pager->getItemsPerPage(), $content, $accounttype);
 
-       if (!DBM::is_result($r)) {
+       if (!DBA::isResult($r)) {
                info(L10n::t('No results.') . EOL);
                return $o;
        }
@@ -159,26 +173,34 @@ function community_content(App $a, $update = 0)
                                }
                                $previousauthor = $item["author-link"];
 
-                               if (($numposts < $maxpostperauthor) && (count($s) < $a->pager['itemspage'])) {
+                               if (($numposts < $maxpostperauthor) && (count($s) < $pager->getItemsPerPage())) {
                                        $s[] = $item;
                                }
                        }
-                       if (count($s) < $a->pager['itemspage']) {
-                               $r = community_getitems($a->pager['start'] + ($count * $a->pager['itemspage']), $a->pager['itemspage'], $content);
+                       if (count($s) < $pager->getItemsPerPage()) {
+                               $r = community_getitems($pager->getStart() + ($count * $pager->getItemsPerPage()), $pager->getItemsPerPage(), $content, $accounttype);
                        }
-               } while ((count($s) < $a->pager['itemspage']) && ( ++$count < 50) && (count($r) > 0));
+               } while ((count($s) < $pager->getItemsPerPage()) && ( ++$count < 50) && (count($r) > 0));
        } else {
                $s = $r;
        }
 
-       $o .= conversation($a, $s, 'community', $update, false, 'commented', local_user());
+       $o .= conversation($a, $s, $pager, 'community', $update, false, 'commented', local_user());
 
        if (!$update) {
-               $o .= alt_pager($a, count($r));
+               $o .= $pager->renderMinimal(count($r));
        }
 
-       $t = get_markup_template("community.tpl");
-       return replace_macros($t, [
+       if (empty($a->page['aside'])) {
+               $a->page['aside'] = '';
+       }
+
+       if (Feature::isEnabled(local_user(), 'trending_tags')) {
+               $a->page['aside'] .= TrendingTags::getHTML($content);
+       }
+
+       $t = Renderer::getMarkupTemplate("community.tpl");
+       return Renderer::replaceMacros($t, [
                '$content' => $o,
                '$header' => '',
                '$show_global_community_hint' => ($content == 'global') && Config::get('system', 'show_global_community_hint'),
@@ -186,24 +208,35 @@ function community_content(App $a, $update = 0)
        ]);
 }
 
-function community_getitems($start, $itemspage, $content)
+function community_getitems($start, $itemspage, $content, $accounttype)
 {
        if ($content == 'local') {
-               $r = dba::p("SELECT `item`.`uri`, `item`.`author-link` FROM `thread`
-                       INNER JOIN `user` ON `user`.`uid` = `thread`.`uid` AND NOT `user`.`hidewall`
-                       INNER JOIN `item` ON `item`.`id` = `thread`.`iid`
+               if (!is_null($accounttype)) {
+                       $sql_accounttype = " AND `user`.`account-type` = ?";
+                       $values = [$accounttype, $start, $itemspage];
+               } else {
+                       $sql_accounttype = "";
+                       $values = [$start, $itemspage];
+               }
+
+               /// @todo Use "unsearchable" here as well (instead of "hidewall")
+               $r = DBA::p("SELECT `item`.`uri`, `author`.`url` AS `author-link` FROM `thread`
+                       STRAIGHT_JOIN `user` ON `user`.`uid` = `thread`.`uid` AND NOT `user`.`hidewall`
+                       STRAIGHT_JOIN `item` ON `item`.`id` = `thread`.`iid`
+                       STRAIGHT_JOIN `contact` AS `author` ON `author`.`id`=`item`.`author-id`
                        WHERE `thread`.`visible` AND NOT `thread`.`deleted` AND NOT `thread`.`moderated`
-                       AND NOT `thread`.`private` AND `thread`.`wall` AND `thread`.`origin`
-                       ORDER BY `thread`.`commented` DESC LIMIT " . intval($start) . ", " . intval($itemspage)
-               );
-               return dba::inArray($r);
+                       AND NOT `thread`.`private` AND `thread`.`wall` AND `thread`.`origin` $sql_accounttype
+                       ORDER BY `thread`.`commented` DESC LIMIT ?, ?", $values);
+               return DBA::toArray($r);
        } elseif ($content == 'global') {
-               $r = dba::p("SELECT `uri` FROM `thread`
-                               INNER JOIN `item` ON `item`.`id` = `thread`.`iid`
-                               INNER JOIN `contact` AS `author` ON `author`.`id`=`item`.`author-id`
-                               WHERE `thread`.`uid` = 0 AND NOT `author`.`hidden` AND NOT `author`.`blocked`
-                               ORDER BY `thread`.`commented` DESC LIMIT " . intval($start) . ", " . intval($itemspage));
-               return dba::inArray($r);
+               if (!is_null($accounttype)) {
+                       $condition = ["`uid` = ? AND NOT `author`.`unsearchable` AND NOT `owner`.`unsearchable` AND `owner`.`contact-type` = ?", 0, $accounttype];
+               } else {
+                       $condition = ["`uid` = ? AND NOT `author`.`unsearchable` AND NOT `owner`.`unsearchable`", 0];
+               }
+
+               $r = Item::selectThreadForUser(0, ['uri'], $condition, ['order' => ['commented' => true], 'limit' => [$start, $itemspage]]);
+               return DBA::toArray($r);
        }
 
        // Should never happen