steeffeen/FancyManiaLinks

View on GitHub
FML/Types/Scriptable.php

Summary

Maintainability
A
0 mins
Test Coverage
F
0%

Avoid excessively long variable names like $scriptActionParameters. Keep variable name length under 20.
Open

    public function setScriptAction($scriptAction, array $scriptActionParameters = null);
Severity: Minor
Found in FML/Types/Scriptable.php by phpmd

LongVariable

Since: 0.2

Detects when a field, formal or local variable is declared with a long name.

Example

class Something {
    protected $reallyLongIntName = -3; // VIOLATION - Field
    public static function main( array $interestingArgumentsList[] ) { // VIOLATION - Formal
        $otherReallyLongName = -5; // VIOLATION - Local
        for ($interestingIntIndex = 0; // VIOLATION - For
             $interestingIntIndex < 10;
             $interestingIntIndex++ ) {
        }
    }
}

Source https://phpmd.org/rules/naming.html#longvariable

Avoid excessively long variable names like $scriptActionParameters. Keep variable name length under 20.
Open

    public function setScriptActionParameters(array $scriptActionParameters = null);
Severity: Minor
Found in FML/Types/Scriptable.php by phpmd

LongVariable

Since: 0.2

Detects when a field, formal or local variable is declared with a long name.

Example

class Something {
    protected $reallyLongIntName = -3; // VIOLATION - Field
    public static function main( array $interestingArgumentsList[] ) { // VIOLATION - Formal
        $otherReallyLongName = -5; // VIOLATION - Local
        for ($interestingIntIndex = 0; // VIOLATION - For
             $interestingIntIndex < 10;
             $interestingIntIndex++ ) {
        }
    }
}

Source https://phpmd.org/rules/naming.html#longvariable

The closing brace for the interface must go on the next line after the body
Open

}
Severity: Minor
Found in FML/Types/Scriptable.php by phpcodesniffer

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

    public function getScriptEvents();
Severity: Minor
Found in FML/Types/Scriptable.php by phpmd

BooleanGetMethodName

Since: 0.2

Looks for methods named 'getX()' with 'boolean' as the return type. The convention is to name these methods 'isX()' or 'hasX()'.

Example

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

Source https://phpmd.org/rules/naming.html#booleangetmethodname

There are no issues that match your filters.

Category
Status