Apicart\FQL\Generator\SQL\Word

Complexity

Total Complexity

7

Size/Duplication

Total Lines	44
Duplicated Lines	0 %

Test Coverage

Coverage

94.44%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	accept()	0	3	2
A	visit()	0	23	4

<?php declare(strict_types = 1);

namespace Apicart\FQL\Generator\SQL;

use Apicart\FQL\Generator\Common\AbstractVisitor;
use Apicart\FQL\Generator\SQL\Resolver\AbstractFilterResolver;
use Apicart\FQL\Token\Node\Group as GroupNode;
use Apicart\FQL\Token\Node\Term;
use Apicart\FQL\Token\Token\Word as WordToken;
use Apicart\FQL\Value\AbstractNode;
use LogicException;

final class Word extends AbstractVisitor
{

    /**
     * @var AbstractFilterResolver
     */
    private $filterResolver;


    public function __construct(AbstractFilterResolver $filterResolver)
    {
        $this->filterResolver = $filterResolver;
    }


    public function accept(AbstractNode $node): bool
    {
        return $node instanceof Term && $node->getToken() instanceof WordToken;
    }


    public function visit(AbstractNode $node, ?AbstractVisitor $subVisitor = null, ?array $options = null): string
    {
        /** @var Term $termNode */
        $termNode = $node;
        /** @var WordToken $token */
        $token = $termNode->getToken();
        $domain = $token->getDomain();

        if ($domain === '') {
            $parent = $options['parent'] ?? false;
            if ($parent instanceof GroupNode) {
                $tokenLeft = $parent->getTokenLeft();
                $domain = $tokenLeft->getDomain();
            }

            if ($domain === '') {
                throw new LogicException('Missing required domain');
            }
        }

        $wordEscaped = preg_replace('/([\\\'"+\-!():#@ ])/', '\\\\$1', $token->getWord());

        return $this->filterResolver->resolve($domain, $wordEscaped);

It seems like $domain can also be of type null; however, parameter $column of Apicart\FQL\Generator\SQL\Resolver\AbstractFilterResolver::resolve() does only seem to accept string, maybe add an additional type check?

    }

}