abrain/einsatzverwaltung

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

Open

    public function __construct($identifier, $name, $metaKey, $icon, $labelWhenActive, $labelWhenInactive)

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

Open

    public function getStateForReport(int $postId): bool
    {
        $get_post_meta = get_post_meta($postId, $this->metaKey, true);
        return 1 == $get_post_meta;
    }

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