From 564ea4e2e4e8d4b5b84ce275ee2bed8dfd26534a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Dawid=20Parafi=C5=84ski?= Date: Wed, 23 Oct 2024 12:14:12 +0200 Subject: [PATCH] IBX-8534: Dropped ContentService::loadRelations usage (#72) * IBX-8534: Dropped ContentService::loadRelations usage * Dropped ContentTypeService argument from service definition --- phpstan-baseline.neon | 10 ---------- .../Resources/config/services/resolvers.yaml | 1 - src/lib/Resolver/ContentResolver.php | 20 +++++++++---------- 3 files changed, 9 insertions(+), 22 deletions(-) diff --git a/phpstan-baseline.neon b/phpstan-baseline.neon index ed36eed..6d4ed56 100644 --- a/phpstan-baseline.neon +++ b/phpstan-baseline.neon @@ -605,11 +605,6 @@ parameters: count: 1 path: src/lib/Resolver/ContentResolver.php - - - message: "#^Method Ibexa\\\\GraphQL\\\\Resolver\\\\ContentResolver\\:\\:findContentRelations\\(\\) should return array\\ but returns iterable\\\\.$#" - count: 1 - path: src/lib/Resolver/ContentResolver.php - - message: "#^Method Ibexa\\\\GraphQL\\\\Resolver\\\\ContentResolver\\:\\:findContentReverseRelations\\(\\) has no return type specified\\.$#" count: 1 @@ -655,11 +650,6 @@ parameters: count: 1 path: src/lib/Resolver/ContentResolver.php - - - message: "#^Property Ibexa\\\\GraphQL\\\\Resolver\\\\ContentResolver\\:\\:\\$contentTypeService is never read, only written\\.$#" - count: 1 - path: src/lib/Resolver/ContentResolver.php - - message: "#^Method Ibexa\\\\GraphQL\\\\Resolver\\\\ContentThumbnailResolver\\:\\:resolveContentThumbnail\\(\\) return type has no value type specified in iterable type array\\.$#" count: 1 diff --git a/src/bundle/Resources/config/services/resolvers.yaml b/src/bundle/Resources/config/services/resolvers.yaml index d25fffc..907c594 100644 --- a/src/bundle/Resources/config/services/resolvers.yaml +++ b/src/bundle/Resources/config/services/resolvers.yaml @@ -27,7 +27,6 @@ services: arguments: $contentService: '@ibexa.siteaccessaware.service.content' $searchService: '@ibexa.siteaccessaware.service.search' - $contentTypeService: '@ibexa.siteaccessaware.service.content_type' Ibexa\GraphQL\Resolver\DomainContentResolver: tags: diff --git a/src/lib/Resolver/ContentResolver.php b/src/lib/Resolver/ContentResolver.php index 0033b1d..733b1aa 100644 --- a/src/lib/Resolver/ContentResolver.php +++ b/src/lib/Resolver/ContentResolver.php @@ -8,10 +8,11 @@ namespace Ibexa\GraphQL\Resolver; use Ibexa\Contracts\Core\Repository\ContentService; -use Ibexa\Contracts\Core\Repository\ContentTypeService; use Ibexa\Contracts\Core\Repository\SearchService; use Ibexa\Contracts\Core\Repository\Values\Content\ContentInfo; use Ibexa\Contracts\Core\Repository\Values\Content\Query; +use Ibexa\Contracts\Core\Repository\Values\Content\Relation; +use Ibexa\Contracts\Core\Repository\Values\Content\RelationList\RelationListItemInterface; use Ibexa\Contracts\Core\Repository\Values\Content\Search\SearchHit; use Ibexa\Contracts\Core\Repository\Values\Content\VersionInfo; @@ -30,16 +31,10 @@ class ContentResolver */ private $searchService; - /** - * @var \Ibexa\Contracts\Core\Repository\ContentTypeService - */ - private $contentTypeService; - - public function __construct(ContentService $contentService, SearchService $searchService, ContentTypeService $contentTypeService) + public function __construct(ContentService $contentService, SearchService $searchService) { $this->contentService = $contentService; $this->searchService = $searchService; - $this->contentTypeService = $contentTypeService; } public function findContentByType($contentTypeId) @@ -63,9 +58,12 @@ static function (SearchHit $searchHit) { */ public function findContentRelations(ContentInfo $contentInfo, $version = null) { - return $this->contentService->loadRelations( - $this->contentService->loadVersionInfo($contentInfo, $version) - ); + return array_filter(array_map( + static fn (RelationListItemInterface $relationListItem): ?Relation => $relationListItem->getRelation(), + $this->contentService->loadRelationList( + $this->contentService->loadVersionInfo($contentInfo, $version) + )->items + )); } public function findContentReverseRelations(ContentInfo $contentInfo)