Goetas\TwitalBundle\Translation\TwitalExtractor

Complexity

Total Complexity

4

Size/Duplication

Total Lines	27
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	extract()	0	12	3

<?php

namespace Goetas\TwitalBundle\Translation;

use Goetas\Twital\TwitalLoader;
use Symfony\Bridge\Twig\Translation\TwigExtractor;
use Symfony\Component\Finder\Finder;
use Symfony\Component\Translation\MessageCatalogue;
use Twig\Environment;

/**
 *
 * @author Asmir Mustafic <goetas@gmail.com>
 *
 */
class TwitalExtractor extends TwigExtractor
{

    protected $twital;

    public function __construct(Environment $twig, TwitalLoader $twital)
    {
        parent::__construct($twig);
        $this->twital = $twital;
    }

    /**
     * {@inheritDoc}
     */
    public function extract($directory, MessageCatalogue $catalogue)
    {
        // load any existing translation files
        $finder = new Finder();
        $files = $finder->files()->in($directory);

It seems like $directory defined by parameter $directory on line 30 can also be of type array; however, Symfony\Component\Finder\Finder::in() does only seem to accept string|array<integer,string>, maybe add an additional type check?

        foreach ($files as $file) {
            if ($adapter = $this->twital->getSourceAdapter($file->getPathname())) {
                $source = $this->twital->getTwital()->compile($adapter, file_get_contents($file->getPathname()));
                $this->extractTemplate($source, $catalogue);
            }
        }
    }
}