Purifier::clean() - Code Metrics - realshadow/request-deserializer - Measure and Improve Code Quality continuously with Scrutinizer

Purifier::clean() A
last analyzed 2017-07-03 12:29 UTC

↳ Parent: Purifier

Complexity

Conditions	3
Paths	3

Size

Total Lines	13
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	13
rs	9.4285
c	0
b	0
f	0
cc	3
eloc	8
nc	3
nop	2

<?php

namespace Realshadow\RequestDeserializer\Validation;

/**
 * Extension of Purifier to make it type aware
 *
 * @package Realshadow\RequestDeserializer\Validation
 * @author Lukáš Homza <[email protected]>
 */
class Purifier extends \Mews\Purifier\Purifier
{

    /**
     * @inheritdoc
     */
    public function clean($dirty, $config = null)
    {
        if (is_array($dirty)) {
            $output = array_map(function ($item) use ($config) {
                return $this->clean($item, $config);
            }, $dirty);
        } else {
            # -- the htmlpurifier uses replace instead of merge, so we merge
            $output = is_string($dirty) ? $this->purifier->purify($dirty, $this->getConfig($config)) : $dirty;
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        }

        return $output;
    }

}


1			<?php
2
3			namespace Realshadow\RequestDeserializer\Validation;
4
5			/**
6			* Extension of Purifier to make it type aware
7			*
8			* @package Realshadow\RequestDeserializer\Validation
9			* @author Lukáš Homza <[email protected]>
10			*/
11			class Purifier extends \Mews\Purifier\Purifier
12			{
13
14			/**
15			* @inheritdoc
16			*/
17			public function clean($dirty, $config = null)
18			{
19			if (is_array($dirty)) {
20			$output = array_map(function ($item) use ($config) {
21			return $this->clean($item, $config);
22			}, $dirty);
23			} else {
24			# -- the htmlpurifier uses replace instead of merge, so we merge
25			$output = is_string($dirty) ? $this->purifier->purify($dirty, $this->getConfig($config)) : $dirty;
			0 ignored issues – show Documentation introduced 2017-07-03 10:06 UTC by Report Bug Copy Issue Report `$this->getConfig($config)` is of type `array`, but the function expects a `object<HTMLPurifier_Config>\|null`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
26			}
27
28			return $output;
29			}
30
31			}
32

realshadow / request-deserializer

Purifier::clean() A last analyzed 2017-07-03 12:29 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

Purifier::clean() A
last analyzed 2017-07-03 12:29 UTC