search

liuyxxy / pe

0 stars 0 forks source link

sort command not performing as expected #2

Open liuyxxy opened 3 years ago

liuyxxy commented 3 years ago

system: catalina

description: the two sort commands end up sorting the entire list

expected and actual result: it is not mentioned in the UG that sort name/sort nric would sort the entire list of patients, thus I was expecting only the search results to be sorted.

step to reproduce:

  1. add in any patient.
  2. search for them.
  3. sort by name (in the screenshots, I sorted by name; however ic does the same thing)
  4. use list to view the entire list of patients.
  5. patients are sorted by name now.

Screen Shot 2020-11-13 at 4.37.12 PM.png ^ initially both aaa and aaa tan are at the end of the list

Screen Shot 2020-11-13 at 4.37.58 PM.png ^ after sorting by name, they are now at the front of the list

nus-pe-bot commented 3 years ago

Team's Response

Developer for this feature states that its an intended feature but was not specific in his UG. The effect does not affect the overall functionality of the command.

Items for the Tester to Verify

:question: Issue type

Team chose [type.DocumentationBug] Originally [type.FunctionalityBug]

Reason for disagreement: [replace this with your explanation]

:question: Issue severity

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

Reason for disagreement: I do not believe that it is a cosmetic flaw since users might be expecting the sort command to function as mentioned in the UG. This would cause, at the very least, a minor inconvenience to the user.