search

seaniy / pe

0 stars 0 forks source link

Invalid error message shown when adding assignment but with extra parameters #9

Open seaniy opened 3 years ago

seaniy commented 3 years ago

Command to recreate: add m/daw a/wda by/17/04/2021 2359 n/21

Should be a command format error as opposed to incorrect date format.

image.png

nus-pe-bot commented 3 years ago

Team's Response

No response provided.

The 'Original' Bug

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

Incorrect error message for edit assignment

Entered Command: edit m/Test Module One a/1 on/04/05/2019 1400

Expected output: Wrong error thrown when using on/ instead of by/ or d/ for edit assignment. Should be error for incorrect command format instead.

image.png

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

Their Response to the 'Original' Bug

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

The alphanumeric error is not an error, as the prefix "on" is not expected here, so it is taken as part of the assignment prefix "a/" especially when it is the most recent valid prefix for the command format. The content of the error message is correct and so we reduced the severity of the issue as it does not affect the overall use of the app, especially with the proper use of the user guide.

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 severity

Team chose [severity.Low] Originally [severity.High]

Reason for disagreement: The content of the error message is not correct as it does not reflect back to the user that he/she is using the wrong prefix format.