search

garylow2001 / pe

0 stars 0 forks source link

Able to edit dentist name to be same as patient name #5

Open garylow2001 opened 1 year ago

garylow2001 commented 1 year ago

image.png

How to replicate: add-dentist n/John Doe p/98765432 s/Orthodontics y/5 followed by edit-dentist 8 n/Alex Yeohh

Actual: There is both an "Alex Yeohh" in patient list and dentist list, which has the same name but with other fields that are different.

Expected: 1) If you are allowed to have the same person as a patient and a dentist, there should be a warning/confirmation to ask if you wish to enter the same person as a patient and dentist so that all the other fields (phone/address/email etc) are also copied over so they are the same and can clearly reflect the same person 2) If you are not allowed to have the same person as a patient and dentist, then there should be a warning that disallows the user to change a person's name to be both a patient and dentist.

nus-se-bot commented 1 year ago

Team's Response

No details provided by team.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: No details were provided by the team as to why this is rejected, despite it being a valid bug as proven in my bug report.

Additional points to re-iterate: 1) This bug is related to duplicate detection as mentioned under the "feature flaw" section of the PE Guide.

image.png

2) There was no mention of duplicate detection in the UG which I could base my assumptions on when dealing with this bug. (such as whether a patient can be a dentist, or what constitutes a "different person")