search

nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Incorrect error message thrown #1355

Open nus-se-script opened 2 weeks ago

nus-se-script commented 2 weeks ago

When updating a user shown below

image.png

The user wants to update their phone number and lesson time but made a error in the spelling of lesson time prefix update jjjjj p/666666 ls/MON-10:00-10:00

However, a wrong error message is thrown. The error indicates a problem with phone number which is wrong

image.png

[original: nus-cs2103-AY2425S1/pe-interim#1870] [original labels: severity.Low type.FunctionalityBug]

EthanLoww commented 2 weeks ago

Team's Response

ls/ is not a valid prefix. As such, the input after the first valid prefix p/ is taken to be 666666 ls/MON... which is invalid, hence the correct error message is shown. It would also be unreasonable for us to check inputs for misspelt prefixes as that would require many checks.

Duplicate status (if any):

--