HeaderRemovePlugin::handleRequest() - Code Metrics - php-http/plugins - Measure and Improve Code Quality continuously with Scrutinizer

HeaderRemovePlugin::handleRequest() A
last analyzed 2019-06-05 09:03 UTC

↳ Parent: HeaderRemovePlugin

Complexity

Conditions	3
Paths	3

Size

Total Lines

Duplication

Lines	10
Ratio	100 %

Code Coverage

Tests	7
CRAP Score	3

Importance

Changes

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

<?php

namespace Http\Client\Plugin;

@trigger_error('The '.__NAMESPACE__.'\HeaderRemovePlugin class is deprecated since version 1.1 and will be removed in 2.0. Use Http\Client\Common\Plugin\HeaderRemovePlugin instead.', E_USER_DEPRECATED);
// For example instead of
@mkdir($dir);

// Better use
if (@mkdir($dir) === false) {
    throw new \RuntimeException('The directory '.$dir.' could not be created.');
}

use Psr\Http\Message\RequestInterface;

/**
 * Removes headers from the request.
 *
 * @author Soufiane Ghzal <[email protected]>
 *
 * @deprecated since since version 1.1, and will be removed in 2.0. Use {@link \Http\Client\Common\Plugin\HeaderRemovePlugin} instead.
 */
class HeaderRemovePlugin implements Plugin

{
    private $headers = [];

    /**
     * @param array $headers headers to remove from the request
     */
    public function __construct(array $headers)
    {
        $this->headers = $headers;
    }

    /**
     * {@inheritdoc}
     */
    public function handleRequest(RequestInterface $request, callable $next, callable $first)
    {
        foreach ($this->headers as $header) {
            if ($request->hasHeader($header)) {
                $request = $request->withoutHeader($header);
            }
        }

        return $next($request);
    }
}


1			<?php
2
3			namespace Http\Client\Plugin;
4
5	1		@trigger_error('The '.__NAMESPACE__.'\HeaderRemovePlugin class is deprecated since version 1.1 and will be removed in 2.0. Use Http\Client\Common\Plugin\HeaderRemovePlugin instead.', E_USER_DEPRECATED);
			0 ignored issues – show Security Best Practice introduced 2016-05-04 07:47 UTC by Report Bug Copy Issue Report It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended. If you suppress an error, we recommend checking for the error condition explicitly: // For example instead of @mkdir($dir); // Better use if (@mkdir($dir) === false) { throw new \RuntimeException('The directory '.$dir.' could not be created.'); } Loading history...
6
7			use Psr\Http\Message\RequestInterface;
8
9			/**
10			* Removes headers from the request.
11			*
12			* @author Soufiane Ghzal <[email protected]>
13			*
14			* @deprecated since since version 1.1, and will be removed in 2.0. Use {@link \Http\Client\Common\Plugin\HeaderRemovePlugin} instead.
15			*/
16		View Code Duplication	class HeaderRemovePlugin implements Plugin
			0 ignored issues – show Deprecated Code introduced 2016-05-04 06:48 UTC by Report Bug Copy Issue Report The interface `Http\Client\Plugin\Plugin` has been deprecated with message: since since version 1.1, and will be removed in 2.0. Use {@link \Http\Client\Common\Plugin} instead. This class, trait or interface has been deprecated. The supplier of the file has supplied an explanatory message. The explanatory message should give you some clue as to whether and when the type will be removed from the class and what other constant to use instead. Loading history... Duplication introduced 2016-01-21 10:28 UTC by Report Bug Copy Issue Report This class seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
17			{
18			private $headers = [];
19
20			/**
21			* @param array $headers headers to remove from the request
22			*/
23	3		public function __construct(array $headers)
24			{
25	3		$this->headers = $headers;
26	3		}
27
28			/**
29			* {@inheritdoc}
30			*/
31	1		public function handleRequest(RequestInterface $request, callable $next, callable $first)
32			{
33	1		foreach ($this->headers as $header) {
34	1		if ($request->hasHeader($header)) {
35	1		$request = $request->withoutHeader($header);
36	1		}
37	1		}
38
39	1		return $next($request);
40			}
41			}
42

php-http / plugins

HeaderRemovePlugin::handleRequest() A last analyzed 2019-06-05 09:03 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

HeaderRemovePlugin::handleRequest() A
last analyzed 2019-06-05 09:03 UTC