nus-cs2103-AY2021S2 / pe-dev-response

0 stars 0 forks source link

Searching by specific fields accepts empty input. #3029

Open nus-pe-bot opened 3 years ago

nus-pe-bot commented 3 years ago

Command Series: add -n John Doe -p 98765432 -e johnd@example.com -c Google -j Software Engineer -a 311, Clementi Ave 2, #02-25 -r University friend -t friends -t owesMoney add -n John -p 98765432 -e johnd@example.com -c Google -j Software Engineer -a 311, Clementi Ave 2, #02-25 -r University friend -t friends -t owesMoney find -n

Explaination: The 'find' command with specific fields accept empty input. But by the format definition in the UG, the name should not be empty. Such command need to be prevented or need to be explained in UG to specify 'find -n' means 'find the person with '-n'' or 'find the person with empty name'.

Screenshot: image.png image.png image.png


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

darkdestry-t commented 3 years ago

Team's Response

Issue relabled as low due to it having low impact/ low probability of occurance. Issue can be easily worked around by supplying a proper search term.

Duplicate status (if any):

--