edit - error message does not match the error

[sylviaokt]

When the index provided is 0, the error returned is Invalid command format, when it should be The person index provided is invalid as it is stated in the UG that INDEX is a positive integer, including 0.

[nus-pe-bot]

Team's Response

No response provided.

The 'Original' Bug

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

Eedit - error message does not match the error

When the index provided is 0, the error returned is Invalid command format, when it should be The event index provided is invalid as it is stated in the UG that INDEX is a positive integer, including 0.

---

_{[original: nus-cs2103-AY2021S2/pe-interim#2508] [original labels: severity.Medium type.FunctionalityBug]}

Their Response to the 'Original' Bug

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

0 is not a positive integer, the inclusive refers to the max integer value 2,147,483,647 and hence 0 is not an index, so the error is correct.

Items for the Tester to Verify

❓ Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

• I disagree

Reason for disagreement: [replace this with your explanation]

---

❓ Issue response

Team chose [response.Rejected]

• I disagree

Reason for disagreement: Under the BIRTHDAY parameter section, it is specified that year must be a positive integer between 0001 and 9999. The same terms positive integer and between was used here, however it started with 0001 and not 0000. On the other hand, the INDEX parameter mentioned in the bug report started off with 0. If the positive integer does not include 0, why did the UG not state clearly that it was between 1 and..., and used 0 instead? The inconsistency in documentation made me believe that there is a bug and that the error message should show The event index provided is invalid.

---

❓ Issue severity

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

• I disagree

Reason for disagreement: [replace this with your explanation]

---