Open nichee opened 4 months ago
Thank you for pointing this bug out. However, similar to issue 346, this is a documentation error on our part. This is because as can be seen from both the user guide command description and format for querying, as well as the error message in the app itself, we intended the find command to only work when searching keywords. However, we failed to update the user guide to match this intended functionality and fixing the bug in the other issue will fix this issue as well. As such we have decided that it is a duplicate issue. (screenshot attached below)
[The team marked this bug as a duplicate of the following bug]
Adding various duplicate names to the person leads to find command not listing down the patients names
Upon adding a couple of patients with the same name but different NRIC.
using Da as the keyword for finding names.
I am met with listing of 0 persons.
This could be the user not knowing if the keyword means that it has to be "David" in this scenario, leading to a misinterpretation
[original: nus-cs2103-AY2324S2/pe-interim#460] [original labels: type.FunctionalityBug severity.Medium]
[This is the team's response to the above 'original' bug]
Thank you for raising this. We acknowledge this is an issue, however, this is just a typo in the user guide and not a functionality bug in our programme. The error message in the actual MediCLI system states that
find: Finds all persons whose names contain any of the specified keywords (case-insensitive) and displays them as a list with index numbers.
(Screenshot attached below). The line you referenced was a typo as the overall description of the command in the user guide also states that it only accepts keywords.Screenshot:
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:
While this is the same type of issue as the other bug, it is not true that fixing that issue would also fix this issue. Both issues would have to be separately fixed (i.e. remove the stated line from not only the "find" command but also the "patient" command.
This is 1) not the exact same bug 2) fixing one doesn't fix the other bug automatically
Expected: "Find am" should display Amy Smith and Am Smit. - Like the "Han will match Hans" in the UG Actual: "Find am" only displays Am Smit Perhaps its due to having two words in the name?