boonhaii / pe

0 stars 0 forks source link

Application does not warn users of possible duplicate entries due to whitespace typo #7

Open boonhaii opened 2 years ago

boonhaii commented 2 years ago

Users are able to add in another student with the same name but is not detected as a duplicate by the app due to the additional whitespace typo.

Screenshot 2021-11-12 at 5.08.25 PM.png

nus-pe-bot commented 2 years ago

Team's Response

Similar issue (Multiple buggy behaviors that are actually caused by the same defect and cannot be fixed independently, i.e. the equals method for Tutee).

The 'Original' Bug

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

Unable to add tutee with same name but different level, school, address

Unable to add tutee with same name but different level, school, address.

image.png

Expected result: should be able to add. Reason: There can be many people with the same name, it is unreasonable to think that only one person with one name will be attending tuition.


[original: nus-cs2103-AY2122S1/pe-interim#4537] [original labels: type.FunctionalityBug severity.Medium]

Their Response to the 'Original' Bug

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

We feel that this is a low severity bug, as given by the definition: A flaw that is unlikely to affect normal operations of the product. Appears only in very rare situations and causes a minor inconvenience only.

Having students with the exact same full name (inclusive of Chinese name, etc.) is not a common occurrence and even so, there are definitely workarounds, such as putting subject names or other identifying features at the back (similar to how phone contacts are distinguished, e.g. James Tan English versus James Tan Math)

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 explanation]


:question: Issue type

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

Reason for disagreement: [replace this with your explanation]