mindplay\kissform\Validators\CheckParser

Complexity

Total Complexity

4

Size/Duplication

Total Lines	35
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	validate()	0	12	3

<?php

namespace mindplay\kissform\Validators;

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

/**
 * Validate a Field by attempting to parse the input value.
 *
 * This is useful for e.g. date or date/time input which may be parsed by {@see DateTimeStringField}.
 */
class CheckParser implements ValidatorInterface
{
    /**
     * @var ParserInterface
     */
    private $parser;

    /**
     * @var string
     */
    private $error;

    /**
     * @param ParserInterface $parser the InputParser (usually a Field) which should attempt to parse the input
     * @param string          $error  error message template (the "{field}" token will be substituted)
     */
    public function __construct(ParserInterface $parser, $error)
    {
        $this->parser = $parser;
        $this->error = $error;
    }

    public function validate(FieldInterface $field, InputModel $model, InputValidation $validation)
    {
        $input = $model->getInput($field);

        if ($input === null) {
            return; // no input
        }

        if ($this->parser->parseInput($input) === null) {

It seems like $input defined by $model->getInput($field) on line 41 can also be of type array; however, mindplay\kissform\Facets\ParserInterface::parseInput() does only seem to accept string, maybe add an additional type check?

            $model->setError($field, strtr($this->error, ["field" => $validation->getLabel($field)]));
        }
    }
}