Apicart\FQL\Tests\Integration\Generator\SQL\Visitor\Phrase::visit()

Complexity

Conditions	2
Paths	2

Size

Total Lines	15
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php declare(strict_types = 1);
	public function visit(AbstractNode $node, ?AbstractVisitor $subVisitor = null, ?array $options = null): string
	{
		/** @var Term $termNode */
		$termNode = $node;
		/** @var PhraseToken $token */
		$token = $termNode->getToken();

		$domain = $token->getDomain();
		if ($domain === '') {
			throw new LogicException('Missing required domain');
		}

		$phraseEscaped = preg_replace("/([\\{$token->getQuote()}])/", '\\\\$1', $token->getPhrase());

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

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

	}