search

AdiMangalam / pe

0 stars 0 forks source link

Code accepts invalid date as entry #1

Open AdiMangalam opened 1 week ago

AdiMangalam commented 1 week ago

image.png

when entering date, the product accepts invalid date like 30th Feb or 31st Feb. The expect result is invalid date input.

nus-se-bot commented 6 days ago

Team's Response

No details provided by team.

The 'Original' Bug

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

Invalid dates accepted

Description

{8D2FCAC6-3D2F-4A02-9CE6-3D413589230B}.png

The program accepts 30th February as a valid input, although in the real-world, it should be rejected as a invalid date.

[original: nus-cs2113-AY2425S1/pe-interim#293] [original labels: type.FeatureFlaw severity.VeryLow]

Their Response to the 'Original' Bug

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

Thank you for bringing this to our attention. We have indeed overlooked this edge case for the month February not having the days 30 and 31.

We deem this a duplicate with #146 as it is the exact same thing.

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.FeatureFlaw`] Originally [`type.FunctionalityBug`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]
## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.Medium`] - [x] I disagree **Reason for disagreement:** It should be prioritized as medium or low, given its impact on data validity and user experience. However, classifying it as very low is not acceptable, as it represents a fundamental error in date validation that cannot be overlooked. This requires attention to ensure accuracy and reliability in the system.