seaniy / pe

0 stars 0 forks source link

Invalid date used but error not highlighted #5

Open seaniy opened 3 years ago

seaniy commented 3 years ago

Invalid date was inputted but error message was not displayed and the addition was successful.

Command used: add m/daw a/project 2 by/31/02/2022 2359

31st Feb should be an invalid date. System autocorrects it to 28th Feb instead.

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]

Invalid dates are accepted without raising error/warning

add m/CS1101S a/mission by/31/06/2022 0000 image.png

The deadline of the specified assignment is set to the latest available date instead (ie. 30/06/2022), without prompting any warning/error. Such a command with invalid date might be a result of typo or some other human error, and it is possible that the user does not intend to add the assignment at the invalid date. eg. typo occurred and the intended date was 31/05/2022 instead, yet the app auto accepts the task deadline to be 30/06/2022, without any prompt to notify the user. User has now unknowingly entered an assignment with a deadline that is a month later than expected, thus causing the user to miss the deadline.

This issue is problematic due to the above explanation, and the fact that this behavior is unexpected as it was not documented in the UG.


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

Their Response to the 'Original' Bug

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

This only affects half of the months in the year and only affects at the end of the month. Furthermore, since our target audience is SOC students, and assignments and exam dates are given by the school which is always valid. Users can still continue to use the product with the current bug.

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]