search

nus-cs2103-AY2021S1 / pe-dev-response

0 stars 0 forks source link

Inconsistent error message #5944

Open nus-pe-bot opened 3 years ago

nus-pe-bot commented 3 years ago

delete -1 shows Unrecognized option: -1 delete 100 shows invalid index

[original: nus-cs2103-AY2021S1/pe-interim#5929]

aidoxe-123 commented 3 years ago

Team's Response

Thank you for the bug report. However, this is the expected behaviour of our app as "-1" will be treated as a flag (refer to UG) , while "100" is treated as an index that may be invalid if size of food list is <100.

Essentially, the parser throws this error since it treats -1 as a flag (like -t, -n, ...).

Therefore, we will be rejecting this as a bug as the "issue" follows the expected behaviour.

Duplicate status (if any):

--