MiddlewareAbstract::getLoggedUser() - Code Metrics - Inspection of "Misc: CS & scrutinizer fixes" - satrun77/tinyissue - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Branch — develop (82845d)

by Mohamed

created 2016-08-09 10:26 UTC

MiddlewareAbstract::getLoggedUser() A

↳ Parent: MiddlewareAbstract

Complexity

Conditions	2
Paths	2

Size

Total Lines	10
Code Lines	5

Duplication

Lines	10
Ratio	100 %

Importance

Changes	1
Bugs	1	Features	0

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

<?php
/*
 * This file is part of the site 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 Tinyissue\Model\User;
use Illuminate\Contracts\Auth\Guard;
use Illuminate\Contracts\Foundation\Application;

/**
 * @author Mohamed Alsharaf <[email protected]>
 */
abstract class MiddlewareAbstract
{
    /**
     * The Guard implementation.
     *
     * @var Guard
     */
    protected $auth;

    /**
     * The application implementation.
     *
     * @var \Illuminate\Contracts\Foundation\Application
     */
    protected $app;

    /**
     * Create a new filter instance.
     *
     * @param Guard                                        $auth
     * @param \Illuminate\Contracts\Foundation\Application $app
     */
    public function __construct(Guard $auth, Application $app)
    {
        $this->auth = $auth;
        $this->app  = $app;
    }

    /**
     * Return instance of the logged user.
     *
     * @return User
     */
    protected function getLoggedUser()

    {
        $user = $this->auth->user();

        if (!$user instanceof User) {
            throw new \DomainException('Unable to find a valid instance of logged user.');
        }

        return $user;
    }
}


1		<?php
2		/*
3		* This file is part of the site package.
4		*
5		* (c) Mohamed Alsharaf <[email protected]>
6		*
7		* For the full copyright and license information, please view the LICENSE
8		* file that was distributed with this source code.
9		*/
10
11		namespace Tinyissue\Http\Middleware;
12
13		use Tinyissue\Model\User;
14		use Illuminate\Contracts\Auth\Guard;
15		use Illuminate\Contracts\Foundation\Application;
16
17		/**
18		* @author Mohamed Alsharaf <[email protected]>
19		*/
20		abstract class MiddlewareAbstract
21		{
22		/**
23		* The Guard implementation.
24		*
25		* @var Guard
26		*/
27		protected $auth;
28
29		/**
30		* The application implementation.
31		*
32		* @var \Illuminate\Contracts\Foundation\Application
33		*/
34		protected $app;
35
36		/**
37		* Create a new filter instance.
38		*
39		* @param Guard $auth
40		* @param \Illuminate\Contracts\Foundation\Application $app
41		*/
42		public function __construct(Guard $auth, Application $app)
43		{
44		$this->auth = $auth;
45		$this->app = $app;
46		}
47
48		/**
49		* Return instance of the logged user.
50		*
51		* @return User
52		*/
53	View Code Duplication	protected function getLoggedUser()
		0 ignored issues – show Duplication introduced 2016-08-09 10:29 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
54		{
55		$user = $this->auth->user();
56
57		if (!$user instanceof User) {
58		throw new \DomainException('Unable to find a valid instance of logged user.');
59		}
60
61		return $user;
62		}
63		}
64

satrun77 / tinyissue

Branch — develop (82845d)

MiddlewareAbstract::getLoggedUser() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like