search

lil1n / pe

0 stars 0 forks source link

No checks for valid date #7

Open lil1n opened 1 year ago

lil1n commented 1 year ago

image.png

command provided: event bus travel fee e/2 t/31/02/2023 12:00

MoneyMind proceeds to save the date as 31/02/2023, however, no such day actually exists

nus-pe-script commented 1 year ago

Team's Response

Seem to be the same defect as and is automatically fixed by fixing #2896

The 'Original' Bug

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

inaccurate date after start up

image.png

image.png

when event command provided: event bus travel fee e/2 t/31/02/2023 12:00 followed by view command

date shown was based on what was provided

but after exiting the application and starting up, the date changed to 28/04/20231

[original: nus-cs2113-AY2223S2/pe-interim#2886] [original labels: type.FunctionalityBug severity.Medium]

Their Response to the 'Original' Bug

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

When the day of the month of a recurring event is entered as 31, for a month that ends on the 30 or before, the program automatically updates the day to the last day of the same month. As for the month updating from February to April is intended and expected, as the tester entered the event as an recurring expense.

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 type Team chose [`type.FunctionalityBug`] Originally [`type.FeatureFlaw`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]