1
0
mirror of https://github.com/flarum/core.git synced 2025-08-01 14:10:37 +02:00

Eager load ListPostsController needed relations (#2717)

* Eager load ListPostsController needed relations
* Add comment explaining the reason for eagerloading
This commit is contained in:
Sami Mazouz
2021-03-22 09:54:18 +01:00
committed by GitHub
parent 512b756804
commit 397c4c10ed

View File

@@ -93,6 +93,17 @@ class ListPostsController extends AbstractListController
$results->areMoreResults() ? null : 0
);
// Eager load discussion for use in the policies,
// eager loading does not affect the JSON response,
// the response only includes relations included in the request.
if (! in_array('discussion', $include)) {
$include[] = 'discussion';
}
if (in_array('user', $include)) {
$include[] = 'user.groups';
}
return $results->getResults()->load($include);
}