search

lebencwb / pe

0 stars 0 forks source link

Index does not handle large integers well #2

Open lebencwb opened 3 years ago

lebencwb commented 3 years ago

When entering a command with an index (e.g. delete INDEX), the index is not parsed correctly when a large positive integer is entered, showing an 'Invalid command format' error message instead of stating that the index provided is invalid (i.e. the index exceeds the number of contacts), even though the UG does not state any restrictions on the maximum integer that can be input.

Steps to reproduce: delete 1000000000000000

Expected:

image.png

(with 1000000000000000 instead of 10).

Actual: image.png

nus-pe-bot commented 3 years ago

Team's Response

Thank you for your report! This is the behaviour inherited from AB3, and thus even though the issue is valid, it will be rejected. If it is a problem, the severity should be low because it only causes a minor inconvenience to users and happens very rarely (regular users will not use such a big number).

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]

:question: Issue severity

Team chose [severity.Low] Originally [severity.Medium]

Reason for disagreement: [replace this with your reason]