JensonWee / pe

0 stars 0 forks source link

Search for patient issue #5

Open JensonWee opened 4 years ago

JensonWee commented 4 years ago

Description: It is noted that the application will allow empty fields for adding patient. However, when one try to search for such patient, the command does not allow. The program does not trim the parameter too, resulting no value been shown.

Steps: 1) "findp" 2) "findp " 3) "findp "

image.png

Steps: 1) "findp a" 2) "findp a" 3) "findp a"

image.png

nus-pe-bot commented 4 years ago

Team's Response

There seem to be 2 parts to this bug report, so I will explain it by parts.

First part: Command does not allow searching empty string. This is because searching for an empty string would result in the entire list being printed, as seen here. Thus, this would be equivalent to listp.

Second part: findp command searches exactly what the user inputs. As seen in the UG under 4.1.5, it states that it searches based on a search value provided by the user.

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]