search

nus-cs2113-AY2223S1 / pe-dev-response

0 stars 0 forks source link

Commands that should take no arguments still accept arguments #364

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

image.png As seen above. This may confuse users as this is behaviour not defined by the User Guide.

[original: nus-cs2113-AY2223S1/pe-interim#338] [original labels: severity.Low type.FunctionalityBug]

yuu-chennn commented 1 year ago

Team's Response

Greetings malcolmang,

Thank you for your bug report. After careful evaluation of your bug report, the development team is rejecting this bug report due to the following reasons:

1) The parsing of commands is handled by the Apache Commons CLI library. This library allows for duplicate arguments as certain applications require the passing of multiple values for the same argument (E.g. Command -a arg1 -a arg2 -a arg3). In our case, the application takes in the first valid argument and the rest will be ignored.

2) The user guide clearly states that "The presence of invalid or extra arguments will be ignored by MoneyGoWhere." under the section "Understanding the Command Syntax".

image.png

Regards, xzynos

Duplicate status (if any):

Duplicate of #843