Code review fixes - mostly FQCN

Author: ViniTou

src/lib/Event/ContentService.php

@@ -133,12 +133,8 @@ public function deleteContent(ContentInfo $contentInfo): iterable
             ? $beforeEvent->getLocations()
             : $this->innerService->deleteContent($contentInfo);
 
-        if (!is_array($locations)) {
-            $locations = iterator_to_array($locations);
-        }
-
         $this->eventDispatcher->dispatch(
-            new DeleteContentEvent($locations, ...$eventData)
+            new DeleteContentEvent(iterator_to_array($locations), ...$eventData)
         );
 
         return $locations;

src/lib/Persistence/Cache/TrashHandler.php

@@ -36,7 +36,7 @@ public function trashSubtree($locationId)
 
         $location = $this->persistenceHandler->locationHandler()->load($locationId);
         $limit = $this->persistenceHandler->contentHandler()->countRelations(
-            $location->contentId,
+            $location->contentId
         );
 
         $reverseRelations = $this->persistenceHandler->contentHandler()->loadRelationList(
@@ -79,7 +79,7 @@ public function recover($trashedId, $newParentId)
         $location = $this->persistenceHandler->locationHandler()->load($return);
 
         $limit = $this->persistenceHandler->contentHandler()->countRelations(
-            $location->contentId,
+            $location->contentId
         );
 
         $reverseRelations = $this->persistenceHandler->contentHandler()->loadRelationList(

src/lib/Repository/Helper/RelationProcessor.php

@@ -190,7 +190,7 @@ public function processFieldRelations(
                 switch ($relationType) {
                     case Relation::FIELD:
                     case Relation::ASSET:
-                        /** @phpstan-var array<int, Relation> $relationEntry */
+                        /** @phpstan-var array<int, \Ibexa\Core\Repository\Values\Content\Relation> $relationEntry */
                         foreach ($relationEntry as $relation) {
                             $this->persistenceHandler->contentHandler()->removeRelation(
                                 $relation->id,
@@ -201,7 +201,7 @@ public function processFieldRelations(
                         break;
                     case Relation::LINK:
                     case Relation::EMBED:
-                        /** @phpstan-var Relation $relationEntry */
+                        /** @phpstan-var \Ibexa\Core\Repository\Values\Content\Relation $relationEntry */
                         $this->persistenceHandler->contentHandler()->removeRelation(
                             $relationEntry->id,
                             $relationType,

src/lib/Repository/Permission/CachedPermissionService.php

@@ -52,7 +52,7 @@ class CachedPermissionService implements PermissionService
      *
      * Value is null if not yet set or cleared.
      *
-     * @var null|bool|\Ibexa\Contracts\Core\Repository\Values\Content\Query\CriterionInterface
+     * @var bool|\Ibexa\Contracts\Core\Repository\Values\Content\Query\CriterionInterface|null
      */
     private $permissionCriterion;
 

tests/integration/Core/Repository/SearchServiceTest.php

@@ -4864,19 +4864,19 @@ protected function simplifySearchResult(SearchResult $result)
                 case $hit->valueObject instanceof Location:
                     /** @phpstan-ignore assign.propertyType */
                     $hit->valueObject = [
-                        /** @phpstan-var Location|Content $hit->valueObject */
+                        /** @phpstan-var \Ibexa\Contracts\Core\Repository\Values\Content\Location|\Ibexa\Contracts\Core\Repository\Values\Content\Content $hit->valueObject */
                         'id' => $hit->valueObject->contentInfo->getId(),
-                        /** @phpstan-var Location|Content $hit->valueObject */
+                        /** @phpstan-var \Ibexa\Contracts\Core\Repository\Values\Content\Location|\Ibexa\Contracts\Core\Repository\Values\Content\Content $hit->valueObject */
                         'title' => $hit->valueObject->contentInfo->getName(),
                     ];
                     break;
 
                 case $hit->valueObject instanceof ContentInfo:
                     /** @phpstan-ignore assign.propertyType */
                     $hit->valueObject = [
-                        /** @phpstan-var ContentInfo $hit->valueObject */
+                        /** @phpstan-var \Ibexa\Contracts\Core\Repository\Values\Content\ContentInfo $hit->valueObject */
                         'id' => $hit->valueObject->id,
-                        /** @phpstan-var ContentInfo $hit->valueObject */
+                        /** @phpstan-var \Ibexa\Contracts\Core\Repository\Values\Content\ContentInfo $hit->valueObject */
                         'title' => $hit->valueObject->name,
                     ];
                     break;