nus-cs2103-AY2021S2 / pe-dev-response

0 stars 0 forks source link

Wrong error given #508

Open nus-pe-bot opened 3 years ago

nus-pe-bot commented 3 years ago

add n/aaaaaaaaa k/abcd p/1234 e/EMAIL@a a/abcd b/01-01-0001 Enters an invalid command but gets a name error image.png


[original: nus-cs2103-AY2021S2/pe-interim#508] [original labels: severity.Medium type.FunctionalityBug]

ivantjh commented 3 years ago

Team's Response

Thank you for your feedback. The team has reviewed this report and decided that this is not an issue.

Reason

This is a defined behaviour.

Elaboration

Since k/ is not a valid prefix, FriendDex assumes the sub-string aaaaaaaaa k/abcd as the argument for the name prefix (n/). This is the intended behaviour of FriendDex and thus not classified as a bug.

Dropping severity

Assuming this is a bug, then it does not affect the normal operations of the application as it is intended.

Duplicate status (if any):

--