mindplay\kissform\Validators\CheckRequired

Complexity

Total Complexity

4

Size/Duplication

Total Lines	25
Duplicated Lines	36 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

100%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	validate()	9	9	3

<?php

namespace mindplay\kissform\Validators;

use mindplay\kissform\Facets\FieldInterface;
use mindplay\kissform\Facets\ValidatorInterface;
use mindplay\kissform\InputModel;
use mindplay\kissform\InputValidation;
use mindplay\lang;

/**
 * Validate required input
 */
class CheckRequired implements ValidatorInterface
{
    /**
     * @var string|null
     */
    private $error;

    /**
     * @param string $error optional custom error message

Should the type for parameter $error not be string|null?

     */
    public function __construct($error = null)
    {
        $this->error = $error;
    }

    public function validate(FieldInterface $field, InputModel $model, InputValidation $validation)

This method seems to be duplicated in your project.

    {
        if ($model->getInput($field) === null) {
            $model->setError(
                $field,
                $this->error ?: lang::text("mindplay/kissform", "required", ["field" => $validation->getLabel($field)])
            );
        }
    }
}