AddCustomProvider::handle() - Code Metrics - Inspection of "add Authenticate middleware" - sfelix-martins/passport-multiauth - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#19)

by Samuel

created 2018-05-07 03:33 UTC

AddCustomProvider::handle() A

↳ Parent: AddCustomProvider

Complexity

Conditions	3
Paths	3

Size

Total Lines	13
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	7
CRAP Score	3

Importance

Changes

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

<?php

namespace SMartins\PassportMultiauth\Http\Middleware;

use Closure;
use Illuminate\Http\Request;

class AddCustomProvider
{
    /**
     * The default provder of api guard.
     *
     * @var string
     */
    protected $defaultApiProvider;

    /**
     * Handle an incoming request. Set the `provider` from `api` guard using a
     * parameter `provider` coming from request. The provider on `apì` guard
     * is used by Laravel Passport to get the correct model on access token
     * creation.
     *
     * @param  \Illuminate\Http\Request  $request
     * @param  \Closure  $next
     * @return mixed
     */
    public function handle(Request $request, Closure $next)
    {
        $this->defaultApiProvider = config('auth.guards.api.provider');

        $params = $request->all();
        if (array_key_exists('provider', $params)) {
            if (! is_null($params['provider'])) {
                config(['auth.guards.api.provider' => $params['provider']]);
            }
        }

        return $next($request);
    }

    public function terminate($request, $response)

    {
        // Reset config provider to default after complete request.
        config(['auth.guards.api.provider' => $this->defaultApiProvider]);
    }
}


1		<?php
2
3		namespace SMartins\PassportMultiauth\Http\Middleware;
4
5		use Closure;
6		use Illuminate\Http\Request;
7
8		class AddCustomProvider
9		{
10		/**
11		* The default provder of api guard.
12		*
13		* @var string
14		*/
15		protected $defaultApiProvider;
16
17		/**
18		* Handle an incoming request. Set the `provider` from `api` guard using a
19		* parameter `provider` coming from request. The provider on `apì` guard
20		* is used by Laravel Passport to get the correct model on access token
21		* creation.
22		*
23		* @param \Illuminate\Http\Request $request
24		* @param \Closure $next
25		* @return mixed
26		*/
27	1	public function handle(Request $request, Closure $next)
28		{
29	1	$this->defaultApiProvider = config('auth.guards.api.provider');
30
31	1	$params = $request->all();
32	1	if (array_key_exists('provider', $params)) {
33	1	if (! is_null($params['provider'])) {
34	1	config(['auth.guards.api.provider' => $params['provider']]);
35		}
36		}
37
38	1	return $next($request);
39		}
40
41		public function terminate($request, $response)
		0 ignored issues – show Unused Code introduced 2018-05-07 03:34 UTC by Report Bug Copy Issue Report The parameter `$request` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history... Unused Code introduced 2018-05-07 03:34 UTC by Report Bug Copy Issue Report The parameter `$response` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
42		{
43		// Reset config provider to default after complete request.
44		config(['auth.guards.api.provider' => $this->defaultApiProvider]);
45		}
46		}
47

sfelix-martins / passport-multiauth

Pull Request — master (#19)

AddCustomProvider::handle() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like