siqirua / pe

0 stars 0 forks source link

Inconsistency between the behavior #9

Open siqirua opened 9 months ago

siqirua commented 9 months ago

When finding with tags first and then finding tasks with name, the finding tasks with name will search all the task list while when finding with tags first and then finding people with name, the finding will only search the filtered contact list.

This could lead to confusion to some users.

In addition, since the finding tasks command will search the entire list, it makes it very inconvenient if the user to locate a task with certain name and tag and this situation may frequently happen is data size grows large.

(The finding with tags here refers to ftag command. The fatag may have similar behavior but I have no time to test)

nus-pe-bot commented 9 months ago

[IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]

Team's Response

Bug #0900

Thank you for your bug report.

The issue raised by the tester is unclear.

Our understanding of the issue is that after running a findTag (ftag) command, the subsequent findTask (ft)command will search within all tasks, but the findPerson (fp) command will only search the filtered contact list but not all the contacts.

However, we were unable to reproduce the bug. Please refer to the video below for details.


video:https://raw.githubusercontent.com/nus-cs2103-AY2324S1/pe/master/files/661041f5-57c8-4107-a689-12b27efb7002.mp4

Steps attempted to reproduce the bug:

  1. Start CoordiMate in a new folder. (so that CoordiMate loads its sample data)
  2. Run the following commands in order:
    1. atagt 1 t/catering
    2. ftag catering
    3. fp Bernice
    4. ft finale

This shows that CoordiMate is able to search for Persons and Tasks outside of the filtered lists.

Conclusion

As the issue raised by the tester is unclear, and we were unable to reproduce the bug, we will reject this bug report as Issue Unclear and downgrade its severity to Low.

We hope that you understand our decision. Thank you.

Items for the Tester to Verify

:question: Issue response

Team chose [response.IssueUnclear]

Reason for disagreement: [replace this with your reason]


## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your reason]