saxulum / saxulum-elasticsearch-querybuilder

src/Converter/IteratableToNodeConverter.php

<?php

declare(strict_types=1);

namespace Saxulum\ElasticSearchQueryBuilder\Converter;

use Saxulum\ElasticSearchQueryBuilder\Node\AbstractNode;
use Saxulum\ElasticSearchQueryBuilder\Node\AbstractParentNode;
use Saxulum\ElasticSearchQueryBuilder\Node\ArrayNode;
use Saxulum\ElasticSearchQueryBuilder\Node\ObjectNode;

final class IteratableToNodeConverter implements IteratableToNodeConverterInterface
{
    /**
     * @var ScalarToNodeConverterInterface
     */
    private $scalarToNodeConverter;

    /**
     * @param ScalarToNodeConverterInterface $scalarToNodeConverter
     */
    public function __construct(ScalarToNodeConverterInterface $scalarToNodeConverter)
    {
        $this->scalarToNodeConverter = $scalarToNodeConverter;
    }

    /**
     * @param array|\Traversable $data
     * @param string             $path
     * @param bool               $allowSerializeEmpty
     *
     * @return AbstractParentNode
     *
     * @throws \InvalidArgumentException
     */
    public function convert($data, string $path = '', bool $allowSerializeEmpty = false): AbstractParentNode
    {
        if (!is_array($data) && !$data instanceof \Traversable) {
            throw new \InvalidArgumentException(sprintf('Params need to be array or %s', \Traversable::class));
        }

        $isArray = $this->isArray($data);
        $parentNode = $this->getParentNode($isArray, $allowSerializeEmpty);

        foreach ($data as $key => $value) {
            $this->addChildNode($parentNode, $key, $value, $path, $isArray, $allowSerializeEmpty);
        }

        return $parentNode;
    }

    /**
     * @param array|\Traversable $data
     *
     * @return bool
     */
    private function isArray($data): bool
    {
        $counter = 0;
        foreach ($data as $key => $value) {
            if ($key !== $counter) {
                return false;
            }

            ++$counter;
        }

        return true;
    }

    /**
     * @param bool $isArray
     * @param bool $allowSerializeEmpty
     *
     * @return AbstractParentNode
     */
    private function getParentNode(bool $isArray, bool $allowSerializeEmpty): AbstractParentNode
    {
        if ($isArray) {
            return ArrayNode::create($allowSerializeEmpty);
        }

        return ObjectNode::create($allowSerializeEmpty);
    }

    /**
     * @param AbstractParentNode $parentNode
     * @param int|string         $key
     * @param mixed              $value
     * @param string             $path
     * @param bool               $isArray
     * @param bool               $allowSerializeEmpty
     */
    private function addChildNode(
        AbstractParentNode $parentNode,
        $key,
        $value,
        string $path,
        bool $isArray,
        bool $allowSerializeEmpty
    ) {
        $subPath = $this->getSubPath($path, $key, $isArray);
        $node = $this->getNode($value, $subPath, $allowSerializeEmpty);

        if ($isArray) {
            $parentNode->add($node);
        } else {
            $parentNode->add((string) $key, $node);
        }
    }

    /**
     * @param string     $path
     * @param string|int $key
     * @param bool       $isArray
     *
     * @return string
     */
    private function getSubPath(string $path, $key, bool $isArray): string
    {
        $key = (string) $key;

        if ($isArray) {
            return $path.'['.$key.']';
        }

        return '' !== $path ? $path.'.'.$key : $key;
    }

    /**
     * @param mixed  $value
     * @param string $path
     * @param bool   $allowSerializeEmpty
     *
     * @return AbstractNode
     */
    private function getNode($value, string $path, bool $allowSerializeEmpty): AbstractNode
    {
        if (is_array($value) || $value instanceof \Traversable) {
            return $this->convert($value, $path, $allowSerializeEmpty);
        }

        return $this->scalarToNodeConverter->convert($value, $path, $allowSerializeEmpty);

The call to ScalarToNodeConverterInterface::convert() has too many arguments starting with $allowSerializeEmpty.

    }
}