Middleware::add() - Code Metrics - Inspection of "Add basic middleware support" - ankitpokhrel/tus-php - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#31)

by Ankit

created 2018-04-08 16:16 UTC

Middleware::add() A

↳ Parent: Middleware

Complexity

Conditions	4
Paths	4

Size

Total Lines	11
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	7
CRAP Score	4

Importance

Changes

Metric	Value
cc	4
eloc	6
nc	4
nop	1
dl	0
loc	11
ccs	7
cts	7
cp	1
crap	4
rs	9.2
c	0
b	0
f	0

<?php

namespace TusPhp\Middleware;

class Middleware
{
    /** @var array */
    protected $globalMiddleware = [];

    /**
     * Middleware constructor.
     */
    public function __construct()
    {
        $this->globalMiddleware = [
            GlobalHeaders::class => new GlobalHeaders(),
            Cors::class => new Cors(),
        ];
    }

    /**
     * Set middleware.
     *
     * @param array $middleware
     *
     * @return Middleware
     */
    public function add(...$middleware) : self
    {
        foreach ($middleware as $m) {
            if ($m instanceof TusMiddleware) {
                $this->globalMiddleware[get_class($m)] = $m;
            } else if (is_string($m)) {
                $this->globalMiddleware[$m] = new $m;
            }
        }

        return $this;
    }

    /**
     * Get registered middleware.
     *
     * @return array
     */
    public function list() : array
    {
        return $this->globalMiddleware;
    }

    /**
     * Skip middleware.
     *
     * @param array ...$middleware
     *
     * @return Middleware
     */
    public function skip(...$middleware) : self
    {
        foreach ($middleware as $m) {
            unset($this->globalMiddleware[$m]);
        }

        return $this;
    }
}


1		<?php
2
3		namespace TusPhp\Middleware;
4
5		class Middleware
6		{
7		/** @var array */
8		protected $globalMiddleware = [];
9
10		/**
11		* Middleware constructor.
12		*/
13	1	public function __construct()
14		{
15	1	$this->globalMiddleware = [
16	1	GlobalHeaders::class => new GlobalHeaders(),
17	1	Cors::class => new Cors(),
18		];
19	1	}
20
21		/**
22		* Set middleware.
23		*
24		* @param array $middleware
25		*
26		* @return Middleware
27		*/
28	1	public function add(...$middleware) : self
29		{
30	1	foreach ($middleware as $m) {
31	1	if ($m instanceof TusMiddleware) {
32	1	$this->globalMiddleware[get_class($m)] = $m;
33	1	} else if (is_string($m)) {
34	1	$this->globalMiddleware[$m] = new $m;
35		}
36		}
37
38	1	return $this;
39		}
40
41		/**
42		* Get registered middleware.
43		*
44		* @return array
45		*/
46	4	public function list() : array
47		{
48	4	return $this->globalMiddleware;
49		}
50
51		/**
52		* Skip middleware.
53		*
54		* @param array ...$middleware
55		*
56		* @return Middleware
57		*/
58	1	public function skip(...$middleware) : self
59		{
60	1	foreach ($middleware as $m) {
61	1	unset($this->globalMiddleware[$m]);
62		}
63
64	1	return $this;
65		}
66		}
67

ankitpokhrel / tus-php

Pull Request — master (#31)

Middleware::add() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like