search

pennhanlee / pe

0 stars 0 forks source link

Able to add in duplicate location if case is changed. #2

Open pennhanlee opened 3 years ago

pennhanlee commented 3 years ago

First location command: addLocation idl/12345 n/NUS a/Address1, 912345

Second location commnad: addLocation idl/12346 n/NUS a/address1, 912345

I am able to add duplicate location if the address defers by the case-sensitivity.

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: Bug report was issued for Duplicate Location. However, the provided reply was in reference to adding person.

:question: Issue type

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

Reason for disagreement: [replace this with your explanation]