Slick\Validator\AbstractValidator

Complexity

Total Complexity

4

Size/Duplication

Total Lines	67
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

100%

Importance

Changes	2
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getMessage()	0	4	1
A	setMessage()	0	5	1
A	addMessage()	0	13	2

<?php

/**
 * This file is part of slick/validator package
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Slick\Validator;

/**
 * AbstractValidator
 *
 * @package Slick\Validator
 * @author  Filipe Silva <silvam.filipe@gmail.com>
 */
abstract class AbstractValidator
{

    /**
     * @var mixed The value to evaluate
     */
    protected $value;

    /**
     * @readwrite
     * @var array
     */
    protected $message = '';

    /**
     * @var array Error messages templates
     */
    protected $messageTemplate = '';

    /**
     * Returns an array of messages that explain why the most recent
     * isValid() call returned false. The array keys are validation failure
     * message identifiers, and the array values are the corresponding
     * human-readable message strings.
     *
     * @return array
     */
    public function getMessage()
    {
        return $this->message;
    }

    /**
     * Sets a custom message for a given identifier
     *
     * @param string $message
     *
     * @return AbstractValidator
     */
    public function setMessage($message)
    {
        $this->messageTemplate = $message;

It seems like $message of type string is incompatible with the declared type array of property $messageTemplate.

        return $this;
    }

    /**
     * Adds a message using a template.
     *
     * @param string $template Message template.
     *
     * @return AbstractValidator
     */
    protected function addMessage($template)

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

    {
        $arguments = func_get_args();
        $template = $arguments[0];

        $arguments[0] = $template;

        $this->message = sizeof($arguments) > 1

It seems like sizeof($arguments) > 1 ? call_user_func_array('sprintf', $arguments) : ($this->message = $template) of type * is incompatible with the declared type array of property $message.

            ? call_user_func_array('sprintf', $arguments)
            : $this->message = $template;

        return $this;
    }

}