fix(symfony,laravel): IriConverter local cache key collision between item and collection ops (#7975)

soyuka · web-flow · commit 2f8249964883 · 2026-05-17T10:24:11.000+02:00
diff --git a/Routing/IriConverter.php b/Routing/IriConverter.php
@@ -122,7 +122,7 @@ public function getIriFromResource(object|string $resource, int $referenceType =
             $operation = $this->operationMetadataFactory->create($context['item_uri_template']);
         }
 
-        $localOperationCacheKey = ($operation?->getName() ?? '').$resourceClass.((\is_string($resource) || $operation instanceof CollectionOperationInterface) ? '_c' : '_i');
+        $localOperationCacheKey = ($operation?->getName() ?? '').$resourceClass.(\is_string($resource) ? '_s' : '_o').($operation instanceof CollectionOperationInterface ? '_c' : '_i');
         if ($operation && isset($this->localOperationCache[$localOperationCacheKey])) {
             return $this->generateSymfonyRoute($resource, $referenceType, $this->localOperationCache[$localOperationCacheKey], $context, $this->localIdentifiersExtractorOperationCache[$localOperationCacheKey] ?? null);
         }

Original file line number	Diff line number	Diff line change
`@@ -122,7 +122,7 @@ public function getIriFromResource(object\|string $resource, int $referenceType =`
`122`	`122`	`$operation = $this->operationMetadataFactory->create($context['item_uri_template']);`
`123`	`123`	`}`
`124`	`124`
`125`		`- $localOperationCacheKey = ($operation?->getName() ?? '').$resourceClass.((\is_string($resource) \|\| $operation instanceof CollectionOperationInterface) ? '_c' : '_i');`
	`125`	`+ $localOperationCacheKey = ($operation?->getName() ?? '').$resourceClass.(\is_string($resource) ? '_s' : '_o').($operation instanceof CollectionOperationInterface ? '_c' : '_i');`
`126`	`126`	`if ($operation && isset($this->localOperationCache[$localOperationCacheKey])) {`
`127`	`127`	`return $this->generateSymfonyRoute($resource, $referenceType, $this->localOperationCache[$localOperationCacheKey], $context, $this->localIdentifiersExtractorOperationCache[$localOperationCacheKey] ?? null);`
`128`	`128`	`}`