Gettext\Extractors\PhpArray

Complexity

Total Complexity

10

Size/Duplication

Total Lines	60
Duplicated Lines	20 %

Coupling/Cohesion

Components	0
Dependencies	2

Importance

Changes	4
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	fromString()	0	4	1
A	fromFile()	12	12	3
B	extract()	0	22	6

<?php

namespace Gettext\Extractors;

use Exception;
use Gettext\Translations;

/**
 * Class to get gettext strings from php files returning arrays.
 */
class PhpArray extends Extractor implements ExtractorInterface
{
    /**
     * Extract the translations from a file.
     *
     * @param array|string      $file         A path of a file or files
     * @param null|Translations $translations The translations instance to append the new translations.
     *
     * @return Translations
     */
    public static function fromFile($file, Translations $translations = null)

This method seems to be duplicated in your project.

    {
        if ($translations === null) {
            $translations = new Translations();
        }

        foreach (static::getFiles($file) as $file) {
            static::extract(include($file), $translations);
        }

        return $translations;
    }

    /**
     * {@inheritdoc}
     */
    public static function fromString($string, Translations $translations = null, $file = '')
    {
        throw new Exception('PhpArray::fromString() cannot be called. Use PhpArray::fromFile()');
    }

    /**
     * Handle an array of translations and append to the Translations instance.
     *
     * @param array        $content
     * @param Translations $translations
     */
    public static function extract(array $content, Translations $translations)
    {
        if (!empty($content['domain'])) {
            $translations->setDomain($content['domain']);
        }

        if (!empty($content['lang'])) {
            $translations->setLanguage($content['language']);
        }

        if (!empty($content['plural-forms'])) {
            $translations->setPluralForms($content['plural-forms']);

The call to setPluralForms() misses a required argument $rule.

        }

        foreach ($content['messages'] as $context => $messages) {
            foreach ($messages as $original => $translations) {
                $translations->insert($context, $original)
                    ->setTranslation(array_shift($translations))
                    ->setPluralTranslations($translations);
            }
        }
    }
}