perf: only load user state when showing full user data

FriendsOfFlarum · Mar 8, 2024 · 27c4f08 · 27c4f08
1 parent 0ada6d0
commit 27c4f08
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/src/Extenders/UserPoliciesRelationship.php b/src/Extenders/UserPoliciesRelationship.php
@@ -12,6 +12,7 @@
 namespace FoF\Terms\Extenders;
 
 use Flarum\Api\Serializer\BasicUserSerializer;
+use Flarum\Api\Serializer\CurrentUserSerializer;
 use Flarum\User\User;
 use FoF\Terms\Repositories\PolicyRepository;
 
@@ -33,6 +34,9 @@ public function __invoke(BasicUserSerializer $serializer, User $user, array $att
 
         if ($request->getAttribute('routeName') === 'users.show' && $serializer->getActor()->can('seeFoFTermsPoliciesState', $user)) {
             $attributes['fofTermsPoliciesState'] = $this->policies->state($user);
+        }
+
+        if ($serializer instanceof CurrentUserSerializer) {
             $attributes['fofTermsPoliciesHasUpdate'] = $this->policies->hasPoliciesUpdate($user);
             $attributes['fofTermsPoliciesMustAccept'] = $this->policies->mustAcceptNewPolicies($user);
         }