rhertogh\Yii2Oauth2Server\controllers\console\client\Oauth2ListClientsAction

Complexity

Total Complexity

2

Size/Duplication

Total Lines	19
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	2
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	run()	0	15	2

<?php

namespace rhertogh\Yii2Oauth2Server\controllers\console\client;

use rhertogh\Yii2Oauth2Server\controllers\console\base\traits\GenerateClientsTableTrait;
use rhertogh\Yii2Oauth2Server\controllers\console\Oauth2ClientController;
use yii\base\Action;
use yii\console\ExitCode;

/**
 * @property Oauth2ClientController $controller
 */
class Oauth2ListClientsAction extends Action
{
    use GenerateClientsTableTrait;

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