Zikula\Bundle\CoreBundle\Command\TranslationKeyToValueCommand::execute()

Complexity

Conditions	11
Paths	140

Size

Total Lines	49
Code Lines	33

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

<?php
    protected function execute(InputInterface $input, OutputInterface $output): int
    {
        $io = new SymfonyStyle($input, $output);

        /** @var KernelInterface $kernel */
        $kernel = $this->getApplication()->getKernel();

The method getKernel() does not exist on Symfony\Component\Console\Application. It seems like you code against a sub-type of Symfony\Component\Console\Application such as Symfony\Bundle\FrameworkBundle\Console\Application.

        $transPaths = [];
        if ($this->defaultTransPath) {
            $transPaths[] = $this->defaultTransPath;
        }
        $currentName = 'default directory';
        // Override with provided Bundle info (this section copied from symfony's translation:update command)
        if (null !== $input->getArgument('bundle')) {
            try {
                $foundBundle = $kernel->getBundle($input->getArgument('bundle'));

It seems like $input->getArgument('bundle') can also be of type null and string[]; however, parameter $name of Symfony\Component\HttpKernel\KernelInterface::getBundle() does only seem to accept string, maybe add an additional type check?

                $bundleDir = $foundBundle->getPath();
                $transPaths = [is_dir($bundleDir . '/Resources/translations') ? $bundleDir . '/Resources/translations' : $bundleDir . '/translations'];
                if ($this->defaultTransPath) {
                    $transPaths[] = $this->defaultTransPath;
                }
                $currentName = $foundBundle->getName();
            } catch (\InvalidArgumentException $e) {
                // such a bundle does not exist, so treat the argument as path
                $path = $input->getArgument('bundle');
                $transPaths = [$path . '/translations'];

Are you sure $path of type null|string|string[] can be used in concatenation?

                if (!is_dir($transPaths[0]) && !isset($transPaths[1])) {
                    throw new InvalidArgumentException(sprintf('"%s" is neither an enabled bundle nor a directory.', $transPaths[0]));
                }
            }
        }
        $io->title('Translation Messages Key to Value Transformer');
        $io->comment(sprintf('Transforming translation files for "<info>%s</info>"', $currentName));
        $finder = new Finder();
        $fs = new Filesystem();
        foreach ($finder->files()->in($transPaths)->name(['*.yaml', '*.yml']) as $file) {
            $io->text(sprintf('<comment>Parsing %s</comment>', $file->getBasename()));
            $messages = Yaml::parseFile($file->getRealPath());
            foreach ($messages as $key => $message) {
                if (null === $message) {
                    $messages[$key] = $key;
                }
            }
            $io->text(sprintf('<info>Dumping %s</info>', $file->getBasename()));
            $fs->dumpFile($file->getRealPath(), Yaml::dump($messages));
        }

        $io->success('Success!');

        return Command::SUCCESS;
    }