petermonky / pe

0 stars 0 forks source link

`filterClients` does not work #9

Open petermonky opened 2 years ago

petermonky commented 2 years ago

image.png

image.png

image.png

The filterClients command does not appear to work at all. No filtering appears to take place regardless of the input parameters.

Expected: Clients with age equal to 23 are retrieved

Actual: All clients are retrieved instead

nus-pe-bot commented 2 years ago

Team's Response

We accepted a similar issue where the user attempted to filter the client list after sorting.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Filtering client by age does not work as expected

image.png

The filter did not filter out the correct clients by age, the issue with sorting seems to persist here (after performing a sort operation, many functionalities do not work as expected).


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

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

While this is indeed a limitation of our application, we specifically warned users regarding the usage of filter and sort at the same time in the User Guide.

image.png

The filter and sort functionalities can only work independently in this release of the application. For this release, we did not push a feature that allows for simultaneous filtering and sorting. However, recognize the limitations that this might cause.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]


:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]


:question: Issue severity

Team chose [severity.Medium] Originally [severity.High]

Reason for disagreement: [replace this with your explanation]