search

ngoivanessa / pe

0 stars 0 forks source link

Invalid name parameter #8

Open ngoivanessa opened 2 years ago

ngoivanessa commented 2 years ago

This input should not be accepted as "est" is just an extension of "/n" and should not be considered as a name

Screenshot 2021-11-12 at 5.01.15 PM.png

nus-se-script commented 2 years ago

Team's Response

Duplicate of #239

The 'Original' Bug

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

Alternate add command input format not specified in UG

Entering the add /m command with inputs without spacing after the / commands still allows the user to add entries. This defers from the UG where it is stated that the add command has the format "add /m [/n MEMBER_NAME] [/s STUDENT_NUMBER] [/g GENDER] [/p PHONE NUMBER]" with a space after the / commands. This can be seen in the examples below where "add /m /ntim/sA7654321a/gf/p91234567" and "add /m /njohn /sA1234567Z /gm /p82345678" is still accepted as valid inputs.

Bug2.JPG Bug2P2.JPG

[original: nus-cs2113-AY2122S1/pe-interim#223] [original labels: severity.Low type.DocumentationBug]

Their Response to the 'Original' Bug

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

Per our UG, we did not explicitly mention that the space was needed to differentiate between the commands; the spacebar is merely intended to help users visualise their data better.

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

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]

:question: Issue type

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

Reason for disagreement: [replace this with your reason]