Sorting changed to "created"

This commit is contained in:
Michael 2023-12-14 06:18:17 +00:00
parent eae1affb21
commit 1cd729531d
8 changed files with 48 additions and 22 deletions

View file

@ -1534,8 +1534,8 @@ CREATE TABLE IF NOT EXISTS `post-user` (
INDEX `event-id` (`event-id`), INDEX `event-id` (`event-id`),
INDEX `psid` (`psid`), INDEX `psid` (`psid`),
INDEX `author-id_uid` (`author-id`,`uid`), INDEX `author-id_uid` (`author-id`,`uid`),
INDEX `author-id_received` (`author-id`,`received`), INDEX `author-id_created` (`author-id`,`created`),
INDEX `owner-id_received` (`owner-id`,`received`), INDEX `owner-id_created` (`owner-id`,`created`),
INDEX `parent-uri-id_uid` (`parent-uri-id`,`uid`), INDEX `parent-uri-id_uid` (`parent-uri-id`,`uid`),
INDEX `uid_wall_received` (`uid`,`wall`,`received`), INDEX `uid_wall_received` (`uid`,`wall`,`received`),
INDEX `uid_contactid` (`uid`,`contact-id`), INDEX `uid_contactid` (`uid`,`contact-id`),
@ -1596,6 +1596,8 @@ CREATE TABLE IF NOT EXISTS `post-thread-user` (
INDEX `post-user-id` (`post-user-id`), INDEX `post-user-id` (`post-user-id`),
INDEX `commented` (`commented`), INDEX `commented` (`commented`),
INDEX `received` (`received`), INDEX `received` (`received`),
INDEX `author-id_created` (`author-id`,`created`),
INDEX `owner-id_created` (`owner-id`,`created`),
INDEX `uid_received` (`uid`,`received`), INDEX `uid_received` (`uid`,`received`),
INDEX `uid_wall_received` (`uid`,`wall`,`received`), INDEX `uid_wall_received` (`uid`,`wall`,`received`),
INDEX `uid_commented` (`uid`,`commented`), INDEX `uid_commented` (`uid`,`commented`),

View file

@ -50,6 +50,8 @@ Indexes
| post-user-id | post-user-id | | post-user-id | post-user-id |
| commented | commented | | commented | commented |
| received | received | | received | received |
| author-id_created | author-id, created |
| owner-id_created | owner-id, created |
| uid_received | uid, received | | uid_received | uid, received |
| uid_wall_received | uid, wall, received | | uid_wall_received | uid, wall, received |
| uid_commented | uid, commented | | uid_commented | uid, commented |

View file

@ -58,8 +58,8 @@ Indexes
| event-id | event-id | | event-id | event-id |
| psid | psid | | psid | psid |
| author-id_uid | author-id, uid | | author-id_uid | author-id, uid |
| author-id_received | author-id, received | | author-id_created | author-id, created |
| owner-id_received | owner-id, received | | owner-id_created | owner-id, created |
| parent-uri-id_uid | parent-uri-id, uid | | parent-uri-id_uid | parent-uri-id, uid |
| uid_wall_received | uid, wall, received | | uid_wall_received | uid, wall, received |
| uid_contactid | uid, contact-id | | uid_contactid | uid, contact-id |

View file

@ -1298,6 +1298,8 @@ class Conversation
usort($parents, [$this, 'sortThrFeaturedReceived']); usort($parents, [$this, 'sortThrFeaturedReceived']);
} elseif (stristr($order, 'pinned_commented')) { } elseif (stristr($order, 'pinned_commented')) {
usort($parents, [$this, 'sortThrFeaturedCommented']); usort($parents, [$this, 'sortThrFeaturedCommented']);
} elseif (stristr($order, 'pinned_created')) {
usort($parents, [$this, 'sortThrFeaturedCreated']);
} elseif (stristr($order, 'received')) { } elseif (stristr($order, 'received')) {
usort($parents, [$this, 'sortThrReceived']); usort($parents, [$this, 'sortThrReceived']);
} elseif (stristr($order, 'commented')) { } elseif (stristr($order, 'commented')) {
@ -1375,6 +1377,24 @@ class Conversation
return strcmp($b['commented'], $a['commented']); return strcmp($b['commented'], $a['commented']);
} }
/**
* usort() callback to sort item arrays by featured and the created key
*
* @param array $a
* @param array $b
* @return int
*/
private function sortThrFeaturedCreated(array $a, array $b): int
{
if ($b['featured'] && !$a['featured']) {
return 1;
} elseif (!$b['featured'] && $a['featured']) {
return -1;
}
return strcmp($b['created'], $a['created']);
}
/** /**
* usort() callback to sort item arrays by the received key * usort() callback to sort item arrays by the received key
* *

View file

@ -1579,7 +1579,7 @@ class Contact
* @return string posts in HTML * @return string posts in HTML
* @throws \Exception * @throws \Exception
*/ */
public static function getPostsFromId(int $cid, int $uid, bool $only_media = false, string $last_received = null): string public static function getPostsFromId(int $cid, int $uid, bool $only_media = false, string $last_created = null): string
{ {
$contact = DBA::selectFirst('contact', ['contact-type', 'network'], ['id' => $cid]); $contact = DBA::selectFirst('contact', ['contact-type', 'network'], ['id' => $cid]);
if (!DBA::isResult($contact)) { if (!DBA::isResult($contact)) {
@ -1596,8 +1596,8 @@ class Contact
$condition = DBA::mergeConditions($condition, ["`$contact_field` = ? AND `gravity` IN (?, ?)", $cid, Item::GRAVITY_PARENT, Item::GRAVITY_COMMENT]); $condition = DBA::mergeConditions($condition, ["`$contact_field` = ? AND `gravity` IN (?, ?)", $cid, Item::GRAVITY_PARENT, Item::GRAVITY_COMMENT]);
if (!empty($last_received)) { if (!empty($last_created)) {
$condition = DBA::mergeConditions($condition, ["`received` < ?", $last_received]); $condition = DBA::mergeConditions($condition, ["`created` < ?", $last_created]);
} }
if ($only_media) { if ($only_media) {
@ -1615,7 +1615,7 @@ class Contact
$pager = new Pager(DI::l10n(), DI::args()->getQueryString(), $itemsPerPage); $pager = new Pager(DI::l10n(), DI::args()->getQueryString(), $itemsPerPage);
$params = ['order' => ['received' => true], 'limit' => [$pager->getStart(), $pager->getItemsPerPage()]]; $params = ['order' => ['created' => true], 'limit' => [$pager->getStart(), $pager->getItemsPerPage()]];
if (DI::pConfig()->get($uid, 'system', 'infinite_scroll')) { if (DI::pConfig()->get($uid, 'system', 'infinite_scroll')) {
$tpl = Renderer::getMarkupTemplate('infinite_scroll_head.tpl'); $tpl = Renderer::getMarkupTemplate('infinite_scroll_head.tpl');
@ -1647,7 +1647,7 @@ class Contact
* @return string posts in HTML * @return string posts in HTML
* @throws \Exception * @throws \Exception
*/ */
public static function getThreadsFromId(int $cid, int $uid, int $update = 0, int $parent = 0, string $last_received = ''): string public static function getThreadsFromId(int $cid, int $uid, int $update = 0, int $parent = 0, string $last_created = ''): string
{ {
$contact = DBA::selectFirst('contact', ['contact-type', 'network'], ['id' => $cid]); $contact = DBA::selectFirst('contact', ['contact-type', 'network'], ['id' => $cid]);
if (!DBA::isResult($contact)) { if (!DBA::isResult($contact)) {
@ -1662,8 +1662,8 @@ class Contact
if (!empty($parent)) { if (!empty($parent)) {
$condition = DBA::mergeConditions($condition, ['parent' => $parent]); $condition = DBA::mergeConditions($condition, ['parent' => $parent]);
} elseif (!empty($last_received)) { } elseif (!empty($last_created)) {
$condition = DBA::mergeConditions($condition, ["`received` < ?", $last_received]); $condition = DBA::mergeConditions($condition, ["`created` < ?", $last_created]);
} }
$contact_field = ((($contact["contact-type"] == self::TYPE_COMMUNITY) || ($contact['network'] == Protocol::MAIL)) ? 'owner-id' : 'author-id'); $contact_field = ((($contact["contact-type"] == self::TYPE_COMMUNITY) || ($contact['network'] == Protocol::MAIL)) ? 'owner-id' : 'author-id');
@ -1683,30 +1683,30 @@ class Contact
$o = ''; $o = '';
} }
$condition1 = DBA::mergeConditions($condition, ["`$contact_field` = ?", $cid]); $condition1 = DBA::mergeConditions($condition, ["`$contact_field` = ? AND `gravity` = ?", $cid, Item::GRAVITY_PARENT]);
$condition2 = DBA::mergeConditions($condition, [ $condition2 = DBA::mergeConditions($condition, [
"`author-id` = ? AND `gravity` = ? AND `vid` = ? AND `protocol` != ? AND `thr-parent-id` = `parent-uri-id`", "`author-id` = ? AND `gravity` = ? AND `vid` = ? AND `protocol` != ? AND `thr-parent-id` = `parent-uri-id`",
$cid, Item::GRAVITY_ACTIVITY, Verb::getID(Activity::ANNOUNCE), Conversation::PARCEL_DIASPORA $cid, Item::GRAVITY_ACTIVITY, Verb::getID(Activity::ANNOUNCE), Conversation::PARCEL_DIASPORA
]); ]);
$sql1 = "SELECT `uri-id`, `received` FROM `post-thread-user-view` WHERE " . array_shift($condition1); $sql1 = "SELECT `uri-id`, `created` FROM `post-thread-user-view` WHERE " . array_shift($condition1);
$sql2 = "SELECT `thr-parent-id` AS `uri-id`, `received` FROM `post-user-view` WHERE " . array_shift($condition2); $sql2 = "SELECT `thr-parent-id` AS `uri-id`, `created` FROM `post-user-view` WHERE " . array_shift($condition2);
$union = array_merge($condition1, $condition2); $union = array_merge($condition1, $condition2);
$sql = $sql1 . " UNION " . $sql2; $sql = $sql1 . " UNION " . $sql2;
$sql .= " ORDER BY `received` DESC LIMIT ?, ?"; $sql .= " ORDER BY `created` DESC LIMIT ?, ?";
$union = array_merge($union, [$pager->getStart(), $pager->getItemsPerPage()]); $union = array_merge($union, [$pager->getStart(), $pager->getItemsPerPage()]);
$items = Post::toArray(DBA::p($sql, $union)); $items = Post::toArray(DBA::p($sql, $union));
if ($pager->getStart() == 0) { if (empty($last_created) && ($pager->getStart() == 0)) {
$fields = ['uri-id', 'thr-parent-id', 'gravity', 'author-id', 'received']; $fields = ['uri-id', 'thr-parent-id', 'gravity', 'author-id', 'created'];
$pinned = Post\Collection::selectToArrayForContact($cid, Post\Collection::FEATURED, $fields); $pinned = Post\Collection::selectToArrayForContact($cid, Post\Collection::FEATURED, $fields);
$items = array_merge($items, $pinned); $items = array_merge($items, $pinned);
} }
$o .= DI::conversation()->render($items, ConversationContent::MODE_CONTACTS, $update, false, 'pinned_received', $uid); $o .= DI::conversation()->render($items, ConversationContent::MODE_CONTACTS, $update, false, 'pinned_created', $uid);
if (!$update) { if (!$update) {
if (DI::pConfig()->get($uid, 'system', 'infinite_scroll')) { if (DI::pConfig()->get($uid, 'system', 'infinite_scroll')) {

View file

@ -113,7 +113,7 @@ class Conversations extends BaseModule
$o = $this->conversation->statusEditor([], 0, true); $o = $this->conversation->statusEditor([], 0, true);
$o .= Contact::getTabsHTML($contact, Contact::TAB_CONVERSATIONS); $o .= Contact::getTabsHTML($contact, Contact::TAB_CONVERSATIONS);
$o .= Model\Contact::getThreadsFromId($contact['id'], $this->userSession->getLocalUserId(), 0, 0, $request['last_received'] ?? ''); $o .= Model\Contact::getThreadsFromId($contact['id'], $this->userSession->getLocalUserId(), 0, 0, $request['last_created'] ?? '');
return $o; return $o;
} }

View file

@ -99,7 +99,7 @@ class Posts extends BaseModule
$o = Contact::getTabsHTML($contact, Contact::TAB_POSTS); $o = Contact::getTabsHTML($contact, Contact::TAB_POSTS);
$o .= Model\Contact::getPostsFromId($contact['id'], $this->userSession->getLocalUserId(), false, $request['last_received'] ?? ''); $o .= Model\Contact::getPostsFromId($contact['id'], $this->userSession->getLocalUserId(), false, $request['last_created'] ?? '');
return $o; return $o;
} }

View file

@ -1551,8 +1551,8 @@ return [
"event-id" => ["event-id"], "event-id" => ["event-id"],
"psid" => ["psid"], "psid" => ["psid"],
"author-id_uid" => ["author-id", "uid"], "author-id_uid" => ["author-id", "uid"],
"author-id_received" => ["author-id", "received"], "author-id_created" => ["author-id", "created"],
"owner-id_received" => ["owner-id", "received"], "owner-id_created" => ["owner-id", "created"],
"parent-uri-id_uid" => ["parent-uri-id", "uid"], "parent-uri-id_uid" => ["parent-uri-id", "uid"],
"uid_wall_received" => ["uid", "wall", "received"], "uid_wall_received" => ["uid", "wall", "received"],
"uid_contactid" => ["uid", "contact-id"], "uid_contactid" => ["uid", "contact-id"],
@ -1602,6 +1602,8 @@ return [
"post-user-id" => ["post-user-id"], "post-user-id" => ["post-user-id"],
"commented" => ["commented"], "commented" => ["commented"],
"received" => ["received"], "received" => ["received"],
"author-id_created" => ["author-id", "created"],
"owner-id_created" => ["owner-id", "created"],
"uid_received" => ["uid", "received"], "uid_received" => ["uid", "received"],
"uid_wall_received" => ["uid", "wall", "received"], "uid_wall_received" => ["uid", "wall", "received"],
"uid_commented" => ["uid", "commented"], "uid_commented" => ["uid", "commented"],