Vanderlee\Comprehend\builder\Ruleset

Complexity

Total Complexity

6

Size/Duplication

Total Lines	36
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__set()	0	3	1
A	call()	0	14	4
A	__unset()	0	3	1

<?php

namespace Vanderlee\Comprehend\builder;

use Vanderlee\Comprehend\Parser\Parser;

/**
 * Description of Ruleset
 *
 * @method void define(array | string $name, Definition | Parser | callable $definition = [])
 * @method bool defined(string[] | string $name)
 * @method void undefine(string[] | string $name)
 * //method static void define(array|string $name, Definition|Parser|callable $definition = [])
 * //method static bool defined(string[]|string $name)
 * //method static void undefine(string[]|string $name)
 *
 * @author Martijn
 */
class Ruleset extends AbstractRuleset
{
    protected static function call(&$rules, $key, $arguments = [])
    {
        switch ($key) {
            case 'define':
                return self::defineRule($rules, ...$arguments);

Are you sure the usage of self::defineRule($rules, $arguments) targeting Vanderlee\Comprehend\builder\AbstractRuleset::defineRule() seems to always return null.

$arguments is expanded, but the parameter $names of Vanderlee\Comprehend\builder\AbstractRuleset::defineRule() does not expect variable arguments.

            case 'defined':
                return self::isRuleDefined($rules, ...$arguments);

$arguments is expanded, but the parameter $names of Vanderlee\Comprehend\builder\AbstractRuleset::isRuleDefined() does not expect variable arguments.

            case 'undefine':
                return self::undefineRule($rules, ...$arguments);

$arguments is expanded, but the parameter $names of Vanderlee\Comprehend\builder\AbstractRuleset::undefineRule() does not expect variable arguments.

Are you sure the usage of self::undefineRule($rules, $arguments) targeting Vanderlee\Comprehend\builder\AbstractRuleset::undefineRule() seems to always return null.

        }

        return parent::call($rules, $key, $arguments);
    }

    /**
     * Define an instance rule
     * @param string $name
     * @param Mixed $definition
     * @throws \Exception
     */
    public function __set($name, $definition)
    {
        self::setRule($this->instanceRules, $name, $definition);
    }

    /**
     * Undefine an instance rule
     * @param string $name
     */
    public function __unset($name)
    {
        unset($this->instanceRules[$name]);
    }

}