[Routing] Don't rebuild cache when controller action body changes

HypeMC · nicolas-grekas · commit 19961cd9a23b · 2025-09-04T16:28:41.000+02:00
diff --git a/Loader/AttributeClassLoader.php b/Loader/AttributeClassLoader.php
@@ -13,7 +13,7 @@
 
 use Symfony\Component\Config\Loader\LoaderInterface;
 use Symfony\Component\Config\Loader\LoaderResolverInterface;
-use Symfony\Component\Config\Resource\FileResource;
+use Symfony\Component\Config\Resource\ReflectionClassResource;
 use Symfony\Component\Routing\Attribute\DeprecatedAlias;
 use Symfony\Component\Routing\Attribute\Route as RouteAttribute;
 use Symfony\Component\Routing\Exception\InvalidArgumentException;
@@ -104,7 +104,7 @@ public function load(mixed $class, ?string $type = null): RouteCollection
 
         $globals = $this->getGlobals($class);
         $collection = new RouteCollection();
-        $collection->addResource(new FileResource($class->getFileName()));
+        $collection->addResource(new ReflectionClassResource($class));
         if ($globals['env'] && $this->env !== $globals['env']) {
             return $collection;
         }
diff --git a/Loader/AttributeDirectoryLoader.php b/Loader/AttributeDirectoryLoader.php
@@ -11,7 +11,7 @@
 
 namespace Symfony\Component\Routing\Loader;
 
-use Symfony\Component\Config\Resource\DirectoryResource;
+use Symfony\Component\Config\Resource\GlobResource;
 use Symfony\Component\Routing\RouteCollection;
 
 /**
@@ -33,7 +33,7 @@ public function load(mixed $path, ?string $type = null): ?RouteCollection
         }
 
         $collection = new RouteCollection();
-        $collection->addResource(new DirectoryResource($dir, '/\.php$/'));
+        $collection->addResource(new GlobResource($dir, '/*.php', true));
         $files = iterator_to_array(new \RecursiveIteratorIterator(
             new \RecursiveCallbackFilterIterator(
                 new \RecursiveDirectoryIterator($dir, \FilesystemIterator::SKIP_DOTS | \FilesystemIterator::FOLLOW_SYMLINKS),
diff --git a/Loader/AttributeFileLoader.php b/Loader/AttributeFileLoader.php
@@ -13,7 +13,7 @@
 
 use Symfony\Component\Config\FileLocatorInterface;
 use Symfony\Component\Config\Loader\FileLoader;
-use Symfony\Component\Config\Resource\FileResource;
+use Symfony\Component\Config\Resource\ReflectionClassResource;
 use Symfony\Component\Routing\RouteCollection;
 
 /**
@@ -52,7 +52,7 @@ public function load(mixed $file, ?string $type = null): ?RouteCollection
                 return null;
             }
 
-            $collection->addResource(new FileResource($path));
+            $collection->addResource(new ReflectionClassResource($refl));
             $collection->addCollection($this->loader->load($class, $type));
         }
 

Original file line number	Diff line number	Diff line change
`@@ -13,7 +13,7 @@`
`13`	`13`
`14`	`14`	`use Symfony\Component\Config\FileLocatorInterface;`
`15`	`15`	`use Symfony\Component\Config\Loader\FileLoader;`
`16`		`-use Symfony\Component\Config\Resource\FileResource;`
	`16`	`+use Symfony\Component\Config\Resource\ReflectionClassResource;`
`17`	`17`	`use Symfony\Component\Routing\RouteCollection;`
`18`	`18`
`19`	`19`	`/**`
`@@ -52,7 +52,7 @@ public function load(mixed $file, ?string $type = null): ?RouteCollection`
`52`	`52`	`return null;`
`53`	`53`	`}`
`54`	`54`
`55`		`- $collection->addResource(new FileResource($path));`
	`55`	`+ $collection->addResource(new ReflectionClassResource($refl));`
`56`	`56`	`$collection->addCollection($this->loader->load($class, $type));`
`57`	`57`	`}`
`58`	`58`