nus-pe-bot
commented
2 years ago Team's Response
This is not a functionality bug because its behavior is specified in our User Guide, and is expected. In the User Guide below, we specified that only name, email and id cannot have duplicates, but deliberately did not mention phone number.
This is also not a feature flaw. The rationale for allowing multiple phone numbers is because it is possible for two patrons to have the same phone number (can be either mobile or home number). Household phone numbers of siblings are often the same, and in some uncommon cases some siblings also share phone numbers.
Items for the Tester to Verify
:question: Issue response
Team chose [response.Rejected]
- [x] I disagree
Reason for disagreement: I disagree. In the current era, everyone has their own mobile phone number and it is very unlikely phone numbers are shared. I believe the odds of an entry being entered wrongly than the odds of 2 individuals having same phone number.
:question: Issue type
Team chose [type.FeatureFlaw]
Originally [type.FunctionalityBug]
- [ ] I disagree
Reason for disagreement: [replace this with your explanation]
Description: Attempted to save the edit the phone numbers of the patrons to be the same.
Step to reproduce: patron edit 1 p/39393944445555666666 patron edit 2 p/39393944445555666666
Expected: Error message stating phone number has already been saved to another patron.
Actual: Program allowed different patrons to have same phone number.