Merge pull request #302 from jolicode/fix/api-platform-iri-collection

fix(api-platform): fix not returning iri when no property to map

Author: Korbeil

CHANGELOG.md

@@ -16,6 +16,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Changed
 - [BC Break] `PropertyTransformerSupportInterface` does not use a `TypesMatching` anymore, you can get the type directly from `SourcePropertyMetadata` or `TargetPropertyMetadata`.
 
+### Fixed
+ - [GH#303](https://github.com/jolicode/automapper/pull/302) Fix api platform not returning an iri when there is no property mapped.
+
 ## [9.5.0] - 2025-09-18
 ### Added
 - [GH#260](https://github.com/jolicode/automapper/pull/260) Add support for identifiers detection and comparison of objects, this allow mappers to detect if objects are equals based on some properties, which allow better deep merge / update of collections. 

castor.composer.lock

@@ -14,15 +14,15 @@
 #[AsTask('cs:check', namespace: 'qa', description: 'Check for coding standards without fixing them')]
 function qa_cs_check()
 {
-    php_cs_fixer(['fix', '--config', __DIR__ . '/.php-cs-fixer.php', '--dry-run', '--diff'], '3.85.1', [
+    php_cs_fixer(['fix', '--config', __DIR__ . '/.php-cs-fixer.php', '--dry-run', '--diff'], '3.92.3', [
         'kubawerlos/php-cs-fixer-custom-fixers' => '^3.21',
     ]);
 }
 
 #[AsTask('cs:fix', namespace: 'qa', description: 'Fix all coding standards', aliases: ['cs'])]
 function qa_cs_fix()
 {
-    php_cs_fixer(['fix', '--config', __DIR__ . '/.php-cs-fixer.php', '-v'], '3.85.1', [
+    php_cs_fixer(['fix', '--config', __DIR__ . '/.php-cs-fixer.php', '-v'], '3.92.3', [
         'kubawerlos/php-cs-fixer-custom-fixers' => '^3.21',
     ]);
 }

castor.php

@@ -1,6 +1,3 @@
-includes:
-	- phpstan-baseline.neon
-
 parameters:
 	level: max
 	paths:

phpstan-baseline.neon

@@ -4,7 +4,6 @@
 
 namespace AutoMapper\EventListener\ApiPlatform;
 
-use ApiPlatform\Api\ResourceClassResolverInterface as LegacyResourceClassResolverInterface;
 use ApiPlatform\Metadata\HttpOperation;
 use ApiPlatform\Metadata\Resource\Factory\ResourceMetadataCollectionFactoryInterface;
 use ApiPlatform\Metadata\ResourceClassResolverInterface;
@@ -21,7 +20,7 @@
 final readonly class JsonLdListener
 {
     public function __construct(
-        private ResourceClassResolverInterface|LegacyResourceClassResolverInterface $resourceClassResolver,
+        private ResourceClassResolverInterface $resourceClassResolver,
         private ResourceMetadataCollectionFactoryInterface $resourceMetadataCollectionFactory,
     ) {
     }

phpstan.neon

@@ -4,8 +4,6 @@
 
 namespace AutoMapper\Provider\ApiPlatform;
 
-use ApiPlatform\Api\IriConverterInterface as LegacyIriConverterInterface;
-use ApiPlatform\Api\ResourceClassResolverInterface as LegacyResourceClassResolverInterface;
 use ApiPlatform\Metadata\IriConverterInterface;
 use ApiPlatform\Metadata\ResourceClassResolverInterface;
 use AutoMapper\MapperContext;
@@ -15,8 +13,8 @@
 final readonly class IriProvider implements ProviderInterface
 {
     public function __construct(
-        private LegacyIriConverterInterface|IriConverterInterface $iriConverter,
-        private LegacyResourceClassResolverInterface|ResourceClassResolverInterface $resourceClassResolver,
+        private IriConverterInterface $iriConverter,
+        private ResourceClassResolverInterface $resourceClassResolver,
     ) {
     }
 

src/EventListener/ApiPlatform/JsonLdListener.php

@@ -9,6 +9,7 @@
 use AutoMapper\Provider\ApiPlatform\IriProvider;
 use AutoMapper\Transformer\ApiPlatform\JsonLdContextTransformer;
 use AutoMapper\Transformer\ApiPlatform\JsonLdIdTransformer;
+use AutoMapper\Transformer\ApiPlatform\JsonLdObjectToIdTransformerFactory;
 
 return static function (ContainerConfigurator $container) {
     $container->services()
@@ -36,5 +37,12 @@
                 service('api_platform.resource_class_resolver'),
             ])
             ->tag('automapper.provider', ['priority' => 0])
+
+        ->set(JsonLdObjectToIdTransformerFactory::class)
+            ->args([
+                service('api_platform.resource_class_resolver'),
+                service('serializer.mapping.class_metadata_factory'),
+            ])
+            ->tag('automapper.transformer_factory', ['priority' => -1003])
     ;
 };

src/Provider/ApiPlatform/IriProvider.php

@@ -53,16 +53,16 @@
             ->tag('automapper.transformer_factory', ['priority' => -1002])
 
         ->set(ObjectTransformerFactory::class)
-            ->tag('automapper.transformer_factory', ['priority' => -1003])
+            ->tag('automapper.transformer_factory', ['priority' => -1004])
 
         ->set(EnumTransformerFactory::class)
-            ->tag('automapper.transformer_factory', ['priority' => -1004])
+            ->tag('automapper.transformer_factory', ['priority' => -1005])
 
         ->set(MixedTransformerFactory::class)
-            ->tag('automapper.transformer_factory', ['priority' => -1005])
+            ->tag('automapper.transformer_factory', ['priority' => -1006])
 
         ->set(CopyTransformerFactory::class)
-            ->tag('automapper.transformer_factory', ['priority' => -1006])
+            ->tag('automapper.transformer_factory', ['priority' => -1007])
 
         ->set(SymfonyUidTransformerFactory::class)
     ;

src/Symfony/Bundle/Resources/config/api_platform.php

@@ -28,7 +28,7 @@ public function __construct(
 
     abstract protected function getAssignExpr(Expr $valuesVar, Expr $outputVar, Expr $loopKeyVar, bool $assignByRef): Expr;
 
-    public function transform(Expr $input, Expr $target, PropertyMetadata $propertyMapping, UniqueVariableScope $uniqueVariableScope, Expr\Variable $source, ?Expr $existingValue = null): array
+    public function transform(Expr $input, Expr $target, PropertyMetadata $propertyMapping, UniqueVariableScope $uniqueVariableScope, Expr $source, ?Expr $existingValue = null): array
     {
         /**
          * $values = [];.