Yiisoft\Validator\Rule\Subset\SubsetHandler::validate()

Complexity

Conditions	5
Paths	5

Size

Total Lines	21
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	10
CRAP Score	5.1158

Importance

Changes

0

<?php
    public function validate(mixed $value, object $rule, ValidatorInterface $validator, ?ValidationContext $context = null): Result
    {
        if (!$rule instanceof Subset) {
            throw new UnexpectedRuleException(Subset::class, $rule);
        }

        $result = new Result();

        if (!is_iterable($value)) {
            $result->addError($rule->iterableMessage);
            return $result;
        }

        if (!ArrayHelper::isSubset($value, $rule->values, $rule->strict)) {
            $values = is_array($rule->values) ? $rule->values : iterator_to_array($rule->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->subsetMessage, ['values' => $valuesString]);
        }

        return $result;
    }