Order InboxNoticeStream by id instead of created

notice.id will give us even really old posts, which were
recently imported. For example if a remote instance had
problems and just managed to post here. Another solution
would be to have a 'notice.imported' field.
This commit is contained in:
Mikael Nordfeldth 2014-05-10 15:06:31 +02:00
parent 04b7194511
commit b0d4825976

View File

@ -119,7 +119,11 @@ class RawInboxNoticeStream extends NoticeStream
$notice->whereAdd(sprintf('notice.id <= %d', $max_id)); $notice->whereAdd(sprintf('notice.id <= %d', $max_id));
} }
$notice->limit($offset, $limit); $notice->limit($offset, $limit);
$notice->orderBy('notice.created DESC'); // notice.id will give us even really old posts, which were
// recently imported. For example if a remote instance had
// problems and just managed to post here. Another solution
// would be to have a 'notice.imported' field and order by it.
$notice->orderBy('notice.id DESC');
if (!$notice->find()) { if (!$notice->find()) {
return array(); return array();