GeoRoutesMiddleware::handle() - Code Metrics - Inspection of "Global middleware" - LaraCrafts/laravel-geo-routes - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#19)

by Choraimy

created 2019-01-26 18:35 UTC

GeoRoutesMiddleware::handle() A

↳ Parent: GeoRoutesMiddleware

Complexity

Conditions	4
Paths	3

Size

Total Lines	14
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	4
CRAP Score	4

Importance

Changes

Metric	Value
cc	4
eloc	7
nc	3
nop	5
dl	0
loc	14
rs	10
c	0
b	0
f	0
ccs	4
cts	4
cp	1
crap	4

<?php

namespace LaraCrafts\GeoRoutes\Http\Middleware;

use Closure;
use Illuminate\Http\Request;
use LaraCrafts\GeoRoutes\DeterminesGeoAccess;

class GeoRoutesMiddleware
{
    use DeterminesGeoAccess;


    protected $countries;
    protected $strategy;

    /**
     * Handle an incoming request.
     *
     * @param \Illuminate\Http\Request $request
     * @param \Closure $next
     * @param string $strategy
     * @param string $countries
     * @param string|null $callback
     * @return mixed
     */
    public function handle(Request $request, Closure $next, string $strategy, string $countries, string $callback = null)
    {
        $this->countries = explode('&', $countries);
        $this->strategy = $strategy;

        if ($this->shouldHaveAccess($request)) {
            return $next($request);
        }

        if ($callback && $callback = unserialize($callback)) {
            return call_user_func_array($callback[0], $callback[1] ?? []);
        }

        return abort(401);
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
    }

    /**
     * Get the countries.
     *
     * @return string[]
     */
    public function getCountries()
    {
        return $this->countries;
    }

    /**
     * Get the strategy.
     *
     * @return string
     */
    public function getStrategy()
    {
        return $this->strategy;
    }
}


1		<?php
2
3		namespace LaraCrafts\GeoRoutes\Http\Middleware;
4
5		use Closure;
6		use Illuminate\Http\Request;
7		use LaraCrafts\GeoRoutes\DeterminesGeoAccess;
8
9		class GeoRoutesMiddleware
10		{
11		use DeterminesGeoAccess;
		0 ignored issues – show Bug introduced 2019-01-26 18:57 UTC by Report Bug Copy Issue Report The trait `LaraCrafts\GeoRoutes\DeterminesGeoAccess` requires the property `$countryCode` which is not provided by `LaraCrafts\GeoRoutes\Htt...are\GeoRoutesMiddleware`. Loading history...
12
13		protected $countries;
14		protected $strategy;
15
16		/**
17		* Handle an incoming request.
18		*
19		* @param \Illuminate\Http\Request $request
20		* @param \Closure $next
21	9	* @param string $strategy
22		* @param string $countries
23	9	* @param string\|null $callback
24		* @return mixed
25	9	*/
26	3	public function handle(Request $request, Closure $next, string $strategy, string $countries, string $callback = null)
27		{
28		$this->countries = explode('&', $countries);
29	6	$this->strategy = $strategy;
30	3
31		if ($this->shouldHaveAccess($request)) {
32		return $next($request);
33	3	}
34
35		if ($callback && $callback = unserialize($callback)) {
36		return call_user_func_array($callback[0], $callback[1] ?? []);
37		}
38
39		return abort(401);
		0 ignored issues – show Bug introduced 2019-01-17 22:17 UTC by Report Bug Copy Issue Report Are you sure the usage of `abort(401)` is correct as it seems to always return `null`. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
40		}
41
42		/**
43		* Get the countries.
44	9	*
45		* @return string[]
46	9	*/
47		public function getCountries()
48	9	{
49	6	return $this->countries;
50		}
51
52	3	/**
53		* Get the strategy.
54		*
55		* @return string
56		*/
57		public function getStrategy()
58		{
59		return $this->strategy;
60		}
61		}
62

LaraCrafts / laravel-geo-routes

Pull Request — master (#19)

GeoRoutesMiddleware::handle() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like