Locale::handle() - Code Metrics - Inspection of "Misc: increment version & update changes doc" - satrun77/tinyissue - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( dec0dd...e17ce1 )

by Mohamed

created 2016-04-22 09:27 UTC

Locale::handle() A

↳ Parent: Locale

Complexity

Conditions	2
Paths	2

Size

Total Lines	8
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	4
CRAP Score	2

Importance

Changes	1
Bugs	0	Features	0

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

<?php

/*
 * This file is part of the Tinyissue package.
 *
 * (c) Mohamed Alsharaf <[email protected]>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Tinyissue\Http\Middleware;

use Closure;
use Illuminate\Contracts\Auth\Guard;
use Illuminate\Http\Request;

/**
 * Locale is a Middleware class to set the locale for the current logged in user.
 *
 * @author Mohamed Alsharaf <[email protected]>
 */
class Locale
{
    /**
     * The Guard implementation.
     *
     * @var Guard
     */
    protected $auth;

    /**
     * Create a new filter instance.
     *
     * @param Guard $auth
     */
    public function __construct(Guard $auth)
    {
        $this->auth = $auth;
    }

    /**
     * Handle an incoming request.
     *
     * @param Request  $request
     * @param \Closure $next
     *
     * @return mixed
     */
    public function handle(Request $request, Closure $next)
    {
        if (!$this->auth->guest()) {
            app()->setLocale($this->auth->user()->language);
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}
        }

        return $next($request);
    }
}


1		<?php
2
3		/*
4		* This file is part of the Tinyissue package.
5		*
6		* (c) Mohamed Alsharaf <[email protected]>
7		*
8		* For the full copyright and license information, please view the LICENSE
9		* file that was distributed with this source code.
10		*/
11
12		namespace Tinyissue\Http\Middleware;
13
14		use Closure;
15		use Illuminate\Contracts\Auth\Guard;
16		use Illuminate\Http\Request;
17
18		/**
19		* Locale is a Middleware class to set the locale for the current logged in user.
20		*
21		* @author Mohamed Alsharaf <[email protected]>
22		*/
23		class Locale
24		{
25		/**
26		* The Guard implementation.
27		*
28		* @var Guard
29		*/
30		protected $auth;
31
32		/**
33		* Create a new filter instance.
34		*
35		* @param Guard $auth
36		*/
37	59	public function __construct(Guard $auth)
38		{
39	59	$this->auth = $auth;
40	59	}
41
42		/**
43		* Handle an incoming request.
44		*
45		* @param Request $request
46		* @param \Closure $next
47		*
48		* @return mixed
49		*/
50	59	public function handle(Request $request, Closure $next)
51		{
52	59	if (!$this->auth->guest()) {
53	52	app()->setLocale($this->auth->user()->language);
		0 ignored issues – show Bug introduced 2016-03-24 20:44 UTC by Report Bug Copy Issue Report Accessing `language` on the interface `Illuminate\Contracts\Auth\Authenticatable` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history...
54		}
55
56	59	return $next($request);
57		}
58		}
59

satrun77 / tinyissue

Push — master ( dec0dd...e17ce1 )

Locale::handle() A

Complexity

Size

Duplication

Code Coverage

Importance

Available Fixes

Duplication Side-by-Side

Filter issues like