rhertogh\Yii2Oauth2Server\controllers\console\client\Oauth2ListClientsAction::run()

Complexity

Conditions	2
Paths	2

Size

Total Lines	15
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	6

Importance

Changes	2
Bugs	0	Features	0

<?php
    public function run()
    {
        $module = $this->controller->module;

        $clients = $module->getClientRepository()->getAllClients();

        if (!$clients) {

The expression $clients of type rhertogh\Yii2Oauth2Server\interfaces\models\Oauth2ClientInterface[] is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

            $this->controller->stdout('No clients defined. Run `yii '
                . $this->controller->uniqueId . '/create` to define one.' . PHP_EOL);
            return ExitCode::OK;
        }

        $this->controller->stdout($this->generateClientsTable($clients));

        return ExitCode::OK;
    }