Apply php-cs-fixer changes

Author: lukmzig

src/SearchIndexAdapter/DefaultSearch/Workspace/QueryService.php

@@ -193,7 +193,8 @@ private function createWorkspacesGroupQuery(string $workspaceType, array $catego
         return $query;
     }
 
-    private function keepDeclinedPathsWithinAllowed(array $declinedPaths, array $allowedPaths): array {
+    private function keepDeclinedPathsWithinAllowed(array $declinedPaths, array $allowedPaths): array
+    {
         foreach ($declinedPaths as $index => $declinedPath) {
             $isIncluded = false;
             foreach ($allowedPaths as $allowedPath) {
@@ -222,6 +223,7 @@ private function addQueryByMainPath(BoolQuery $query, array $allowedPaths): void
                     ],
                 ]
             );
+
             return;
         }
 
@@ -410,7 +412,7 @@ private function getAllDeclinedParentPaths(array $declinedPaths, array $allowedP
             return [];
         }
 
-        $allowedParentPaths = $this->pathService->getAllParentPaths($allowedPaths,  false);
+        $allowedParentPaths = $this->pathService->getAllParentPaths($allowedPaths, false);
         $declinedParentPaths = [];
         foreach ($allowedParentPaths as $allowedParentPath) {
             foreach ($declinedPaths as $declinedPath) {
@@ -427,4 +429,4 @@ private function getAllDeclinedParentPaths(array $declinedPaths, array $allowedP
 
         return $declinedParentPaths;
     }
-}
+}

src/Service/PathService.php

@@ -130,4 +130,4 @@ private function getParentPath(string $path): string
 
         return implode('/', $pathParts);
     }
-}
+}

tests/Functional/Search/Modifier/Filter/Workspace/WorkspaceQueryHandlerTest.php

@@ -272,7 +272,7 @@ public function testHandleWorkspaceQueryCombineIncludeExclude(): void
             '/test-asset-folder-1',
             '/test-asset-folder-1/sub-folder-1',
             '/test-asset-folder-1/sub-folder-1/sub-sub-folder-1',
-            '/test-asset-folder-1/sub-folder-1/sub-sub-folder-1/sub-sub-sub-folder-1'
+            '/test-asset-folder-1/sub-folder-1/sub-sub-folder-1/sub-sub-sub-folder-1',
         ], $user);
 
         $user = $this->createUserWithAssetWorkspaces([
@@ -288,7 +288,7 @@ public function testHandleWorkspaceQueryCombineIncludeExclude(): void
             '/test-asset-folder-1',
             '/test-asset-folder-1/sub-folder-1',
             '/test-asset-folder-1/sub-folder-1/sub-sub-folder-1',
-            '/test-asset-folder-1/sub-folder-1/sub-sub-folder-1/sub-sub-sub-folder-1'
+            '/test-asset-folder-1/sub-folder-1/sub-sub-folder-1/sub-sub-sub-folder-1',
         ], $user);
 
         $user = $this->createUserWithAssetWorkspaces([

tests/Functional/Service/PermissionServiceTest.php

@@ -29,12 +29,19 @@
 class PermissionServiceTest extends Unit
 {
     protected IndexTester $tester;
+
     private User $user;
+
     private Asset $asset1;
+
     private Asset $asset2;
+
     private Folder $folder1;
+
     private Folder $folder2;
+
     private PermissionServiceInterface $permissionService;
+
     private AssetSearchServiceInterface $assetSearchService;
 
     protected function _before(): void
@@ -66,7 +73,7 @@ public function testBasicUserPermissions(): void
                     path: '/',
                     permissions: ['list' => true],
                     type: AssetWorkspace::WORKSPACE_TYPE
-                )
+                ),
             ]
         );
 
@@ -88,7 +95,7 @@ public function testBasicUserNestedPermissions(): void
                     path: $this->folder1->getFullPath(),
                     permissions: ['list' => false],
                     type: AssetWorkspace::WORKSPACE_TYPE
-                )
+                ),
             ]
         );
 
@@ -101,7 +108,6 @@ public function testBasicUserNestedPermissions(): void
         $asset1Permissions = $this->permissionService->getAssetPermissions($resultAsset1, $this->user);
         $asset2Permissions = $this->permissionService->getAssetPermissions($resultAsset2, $this->user);
 
-
         $this->assertTrue($rootPermissions->isList());
         $this->assertFalse($asset1Permissions->isList());
         $this->assertFalse($asset2Permissions->isList());
@@ -125,7 +131,7 @@ public function testUserPermissionsCombineIncludeExclude(): void
                     path: $this->folder2->getFullPath(),
                     permissions: ['list' => true],
                     type: AssetWorkspace::WORKSPACE_TYPE
-                )
+                ),
             ]
         );
 
@@ -142,7 +148,6 @@ public function testUserPermissionsCombineIncludeExclude(): void
         $folder2Permissions = $this->permissionService->getAssetPermissions($resultFolder2, $this->user);
         $asset2Permissions = $this->permissionService->getAssetPermissions($resultAsset2, $this->user);
 
-
         $this->assertTrue($rootPermissions->isList());
         $this->assertFalse($folder1Permissions->isList());
         $this->assertFalse($asset1Permissions->isList());

tests/Support/Helper/Service.php

@@ -24,12 +24,11 @@
 use Pimcore\Bundle\GenericDataIndexBundle\Permission\Workspace\DataObjectWorkspace;
 use Pimcore\Bundle\GenericDataIndexBundle\Permission\Workspace\DocumentWorkspace;
 use Pimcore\Model\User\Workspace\Asset as WorkspaceAsset;
-use Pimcore\Model\User\Workspace\Document as WorkspaceDocument;
 use Pimcore\Model\User\Workspace\DataObject as WorkspaceDataObject;
+use Pimcore\Model\User\Workspace\Document as WorkspaceDocument;
 
 class Service extends Module
 {
-
     /**
      * @throws InvalidArgumentException
      */