SwitchChoose::applyToNode() - Code Metrics - Inspection of "Enricher schema" - legalthings/data-enricher - Measure and Improve Code Quality continuously with Scrutinizer

Failed Conditions

Pull Request — master (#32)

by Moesjarraf

created 2018-04-03 09:58 UTC

SwitchChoose::applyToNode() C

↳ Parent: SwitchChoose

Complexity

Conditions	8
Paths	11

Size

Total Lines	30
Code Lines	15

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	9
CRAP Score	8

Importance

Changes

Metric	Value
c	0
b	0
f	0
dl	0
loc	30
ccs	9
cts	9
cp	1
rs	5.3846
cc	8
eloc	15
nc	11
nop	1
crap	8

<?php

namespace LegalThings\DataEnricher\Processor;

use LegalThings\DataEnricher\Node;
use LegalThings\DataEnricher\Processor;
use LegalThings\DataEnricher\Processor\Helper;

/**
 * Choose one of the child properties based on a property in the document
 */
class SwitchChoose implements Processor
{
    use Processor\Implementation,
        Helper\GetByReference
    {
        Helper\GetByReference::withSourceAndTarget insteadof Processor\Implementation;
    }
    
    /**
     * Apply processing to a single node
     * 
     * @param Node $node
     */
    public function applyToNode(Node $node)
    {
        $instruction = $node->getInstruction($this);

        // for bc
        if (is_string($instruction)) {
            $cases = $node->getResult();

            $result = $this->choose($instruction, $cases);
/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}
            return $node->setResult($result);
        }
        
        if (is_array($instruction) || is_object($instruction)) {
            $instruction = (object)$instruction;
        }
        
        if (!isset($instruction->on) || !isset($instruction->options)) {
            return;
        }
        
        $instruction->options = (object)$instruction->options;
        
        $result = isset($instruction->default) ? $instruction->default : null;
        
        if (isset($instruction->options->{$instruction->on})) {
            $result = $instruction->options->{$instruction->on};
        }
        
        $node->setResult($result);
    }
    
    /**
     * Choose on of the cases
     * 
     * @param string $ref    Property name of source
     * @param object $cases
     * @return mixed
     */
    protected function choose($ref, $cases)
    {
        $test = $this->getByReference($ref, $this->source, $this->target);
        return isset($cases->$test) ? $cases->$test : null;
    }
}


1		<?php
2
3		namespace LegalThings\DataEnricher\Processor;
4
5		use LegalThings\DataEnricher\Node;
6		use LegalThings\DataEnricher\Processor;
7		use LegalThings\DataEnricher\Processor\Helper;
8
9		/**
10		* Choose one of the child properties based on a property in the document
11		*/
12		class SwitchChoose implements Processor
13		{
14		use Processor\Implementation,
15		Helper\GetByReference
16		{
17		Helper\GetByReference::withSourceAndTarget insteadof Processor\Implementation;
18		}
19
20		/**
21		* Apply processing to a single node
22		*
23		* @param Node $node
24		*/
25	1	public function applyToNode(Node $node)
26		{
27	1	$instruction = $node->getInstruction($this);
28	1
29		// for bc
30	1	if (is_string($instruction)) {
31	1	$cases = $node->getResult();
32	1
33		$result = $this->choose($instruction, $cases);
		0 ignored issues – show Bug introduced 2018-03-06 16:14 UTC by Report Bug Copy Issue Report It seems like `$cases` defined by `$node->getResult()` on line `31` can also be of type `array`; however, `LegalThings\DataEnricher...\SwitchChoose::choose()` does only seem to accept `object`, 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...
34		return $node->setResult($result);
35		}
36
37		if (is_array($instruction) \|\| is_object($instruction)) {
38		$instruction = (object)$instruction;
39		}
40
41	1	if (!isset($instruction->on) \|\| !isset($instruction->options)) {
42		return;
43	1	}
44	1
45		$instruction->options = (object)$instruction->options;
46
47		$result = isset($instruction->default) ? $instruction->default : null;
48
49		if (isset($instruction->options->{$instruction->on})) {
50		$result = $instruction->options->{$instruction->on};
51		}
52
53		$node->setResult($result);
54		}
55
56		/**
57		* Choose on of the cases
58		*
59		* @param string $ref Property name of source
60		* @param object $cases
61		* @return mixed
62		*/
63		protected function choose($ref, $cases)
64		{
65		$test = $this->getByReference($ref, $this->source, $this->target);
66		return isset($cases->$test) ? $cases->$test : null;
67		}
68		}
69

legalthings / data-enricher

Pull Request — master (#32)

SwitchChoose::applyToNode() C

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like