nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Did not handle invalid input for fields in filter #3566

Open nus-pe-bot opened 1 week ago

nus-pe-bot commented 1 week ago

Problem

When user input an invalid field for filter, there isn't any error messages warning the user.

Steps to reproduce

filter m/m

Expected

Error: the module m does not exist.

Actual

0 persons listed!

Screenshot

Screenshot 2024-11-15 at 4.46.49 PM.png

Potential improvement

add error message


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

tanyqcollin commented 1 week ago

Team's Response

The example you gave was filter a person with module m, and this did not match any existing records, the system correctly indicated that 0 person listed!. This is a valid outcome as the filter simply narrow down the list based on the given input, regardless of whether the input matches existing data.

Duplicate status (if any):

--