TopKec / pe

0 stars 0 forks source link

Incorrect handling of leap years #1

Open TopKec opened 5 months ago

TopKec commented 5 months ago

Setting the /tt field to invalid leap years still create a new an interview time, but at the incorrect time.

For example, the following command:

add cn/Google n/John Doe p/98765432 e/johnd@example.com a/311, Clementi Ave 2, #02-25 tt/290221002359 i/Birthday: 12 May 2001 s/5000 pl/Java t/friends t/owesMoney pri/2

Incorrectly determines 29 Feb 2100 as a valid date, instead choosing to depict it as occurring the previous day, as seen below:

nus-se-bot commented 5 months ago

Team's Response

Already stated in UG

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

[ ] I disagree

Reason for disagreement: [replace this with your explanation]

## :question: Issue type Team chose [`type.FeatureFlaw`] Originally [`type.FunctionalityBug`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]