search

nus-cs2113-AY2122S1 / pe-dev-response

0 stars 0 forks source link

Adding ingredient with special character error message is inconsistent #2948

Open nus-se-bot opened 2 years ago

nus-se-bot commented 2 years ago

enter: add i/yoo/ expected: error msg about speical char

Normally this happens: enter: add i/yoo :D expected: error msg about special char

image.png

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

bernardboey commented 2 years ago

Team's Response

This is expected and correct behaviour. The error message from our program is correct, as it is intended to warn about invalid command formats (e.g. the user might have intended for add i/chicken u/kg but accidentally typed it as add i/chickenu/kg). In such a case, the invalid command format message is more appropriate. It would not be reasonable to distinguish between the user's intention for adding the / as part of the name vs as part of a command format.

Additionally, this issue does not affect usage of the product.

Duplicate status (if any):

--