zacwong2151 / pe

0 stars 0 forks source link

Incorrect error message for very large integers for some commands #9

Open zacwong2151 opened 9 months ago

zacwong2151 commented 9 months ago

image.png

when trying to delete an integer larger than 32bit, the error shows invalid command format when the user expects to see the date index provided is invalid. Same applies for star and unstar commands

soc-se-bot commented 9 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]

delete : Inconsistent error messages when valid indexes are provided

Screenshot 2023-11-17 at 4.23.40 PM.png

Screenshot 2023-11-17 at 4.23.58 PM.png

delete 5 and delete 9999999999999999999999999999999999999999 are not handled the same despite both being positive indexes.


[original: nus-cs2103-AY2324S1/pe-interim#3745] [original labels: severity.Low type.FunctionalityBug]

Their Response to the 'Original' Bug

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

image.png

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