search

nus-cs2113-AY2223S1 / pe-dev-response

0 stars 0 forks source link

Convert-currency function accepts mutliple -x flags #844

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

Screenshot 2022-11-11 at 5.18.13 PM.png

Expected output should have been : arguments entered are invalid

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

jeyvia commented 1 year ago

Team's Response

Greetings karthikstar,

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