LegalThings\DataEnricher\Processor\SwitchChoose

Complexity

Total Complexity

10

Size/Duplication

Total Lines	57
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

94.74%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	choose()	0	5	2
B	applyToNode()	0	30	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);

It seems like $cases defined by $node->getResult() on line 31 can also be of type array; however, LegalThings\DataEnricher\Processor\SwitchChoose::choose() does only seem to accept object, maybe add an additional type check?

            return $node->setResult($result);
        }
        
        if (is_array($instruction) || is_object($instruction)) {
            $instruction = (object)$instruction;
        }
        
        if (!isset($instruction->on) || !isset($instruction->options)) {
            return;
        }
        
        $instruction->options = (array)$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;
    }
}