search

swongts / pe

0 stars 0 forks source link

Unable to detect duplicate prefixes #4

Open swongts opened 2 years ago

swongts commented 2 years ago

image.png

nus-pe-script commented 2 years ago

Team's Response

This is a design choice and was implemented on purpose. We only detect duplicate functionality when the command could take in that parameter. To illustrate an example, for the command you wrote, since the main add command function is: add [/MEALTYPE] FOOD_NAME, the FOOD_NAME does not require any optional parameters with a backslash and is accepted as a valid command. However, if you misspelled "/dinner" as "/dinz", the command would have been rejected.

This behaviour was mentioned in the UG

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Both /dinner and /snack are valid prefixes. Perhaps this could be considered as a feature flaw instead, since duplicate detections is part of that.