QueryDetector::get() - Code Metrics - Inspection of "Merge pull request #7 from thinktomorrow/2.0" - thinktomorrow/locale - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 0eceaa...aee3a6 )

by Ben

created 2018-04-19 07:02 UTC

QueryDetector::get() A

↳ Parent: QueryDetector

Complexity

Conditions	4
Paths	4

Size

Total Lines	12
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	6
CRAP Score	4

Importance

Changes

Metric	Value
cc	4
eloc	6
nc	4
nop	2
dl	0
loc	12
ccs	6
cts	6
cp	1
crap	4
rs	9.2
c	0
b	0
f	0

<?php

namespace Thinktomorrow\Locale\Detectors;

use Illuminate\Http\Request;
use Thinktomorrow\Locale\Scope;
use Thinktomorrow\Locale\Values\Config;
use Thinktomorrow\Locale\Values\Locale;

class QueryDetector implements Detector
{
    /**
     * @var Request
     */
    private $request;

    public function __construct(Request $request)

    {
        $this->request = $request;
    }

    public function get(Scope $scope, Config $config): ?Locale
    {
        if (!isset($config['query_key'])) {
            return null;
        }

        if (!$queryValue = $this->request->get($config['query_key'])) {
            return null;
        }

        return ($scope->validateLocale(Locale::from($queryValue))) ? Locale::from($queryValue) : null;
    }
}


1		<?php
2
3		namespace Thinktomorrow\Locale\Detectors;
4
5		use Illuminate\Http\Request;
6		use Thinktomorrow\Locale\Scope;
7		use Thinktomorrow\Locale\Values\Config;
8		use Thinktomorrow\Locale\Values\Locale;
9
10		class QueryDetector implements Detector
11		{
12		/**
13		* @var Request
14		*/
15		private $request;
16
17	64	public function __construct(Request $request)
		0 ignored issues – show Bug introduced 2018-02-12 08:23 UTC by Report Bug Copy Issue Report You have injected the Request via parameter `$request`. This is generally not recommended as there might be multiple instances during a request cycle (f.e. when using sub-requests). Instead, it is recommended to inject the RequestStack and retrieve the current request each time you need it via `getCurrentRequest()`. Loading history...
18		{
19	64	$this->request = $request;
20	64	}
21
22	64	public function get(Scope $scope, Config $config): ?Locale
23		{
24	64	if (!isset($config['query_key'])) {
25	1	return null;
26		}
27
28	63	if (!$queryValue = $this->request->get($config['query_key'])) {
29	61	return null;
30		}
31
32	3	return ($scope->validateLocale(Locale::from($queryValue))) ? Locale::from($queryValue) : null;
33		}
34		}
35

thinktomorrow / locale

Push — master ( 0eceaa...aee3a6 )

QueryDetector::get() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like