aaronliwh / pe

0 stars 0 forks source link

Incorrect error message for invalid date #5

Open aaronliwh opened 1 year ago

aaronliwh commented 1 year ago

image.png

Input : addtut g/T08 c/UML diagrams t/2022-06-31 1400 Expected : Time should only take certain format, and it should not be blank Actual: This tutorial already exists in the address book

The command " addtut g/T08 c/UML diagrams t/2022-06-30 1400 " was previously added

The error message is wrong, as the inputted tutorial date does not exist in the address book, but is instead invalid (The month of June has 30 months). The error message suggests the user already has a tutorial on June 31st, which may cause some minor confusion and inconvenience

soc-pe-bot commented 1 year ago

Team's Response

No details provided by team.

The 'Original' Bug

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

Incorrect date when adding a tutorial using the addtut command

As per the user guide,

the only special cases for Date are as below

image.png

However, when you try adding other dates that does not exist (such as 31st April), it will be inputted as the last day of the month instead.

e.g. if try to do addtut g/wegweg c/ewgwe t/2022-04-31 0001 the following will be shown instead of an error or displaying the correct dare

image.png


[original: nus-cs2103-AY2223S1/pe-interim#5753] [original labels: type.FunctionalityBug severity.Low]

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]