Allow duplicated Phone Number

[choowengyan]

As shown in the screenshot below, inserting same phone number for different person is allowed, which is not supposed to be in real-world scenario. In real-world scenario, no more than one person should have the same phone number and phone number should be unique.

As shown below, different person were added with the same phone number, and the program allows it. (Client Index 7, 8 and 9)

[nus-se-bot]

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Allow adding duplicated email address for add command

As shown in the screenshot below, the program allow user to insert duplicated email address for different person (Client index 12 and 13).

In real-world scenario, email address should be unique, and no more than one person can have the same email address. In this case, it will cause the user to send information to the wrong person as the email address entered are duplicated and caused inconvenience to the user to double check the accuracy of insertion.

It would be great for the team to detect duplication in the insertion of email address such that duplicated email address is not allowed/ warning message can be shown.

---

_{[original: nus-cs2103-AY2324S2/pe-interim#390] [original labels: severity.Medium type.FeatureFlaw]}

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

In our documentation, we stated the following:

We downgraded the severity to low, as this scenario only happens rarely (people do not edit their client's emails to be duplicates of other client's emails often, if not at all).

This case also qualifies for the highlighted case, as duplicate detection based on email is currently not a feature, although it has been identified as a planned enhancement, and trying to use the missing feature does not cause the software to crash.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

• [x] I disagree

Reason for disagreement: [The reported issue is about duplication of phone number, but the duplicated of another issue (original bug) is about duplication of email address. I believe that they are considered as two different issues.]

---

## :question: Issue response Team chose [`response.NotInScope`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]

---