Marcosh\PhpValidationDSL\Basic\IsIterable

Complexity

Total Complexity

3

Size/Duplication

Total Lines	20
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	withTranslator()	0	4	1
A	__construct()	0	8	2

<?php

declare(strict_types=1);

namespace Marcosh\PhpValidationDSL\Basic;

use Marcosh\PhpValidationDSL\Translator\Translator;
use Marcosh\PhpValidationDSL\Validation;
use function is_callable;

final class IsIterable extends ComposingAssertion implements Validation
{
    public const NOT_AN_ITERABLE = 'is-iterable.not-an-iterable';

    public function __construct(?callable $errorFormatter = null)
    {
        $this->isAsAsserted = IsAsAsserted::withAssertionAndErrorFormatter(
            'is_iterable',
            is_callable($errorFormatter) ?

It seems like is_callable($errorFormatter) ? $errorFormatter : function(...) { /* ... */ } can also be of type null; however, parameter $errorFormatter of Marcosh\PhpValidationDSL\Basic\IsAsAsserted::withAssertionAndErrorFormatter() does only seem to accept callable, maybe add an additional type check?

                $errorFormatter :
                function ($data) {

The parameter $data is not used and could be removed.

                    return [self::NOT_AN_ITERABLE];
                }
        );
    }

    public static function withTranslator(Translator $translator): self
    {
        return new self(function ($data) use ($translator) {

The parameter $data is not used and could be removed.

            return [$translator->translate(self::NOT_AN_ITERABLE)];
        });
    }
}