Yiisoft\Validator\Rule\CompositeHandler

Complexity

Total Complexity

3

Size/Duplication

Total Lines	17
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	validate()	0	13	3

<?php

declare(strict_types=1);

namespace Yiisoft\Validator\Rule;

use Yiisoft\Validator\Exception\UnexpectedRuleException;
use Yiisoft\Validator\Result;
use Yiisoft\Validator\Rule\Trait\PreValidateTrait;
use Yiisoft\Validator\ValidationContext;

/**
 * Can be used to group rules for validation by `skipOnEmpty`, `skipOnError` or `when`.
 *
 * For example, we have same when closure:
 * ```
 * 'name' => [
 *     new Required(when: fn() => $this->useName),
 *     new HasLength(min: 1, max: 50, skipOnEmpty: true, when: fn() => $this->useName),
 * ],
 * ```
 * So we can configure it like this:
 * ```
 * 'name' => [
 *     new Composite(
 *         rules: [
 *             new Required(),
 *             new HasLength(min: 1, max: 50, skipOnEmpty: true),
 *         ],
 *         when: fn() => $this->useName,
 *     ),
 * ],
 * ```
 */
final class CompositeHandler implements RuleHandlerInterface
{
    use PreValidateTrait;

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

        $context->setParameter($this->parameterPreviousRulesErrored, true);

The method setParameter() does not exist on null.

        if ($this->preValidate($value, $context, $rule)) {

It seems like $context can also be of type null; however, parameter $context of Yiisoft\Validator\Rule\CompositeHandler::preValidate() does only seem to accept Yiisoft\Validator\ValidationContext, maybe add an additional type check?

            return new Result();
        }

        return $context->getValidator()->validate($value, $rule->getRules());
    }
}