Merge pull request #1729 from soyuka/fix-cache-subresource

dunglas · web-flow · commit e60243b99525 · 2018-02-23T11:40:29.000+01:00
fix route name resolver cache with subresources
diff --git a/src/Bridge/Symfony/Routing/CachedRouteNameResolver.php b/src/Bridge/Symfony/Routing/CachedRouteNameResolver.php
@@ -39,7 +39,8 @@ public function __construct(CacheItemPoolInterface $cacheItemPool, RouteNameReso
      */
     public function getRouteName(string $resourceClass, $operationType /**, array $context = []**/): string
     {
-        $cacheKey = self::CACHE_KEY_PREFIX.md5(serialize([$resourceClass, $operationType]));
+        $context = \func_num_args() > 2 ? func_get_arg(2) : [];
+        $cacheKey = self::CACHE_KEY_PREFIX.md5(serialize([$resourceClass, $operationType, $context['subresource_resources'] ?? null]));
 
         try {
             $cacheItem = $this->cacheItemPool->getItem($cacheKey);

Original file line number	Diff line number	Diff line change
`@@ -39,7 +39,8 @@ public function __construct(CacheItemPoolInterface $cacheItemPool, RouteNameReso`
`39`	`39`	`*/`
`40`	`40`	`public function getRouteName(string $resourceClass, $operationType /, array $context = []/): string`
`41`	`41`	`{`
`42`		`- $cacheKey = self::CACHE_KEY_PREFIX.md5(serialize([$resourceClass, $operationType]));`
	`42`	`+ $context = \func_num_args() > 2 ? func_get_arg(2) : [];`
	`43`	`+ $cacheKey = self::CACHE_KEY_PREFIX.md5(serialize([$resourceClass, $operationType, $context['subresource_resources'] ?? null]));`
`43`	`44`
`44`	`45`	`try {`
`45`	`46`	`$cacheItem = $this->cacheItemPool->getItem($cacheKey);`