Bug with unarchiving a person at the specified index

[Tsenrae]

Steps to reproduce:

1. unarchive +1

Expected: Person is unarchived as +1 is a positive integer

Actual: Error message saying that person can't be unarchived

[nus-pe-bot]

Team's Response

In the UG, we have stated that the input must be a positive integer, and provided correct examples of valid indexes.

+ is not a number, but a symbol, and the input +1 is clearly not a valid input. Thus, the error message displayed is the expected behavior of our app.

Furthermore, preventing the input +1 does not cause any additional inconvenience for the user, as there exists an even more convenient input 1 which allows the user to achieve the same outcome.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

• [x] I disagree

Reason for disagreement: Your point about stating that the input must be a positive integer is wrong as +1 is clearly a positive integer. The error message above clearly states that INDEX (must be a positive integer). If intentions were clearer, it should be must be a unsigned positive integer. This would then make +1 an invalid input.

Moreover, the multiple examples states (1, 2, 3...), however, it is still up the reader's interpretation of what a positive integer is. It can't be dependent solely on the example numbers, as then the argument can be that since '4' is not listed in the examples, I can't use '4' as a valid input. Hence, it is not about the examples but the definition of the parameter, which in this case is a positive integer and '+1' is a positive integer.

As for your point that preventing the input +1 does not cause any addtional inconvenience for the user, this point is also wrong as you are clearly inconveniencing them. The user would think +1 is a valid input due to your User Guide saying you accept positive integers, and would be confused when they are shown an error message that is also incorrect. Just because there is a more convenient input 1 that allows the user to achieve the same outcome, does not mean that the user is not hindered when a valid input like +1 is not allowed, though they mean the same thing. What if the user is more math-oriented, and is more familiar with a more mathematical way of saying 1 like +1? They would obviously favour the input +1 instead, and be hindered when they find that their input is rejected by your app.

---