[jira] Updated: (CLI-6) [cli] Unknown options are ignored instead of throwing UnrecognizedOptionException

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[jira] Updated: (CLI-6) [cli] Unknown options are ignored instead of throwing UnrecognizedOptionException

Gary D. Gregory (Jira)

     [ https://issues.apache.org/jira/browse/CLI-6?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Brian Egge updated CLI-6:
-------------------------

    Component/s:     (was: CLI-2.x)
                 CLI-1.x

The patch is for CLI 1.x.  This issue has been resolved in CLI-71, and should be working as expected in CLI 1.1.

> [cli] Unknown options are ignored instead of throwing UnrecognizedOptionException
> ---------------------------------------------------------------------------------
>
>                 Key: CLI-6
>                 URL: https://issues.apache.org/jira/browse/CLI-6
>             Project: Commons CLI
>          Issue Type: Bug
>          Components: CLI-1.x
>    Affects Versions: 1.0
>         Environment: Operating System: All
> Platform: Other
>            Reporter: Peter Kunszt
>             Fix For: 2.0
>
>         Attachments: patch
>
>
> CLI does not handle unknown options, i.e. throw an 'UnrecognizedOptionException'.
> The reason is that the logic of PosixParser.processOptionToken() is incomplete:
>         if (this.options.hasOption(token))
>         {
>             currentOption = this.options.getOption(token);
>             tokens.add(token);
>         }
>         else if (stopAtNonOption)
>         {
>             eatTheRest = true;
>         }
> ie. if the stopAtNonOption is not set and the token is unknown, it
> will be ignored. there has to be a default case added:
>           else {
>             tokens.add(token);
>         }
> in which case the caller will throw the proper exception, because
> in Parse there is another check whether the option exists.
> i would've submitted a patch but did not find where to, sorry.
> peter

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.