steeffffeen/FancyManiaLinks

CustomUI has 21 functions (exceeds 20 allowed). Consider refactoring.

Open

class CustomUI
{

    /**
     * @var bool $globalVisible If the UI should be shown at all

Missing class import via use statement (line '265', column '43').

Open

        $domDocument                = new \DOMDocument("1.0", "utf-8");

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

function make() {
    return new \stdClass();
}

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

Open

    protected $checkpointListVisible = null;

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

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++ ) {
        }
    }
}

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

Open

}

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    public function getChallengeInfoVisible()
    {
        return $this->challengeInfoVisible;
    }

• 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
}

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

Open

    public function getCheckpointListVisible()
    {
        return $this->checkpointListVisible;
    }

• 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
}

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

Open

    public function getGlobalVisible()
    {
        return $this->globalVisible;
    }

• 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
}

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

Open

    public function getChatVisible()
    {
        return $this->chatVisible;
    }

• 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
}

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

Open

    public function getRoundScoresVisible()
    {
        return $this->roundScoresVisible;
    }

• 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
}

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

Open

    public function getNetInfosVisible()
    {
        return $this->netInfosVisible;
    }

• 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
}

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

Open

    public function getNoticeVisible()
    {
        return $this->noticeVisible;
    }

• 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
}

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

Open

    public function getScoretableVisible()
    {
        return $this->scoretableVisible;
    }

• 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
}