Hash::arrayFilterRecursive() - Code Metrics - Inspection of "Merge pull request #607 from libgraviton/feature/E..." - libgraviton/graviton - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — develop ( bc4b3b...74f664 )

by Narcotic

created 2017-10-04 11:52 UTC

Hash::arrayFilterRecursive() B

↳ Parent: Hash

Complexity

Conditions	5
Paths	4

Size

Total Lines	12
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	30

Importance

Changes

Metric	Value
dl	0
loc	12
rs	8.8571
c	0
b	0
f	0
ccs	0
cts	7
cp	0
cc	5
eloc	7
nc	4
nop	1
crap	30

<?php
/**
 * Hash class file
 */

namespace Graviton\DocumentBundle\Entity;

/**
 * Special type for hash fields
 *
 * @author   List of contributors <https://github.com/libgraviton/graviton/graphs/contributors>
 * @license  http://opensource.org/licenses/gpl-license.php GNU Public License
 * @link     http://swisscom.ch
 */
class Hash extends \ArrayObject implements \JsonSerializable
{
    /**
     * Specify data which should be serialized to JSON
     *
     * @return object
     */
    public function jsonSerialize()
    {
        return (object) $this->arrayFilterRecursive($this->getArrayCopy());
    }

    /**
     * Clean up, remove empty positions of second level and above
     *
     * @param array $input to be cleaned up
     * @return array
     */
    private function arrayFilterRecursive($input)

    {
        if (empty($input)) {
            return [];
        }
        foreach ($input as &$value) {
            if (is_array($value) || is_object($value)) {
                $value = $this->arrayFilterRecursive($value);
/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}
            }
        }
        return array_filter($input, [$this, 'cleanUpArray']);
    }

    /**
     * Remove NULL values or Empty array object
     * @param mixed $var object field value
     * @return bool
     */
    private function cleanUpArray($var)
    {
        $empty = empty($var);
        if ($empty && (
                is_int($var) ||
                is_bool($var) ||
                is_float($var) ||
                is_integer($var) ||
                is_string($var))
        ) {
            return true;
        }
        return !$empty;
    }
}


1			<?php
2			/**
3			* Hash class file
4			*/
5
6			namespace Graviton\DocumentBundle\Entity;
7
8			/**
9			* Special type for hash fields
10			*
11			* @author List of contributors <https://github.com/libgraviton/graviton/graphs/contributors>
12			* @license http://opensource.org/licenses/gpl-license.php GNU Public License
13			* @link http://swisscom.ch
14			*/
15			class Hash extends \ArrayObject implements \JsonSerializable
16			{
17			/**
18			* Specify data which should be serialized to JSON
19			*
20			* @return object
21			*/
22			public function jsonSerialize()
23			{
24			return (object) $this->arrayFilterRecursive($this->getArrayCopy());
25			}
26
27			/**
28			* Clean up, remove empty positions of second level and above
29			*
30			* @param array $input to be cleaned up
31			* @return array
32			*/
33			private function arrayFilterRecursive($input)
			0 ignored issues – show Unused Code introduced 2017-08-04 06:46 UTC by Report Bug Copy Issue Report This method is not used, and could be removed. Loading history...
34			{
35			if (empty($input)) {
36			return [];
37			}
38			foreach ($input as &$value) {
39			if (is_array($value) \|\| is_object($value)) {
40			$value = $this->arrayFilterRecursive($value);
			0 ignored issues – show Bug introduced 2017-04-28 15:14 UTC by Report Bug Copy Issue Report It seems like `$value` can also be of type `object`; however, `Graviton\DocumentBundle\...:arrayFilterRecursive()` does only seem to accept `array`, maybe add an additional type check? If a method or function can return multiple different values and unless you are sure that you only can receive a single value in this context, we recommend to add an additional type check: /** * @return array\|string */ function returnsDifferentValues($x) { if ($x) { return 'foo'; } return array(); } $x = returnsDifferentValues($y); if (is_array($x)) { // $x is an array. } If this a common case that PHP Analyzer should handle natively, please let us know by opening an issue. Loading history...
41			}
42			}
43			return array_filter($input, [$this, 'cleanUpArray']);
44			}
45
46			/**
47			* Remove NULL values or Empty array object
48			* @param mixed $var object field value
49			* @return bool
50			*/
51			private function cleanUpArray($var)
52			{
53			$empty = empty($var);
54			if ($empty && (
55			is_int($var) \|\|
56			is_bool($var) \|\|
57			is_float($var) \|\|
58			is_integer($var) \|\|
59			is_string($var))
60			) {
61			return true;
62			}
63			return !$empty;
64			}
65			}
66

libgraviton / graviton

Push — develop ( bc4b3b...74f664 )

Hash::arrayFilterRecursive() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like