github-mirror/framework
2
0
Fork 0
mirror of https://github.com/flarum/framework.git synced 2024-11-25 04:44:23 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

Don't include post content in the "basic" serializer

Browse Source 
Currently all of a post's replies are loaded in full whenever the post
is loaded, which is kind of overkill - we really just need to know that
they exist (and who posted them) in order to render the "X replied to
this" line.
This commit is contained in:
Toby Zerner 2017-09-19 19:10:07 +09:30
parent c31c1ea062
commit 37cf95f94d
2 changed files with 3 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/Api/Serializer/DiscussionBasicSerializer.php
View File

@ -54,7 +54,7 @@ class DiscussionBasicSerializer extends AbstractSerializer
*/
protected function startPost($discussion)
{
return $this->hasOne($discussion, 'Flarum\Api\Serializer\PostBasicSerializer');
return $this->hasOne($discussion, 'Flarum\Api\Serializer\PostSerializer');
}
/**
@ -70,7 +70,7 @@ class DiscussionBasicSerializer extends AbstractSerializer
*/
protected function lastPost($discussion)
{
return $this->hasOne($discussion, 'Flarum\Api\Serializer\PostBasicSerializer');
return $this->hasOne($discussion, 'Flarum\Api\Serializer\PostSerializer');
}
/**
@ -86,6 +86,6 @@ class DiscussionBasicSerializer extends AbstractSerializer
*/
protected function relevantPosts($discussion)
{
return $this->hasMany($discussion, 'Flarum\Api\Serializer\PostBasicSerializer');
return $this->hasMany($discussion, 'Flarum\Api\Serializer\PostSerializer');
}
}

6
src/Api/Serializer/PostBasicSerializer.php
View File

@ -43,12 +43,6 @@ class PostBasicSerializer extends AbstractSerializer
'contentType' => $post->type
];
if ($post instanceof CommentPost) {
$attributes['contentHtml'] = $post->content_html;
} else {
$attributes['content'] = $post->content;
}
return $attributes;
}