IBX-9727: Added type-hints and adapted codebase to PHP8+ within contracts

Author: konradoboza

dependencies.json

@@ -1086,18 +1086,6 @@ parameters:
 			count: 1
 			path: src/bundle/Controller/URLWildcardController.php
 
-		-
-			message: '#^Parameter \#1 \$locationIds of method Ibexa\\Contracts\\AdminUi\\UniversalDiscovery\\Provider\:\:getLocations\(\) expects array\<int\>, list\<string\> given\.$#'
-			identifier: argument.type
-			count: 1
-			path: src/bundle/Controller/UniversalDiscoveryController.php
-
-		-
-			message: '#^Parameter \#1 \$locationList of class Ibexa\\AdminUi\\REST\\Value\\UniversalDiscovery\\LocationListData constructor expects array\<array\{location\: Ibexa\\Contracts\\Core\\Repository\\Values\\Content\\Location, permissions\: array\{create\: array\{hasAccess\: bool, restrictedContentTypeIds\: array\<int\>, restrictedLanguageCodes\: array\<string\>\}, edit\: array\{hasAccess\: bool, restrictedContentTypeIds\: array\<int\>, restrictedLanguageCodes\: array\<string\>\}\}\}\>, array\<Ibexa\\Contracts\\Core\\Repository\\Values\\Content\\Location\> given\.$#'
-			identifier: argument.type
-			count: 1
-			path: src/bundle/Controller/UniversalDiscoveryController.php
-
 		-
 			message: '#^Cannot access property \$languageCode on Ibexa\\Contracts\\Core\\Repository\\Values\\Content\\Language\|null\.$#'
 			identifier: property.nonObject
@@ -8707,7 +8695,7 @@ parameters:
 			path: src/lib/UniversalDiscovery/UniversalDiscoveryProvider.php
 
 		-
-			message: '#^Method Ibexa\\AdminUi\\UniversalDiscovery\\UniversalDiscoveryProvider\:\:getLocations\(\) should return array\<Ibexa\\Contracts\\Core\\Repository\\Values\\Content\\Location\> but returns list\<array\<string, array\<string, mixed\>\|Ibexa\\Contracts\\Core\\Repository\\Values\\Content\\Location\>\>\.$#'
+			message: '#^Method Ibexa\\AdminUi\\UniversalDiscovery\\UniversalDiscoveryProvider\:\:getLocations\(\) should return array\<array\{location\: Ibexa\\Contracts\\Core\\Repository\\Values\\Content\\Location, permissions\: array\{create\: array\{hasAccess\: bool, restrictedContentTypeIds\: array\<int\>, restrictedLanguageCodes\: array\<string\>\}, edit\: array\{hasAccess\: bool, restrictedContentTypeIds\: array\<int\>, restrictedLanguageCodes\: array\<string\>\}\}\}\> but returns list\<array\{location\: Ibexa\\Contracts\\Core\\Repository\\Values\\Content\\Location, permissions\: array\<string, mixed\>\}\>\.$#'
 			identifier: return.type
 			count: 1
 			path: src/lib/UniversalDiscovery/UniversalDiscoveryProvider.php
@@ -8760,6 +8748,12 @@ parameters:
 			count: 1
 			path: src/lib/UniversalDiscovery/UniversalDiscoveryProvider.php
 
+		-
+			message: '#^Parameter \#1 \$value of class Ibexa\\Contracts\\Core\\Repository\\Values\\Content\\Query\\Criterion\\LocationId constructor expects array\<int\>\|int, list\<string\> given\.$#'
+			identifier: argument.type
+			count: 1
+			path: src/lib/UniversalDiscovery/UniversalDiscoveryProvider.php
+
 		-
 			message: '#^Parameter \#2 \$locations of method Ibexa\\AdminUi\\UniversalDiscovery\\UniversalDiscoveryProvider\:\:moveSelectedLocationOnTop\(\) expects array, Ibexa\\Contracts\\Core\\Repository\\Values\\Content\\Location given\.$#'
 			identifier: argument.type

phpstan-baseline.neon

@@ -16,14 +16,11 @@
 use Ibexa\Rest\Server\Controller;
 use Symfony\Component\HttpFoundation\Request;
 
-class UniversalDiscoveryController extends Controller
+final class UniversalDiscoveryController extends Controller
 {
-    private Provider $provider;
-
     public function __construct(
-        Provider $provider
+        private readonly Provider $provider
     ) {
-        $this->provider = $provider;
     }
 
     public function locationsAction(Request $request): LocationListData

src/bundle/Controller/UniversalDiscoveryController.php

@@ -83,9 +83,23 @@ public function getLocationGridViewData(
     ): array;
 
     /**
-     * @param int[] $locationIds
+     * @param list<string> $locationIds
      *
-     * @return \Ibexa\Contracts\Core\Repository\Values\Content\Location[]
+     * @return array<array{
+     *     location: \Ibexa\Contracts\Core\Repository\Values\Content\Location,
+     *     permissions: array{
+     *       create: array{
+     *         hasAccess: bool,
+     *         restrictedContentTypeIds: array<int>,
+     *         restrictedLanguageCodes: array<string>
+     *       },
+     *       edit: array{
+     *         hasAccess: bool,
+     *         restrictedContentTypeIds: array<int>,
+     *         restrictedLanguageCodes: array<string>
+     *       }
+     *     }
+     * }>
      */
     public function getLocations(array $locationIds): array;