search

pennhanlee / pe

0 stars 0 forks source link

Duplicate Person can be added if Case is not sensitive #1

Open pennhanlee opened 3 years ago

pennhanlee commented 3 years ago

First Command: addPerson idp/12345 n/Mary p/91234567 e/johnd@example.com a/John street,block 123, #01-01 q/2020-10-10 i/false

Second Command: addPerson idp/12345 n/mary p/91234567 e/johnd@example.com a/John street,block 123, #01-01 q/2020-10-10 i/false

In the UG. It states that persons are duplicates if they have the same name and email/phone number are the same.

In this case, when the case is not sensitive, the duplicate can be added.

image.png

image.png

nus-pe-bot commented 3 years ago

Team's Response

This aspect of accepting people because of name has been inherited from addressbook and thus would be rejected.

In addition, the functionality describes follows our user guide in which we do add the person since the name is different. Hence, this should qualify as a feature flaw.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]

:question: Issue type

Team chose [type.FeatureFlaw] Originally [type.FunctionalityBug]

Reason for disagreement: [replace this with your explanation]