mirror/php-flarum
1
0
Fork 0
mirror of https://github.com/flarum/core.git synced 2025-07-28 12:10:51 +02:00
Code Issues Releases Wiki Activity

Sort included posts

Browse Source 
They can be out of order in the payload due to relationship loading,
e.g. post #1 includes post #14 that has mentioned it, therefore #14
will be the first post in the payload. The new post stream doesn’t take
kindly to out of order posts.
This commit is contained in:
Toby Zerner
2015-05-29 18:55:19 +09:30
parent 5314d2b512
commit 741da52ccd
1 changed files with 1 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
framework/core/js/forum/src/components/discussion-page.js
View File

@@ -82,6 +82,7 @@ export default class DiscussionPage extends mixin(Component, evented) {
includedPosts.push(app.store.getById('posts', record.id)); includedPosts.push(app.store.getById('posts', record.id));
} }
}); });
includedPosts.sort((a, b) => a.id() - b.id());
this.stream = new PostStream({ discussion, includedPosts }); this.stream = new PostStream({ discussion, includedPosts });
this.stream.on('positionChanged', this.positionChanged.bind(this)); this.stream.on('positionChanged', this.positionChanged.bind(this));