search

owx0130 / pe

0 stars 0 forks source link

phone number does not allow whitespace in between #9

Open owx0130 opened 5 months ago

owx0130 commented 5 months ago

image.png

many numbers in singapore follow this format with a whitespace in the middle, but this is not accepted by the application. this qualifies for overzealous input validation in my opinion, see below:

image.png

soc-se-bot commented 5 months ago

Team's Response

No details provided by team.

The 'Original' Bug

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

Unable to input phone number with spaces

In the UG, it was specified that the user needs to input an 8-digit phone number, however there is no indication that there shouldn't be any space. The app, however, cannot receive any blank spaces.

image.png

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

Their Response to the 'Original' Bug

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

Regardless of the spacing issue, the product is still usable, and numbers are still able to be input into the program. Furthermore the spaces are generally printed to improve readability when showing it to human readers, not when entering data. Thus relabelling as not in scope.

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 response Team chose [`response.NotInScope`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]
## :question: Issue type Team chose [`type.FeatureFlaw`] Originally [`type.FunctionalityBug`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]