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

["Henri Yandell (JIRA)" <ji...@apache.org>]

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

Henri Yandell commented on CLI-6:
---------------------------------

Is a patch for 2.0 still needed?

> [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.