search

muhammad-faruq / pe

0 stars 0 forks source link

INDEX cannot be in any order #14

Open muhammad-faruq opened 2 years ago

muhammad-faruq commented 2 years ago

image.png

It is stated that words in upper case are parameters meaning INDEX is a parameter. Then it is stated that parameters can be in any order. This is certainly not the case as the INDEX parameter needs has to be ordered.

nus-pe-bot commented 2 years ago

Team's Response

This does not hamper the understanding of majority of the readers as the many command examples show that the index is placed at the front of the commands. The examples n/NAME hp/PHONE_NUMBER, hp/PHONE_NUMBER n/NAME also implies that only the parameters with prefixes are considered for this case.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: Once again User Guide is meant to tell the user what to do instead of implying things to the user based on what it writes. It is not a valid argument to state that it can be implied from the User Guide that a command should be as such since there is only so few examples given in the user guide which makes pattern recognition not as simple for the user. Besides it is not for the user to try and discern patterns and assume how the application works but rather be clear on how it works based on what is stated in the UG.

This is most definitely not a cosmetic issue and hence it should still remain as Severity:Low.