Goetas\TwitalBundle\Translation\Jms\TwitalExtractor

Complexity

Total Complexity

6

Size/Duplication

Total Lines	53
Duplicated Lines	41.51 %

Coupling/Cohesion

Components	1
Dependencies	5

Test Coverage

Coverage

26.32%

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	visitPhpFile()	0	3	1
A	visitTwigFile()	11	11	2
A	visitFile()	11	11	2

<?php

namespace Goetas\TwitalBundle\Translation\Jms;

use Goetas\Twital\TwitalLoader;
use JMS\TranslationBundle\Model\MessageCatalogue;
use JMS\TranslationBundle\Translation\Extractor\File\TwigFileExtractor;
use JMS\TranslationBundle\Translation\Extractor\FileVisitorInterface;
use Twig\Source;

/**
 *
 * @author Asmir Mustafic <goetas@gmail.com>
 *
 */
class TwitalExtractor implements FileVisitorInterface
{
    /**
     *
     * @var TwigFileExtractor
     */
    private $twigFileExtractor;
    /**
     *
     * @var \Goetas\Twital\TwitalLoader
     */
    private $twitalLoader;
    /**
     *
     * @var \Twig_Environment
     */
    private $twig;

    public function __construct(\Twig_Environment $twig, TwitalLoader $twitalLoader, TwigFileExtractor $twigFileExtractor)
    {
        $this->twig = $twig;
        $this->twitalLoader = $twitalLoader;
        $this->twigFileExtractor = $twigFileExtractor;
    }

    public function visitPhpFile(\SplFileInfo $file, MessageCatalogue $catalogue, array $ast)
    {
    }

    public function visitTwigFile(\SplFileInfo $file, MessageCatalogue $catalogue, \Twig_Node $ast)

This method seems to be duplicated in your project.

    {
        if ($adapter = $this->twitalLoader->getSourceAdapter((string)$file)) {

            $source = $this->twitalLoader->getTwital()->compile($adapter, file_get_contents((string)$file));

            $ast = $this->twig->parse($this->twig->tokenize(new Source($source, (string)$file)));
        }

        $this->twigFileExtractor->visitTwigFile($file, $catalogue, $ast);
    }

    function visitFile(\SplFileInfo $file, MessageCatalogue $catalogue)

It is generally recommended to explicitly declare the visibility for methods.

This method seems to be duplicated in your project.

    {
        if ($adapter = $this->twitalLoader->getSourceAdapter((string)$file)) {

            $source = $this->twitalLoader->getTwital()->compile($adapter, file_get_contents((string)$file));

            $ast = $this->twig->parse($this->twig->tokenize(new Source($source, (string)$file)));

            $this->twigFileExtractor->visitTwigFile($file, $catalogue, $ast);
        }
    }
}