From e20ea092d2ba4496d0c8f9a907d1de62a0f3348b Mon Sep 17 00:00:00 2001 From: Michael Vogel Date: Tue, 21 Aug 2018 17:39:49 +0200 Subject: [PATCH] You can now filter the community page by account types (#5649) * You can now filter the community page by account types * Better use "null" --- mod/community.php | 51 ++++++++++++++++++++++++++++++++++++++++------- 1 file changed, 44 insertions(+), 7 deletions(-) diff --git a/mod/community.php b/mod/community.php index f148191bbc..9c9fb43900 100644 --- a/mod/community.php +++ b/mod/community.php @@ -10,6 +10,7 @@ use Friendica\Core\Config; use Friendica\Core\L10n; use Friendica\Core\PConfig; use Friendica\Database\DBA; +use Friendica\Model\Contact; function community_init(App $a) { @@ -35,6 +36,25 @@ function community_content(App $a, $update = 0) return; } + $accounttype = null; + + if ($a->argc > 2) { + switch ($a->argv[2]) { + case 'person': + $accounttype = Contact::ACCOUNT_TYPE_PERSON; + break; + case 'organisation': + $accounttype = Contact::ACCOUNT_TYPE_ORGANISATION; + break; + case 'news': + $accounttype = Contact::ACCOUNT_TYPE_NEWS; + break; + case 'community': + $accounttype = Contact::ACCOUNT_TYPE_COMMUNITY; + break; + } + } + if ($a->argc > 1) { $content = $a->argv[1]; } else { @@ -135,7 +155,7 @@ function community_content(App $a, $update = 0) $a->set_pager_itemspage($itemspage_network); - $r = community_getitems($a->pager['start'], $a->pager['itemspage'], $content); + $r = community_getitems($a->pager['start'], $a->pager['itemspage'], $content, $accounttype); if (!DBA::isResult($r)) { info(L10n::t('No results.') . EOL); @@ -164,7 +184,7 @@ function community_content(App $a, $update = 0) } } if (count($s) < $a->pager['itemspage']) { - $r = community_getitems($a->pager['start'] + ($count * $a->pager['itemspage']), $a->pager['itemspage'], $content); + $r = community_getitems($a->pager['start'] + ($count * $a->pager['itemspage']), $a->pager['itemspage'], $content, $accounttype); } } while ((count($s) < $a->pager['itemspage']) && ( ++$count < 50) && (count($r) > 0)); } else { @@ -186,23 +206,40 @@ function community_content(App $a, $update = 0) ]); } -function community_getitems($start, $itemspage, $content) +function community_getitems($start, $itemspage, $content, $accounttype) { if ($content == 'local') { + if (!is_null($accounttype)) { + $sql_accounttype = " AND `user`.`account-type` = ?"; + $values = [$accounttype, $start, $itemspage]; + } else { + $sql_accounttype = ""; + $values = [$start, $itemspage]; + } + $r = DBA::p("SELECT `item`.`uri`, `author`.`url` AS `author-link` FROM `thread` INNER JOIN `user` ON `user`.`uid` = `thread`.`uid` AND NOT `user`.`hidewall` INNER JOIN `item` ON `item`.`id` = `thread`.`iid` INNER 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 ?, ?", $start, $itemspage); + 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') { + if (!is_null($accounttype)) { + $sql_accounttype = " AND `owner`.`contact-type` = ?"; + $values = [$accounttype, $start, $itemspage]; + } else { + $sql_accounttype = ""; + $values = [$start, $itemspage]; + } + $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 ?, ?", $start, $itemspage); + INNER JOIN `contact` AS `owner` ON `owner`.`id`=`item`.`owner-id` + WHERE `thread`.`uid` = 0 AND NOT `author`.`hidden` AND NOT `author`.`blocked` $sql_accounttype + ORDER BY `thread`.`commented` DESC LIMIT ?, ?", $values); return DBA::toArray($r); }