Locastic\ApiPlatformTranslationBundle\Translation\Translator

Complexity

Total Complexity

11

Size/Duplication

Total Lines	96
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	trans()	0	8	2
A	transChoice()	0	8	2
A	loadCurrentLocale()	0	19	4
A	setLocale()	0	4	1
A	getLocale()	0	4	1

<?php

declare(strict_types=1);

namespace Locastic\ApiPlatformTranslationBundle\Translation;

use Symfony\Component\HttpFoundation\RequestStack;
use Symfony\Component\Translation\TranslatorInterface;

/**
 * Class Translator
 *
 * @package Locastic\ApiPlatformTranslationBundle\Translation
 */
class Translator implements TranslatorInterface
{
    /**
     * @var TranslatorInterface
     */
    private $translator;

    /**
     * @var RequestStack
     */
    private $requestStack;

    /**
     * @var string
     */
    private $defaultLocale;

    /**
     * Translator constructor.
     *
     * @param TranslatorInterface $translator
     * @param RequestStack $requestStack
     * @param string $defaultLocale
     */
    public function __construct(TranslatorInterface $translator, RequestStack $requestStack, string $defaultLocale)
    {
        $this->translator = $translator;
        $this->requestStack = $requestStack;
        $this->defaultLocale = $defaultLocale;
    }

    /**
     * {@inheritdoc}
     */
    public function trans($id, array $parameters = array(), $domain = null, $locale = null): string
    {
        if (!$locale) {

The expression $locale of type string|null is loosely compared to false; this is ambiguous if the string can be empty. You might want to explicitly use === null instead.

            $locale = $this->loadCurrentLocale();

Consider using a different name than the parameter $locale. This often makes code more readable.

        }

        return $this->translator->trans($id, $parameters, $domain, $locale);
    }

    /**
     * {@inheritdoc}
     */
    public function transChoice($id, $number, array $parameters = array(), $domain = null, $locale = null): string
    {
        if (!$locale) {

The expression $locale of type string|null is loosely compared to false; this is ambiguous if the string can be empty. You might want to explicitly use === null instead.

            $locale = $this->loadCurrentLocale();

Consider using a different name than the parameter $locale. This often makes code more readable.

        }

        return $this->translator->transChoice($id, $number, $parameters, $domain, $locale);
    }

    /**
     * @return mixed|string
     */
    public function loadCurrentLocale(): string
    {
        $request = $this->requestStack->getCurrentRequest();

        if (!$request) {
            return $this->defaultLocale;
        }

        $localeCode = $request->query->get('locale');

        if (!$localeCode) {
            $preferredLanguage = $request->getPreferredLanguage();
            $preferredLanguage = '' === $preferredLanguage ? null : $preferredLanguage;

            return $preferredLanguage ?? $this->defaultLocale;
        }

        return $localeCode;
    }

    /**
     * {@inheritdoc}
     * @codeCoverageIgnore
     */
    public function setLocale($locale = 'en'): void
    {
        $this->translator->setLocale($locale);
    }

    /**
     * {@inheritdoc}
     * @codeCoverageIgnore
     */
    public function getLocale(): string
    {
        return $this->translator->getLocale();
    }
}