Added default names in the commands for lazy loading (#668)

* added default names in the commands for lazy loading * fixed codesniffer violation * remove setName, because its done automatically by symfony
FriendsOfSymfony · Oct 22, 2020 · 1e6afb6 · 1e6afb6
1 parent 0a877e4
commit 1e6afb6
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/Command/CleanCommand.php b/Command/CleanCommand.php
@@ -21,15 +21,17 @@
 
 class CleanCommand extends Command
 {
+    protected static $defaultName = 'fos:oauth-server:clean';
+
     private $accessTokenManager;
     private $refreshTokenManager;
     private $authCodeManager;
 
     public function __construct(
         TokenManagerInterface $accessTokenManager,
         TokenManagerInterface $refreshTokenManager,
-        AuthCodeManagerInterface $authCodeManager)
-    {
+        AuthCodeManagerInterface $authCodeManager
+    ) {
         parent::__construct();
 
         $this->accessTokenManager = $accessTokenManager;
@@ -45,7 +47,6 @@ protected function configure()
         parent::configure();
 
         $this
-            ->setName('fos:oauth-server:clean')
             ->setDescription('Clean expired tokens')
             ->setHelp(<<<EOT
 The <info>%command.name%</info> command will remove expired OAuth2 tokens.

diff --git a/Command/CreateClientCommand.php b/Command/CreateClientCommand.php
@@ -22,6 +22,8 @@
 
 class CreateClientCommand extends Command
 {
+    protected static $defaultName = 'fos:oauth-server:create-client';
+
     private $clientManager;
 
     public function __construct(ClientManagerInterface $clientManager)
@@ -39,7 +41,6 @@ protected function configure()
         parent::configure();
 
         $this
-            ->setName('fos:oauth-server:create-client')
             ->setDescription('Creates a new client')
             ->addOption(
                 'redirect-uri',