iranianpep/code-jetter

RouteInfo has 23 functions (exceeds 20 allowed). Consider refactoring.

Open

class RouteInfo
{
    private $routeType;
    private $requestMethod;
    private $url;

Method __construct has 10 arguments (exceeds 4 allowed). Consider refactoring.

Open

        $routeType,
        $requestMethod,
        $url,
        $component,
        $controller,

The method __construct has 10 parameters. Consider reducing the number of parameters to less than 10.

Open

    public function __construct(
        $routeType,
        $requestMethod,
        $url,
        $component,

• Exclude checks
  • Disable engine
  • Disable check

The 'getCheckAntiCSRFToken()' method which returns a boolean should be named 'is...()' or 'has...()'

Open

    public function getCheckAntiCSRFToken()
    {
        return !isset($this->checkAntiCSRFToken) ? false : $this->checkAntiCSRFToken;
    }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    /**
     * @return boolean
     */
    public function getFoo() {} // bad
    /**
     * @return bool
     */
    public function isFoo(); // ok
    /**
     * @return boolean
     */
    public function getFoo($bar); // ok, unless checkParameterizedMethods=true
}