Language::getCurrent() - Code Metrics - Inspection of "Feature formbuilder" - FloKnapp/faulancer - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Pull Request — master (#19)

by Flo

created 2017-08-23 01:03 UTC

Language::getCurrent() A

↳ Parent: Language

Complexity

Conditions	2
Paths	2

Size

Total Lines	12
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	12
rs	9.4285
c	0
b	0
f	0
cc	2
eloc	6
nc	2
nop	1

<?php
/**
 * Class LanguageLink | LanguageLink.php
 * @package Faulancer\View\Helper
 * @author  Florian Knapp <[email protected]>
 */
namespace Faulancer\View\Helper;

use Faulancer\Service\Config;
use Faulancer\Service\SessionManagerService;
use Faulancer\Session\SessionManager;
use Faulancer\View\AbstractViewHelper;

/**
 * Class LanguageLink
 */
class Language extends AbstractViewHelper
{

    /** @var array */
    protected $languageTextMapping = [
        'de' => 'Deutsch',
        'en' => 'English',
        'hr' => 'Hrvatski',
        'fr' => 'France'
    ];

    /**
     * @return Language $this
     */
    public function __invoke()
    {
        return $this;
    }

    /**
     * @param bool $codeOnly
     * @return string
     */
    public function getCurrent($codeOnly = true)
    {
        /** @var SessionManager $sessionManager */
        $sessionManager = $this->getServiceLocator()->get(SessionManagerService::class);
        $code           = $sessionManager->get('language') ?? 'de';


        if ($codeOnly) {
            return $code;
        }

        return $this->languageTextMapping[$code];
    }

    /**
     * @return string
     */
    public function getLinks()
    {
        /** @var Config $config */
        $config       = $this->getServiceLocator()->get(Config::class);
        $translations = $config->get('translation');

        $result  = [];
        $pattern = '<a rel="alternate" hreflang="%s" class="lang %s" href="?lang=%s">%s</a>';

        foreach ($translations as $key => $content) {
            $result[] = sprintf($pattern, strtolower($key), strtolower($key), $key, $this->languageTextMapping[$key]);
        }

        return implode('', $result);
    }

}

1			<?php
2			/**
3			* Class LanguageLink \| LanguageLink.php
4			* @package Faulancer\View\Helper
5			* @author Florian Knapp <[email protected]>
6			*/
7			namespace Faulancer\View\Helper;
8
9			use Faulancer\Service\Config;
10			use Faulancer\Service\SessionManagerService;
11			use Faulancer\Session\SessionManager;
12			use Faulancer\View\AbstractViewHelper;
13
14			/**
15			* Class LanguageLink
16			*/
17			class Language extends AbstractViewHelper
18			{
19
20			/** @var array */
21			protected $languageTextMapping = [
22			'de' => 'Deutsch',
23			'en' => 'English',
24			'hr' => 'Hrvatski',
25			'fr' => 'France'
26			];
27
28			/**
29			* @return Language $this
30			*/
31			public function __invoke()
32			{
33			return $this;
34			}
35
36			/**
37			* @param bool $codeOnly
38			* @return string
39			*/
40			public function getCurrent($codeOnly = true)
41			{
42			/** @var SessionManager $sessionManager */
43			$sessionManager = $this->getServiceLocator()->get(SessionManagerService::class);
44			$code = $sessionManager->get('language') ?? 'de';
			0 ignored issues – show Bug Compatibility introduced 2017-08-13 12:57 UTC by Report Bug Copy Issue Report The expression `$sessionManager->get('language') ?? 'de';` of type `string\|array` adds the type `array` to the return on line 47 which is incompatible with the return type documented by `Faulancer\View\Helper\Language::getCurrent` of type `string`. Loading history...
45
46			if ($codeOnly) {
47			return $code;
48			}
49
50			return $this->languageTextMapping[$code];
51			}
52
53			/**
54			* @return string
55			*/
56			public function getLinks()
57			{
58			/** @var Config $config */
59			$config = $this->getServiceLocator()->get(Config::class);
60			$translations = $config->get('translation');
61
62			$result = [];
63			$pattern = '<a rel="alternate" hreflang="%s" class="lang %s" href="?lang=%s">%s</a>';
64
65			foreach ($translations as $key => $content) {
66			$result[] = sprintf($pattern, strtolower($key), strtolower($key), $key, $this->languageTextMapping[$key]);
67			}
68
69			return implode('', $result);
70			}
71
72			}

FloKnapp / faulancer

Pull Request — master (#19)

Language::getCurrent() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like