Ffcms\Core\Network\Request\MultiLanguageFeatures

Complexity

Total Complexity

20

Size/Duplication

Total Lines	107
Duplicated Lines	0 %

Importance

Changes	3
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	setLanguage()	0	8	2
A	setLanguageFromBrowser()	0	24	6
A	languageInPath()	0	3	1
B	runMultiLanguage()	0	34	10
A	getLanguage()	0	3	1

<?php

namespace Ffcms\Core\Network\Request;

use Ffcms\Core\App;
use Ffcms\Core\Helper\Type\Any;
use Ffcms\Core\Helper\Type\Arr;
use Ffcms\Core\Helper\Type\Str;
use Ffcms\Templex\Url\UrlRepository;

The type Ffcms\Templex\Url\UrlRepository was not found. Maybe you did not declare it correctly or list all dependencies?

use Symfony\Component\HttpFoundation\ParameterBag;
use Symfony\Component\HttpFoundation\RedirectResponse;

/**
 * Trait MultiLanguageFeatures. Multilanguage features for Request controller.
 * @package Ffcms\Core\Network\Request
 * @property ParameterBag $query
 */
trait MultiLanguageFeatures
{
    protected $language;
    protected $languageInPath = false;

    /**
     * Build multi language pathway binding.
     * @return void
     */
    private function runMultiLanguage(): void
    {
        // check if multi-language is enabled
        if (!App::$Properties->get('multiLanguage')) {
            $this->language = App::$Properties->get('singleLanguage');
            return;
        }

        // check if domain-lang binding is enabled
        if (Any::isArray(App::$Properties->get('languageDomainAlias'))) {
            /** @var array $domainAlias */
            $domainAlias = App::$Properties->get('languageDomainAlias');
            if (Any::isArray($domainAlias) && !Str::likeEmpty($domainAlias[$this->getHost()])) {

It seems like getHost() must be provided by classes using this trait. How about adding it as abstract method to this trait?

                $this->language = $domainAlias[$this->getHost()];
            }
            return;
        }

        // try to find language in pathway
        foreach (App::$Properties->get('languages') as $lang) {
            if (Str::startsWith('/' . $lang, $this->getPathInfo())) {

It seems like getPathInfo() must be provided by classes using this trait. How about adding it as abstract method to this trait?

                $this->language = $lang;
                $this->languageInPath = true;
            }
        }

        // try to find in ?lang get
        if (!$this->language && Arr::in($this->query->get('lang'), App::$Properties->get('languages'))) {

It seems like Ffcms\Core\App::Properties->get('languages') can also be of type false; however, parameter $haystack of Ffcms\Core\Helper\Type\Arr::in() does only seem to accept array|null, maybe add an additional type check?

            $this->language = $this->query->get('lang');
        }

        // language still not defined?!
        if (!$this->language) {
            $this->setLanguageFromBrowser();
        }
    }

    /**
     * Set language from browser headers
     * @return void
     */
    private function setLanguageFromBrowser(): void
    {
        $userLang = App::$Properties->get('singleLanguage');
        $browserAccept = $this->getLanguages();

The method getLanguages() does not exist on Ffcms\Core\Network\Request\MultiLanguageFeatures. Did you maybe mean getLanguage()?

        if (Any::isArray($browserAccept) && count($browserAccept) > 0) {
            foreach ($browserAccept as $bLang) {
                if (Arr::in($bLang, App::$Properties->get('languages'))) {

It seems like Ffcms\Core\App::Properties->get('languages') can also be of type false; however, parameter $haystack of Ffcms\Core\Helper\Type\Arr::in() does only seem to accept array|null, maybe add an additional type check?

                    $userLang = $bLang;
                    break; // stop calculating, language is founded in priority
                }
            }
        }

        // parse query string
        $queryString = null;
        if (count($this->query->all()) > 0) {
            $queryString = '?' . http_build_query($this->query->all());
        }

        // build response with redirect to language-based path
        $redirectUrl = $this->getSchemeAndHttpHost() . $this->basePath . '/' . $userLang . $this->getPathInfo() . $queryString;

It seems like getSchemeAndHttpHost() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $response = new RedirectResponse($redirectUrl);
        $response->send();
        exit();

Using exit here is not recommended.

    }

    /**
     * Get current language
     * @return string|null
     */
    public function getLanguage(): ?string
    {
        return $this->language;
    }

    /**
     * Set current language
     * @param string $lang
     * @return bool
     */
    public function setLanguage($lang): bool
    {
        if (Arr::in($lang, App::$Properties->get('languages'))) {

It seems like Ffcms\Core\App::Properties->get('languages') can also be of type false; however, parameter $haystack of Ffcms\Core\Helper\Type\Arr::in() does only seem to accept array|null, maybe add an additional type check?

            $this->language = $lang;
            return true;
        }

        return false;
    }

    /**
     * Check if language used in path
     * @return bool
     */
    public function languageInPath(): bool
    {
        return (bool)$this->languageInPath;
    }
}