bug #964 [StimulusBundle] Fixing bug where new custom controllers were not seen due to cache (weaverryan)

weaverryan · weaverryan · commit 6ee3f52b93d2 · 2023-06-26T13:46:34.000-04:00
This PR was merged into the 2.x branch. Discussion ---------- [StimulusBundle] Fixing bug where new custom controllers were not seen due to cache | Q | A | ------------- | --- | Bug fix? | yes | New feature? | no | Tickets | Fix #961 | License | MIT This requires symfony/symfony#50749 Cheers! Commits ------- 4503e3d [StimulusBundle] Fixing bug where new custom controlles were not seen due to cache
diff --git a/src/StimulusBundle/src/AssetMapper/ControllersMapGenerator.php b/src/StimulusBundle/src/AssetMapper/ControllersMapGenerator.php
@@ -48,6 +48,11 @@ public function getControllersJsonPath(): string
         return $this->controllersJsonPath;
     }
 
+    public function getControllerPaths(): array
+    {
+        return $this->controllerPaths;
+    }
+
     /**
      * @return array<string, MappedControllerAsset>
      */
diff --git a/src/StimulusBundle/src/AssetMapper/StimulusLoaderJavaScriptCompiler.php b/src/StimulusBundle/src/AssetMapper/StimulusLoaderJavaScriptCompiler.php
@@ -43,7 +43,13 @@ public function compile(string $content, MappedAsset $asset, AssetMapperInterfac
         $eagerControllerParts = [];
         $lazyControllers = [];
         $loaderPublicPath = $asset->publicPathWithoutDigest;
+
+        // add file dependencies so the cache rebuilds
         $asset->addFileDependency($this->controllersMapGenerator->getControllersJsonPath());
+        foreach ($this->controllersMapGenerator->getControllerPaths() as $controllerDir) {
+            $asset->addFileDependency($controllerDir);
+        }
+
         foreach ($this->controllersMapGenerator->getControllersMap() as $name => $mappedControllerAsset) {
             $controllerPublicPath = $mappedControllerAsset->asset->publicPathWithoutDigest;
             $relativeImportPath = $this->createRelativePath($loaderPublicPath, $controllerPublicPath);
