PlaygroundCore\Service\Country

Complexity

Total Complexity

17

Size/Duplication

Total Lines	92
Duplicated Lines	14.13 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes	3
Bugs	1	Features	0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	getTranslatedTo()	0	4	1
A	setTranslatedTo()	0	6	1
A	getPath()	0	8	2
A	getCorePath()	0	8	2
A	getAllCountries()	7	17	4
B	getCountry()	6	24	6

<?php
namespace PlaygroundCore\Service;

use Zend\ServiceManager\ServiceManager;
use ZfcBase\EventManager\EventProvider;
use Zend\ServiceManager\ServiceLocatorInterface;

final class Country extends EventProvider
{
    private $translatedTo;

    private $path;

    private $corePath;

    /**
     *
     * @var ServiceManager
     */
    protected $serviceLocator;

    public function __construct(ServiceLocatorInterface $locator)
    {
        $this->serviceLocator = $locator;

$locator is of type object<Zend\ServiceManager\ServiceLocatorInterface>, but the property $serviceLocator was declared to be of type object<Zend\ServiceManager\ServiceManager>. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

    }

    public function getTranslatedTo()
    {
        return $this->translatedTo;
    }

    public function setTranslatedTo($translatedTo)
    {
        $this->translatedTo = (string) $translatedTo;

        return $this;
    }

    public function getPath()
    {
        if(empty($this->path)){
            $this->path = str_replace('\\', '/', getcwd()) . '/language/countries';
        }

        return $this->path;
    }

    public function getCorePath()
    {
        if(empty($this->corePath)){
            $this->corePath = __DIR__ . '/../../../language/countries';
        }

        return $this->corePath;
    }

    public function getAllCountries($translatedTo = null)
    {
        if (null === $translatedTo) {
            $translatedTo = $this->serviceLocator->get('translator')->getLocale();
        }
        
        $fileName = $this->getPath().'/'.$translatedTo.'.php';
        if (! file_exists($fileName)) {

This code seems to be duplicated across your project.

            $fileName = $this->getCorePath().'/'.$translatedTo.'.php';

            if (! file_exists($fileName)) {
                throw new \InvalidArgumentException("Language $translatedTo not found.");
            }
        }

        return include $fileName;
    }

    public function getCountry($country, $translatedTo = null)
    {
        if (null === $translatedTo) {
            $translatedTo = $this->serviceLocator->get('translator')->getLocale();
        }
        $fileName = $this->getPath().'/'.$translatedTo.'.php';
        if (! file_exists($fileName)) {

This code seems to be duplicated across your project.

            $fileName = $this->getCorePath().'/'.$translatedTo.'.php';
            if (! file_exists($fileName)) {
                throw new \InvalidArgumentException("Language $translatedTo not found.");
            }
        }

        $list = include $fileName;
        if (!is_array($list)) {
            throw new \InvalidArgumentException("Language $translatedTo not found.");
        }
        $country = strtoupper($country);
        if (!array_key_exists($country, $list)) {
            throw new \InvalidArgumentException("Country $country not found for $translatedTo.");
        }

        return $list[$country];
    }
}