Merge pull request #7362 from alrossi/fix/9.2/cli-illegal-arg-on-call

dcache-cli: convert IllegalArgumentException to CommandException on …
dCache · Oct 4, 2023 · 195fbb0 · 195fbb0
2 parents ef878f4 + 1378259
commit 195fbb0
Showing 1 changed file with 1 addition and 4 deletions.
diff --git a/modules/common-cli/src/main/java/org/dcache/util/cli/AnnotatedCommandExecutor.java b/modules/common-cli/src/main/java/org/dcache/util/cli/AnnotatedCommandExecutor.java
@@ -123,15 +123,12 @@ public Serializable execute(Args arguments) throws CommandException {
             for (Handler handler : _handlers) {
                 handler.apply(command, arguments);
             }
+            return command.call();
         } catch (IllegalAccessException e) {
             throw new RuntimeException("This is a bug. Please notify " +
                   "support@dcache.org", e);
         } catch (IllegalArgumentException e) {
             throw new CommandSyntaxException(e.getMessage());
-        }
-
-        try {
-            return command.call();
         } catch (CommandException e) {
             throw e;
         } catch (RuntimeException e) {