<?php
/**
- * @copyright Copyright (C) 2020, Friendica
+ * @copyright Copyright (C) 2010-2022, the Friendica project
*
* @license GNU AGPL version 3 or any later version
*
use Friendica\Content\Text\HTML;
use Friendica\Content\Widget;
use Friendica\Content\Widget\TrendingTags;
-use Friendica\Core\ACL;
use Friendica\Core\Renderer;
use Friendica\Core\Session;
use Friendica\Database\DBA;
use Friendica\DI;
use Friendica\Model\Item;
+use Friendica\Model\Post;
use Friendica\Model\User;
use Friendica\Network\HTTPException;
{
protected static $page_style;
protected static $content;
- protected static $accounttype;
+ protected static $accountTypeString;
+ protected static $accountType;
protected static $itemsPerPage;
- protected static $since_id;
+ protected static $min_id;
protected static $max_id;
protected static $item_id;
- public static function content(array $parameters = [])
+ protected function content(array $request = []): string
{
- self::parseRequest($parameters);
+ $this->parseRequest();
if (DI::pConfig()->get(local_user(), 'system', 'infinite_scroll')) {
$tpl = Renderer::getMarkupTemplate('infinite_scroll_head.tpl');
Nav::setSelected('community');
- DI::page()['aside'] .= Widget::accounts('community/' . self::$content, $parameters['accounttype'] ?? '');
+ DI::page()['aside'] .= Widget::accounttypes('community/' . self::$content, self::$accountTypeString);
if (local_user() && DI::config()->get('system', 'community_no_sharer')) {
$path = self::$content;
- if (!empty($parameters['accounttype'])) {
- $path .= '/' . $parameters['accounttype'];
+ if (!empty($this->parameters['accounttype'])) {
+ $path .= '/' . $this->parameters['accounttype'];
}
$query_parameters = [];
- if (!empty($_GET['since_id'])) {
- $query_parameters['since_id'] = $_GET['since_id'];
+ if (!empty($_GET['min_id'])) {
+ $query_parameters['min_id'] = $_GET['min_id'];
}
if (!empty($_GET['max_id'])) {
$query_parameters['max_id'] = $_GET['max_id'];
// We need the editor here to be able to reshare an item.
if (Session::isAuthenticated()) {
- $x = [
- 'is_owner' => true,
- 'allow_location' => DI::app()->user['allow_location'],
- 'default_location' => DI::app()->user['default-location'],
- 'nickname' => DI::app()->user['nickname'],
- 'lockstate' => (is_array(DI::app()->user) && (strlen(DI::app()->user['allow_cid']) || strlen(DI::app()->user['allow_gid']) || strlen(DI::app()->user['deny_cid']) || strlen(DI::app()->user['deny_gid'])) ? 'lock' : 'unlock'),
- 'acl' => ACL::getFullSelectorHTML(DI::page(), DI::app()->user, true),
- 'bang' => '',
- 'visitor' => 'block',
- 'profile_uid' => local_user(),
- ];
- $o .= status_editor(DI::app(), $x, 0, true);
+ $o .= DI::conversation()->statusEditor([], 0, true);
}
}
return $o;
}
- $o .= conversation(DI::app(), $items, 'community', false, false, 'commented', local_user());
+ $o .= DI::conversation()->create($items, 'community', false, false, 'commented', local_user());
$pager = new BoundariesPager(
DI::l10n(),
/**
* Computes module parameters from the request and local configuration
*
- * @param array $parameters
* @throws HTTPException\BadRequestException
* @throws HTTPException\ForbiddenException
*/
- protected static function parseRequest(array $parameters)
+ protected function parseRequest()
{
if (DI::config()->get('system', 'block_public') && !Session::isAuthenticated()) {
throw new HTTPException\ForbiddenException(DI::l10n()->t('Public access denied.'));
throw new HTTPException\ForbiddenException(DI::l10n()->t('Access denied.'));
}
- self::$accounttype = User::getAccountTypeByString($parameters['accounttype'] ?? '');
+ self::$accountTypeString = $_GET['accounttype'] ?? $this->parameters['accounttype'] ?? '';
+ self::$accountType = User::getAccountTypeByString(self::$accountTypeString);
- self::$content = $parameters['content'] ?? '';
+ self::$content = $this->parameters['content'] ?? '';
if (!self::$content) {
if (!empty(DI::config()->get('system', 'singleuser'))) {
// On single user systems only the global page does make sense
DI::config()->get('system', 'itemspage_network'));
}
- // now that we have the user settings, see if the theme forces
- // a maximum item number which is lower then the user choice
- if ((DI::app()->force_max_items > 0) && (DI::app()->force_max_items < self::$itemsPerPage)) {
- self::$itemsPerPage = DI::app()->force_max_items;
- }
-
if (!empty($_GET['item'])) {
- $item = Item::selectFirst(['parent'], ['id' => $_GET['item']]);
+ $item = Post::selectFirst(['parent'], ['id' => $_GET['item']]);
self::$item_id = $item['parent'] ?? 0;
} else {
self::$item_id = 0;
}
- self::$since_id = $_GET['since_id'] ?? null;
+ self::$min_id = $_GET['min_id'] ?? null;
self::$max_id = $_GET['max_id'] ?? null;
self::$max_id = $_GET['last_commented'] ?? self::$max_id;
}
*/
protected static function getItems()
{
- $items = self::selectItems(self::$since_id, self::$max_id, self::$item_id, self::$itemsPerPage);
+ $items = self::selectItems(self::$min_id, self::$max_id, self::$item_id, self::$itemsPerPage);
$maxpostperauthor = (int) DI::config()->get('system', 'max_author_posts_community_page');
if ($maxpostperauthor != 0 && self::$content == 'local') {
// If we're looking at a "previous page", the lookup continues forward in time because the list is
// sorted in chronologically decreasing order
- if (isset(self::$since_id)) {
- self::$since_id = $items[0]['commented'];
+ if (isset(self::$min_id)) {
+ self::$min_id = $items[0]['commented'];
} else {
// In any other case, the lookup continues backwards in time
self::$max_id = $items[count($items) - 1]['commented'];
}
- $items = self::selectItems(self::$since_id, self::$max_id, self::$item_id, self::$itemsPerPage);
+ $items = self::selectItems(self::$min_id, self::$max_id, self::$item_id, self::$itemsPerPage);
}
} else {
$selected_items = $items;
/**
* Database query for the comunity page
*
- * @param $since_id
+ * @param $min_id
* @param $max_id
* @param $itemspage
* @return array
* @throws \Exception
* @TODO Move to repository/factory
*/
- private static function selectItems($since_id, $max_id, $item_id, $itemspage)
+ private static function selectItems($min_id, $max_id, $item_id, $itemspage)
{
- $r = false;
-
if (self::$content == 'local') {
- if (!is_null(self::$accounttype)) {
- $condition = ["`wall` AND `origin` AND `private` = ? AND `owner`.`contact-type` = ?", Item::PUBLIC, self::$accounttype];
+ if (!is_null(self::$accountType)) {
+ $condition = ["`wall` AND `origin` AND `private` = ? AND `owner-contact-type` = ?", Item::PUBLIC, self::$accountType];
} else {
$condition = ["`wall` AND `origin` AND `private` = ?", Item::PUBLIC];
}
} elseif (self::$content == 'global') {
- if (!is_null(self::$accounttype)) {
- $condition = ["`uid` = ? AND `private` = ? AND `owner`.`contact-type` = ?", 0, Item::PUBLIC, self::$accounttype];
+ if (!is_null(self::$accountType)) {
+ $condition = ["`uid` = ? AND `private` = ? AND `owner-contact-type` = ?", 0, Item::PUBLIC, self::$accountType];
} else {
$condition = ["`uid` = ? AND `private` = ?", 0, Item::PUBLIC];
}
return [];
}
+ $params = ['order' => ['commented' => true], 'limit' => $itemspage];
+
if (!empty($item_id)) {
- $condition[0] .= " AND `iid` = ?";
+ $condition[0] .= " AND `id` = ?";
$condition[] = $item_id;
} else {
if (local_user() && !empty($_REQUEST['no_sharer'])) {
- $condition[0] .= " AND NOT EXISTS (SELECT `uri-id` FROM `thread` AS t1 WHERE `t1`.`uri-id` = `thread`.`uri-id` AND `t1`.`uid` = ?)";
+ $condition[0] .= " AND NOT EXISTS (SELECT `uri-id` FROM `post-user` WHERE `post-user`.`uri-id` = `post-thread-user-view`.`uri-id` AND `post-user`.`uid` = ?)";
$condition[] = local_user();
}
$condition[] = $max_id;
}
- if (isset($since_id)) {
+ if (isset($min_id)) {
$condition[0] .= " AND `commented` > ?";
- $condition[] = $since_id;
+ $condition[] = $min_id;
+
+ // Previous page case: we want the items closest to min_id but for that we need to reverse the query order
+ if (!isset($max_id)) {
+ $params['order']['commented'] = false;
+ }
}
}
- $r = Item::selectThreadForUser(0, ['uri', 'commented', 'author-link'], $condition, ['order' => ['commented' => true], 'limit' => $itemspage]);
+ $r = Post::selectThreadForUser(0, ['uri-id', 'commented', 'author-link'], $condition, $params);
+
+ $items = Post::toArray($r);
+ if (empty($items)) {
+ return [];
+ }
+
+ // Previous page case: once we get the relevant items closest to min_id, we need to restore the expected display order
+ if (empty($item_id) && isset($min_id) && !isset($max_id)) {
+ $items = array_reverse($items);
+ }
- return DBA::toArray($r);
+ return $items;
}
}