Ansel-Ch / pe

0 stars 0 forks source link

Duplicate persons inconsistent check #1

Open Ansel-Ch opened 4 days ago

Ansel-Ch commented 4 days ago

Duplicate Person regex check

I realised that no two persons can have the same name or phone, however they could share the same email which is not consistent with the fact that there is a check only for duplicate phones although email is another form of unique contact.

To Reproduce

Use an add command with a unique name and phone but an email that is available in the list.

Expected behavior

Due to the overzealous check of duplicate phones, there should also be a check for the second form of communication which is emails. The issue this might cause is the user might contact the wrong email when erroneous email is associated with a different person.

Severity

Medium

Reference – cs2103 bug severity levels

Screenshots

image.png

Desktop

OS: Microsoft Windows Resolution: 2560 X 1440 Version: Windows 11 Home 24H2

nus-pe-script commented 1 day ago

Your response not required for this bug as the team has accepted the bug as it is.