Marcosh\PhpValidationDSL\Basic\NonEmpty

Complexity

Total Complexity

3

Size/Duplication

Total Lines	21
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	10	2
A	withTranslator()	0	3	1

<?php

declare(strict_types=1);

namespace Marcosh\PhpValidationDSL\Basic;

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

final class NonEmpty extends ComposingAssertion implements Validation
{
    public const EMPTY = 'non-empty.empty';

    public function __construct(?callable $errorFormatter = null)
    {
        $this->isAsAsserted = IsAsAsserted::withAssertionAndErrorFormatter(
            function ($data) {
                return !empty($data);
            },
            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::EMPTY];
                }
        );
    }

    public static function withTranslator(Translator $translator): self
    {
        return self::withTranslatorAndMessage($translator, self::EMPTY);
    }
}