Translation\Converter\Command\ConvertCommand

Complexity

Total Complexity

18

Size/Duplication

Total Lines	72
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	11

Test Coverage

Coverage

0%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	configure()	0	10	1
B	execute()	0	25	6
B	getReader()	0	25	11

<?php

/*
 * This file is part of the PHP Translation package.
 *
 * (c) PHP Translation team <tobias.nyholm@gmail.com>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Translation\Converter\Command;

use Symfony\Component\Console\Command\Command;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Input\InputOption;
use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\Translation\Loader;
use Translation\Converter\Reader\JmsReader;
use Translation\Converter\Service\Converter;

/**
 * @author Tobias Nyholm <tobias.nyholm@gmail.com>
 */
class ConvertCommand extends Command
{
    const SUPPORTED_FORMATS = ['jms', 'csv', 'ini', 'json', 'mo', 'php', 'yml', 'xlf'];

    protected function configure()
    {
        $this
            ->setName('translation:convert')
            ->setDescription('Convert your existing translation files to Xliff')
            ->addArgument('input_dir', InputArgument::REQUIRED, 'Where are your existing translations?')
            ->addArgument('output_dir', InputArgument::REQUIRED, 'Where should we put the new translations?')
            ->addOption('format', null, InputOption::VALUE_REQUIRED, 'The format to convert from. Supported formats are: '.implode(', ', self::SUPPORTED_FORMATS))
        ;
    }

    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $files = scandir($input->getArgument('input_dir'));

        $formats = [];
        $locales = [];
        foreach ($files as $file) {
            if (preg_match('|.+\.(.+)\.(.+)|', $file, $matches)) {
                $locales[] = $matches[1];
                $formats[] = $matches[2];
            }
        }

        if (null === $format = $input->getOption('format')) {
            $formats = array_unique($formats);
            if (1 !== count($formats)) {
                throw new \InvalidArgumentException('More than one format found. Please specify a format with --format=xxx');
            }
            $format = reset($formats);
        }

        $reader = $this->getReader($format);
        $converter = new Converter($reader, 'jms' === $format ? ['xlf', 'xliff'] : $format);
        $converter->convert($input->getArgument('input_dir'), $input->getArgument('output_dir'), array_unique($locales));

It seems like $input->getArgument('input_dir') targeting Symfony\Component\Console\Input\InputInterface::getArgument() can also be of type array<integer,string> or null; however, Translation\Converter\Service\Converter::convert() does only seem to accept string, maybe add an additional type check?

It seems like $input->getArgument('output_dir') targeting Symfony\Component\Console\Input\InputInterface::getArgument() can also be of type array<integer,string> or null; however, Translation\Converter\Service\Converter::convert() does only seem to accept string, maybe add an additional type check?

    }

    /**
     * @param string $format
     *
     * @return Loader\LoaderInterface
     */
    private function getReader($format)
    {
        switch ($format) {
            case 'jms':
                return new JmsReader();
            case 'csv':
                return new Loader\CsvFileLoader();
            case 'ini':
                return new Loader\IniFileLoader();
            case 'json':
                return new Loader\JsonFileLoader();
            case 'mo':
                return new Loader\MoFileLoader();
            case 'php':
                return new Loader\PhpFileLoader();
            case 'yml':
            case 'yaml':
                return new Loader\YamlFileLoader();
            case 'xlf':
            case 'xliff':
                return new Loader\XliffFileLoader();
            default:
                throw new \InvalidArgumentException(sprintf('Format "%s" is not a valid format', $format));
        }
    }
}