From 781f307190b86a8197353335e8b7e0bdded76208 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Vedran=20Miho=C4=8Dinec?= <vedran.mihocinec@gmail.com>
Date: Mon, 30 Mar 2020 22:10:54 +0200
Subject: [PATCH] Fix UpdateClientCommand options

---
 Command/UpdateClientCommand.php | 106 ++++++++++++++++++++++----------
 1 file changed, 75 insertions(+), 31 deletions(-)

diff --git a/Command/UpdateClientCommand.php b/Command/UpdateClientCommand.php
index 624c7be1..5b7915e8 100644
--- a/Command/UpdateClientCommand.php
+++ b/Command/UpdateClientCommand.php
@@ -39,29 +39,30 @@ protected function configure(): void
             ->addOption(
                 'redirect-uri',
                 null,
-                InputOption::VALUE_REQUIRED | InputOption::VALUE_IS_ARRAY,
-                'Sets redirect uri for client. Use this option multiple times to set multiple redirect URIs.',
-                []
+                InputOption::VALUE_OPTIONAL | InputOption::VALUE_IS_ARRAY,
+                'Sets redirect uri for client. Use this option multiple times to set multiple redirect URIs. Use it without value to remove existing values.',
+                [0]
             )
             ->addOption(
                 'grant-type',
                 null,
-                InputOption::VALUE_REQUIRED | InputOption::VALUE_IS_ARRAY,
-                'Sets allowed grant type for client. Use this option multiple times to set multiple grant types.',
-                []
+                InputOption::VALUE_OPTIONAL | InputOption::VALUE_IS_ARRAY,
+                'Sets allowed grant type for client. Use this option multiple times to set multiple grant types. Use it without value to remove existing values.',
+                [0]
             )
             ->addOption(
                 'scope',
                 null,
-                InputOption::VALUE_REQUIRED | InputOption::VALUE_IS_ARRAY,
-                'Sets allowed scope for client. Use this option multiple times to set multiple scopes.',
-                []
+                InputOption::VALUE_OPTIONAL | InputOption::VALUE_IS_ARRAY,
+                'Sets allowed scope for client. Use this option multiple times to set multiple scopes. Use it without value to remove existing values.',
+                [0]
             )
             ->addOption(
-                'deactivated',
+                'active',
                 null,
-                InputOption::VALUE_NONE,
-                'If provided, it will deactivate the given client.'
+                InputOption::VALUE_REQUIRED,
+                'Client active state, 1 for active, 0 for inactive',
+                null
             )
             ->addArgument(
                 'identifier',
@@ -90,26 +91,69 @@ protected function execute(InputInterface $input, OutputInterface $output): int
 
     private function updateClientFromInput(Client $client, InputInterface $input): Client
     {
-        $client->setActive(!$input->getOption('deactivated'));
-
-        $redirectUris = array_map(
-            static function (string $redirectUri): RedirectUri { return new RedirectUri($redirectUri); },
-            $input->getOption('redirect-uri')
-        );
-        $client->setRedirectUris(...$redirectUris);
-
-        $grants = array_map(
-            static function (string $grant): Grant { return new Grant($grant); },
-            $input->getOption('grant-type')
-        );
-        $client->setGrants(...$grants);
-
-        $scopes = array_map(
-            static function (string $scope): Scope { return new Scope($scope); },
-            $input->getOption('scope')
-        );
-        $client->setScopes(...$scopes);
+        $active = $input->getOption('active');
+
+        if (null !== $active) {
+            $client->setActive((bool) $active);
+        }
+
+        $redirectUrisArray = $this->getNullableOption($input, 'redirect-uri');
+
+        if (null !== $redirectUrisArray) {
+            $redirectUris = array_map(
+                static function (string $redirectUri): RedirectUri {
+                    return new RedirectUri($redirectUri);
+                },
+                $redirectUrisArray
+            );
+            $client->setRedirectUris(...$redirectUris);
+        }
+
+        $grantsArray = $this->getNullableOption($input, 'grant-type');
+
+        if (null !== $grantsArray) {
+            $grants = array_map(
+                static function (string $grant): Grant {
+                    return new Grant($grant);
+                },
+                $grantsArray
+            );
+            $client->setGrants(...$grants);
+        }
+
+        $scopesArray = $this->getNullableOption($input, 'scope');
+
+        if (null !== $scopesArray) {
+            $scopes = array_map(
+                static function (string $scope): Scope {
+                    return new Scope($scope);
+                },
+                $scopesArray
+            );
+            $client->setScopes(...$scopes);
+        }
 
         return $client;
     }
+
+    private function getNullableOption(InputInterface $input, string $name): ?array
+    {
+        $value = $input->getOption($name);
+
+        if (
+            array_key_exists(0, $value)
+            && 0 === $value[0] //if user has entered some value it will always be string so it is fine to rely on 0
+        ) {
+            return null;
+        }
+
+        if (
+            array_key_exists(0, $value)
+            && null === $value[0] //when option has mode InputOption::VALUE_OPTIONAL | InputOption::VALUE_IS_ARRAY and no value is sent, option will have value [null]
+        ) {
+            return [];
+        }
+
+        return $value;
+    }
 }