soc-pe-bot
commented
2 years ago [IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]
Team's Response
Yes it would have been better to have a standard, however its not common for a user to be including additional spaces in between words of a defined command. These commands are pre-defined in the user guide hence the occurence of this issue is rare. Hence it is of low severity.
This issue is similar to issue #365.
The 'Original' Bug
[The team marked this bug as a duplicate of the following bug]
Application accepts whitespace in front of user command but does not allows accept whitespace in the middle
The application allows users to delete doctors using a command with too much whitespace at the front:
However, having whitespace in the middle is not always permitted.
Perhaps you could choose one standard to obey.
[original: nus-cs2113-AY2122S2/pe-interim#410] [original labels: severity.Medium type.FunctionalityBug]
Their Response to the 'Original' Bug
[This is the team's response to the above 'original' bug]
Ye I do agree that there should be a standard to obey. However I think this is a low severity bug as users will not frequently use additional spacings between words in a given command. This bug only occurs in rare instances.
Items for the Tester to Verify
:question: 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 reason]
:question: Issue severity
Team chose [severity.Low]
Originally [severity.Medium]
- [ ] I disagree
Reason for disagreement: [replace this with your reason]
If there are too many spaces, the application catches it.
However, if there are too few spaces, the application allows for it.
Perhaps, for the sake of consistency, you could choose one standard to obey.