Imedia\Ammit\UI\Resolver\Validator\Implementation\Pure\BooleanValidatorTrait

Complexity

Total Complexity

5

Size/Duplication

Total Lines	45
Duplicated Lines	44.44 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

86.67%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
B	mustBeBoolean()	20	33	5

<?php

namespace Imedia\Ammit\UI\Resolver\Validator\Implementation\Pure;

use Assert\InvalidArgumentException;
use Imedia\Ammit\Domain\BooleanValidation;
use Imedia\Ammit\UI\Resolver\UIValidationEngine;
use Imedia\Ammit\UI\Resolver\Validator\UIValidatorInterface;

/**
 * @author Guillaume MOREL <g.morel@imediafrance.fr>
 */
trait BooleanValidatorTrait
{
    /** @var UIValidationEngine */
    protected $validationEngine;

    /**
     * Exceptions are caught in order to be processed later
     * @param mixed $value Boolean ?
     *
     * @return boolean Value casted into boolean or false
     */
    public function mustBeBoolean($value, string $propertyPath = null, UIValidatorInterface $parentValidator = null, string $exceptionMessage = null): bool
    {
        $this->validationEngine->validateFieldValue(
            $parentValidator ?: $this,
            function() use ($value, $propertyPath, $exceptionMessage) {

This code seems to be duplicated across your project.

                $booleanValidation = new BooleanValidation();
                if ($booleanValidation->isBooleanValid($value)) {
                    return;
                }

                if (null === $exceptionMessage) {
                    $exceptionMessage = sprintf(

Consider using a different name than the imported variable $exceptionMessage, or did you forget to import by reference?

                        'Value "%s" is not a boolean.',
                        $value
                    );
                }

                throw new InvalidArgumentException(
                    $exceptionMessage,
                    0,
                    $propertyPath,
                    $value
                );
            }
        );

        // Otherwise "false" would return true
        if (in_array($value, [true, 'true', '1', 1], true)) {

This if statement, and the following return statement can be replaced with return in_array($value, array(true, 'true', '1', 1), true);.

            return true;
        }

        return false;
    }
}