search

azumieflare / pe

0 stars 0 forks source link

Date validation is not properly implemented #7

Open azumieflare opened 2 months ago

azumieflare commented 2 months ago

When attempting to add events, the command addevent en/Test ed/31-02-2000 ec/Staff is accepted even though it is not a valid date. This could cause users to add in invalid data without notice and cause inconvenience.

image.png

nus-pe-bot commented 2 months ago

Team's Response

No details provided by team.

The 'Original' Bug

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

Adding of invalid dates is allowed for addevent

image.png

Invalid dates such as 31st Feb 2024 are allowed.

[original: nus-cs2103-AY2324S2/pe-interim#3408] [original labels: type.FunctionalityBug severity.Medium]

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]

## :question: Issue type Team chose [`type.FunctionalityBug`] Originally [`type.FeatureFlaw`] - [x] I disagree **Reason for disagreement:** I believe that this is not a functionality bug as the feature still works as intended, but just missing proper date validation. As such, it should be considered a feature flaw instead.
## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]