LocaleSelector::handle() - Code Metrics - Inspection of "update .scrutinizer.yml" - kingscode/laravel-api-starter - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Push — master ( ffc71e...1a0a2f )

by Koen

created 2019-07-31 14:45 UTC

LocaleSelector::handle() A

↳ Parent: LocaleSelector

Complexity

Conditions	2
Paths	2

Size

Total Lines	11
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	2
eloc	5
nc	2
nop	2
dl	0
loc	11
rs	10
c	0
b	0
f	0

<?php

namespace App\Http\Middleware;

use Closure;
use Illuminate\Contracts\Config\Repository;
filter:
    dependency_paths: ["lib/*"]
use Illuminate\Contracts\Translation\Translator;
filter:
    dependency_paths: ["lib/*"]
use Illuminate\Http\Request;
filter:
    dependency_paths: ["lib/*"]

final class LocaleSelector
{
    /**
     * @var \Illuminate\Contracts\Config\Repository
     */
    protected $config;

    /**
     * @var \Illuminate\Contracts\Translation\Translator
     */
    protected $translator;

    /**
     * SetLocaleFromHeader constructor.
     *
     * @param  \Illuminate\Contracts\Config\Repository      $config
     * @param  \Illuminate\Contracts\Translation\Translator $translator
     */
    public function __construct(Repository $config, Translator $translator)
    {
        $this->config = $config;
        $this->translator = $translator;
    }

    /**
     * Handle an incoming request.
     *
     * @param  \Illuminate\Http\Request $request
     * @param  \Closure                 $next
     * @return mixed
     */
    public function handle(Request $request, Closure $next)
    {
        if ($request->headers->has('Accept-Language')) {
            $locale = $request->headers->get('Accept-Language');

            $this->config->set('app.locale', $locale);

            $this->translator->setLocale($locale);
        }

        return $next($request);
    }
}


1			<?php
2
3			namespace App\Http\Middleware;
4
5			use Closure;
6			use Illuminate\Contracts\Config\Repository;
			0 ignored issues – show Bug introduced 2019-07-31 14:49 UTC by Report Bug Copy Issue Report The type `Illuminate\Contracts\Config\Repository` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
7			use Illuminate\Contracts\Translation\Translator;
			0 ignored issues – show Bug introduced 2019-07-31 14:49 UTC by Report Bug Copy Issue Report The type `Illuminate\Contracts\Translation\Translator` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
8			use Illuminate\Http\Request;
			0 ignored issues – show Bug introduced 2019-07-31 14:49 UTC by Report Bug Copy Issue Report The type `Illuminate\Http\Request` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
9
10			final class LocaleSelector
11			{
12			/**
13			* @var \Illuminate\Contracts\Config\Repository
14			*/
15			protected $config;
16
17			/**
18			* @var \Illuminate\Contracts\Translation\Translator
19			*/
20			protected $translator;
21
22			/**
23			* SetLocaleFromHeader constructor.
24			*
25			* @param \Illuminate\Contracts\Config\Repository $config
26			* @param \Illuminate\Contracts\Translation\Translator $translator
27			*/
28			public function __construct(Repository $config, Translator $translator)
29			{
30			$this->config = $config;
31			$this->translator = $translator;
32			}
33
34			/**
35			* Handle an incoming request.
36			*
37			* @param \Illuminate\Http\Request $request
38			* @param \Closure $next
39			* @return mixed
40			*/
41			public function handle(Request $request, Closure $next)
42			{
43			if ($request->headers->has('Accept-Language')) {
44			$locale = $request->headers->get('Accept-Language');
45
46			$this->config->set('app.locale', $locale);
47
48			$this->translator->setLocale($locale);
49			}
50
51			return $next($request);
52			}
53			}
54

kingscode / laravel-api-starter

Push — master ( ffc71e...1a0a2f )

LocaleSelector::handle() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like