Merge pull request #101 from inquam/dev

Only evaluate 'needs' constraints of an option if that option is actually used.
nategood · Jul 10, 2019 · 0bfb6e3 · 0bfb6e3
2 parents ba1ba17 + 06b2d52
commit 0bfb6e3
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/Commando/Command.php b/src/Commando/Command.php
@@ -525,10 +525,11 @@ public function parse()
                 }
             }
 
-            // See if our options have what they require
+            // See if our options have what they require.
+            // But only do so if the option is actually used
             foreach ($this->options as $option) {
                 $needs = $option->hasNeeds($this->options);
-                if ($needs !== true) {
+                if ($needs !== true && array_key_exists($option->getName(), $keyvals)) {
                     throw new \InvalidArgumentException(
                         'Option "' . $option->getName() . '" does not have required option(s): ' . implode(', ', $needs)
                     );