UdhayaShan1 / pe

0 stars 0 forks source link

`COMMAND` is case-sensitive #7

Open UdhayaShan1 opened 7 months ago

UdhayaShan1 commented 7 months ago

I believe command words should case insensitive, ADD should be parsed no differently from add. In this case, invalid command is shown.

Unable to take screenshot as the suggestion box for CLI keeps popping up. image.png

This should return same result as add n/...

soc-pe-bot commented 7 months ago

Team's Response

image.png

It is stated in the UG that commands are to be entered in lower cases.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: By rejecting this report, the developers are not acknowledging the root issue of overzealous case sensitive command words which is clearly an issue to rectify. This should not be "expected and correct behavior"

image.png

-I believe this is unreasonable for a CLI application as well as This is also overzealous validation of my command words, suppose a user has CAPS on and types command words as caps ADD n/DENZEL e/EMAIL.... We should expect any CLI app to accept keywords in case-insensitive manner as many users may mix up their command words knowingly or not. This should cause a hindrance to most users.

Since this is a possible issue that should be rectified in future iterations, I believe there is no grounds to reject this downright.


## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.Low`] - [x] I disagree **Reason for disagreement:** I disagree, this is not a cosmetic issue in any way. ![image.png](https://raw.githubusercontent.com/UdhayaShan1/pe/main/files/cd78940b-92f6-46e2-bb29-13687fd40e3e.png) This is at least a FeatureFlaw by ![image.png](https://raw.githubusercontent.com/UdhayaShan1/pe/main/files/d826ddf9-7d1b-4ec7-bf49-4ea81c99663e.png) Hence, it should be at least a `Low`.