Issue CollaboraOnline#49: Remove redundant ->cachePerPermission() whe…

…n ->cachePerUser() is already present.
donquixote · Nov 20, 2024 · bf2c338 · bf2c338
1 parent 35d11b3
commit bf2c338
Showing 1 changed file with 0 additions and 2 deletions.
diff --git a/collabora_online.module b/collabora_online.module
@@ -146,7 +146,6 @@ function collabora_online_media_access(MediaInterface $media, string $operation,
                     ->setReason("The user has the '$preview_own_permission' permission, but is not the owner of the media item.");
             }
             return $access_result
-                ->cachePerPermissions()
                 ->cachePerUser()
                 ->addCacheableDependency($media);
 
@@ -170,7 +169,6 @@ function collabora_online_media_access(MediaInterface $media, string $operation,
                 $access_result = AccessResult::allowed();
             }
             return $access_result
-                ->cachePerPermissions()
                 ->cachePerUser()
                 ->addCacheableDependency($media);