Saxulum\ElasticSearchQueryBuilder\Node\ObjectNode

Complexity

Total Complexity

8

Size/Duplication

Total Lines	43
Duplicated Lines	11.63 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes	6
Bugs	1	Features	2

2 Methods

Rating	Name	Duplication	Size	Complexity
A	add()	0	16	3
B	serialize()	5	15	5

<?php

namespace Saxulum\ElasticSearchQueryBuilder\Node;

class ObjectNode extends AbstractParentNode
{
    /**
     * @param string $name
     * @param AbstractNode $node
     * @return $this
     */
    public function add($name, AbstractNode $node)
    {
        if (null !== $this->reflectionProperty->getValue($node)) {
            throw new \InvalidArgumentException('Node already got a parent!');
        }

        if (array_key_exists($name, $this->children)) {
            throw new \InvalidArgumentException(sprintf('There is allready a child for name %s', $name));
        }

        $this->reflectionProperty->setValue($node, $this);
        
        $this->children[$name] = $node;

        return $this;
    }

    /**
     * @return \stdClass|null
     */
    public function serialize()
    {
        $serialized = new \stdClass();
        foreach ($this->children as $name => $child) {

This code seems to be duplicated across your project.

            if (null !== $serializedChild = $child->serialize()) {
                $serialized->$name = $serializedChild;
            }
        }

        if (!$this->allowEmpty && [] === (array) $serialized) {
            return null;
        }

        return $serialized;
    }
}