Jarrod-Bob / pe

0 stars 0 forks source link

Find Command Bug #5

Open Jarrod-Bob opened 4 years ago

Jarrod-Bob commented 4 years ago

image.png

As per the UG, trying to find whether my data's name conatins 'sadsa' would return no transactions even though I have at least 1 transaction whose name 'contains' 'sadsa'. Users might be confused about the functionality of this command as it does not follow what is implied.

image.png

Replication:

  1. add n/ sadsa\sadsa\/
  2. find sadsa
nus-pe-bot commented 4 years ago

Team's Response

image.png

According to the userguide, characters separated by '/' or '\' do not constitute separate words. Words are characters separated by spaces. Therefore "sadsa\sadsa\/" is one word, so "sadsa" could not be found.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Based on your image above, a user might be naturally inclined to think that "contains" mean that the name contains a certain substring of the keyword given.

The line "Clears all filter and display all transactions whose name matches any of the keyword" is also misleading as the meaning of match is not clear.

I am ok with dropping this in Severity to either Low or VeryLow, but it is something that should not be rejected since it is not being made clear.

sadsa does match one portion of the name sadsa\sadsa/, but even if it should match the name, then it seems to suggest either

  1. the keyword has to be the exact name itself
  2. The clarity of the UG needs to be improved upon.