HttpMethodBypass::addMethods() - Code Metrics - Inspection of "Merge branch '4'" - silverstripe/silverstripe-framework - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( ddbf8b...086098 )

by Robbie

created 2019-06-18 04:58 UTC

HttpMethodBypass::addMethods() A

↳ Parent: HttpMethodBypass

Complexity

Conditions	4
Paths	3

Size

Total Lines	22
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	4
eloc	11
nc	3
nop	1
dl	0
loc	22
rs	9.9
c	0
b	0
f	0

<?php

namespace SilverStripe\Control\Middleware\ConfirmationMiddleware;

use SilverStripe\Control\HTTPRequest;

/**
 * Allows to bypass requests of a particular HTTP method
 */
class HttpMethodBypass implements Bypass
{
    /**
     * HTTP Methods to bypass
     *
     * @var string[]
     */
    private $methods = [];

    /**
     * Initialize the bypass with HTTP methods
     *
     * @param string[] ...$method
     */
    public function __construct(...$methods)
    {
        $this->addMethods(...$methods);
    }

    /**
     * Returns the list of methods
     *
     * @return string[]
     */
    public function getMethods()
    {
        return $this->methods;
    }

    /**
     * Add new HTTP methods to the list
     *
     * @param string[] ...$methods
     *
     * return $this
     */
    public function addMethods(...$methods)
    {
        // uppercase and exclude empties
        $methods = array_reduce(
            $methods,
            static function &(&$result, $method) {
                $method = strtoupper(trim($method));
                if (strlen($method)) {
                    $result[] = $method;
                }
                return $result;
            },
            []
        );

        foreach ($methods as $method) {
            if (!in_array($method, $this->methods, true)) {
                $this->methods[] = $method;
            }
        }

        return $this;
    }

    /**
     * Returns true if the current process is running in CLI mode
     *
     * @param HTTPRequest $request
     *
     * @return bool
     */
    public function checkRequestForBypass(HTTPRequest $request)
    {
        return in_array($request->httpMethod(), $this->methods, true);
    }
}


1			<?php
2
3			namespace SilverStripe\Control\Middleware\ConfirmationMiddleware;
4
5			use SilverStripe\Control\HTTPRequest;
6
7			/**
8			* Allows to bypass requests of a particular HTTP method
9			*/
10			class HttpMethodBypass implements Bypass
11			{
12			/**
13			* HTTP Methods to bypass
14			*
15			* @var string[]
16			*/
17			private $methods = [];
18
19			/**
20			* Initialize the bypass with HTTP methods
21			*
22			* @param string[] ...$method
23			*/
24			public function __construct(...$methods)
25			{
26			$this->addMethods(...$methods);
27			}
28
29			/**
30			* Returns the list of methods
31			*
32			* @return string[]
33			*/
34			public function getMethods()
35			{
36			return $this->methods;
37			}
38
39			/**
40			* Add new HTTP methods to the list
41			*
42			* @param string[] ...$methods
43			*
44			* return $this
45			*/
46			public function addMethods(...$methods)
47			{
48			// uppercase and exclude empties
49			$methods = array_reduce(
50			$methods,
51			static function &(&$result, $method) {
52			$method = strtoupper(trim($method));
53			if (strlen($method)) {
54			$result[] = $method;
55			}
56			return $result;
57			},
58			[]
59			);
60
61			foreach ($methods as $method) {
62			if (!in_array($method, $this->methods, true)) {
63			$this->methods[] = $method;
64			}
65			}
66
67			return $this;
68			}
69
70			/**
71			* Returns true if the current process is running in CLI mode
72			*
73			* @param HTTPRequest $request
74			*
75			* @return bool
76			*/
77			public function checkRequestForBypass(HTTPRequest $request)
78			{
79			return in_array($request->httpMethod(), $this->methods, true);
80			}
81			}
82

silverstripe / silverstripe-framework

Push — master ( ddbf8b...086098 )

HttpMethodBypass::addMethods() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like