search

Kaya3842 / pe

0 stars 0 forks source link

Person duplicates did not account for case sensitivity #1

Open Kaya3842 opened 6 months ago

Kaya3842 commented 6 months ago

In the UG it was stated that " Duplicate clients with the same name are not allowed". However, the duplicates did not account for case sensitivity, therefore user can still add a same contact with different uppercases and lowercases.

image.png

soc-pe-bot commented 6 months ago

[IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]

Team's Response

No details provided by team.

The 'Original' Bug

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

Name is case sensitive

When adding a new client, clients with the same name but different capitalizations are considered as different people. The user guide specifically mentioned that duplicate clients with the same name should not be allowed, but they should at least restrict this behavior or give an error message when given with such an input

Steps to reproduce:

  1. add n/Betsy Crowe t/friend e/betsycrowe@example.com a/Newgate Prison p/1234567 t/criminal
  2. add n/Betsy crowe t/friend e/betsycrowe@example.com a/Newgate Prison p/1234567 t/criminal

[original: nus-cs2103-AY2324S2/pe-interim#4091] [original labels: severity.Low type.FeatureFlaw]

Their Response to the 'Original' Bug

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

No details provided by team.

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)

Reason for disagreement: [replace this with your reason]

## :question: Issue type Team chose [`type.FeatureFlaw`] Originally [`type.FunctionalityBug`] - [ ] I disagree **Reason for disagreement:** [replace this with your reason]