search

nus-cs2113-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Find features display all name when there is no input #607

Open nus-pe-bot opened 2 weeks ago

nus-pe-bot commented 2 weeks ago

Steps to recreate

Expected output

Actual output Screenshot 2024-11-15 162033.png

I think there needs the be a check and trim of anything after and validate the input

[original: nus-cs2113-AY2425S1/pe-interim#380] [original labels: severity.Medium type.FunctionalityBug]

lowjunchen commented 2 weeks ago

Team's Response

Thank you for the report. If a space is indicated as something to find, our program assumes you want to return all. Therefore, it should act as a 'list', which in fact works better than returning an error message. This is because in normal operations, a word to search should have been added. This is normal behavior and should have been added in the user guide.

Consider the case where no space is added. Our program will consider it to be an invalid input as the user has not indicated anything to find.

Our team decided to change the severity to LOW as the product is still usable, image.png

Duplicate status (if any):

--