rhosocial/yii2-organization

Space found before comma in function call

Open

            $this->removeMemberSuccessMessage = Yii::t('organization' ,'Member Removed.');

• Exclude checks
  • Disable engine
  • Disable check

Space found before comma in function call

Open

            $this->updateSuccessMessage = Yii::t('user' ,'Updated.');

• Exclude checks
  • Disable engine
  • Disable check

Space found before comma in function call

Open

            $this->organizationSetUpSuccessMessage = Yii::t('organization' ,'Organization Set Up.');

• Exclude checks
  • Disable engine
  • Disable check

The method removeAdministrator has a boolean flag argument $keep, which is a certain sign of a Single Responsibility Principle violation.

Open

    public function removeAdministrator(&$member, $keep = true)

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

class Foo {
    public function bar($flag = true) {
    }
}

Avoid too many return statements within this method.

Open

                return Url::to(['update', 'id' => $model->getID()]);

Avoid too many return statements within this method.

Open

                return Yii::$app->authManager->checkAccess($this->operator->getGUID(), $permission, ['organization' => $model]);

Avoid too many return statements within this method.

Open

                    return false;

Avoid too many return statements within this method.

Open

        return $result;

Avoid too many return statements within this method.

Open

                return Url::to(['exit', 'id' => $model->getID()]);

Avoid too many return statements within this method.

Open

                return true;

Avoid too many return statements within this method.

Open

                return Yii::$app->authManager->checkAccess($this->operator->getGUID(), (new ManageMember)->name, ['organization' => $model->organization]);

Avoid too many return statements within this method.

Open

                return Url::to(['revoke', 'id' => $model->getID()]);

The method setSetting has a boolean flag argument $unique, which is a certain sign of a Single Responsibility Principle violation.

Open

    public function setSetting($item, $value, $unique = false)

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

class Foo {
    public function bar($flag = true) {
    }
}

The method setIsOnlyAcceptCurrentOrgMember has a boolean flag argument $value, which is a certain sign of a Single Responsibility Principle violation.

Open

    public function setIsOnlyAcceptCurrentOrgMember($value = true)

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

class Foo {
    public function bar($flag = true) {
    }
}

Avoid too many return statements within this method.

Open

            return '#';

The method setIsDisallowMemberJoinOther has a boolean flag argument $value, which is a certain sign of a Single Responsibility Principle violation.

Open

    public function setIsDisallowMemberJoinOther($value = true)

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

class Foo {
    public function bar($flag = true) {
    }
}

Avoid too many return statements within this method.

Open

                return Url::to(['settings', 'id' => $model->getID()]);

The method setIsOnlyAcceptSuperiorOrgMember has a boolean flag argument $value, which is a certain sign of a Single Responsibility Principle violation.

Open

    public function setIsOnlyAcceptSuperiorOrgMember($value = true)

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

class Foo {
    public function bar($flag = true) {
    }
}

Avoid too many return statements within this method.

Open

        return $this->controller->redirect(['member', 'org' => $org]);

The method setIsExcludeOtherMembers has a boolean flag argument $value, which is a certain sign of a Single Responsibility Principle violation.

Open

    public function setIsExcludeOtherMembers($value = true)

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

class Foo {
    public function bar($flag = true) {
    }
}