Yiisoft\Validator\Rule\SubsetHandler

Complexity

Total Complexity

5

Size/Duplication

Total Lines	36
Duplicated Lines	0 %

Test Coverage

Coverage

72.21%

Importance

Changes	3
Bugs	1	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	validate()	0	34	5

<?php

declare(strict_types=1);

namespace Yiisoft\Validator\Rule;

use Traversable;
use Yiisoft\Arrays\ArrayHelper;
use Yiisoft\Validator\Exception\UnexpectedRuleException;
use Yiisoft\Validator\Result;
use Yiisoft\Validator\RuleHandlerInterface;
use Yiisoft\Validator\ValidationContext;

final class SubsetHandler implements RuleHandlerInterface
{
    public function validate(mixed $value, object $rule, ValidationContext $context): Result
    {
        if (!$rule instanceof Subset) {
            throw new UnexpectedRuleException(Subset::class, $rule);
        }

        $result = new Result();
        if (!is_iterable($value)) {
            $result->addError($rule->getIterableMessage(), [
                'attribute' => $context->getAttribute(),
                'valueType' => get_debug_type($value),
            ]);

            return $result;
        }

        if (!ArrayHelper::isSubset($value, $rule->getValues(), $rule->isStrict())) {
            $values = $rule->getValues();
            if ($values instanceof Traversable) {
                $values = iterator_to_array($values);
            }

            $valuesString = '"' . implode('", "', $values) . '"';

It seems like $values can also be of type iterable; however, parameter $pieces of implode() does only seem to accept array, maybe add an additional type check?

            $result->addError(
                $rule->getSubsetMessage(),
                [
                    'attribute' => $context->getAttribute(),
                    'values' => $valuesString,
                ],
            );
        }

        return $result;
    }
}