bhnuka / pe

0 stars 0 forks source link

findTask cannot take in blank space #5

Open bhnuka opened 10 months ago

bhnuka commented 10 months ago

findTask does not allow the user to enter a blank space as the task, despite the fact that a blank space is accepted as input for part of the name. As "finale night" allows you to find name by "night", even though it is just a part of the name, a blank space " " should return names with blank spaces (i.e contacts where the title or note has more than just a one word). This is according to the remarks for title/note provided under field, where a task can contain any characters, including spaces. As such, a blank space should be an acceptable input.

image.png

nus-se-script commented 9 months ago

Team's Response

Bug #4847

Thank you for your bug report.

In the Note under the Usage section of the User Guide, we have stated that whitespaces at the start and end of fields are trimmed.

Bug4845.png

As KEYWORD are fields for the findTask command, leading and trailing whitespaces in KEYWORD are also trimmed.

Therefore, having a KEYWORD consisting of only whitespaces is equivalent to having no KEYWORD at all.

As such, the findTask command will show an error about an invalid command format, as no KEYWORD is provided.

Hence, we believe that the current behaviour of the findTask command is correct.

Nevertheless, we acknowledge that the User Guide is too vague regarding the accepted format of KEYWORD. We should have included the fact that the KEYWORD cannot consist entirely of whitespaces.

We hope that you understand our decision. Thank you.

Team chose [type.DocumentationBug] Originally [type.FunctionalityBug]

Reason for disagreement: [replace this with your explanation]