diff --git a/src/Symfony/Bundle/FrameworkBundle/Command/CacheClearCommand.php b/src/Symfony/Bundle/FrameworkBundle/Command/CacheClearCommand.php index faf58103e8a7..28dac98e3591 100644 --- a/src/Symfony/Bundle/FrameworkBundle/Command/CacheClearCommand.php +++ b/src/Symfony/Bundle/FrameworkBundle/Command/CacheClearCommand.php @@ -66,8 +66,7 @@ protected function execute(InputInterface $input, OutputInterface $output) } $kernel = $this->getContainer()->get('kernel'); - $output->writeln($this->getClearingCacheMessage($output, $kernel)); - + $output->writeln(sprintf('Clearing the cache for the %s environment with debug %s', $kernel->getEnvironment(), var_export($kernel->isDebug(), true))); $this->getContainer()->get('cache_clearer')->clear($realCacheDir); if ($input->getOption('no-warmup')) { @@ -78,11 +77,15 @@ protected function execute(InputInterface $input, OutputInterface $output) $warmupDir = substr($realCacheDir, 0, -1).'_'; if ($filesystem->exists($warmupDir)) { - $this->writelnIfVerbose($output, 'Clearing outdated warmup directory...'); + if ($output->isVerbose()) { + $output->writeln(' Clearing outdated warmup directory'); + } $filesystem->remove($warmupDir); } - $this->writelnIfVerbose($output, 'Warming up cache...'); + if ($output->isVerbose()) { + $output->writeln(' Warming up cache'); + } $this->warmup($warmupDir, $realCacheDir, !$input->getOption('no-optional-warmers')); $filesystem->rename($realCacheDir, $oldCacheDir); @@ -90,43 +93,19 @@ protected function execute(InputInterface $input, OutputInterface $output) sleep(1); // workaround for Windows PHP rename bug } $filesystem->rename($warmupDir, $realCacheDir); - $this->writelnIfVerbose($output, 'Warm up completed...'); } - $this->writelnIfVerbose($output, 'Removing old cache directory...'); - $filesystem->remove($oldCacheDir); - $this->writelnIfVerbose($output, 'Completed clearing cache' . ($input->getOption('no-warmup') ? "!" : " and warmup!")); - } + if ($output->isVerbose()) { + $output->writeln(' Removing old cache directory'); + } - /** - * @param OutputInterface $output - * @param KernelInterface $kernel - * @return string - */ - protected function getClearingCacheMessage(OutputInterface $output, KernelInterface $kernel){ - $message = 'Clearing the cache for the %s environment with debug %s'; - $message .= $this->isVerbose($output) ? ":" : ""; - return sprintf($message, $kernel->getEnvironment(), var_export($kernel->isDebug(), true)); - } + $filesystem->remove($oldCacheDir); - /** - * @param OutputInterface $output - * @param string $message - */ - protected function writelnIfVerbose(OutputInterface $output, $message){ - if ($this->isVerbose($output)){ - $output->writeln($message); + if ($output->isVerbose()) { + $output->writeln(' Done'); } } - /** - * @param OutputInterface $output - * @return bool - */ - protected function isVerbose(OutputInterface $output){ - return OutputInterface::VERBOSITY_VERBOSE <= $output->getVerbosity(); - } - /** * @param string $warmupDir * @param string $realCacheDir