JaegerApp\Validate

Complexity

Total Complexity

16

Size/Duplication

Total Lines	137
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes	1
Bugs	0	Features	0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	16	2
B	loadRules()	0	29	6
A	val()	0	12	2
A	hasErrors()	0	8	2
A	getErrorMessages()	0	4	1
A	getCustomRules()	0	4	1
A	setRegex()	0	5	1
A	getRegex()	0	4	1

<?php
/**
 * Jaeger
 *
 * @copyright	Copyright (c) 2015-2016, mithra62
 * @link		http://jaeger-app.com
 * @version		1.0
 * @filesource 	./Validate.php
 */
namespace JaegerApp;

use Valitron\Validator;
use JaegerApp\Exceptions\ValidateException;

/**
 * mithra62 - Validation Object
 *
 * Object to manage system settings
 *
 * @package Validate
 * @author Eric Lamb <eric@mithra62.com>
 */
class Validate extends Validator
{

    /**
     * The regex engine
     * 
     * @var \mithra62\Regex
     */
    protected $regex = null;

    /**
     * Overrides the initial validation constructor
     * 
     * @throws \InvalidArgumentException
     */
    public function __construct()
    {
        $langDir = static::langDir();
        
        $lang = static::lang();
        // Load language file in directory
        $langFile = rtrim($langDir, '/') . '/' . $lang . '.php';
        if (stream_resolve_include_path($langFile)) {
            $langMessages = include $langFile;
            static::$_ruleMessages = array_merge(static::$_ruleMessages, $langMessages);
        } else {
            throw new \InvalidArgumentException("fail to load language file '$langFile'");
        }
        
        $this->loadRules();
    }

    /**
     * Adds all the custom rules
     * 
     * @throws ValidateException
     */
    protected function loadRules()
    {
        $old_cwd = getcwd();
        chdir(__DIR__);
        $path = 'Validate' . DIRECTORY_SEPARATOR . 'Rules';
        if (! is_dir($path)) {
            throw new ValidateException("Rules Directory " . $path . " isn't a directory...");
        }
        
        $files = new \RecursiveIteratorIterator(new \RecursiveDirectoryIterator($path), \RecursiveIteratorIterator::LEAVES_ONLY);
        
        foreach ($files as $file) {
            if (! $file->isDir()) {
                $name = '\\JaegerApp\\' . str_replace('/', '\\', str_replace('.php', '', $file->getPathname()));
                $class = $name;
                if (class_exists($class)) {
                    $rule = new $class();
                    if ($rule instanceof Validate\RuleInterface) {
                        self::addRule($rule->getName(), array(
                            $rule,
                            'validate'
                        ), $rule->getErrorMessage());

$rule->getErrorMessage() is of type array, but the function expects a string.

                    }
                }
            }
        }
        
        chdir($old_cwd);
    }

    /**
     * Processes the validation group
     * 
     * @param array $data            
     * @return \Valitron\boolean
     */
    public function val(array $data)
    {
        $this->_fields = $data;
        $labels = array();
        foreach ($data as $key => $value) {
            $labels[$key] = ucwords(str_replace('_', ' ', $key));
        }
        
        $this->labels($labels);
        
        return parent::validate();

It seems like you call parent on a different method (validate() instead of val()). Are you sure this is correct? If so, you might want to change this to $this->validate().

    }

    /**
     * Checks if there are errors and returns a boolean
     * 
     * @return boolean
     */
    public function hasErrors()
    {
        if (count($this->errors()) != '0') {
            return true;
        }
        
        return false;
    }

    /**
     * Returns the error messages
     * 
     * @return Ambigous <\Valitron\array, \Valitron\bool>
     */
    public function getErrorMessages()
    {
        return $this->errors();
    }

    /**
     * Returns an array of all the custom rules
     * 
     * @return array
     */
    public function getCustomRules()
    {
        return static::$_rules;
    }

    /**
     * Sets the regular expression engine
     * 
     * @param \mithra62\Regex $regex            
     * @return \mithra62\Validate
     */
    public function setRegex(\JaegerApp\Regex $regex)
    {
        $this->regex = $regex;

It seems like $regex of type object<JaegerApp\Regex> is incompatible with the declared type object<mithra62\Regex> of property $regex.

        return $this;
    }
    
    public function getRegex()
    {
        return $this->regex;
    }
}