[Insight] exit() and die() functions should be avoided - in Bundle/…/Command/UpdateToI18nCommand.php, line 31

[FloSeas]

in Bundle/I18nBundle/Command/UpdateToI18nCommand.php, line 31

This line stops the execution flow, without explanation. If this is for debug, you should remove it. If this is to deal with an error, use exceptions instead.

    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $defaultLocale = $input->getArgument('default-locale');
        $defaultLocale = $defaultLocale ? $defaultLocale : 'fr';
        $this->doUpdate($output, $defaultLocale);
        exit;
    }

    protected function doUpdate(OutputInterface $output, $locale) 
    {
        $container = $this->getApplication()->getKernel()->getContainer();

---

Posted from SensioLabsInsight