[PE-D][Tester D] Application Mismanages Phone Numbers Containing Non-Numeric Characters

[nus-se-script]

Description:
The application fails to recognize common phone number conventions that include descriptors such as "(office)", "(home)", etc. These are commonly used formats and should be accepted by the application.

Steps to Reproduce:

Launch the application.
Attempt to add a new contact with a phone number descriptor using the command: add n/John Doe p/98765432 (office) e/[email protected] m/CS2103 f/Computing v/311, Clementi Ave 2, #2-25 t/friends t/owesMoney a/mon 12:00 14:00 a/thur 12:00 14:00.
Expected Result:

The contact should be added successfully with the phone number including the descriptor "(office)".
Actual Result:

The application rejects the command with an error message: "Phone numbers should only contain numbers, and it should be at least 3 digits long."
Severity:

Medium: This limitation can inconvenience users who use phone number descriptors for better organization and clarity, leading to possible avoidance of using the application for managing contacts.
Screenshots:

Suggestion:

Adjust the application's phone number validation logic to accept these common phone number formats with descriptors. This would likely involve changes to both the back-end validation and the user guide documentation to explicitly state that such formats are supported.

---

Labels: severity.Medium type.FunctionalityBug
original: Shauryan123/ped#7

[whitesnowx]

Intentional Design: Phone Descriptor not need for our purpose

Followup: #233

• Include Plausible Workaround