sudarshan2401 / pe

0 stars 0 forks source link

Unable to add person with "s/o" and "-" in their names #2

Open sudarshan2401 opened 8 months ago

sudarshan2401 commented 8 months ago

Command: add Description: Unable to add people with "s/o" and "-" in their names Steps to Reproduce:

  1. add n/tom s/o muthu p/82775346 e/sarah.woo@gmail.com
  2. add n/wang e-jin p/82775346 e/sarah.woo@gmail.com

Expected Behavior: Accomodate for people with non alphanumerical characters in their names due to being in a context where people do have non alphanumerical characters in their names. Actual Behavior: Application throws error "Names should only contain alphanumeric characters and spaces, and it should not be blank"

image.png

nus-se-bot commented 7 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]

Name input only allows alphanumeric characters

Great job in specifying in the UG that

String, alphanumeric values and at least 2 characters long.

However, as this app is intended to organise contacts, it should be possible to input other Unicode characters for the name field. This is especially the case in that this is a generic contacts management app, and I might have contacts from different countries.

For example, add n/María José Carreño Quiñones p/12345678 e/john.doe@example.com is not an acceptable input, but it is possible that a person's name includes non-alphanumeric characters.

Other names that you might wish to consider: John s/o Doe 毛泽东 María José Carreño Quiñones John Q. Public. John Hellen-Doe Борис Николаевич Ельцин (Boris Nikolayevich Yeltsin)


[original: nus-cs2103-AY2324S1/pe-interim#2777] [original labels: severity.Low type.FeatureFlaw]

Their Response to the 'Original' Bug

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

No details provided by team.

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]