CheckIfAdmin::handle() - Code Metrics - Inspection of "corrects merge conflict resolution error" - GCTC-NTGC/TalentCloud - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — feature/manager_localization2 ( 3f9cc8...df7854 )

by Xander

created 2019-02-20 21:23 UTC

CheckIfAdmin::handle() A

↳ Parent: CheckIfAdmin

Complexity

Conditions	3
Paths	3

Size

Total Lines	11
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	12

Importance

Changes

Metric	Value
eloc	5
dl	0
loc	11
ccs	0
cts	6
cp	0
rs	10
c	0
b	0
f	0
cc	3
nc	3
nop	2
crap	12

<?php


namespace App\Http\Middleware;

use Closure;

class CheckIfAdmin

{
    /**
     * Checked that the logged in user is an administrator.
     *
     * --------------
     * VERY IMPORTANT
     * --------------
     * If you have both regular users and admins inside the same table,
     * change the contents of this method to check that the logged in user
     * is an admin, and not a regular user.
     *
     * @param [type] $user [description]

     *
     * @return bool [description]
     */
    private function checkIfUserIsAdmin($user)

    {
        return ($user->hasRole('admin'));
    }

    /**
     * Answer to unauthorized access request.
     *
     * @param [type] $request [description]

     *
     * @return [type] [description]

     */
    private function respondToUnauthorizedRequest($request)

    {
        if ($request->ajax() || $request->wantsJson()) {
            return response(trans('backpack::base.unauthorized'), 401);
        } else {
            return redirect()->guest(backpack_url('login'));
        }
    }

    /**
     * Handle an incoming request.
     *
     * @param \Illuminate\Http\Request $request

     * @param \Closure                 $next

     *
     * @return mixed
     */
    public function handle($request, Closure $next)
    {
        if (backpack_auth()->guest()) {
            return $this->respondToUnauthorizedRequest($request);
        }

        if (!$this->checkIfUserIsAdmin(backpack_user())) {
            return $this->respondToUnauthorizedRequest($request);
        }

        return $next($request);
    }
}


1			<?php
2
			0 ignored issues – show Coding Style introduced 2019-01-29 14:21 UTC by Report Bug Copy Issue Report Missing file doc comment Loading history...
3			namespace App\Http\Middleware;
4
5			use Closure;
6
7			class CheckIfAdmin
			0 ignored issues – show Coding Style introduced 2019-01-29 14:21 UTC by Report Bug Copy Issue Report Missing doc comment for class CheckIfAdmin Loading history...
8			{
9			/**
10			* Checked that the logged in user is an administrator.
11			*
12			* --------------
13			* VERY IMPORTANT
14			* --------------
15			* If you have both regular users and admins inside the same table,
16			* change the contents of this method to check that the logged in user
17			* is an admin, and not a regular user.
18			*
19			* @param [type] $user [description]
			0 ignored issues – show Documentation Bug introduced 2019-01-29 14:21 UTC by Report Bug Copy Issue Report The doc comment `[type]` at position `0` could not be parsed: Unknown type name '[' at position 0 in [type]. Loading history...
20			*
21			* @return bool [description]
22			*/
23			private function checkIfUserIsAdmin($user)
			0 ignored issues – show Coding Style introduced 2019-01-29 14:21 UTC by Report Bug Copy Issue Report Private method name "CheckIfAdmin::checkIfUserIsAdmin" must be prefixed with an underscore Loading history...
24			{
25			return ($user->hasRole('admin'));
26			}
27
28			/**
29			* Answer to unauthorized access request.
30			*
31			* @param [type] $request [description]
			0 ignored issues – show Documentation Bug introduced 2019-01-29 14:21 UTC by Report Bug Copy Issue Report The doc comment `[type]` at position `0` could not be parsed: Unknown type name '[' at position 0 in [type]. Loading history...
32			*
33			* @return [type] [description]
			0 ignored issues – show Documentation Bug introduced 2019-01-29 14:21 UTC by Report Bug Copy Issue Report The doc comment `[type]` at position `0` could not be parsed: Unknown type name '[' at position 0 in [type]. Loading history...
34			*/
35			private function respondToUnauthorizedRequest($request)
			0 ignored issues – show Coding Style introduced 2019-01-29 14:21 UTC by Report Bug Copy Issue Report Private method name "CheckIfAdmin::respondToUnauthorizedRequest" must be prefixed with an underscore Loading history...
36			{
37			if ($request->ajax() \|\| $request->wantsJson()) {
38			return response(trans('backpack::base.unauthorized'), 401);
39			} else {
40			return redirect()->guest(backpack_url('login'));
41			}
42			}
43
44			/**
45			* Handle an incoming request.
46			*
47			* @param \Illuminate\Http\Request $request
			0 ignored issues – show Coding Style introduced 2018-09-14 15:18 UTC by Report Bug Copy Issue Report Missing parameter comment Loading history...
48			* @param \Closure $next
			0 ignored issues – show Coding Style introduced 2019-01-29 14:21 UTC by Report Bug Copy Issue Report Missing parameter comment Loading history...
49			*
50			* @return mixed
51			*/
52			public function handle($request, Closure $next)
53			{
54			if (backpack_auth()->guest()) {
55			return $this->respondToUnauthorizedRequest($request);
56			}
57
58			if (!$this->checkIfUserIsAdmin(backpack_user())) {
59			return $this->respondToUnauthorizedRequest($request);
60			}
61
62			return $next($request);
63			}
64			}
65

GCTC-NTGC / TalentCloud

Push — feature/manager_localization2 ( 3f9cc8...df7854 )

CheckIfAdmin::handle() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like