nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Duplicate detection for patients does not match UG #590

Open nus-se-script opened 1 day ago

nus-se-script commented 1 day ago

Screenshot 2024-11-15 at 4.36.15 PM.png Screenshot 2024-11-15 at 4.40.09 PM.png

Issue: The UG states that:

Duplicate patients (same name, phone, or email) are not allowed. I would understand this to mean that if 2 patients have the same name, or same phone, or same email, then they would be considered duplicates and are not allowed. However, as seen in the screenshot attached, it is possible to add patients with the same phone number, but different name and email, or same email, but different name and phone number

To replicate, enter the following commands: createP n/P1 p/87654321 e/p1@gmail.com a/8 Kent Ridge Drive 359332 createP n/P2 p/98765432 e/p1@gmail.com a/8 Kent Ridge Drive 359332 createP n/P3 p/98765432 e/p2@gmail.com a/8 Kent Ridge Drive 359332

Could also be considered a documentation bug


[original: nus-cs2103-AY2425S1/pe-interim#838] [original labels: severity.Low type.FunctionalityBug]

Nigeltzy commented 2 hours ago

Team's Response

No details provided by team.

Duplicate status (if any):

--