bug #23195 [FrameworkBundle] [Command] Clean bundle directory, fixes #…

…23177 (NicolasPion) This PR was squashed before being merged into the 2.7 branch (closes #23195). Discussion ---------- [FrameworkBundle] [Command] Clean bundle directory, fixes #23177 | Q | A | ------------- | --- | Branch? | 2.7  | Bug fix? | yes | New feature? | no  | BC breaks? | no | Deprecations? | no  | Tests pass? | no | Fixed tickets | #23177 | License | MIT This PR fix #23177 when running an assets:install, it will remove directorys who do not have anymore a valid Bundle Commits ------- 180f178 [FrameworkBundle] [Command] Clean bundle directory, fixes #23177
symfony · Jun 16, 2017 · 436d5e4 · 436d5e4
2 parents c8884e7 + 180f178
commit 436d5e4
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/src/Symfony/Bundle/FrameworkBundle/Command/AssetsInstallCommand.php b/src/Symfony/Bundle/FrameworkBundle/Command/AssetsInstallCommand.php
@@ -89,6 +89,7 @@ protected function execute(InputInterface $input, OutputInterface $output)
             $output->writeln('Installing assets as <comment>hard copies</comment>.');
         }
 
+        $validAssetDirs = array();
         foreach ($this->getContainer()->get('kernel')->getBundles() as $bundle) {
             if (is_dir($originDir = $bundle->getPath().'/Resources/public')) {
                 $targetDir = $bundlesDir.preg_replace('/bundle$/', '', strtolower($bundle->getName()));
@@ -131,6 +132,13 @@ protected function execute(InputInterface $input, OutputInterface $output)
                 } else {
                     $this->hardCopy($originDir, $targetDir);
                 }
+                $validAssetDirs[] = $targetDir;
+            }
+        }
+        // remove the assets of the bundles that no longer exist
+        foreach (new \FilesystemIterator($bundlesDir) as $dir) {
+            if (!in_array($dir, $validAssetDirs)) {
+                $filesystem->remove($dir);
             }
         }
     }