CodyChew / pe

0 stars 0 forks source link

cannot find for items in menu #2

Open CodyChew opened 3 years ago

CodyChew commented 3 years ago

steps to reproduce: 1)enter a valid vendor with milo in the menu e.g. vendor 1 and show menu using menu command result: image.png

2)type find "milo result: nothing found

image.png

3) type find milo result: no milo found (error) image.png

nus-se-bot commented 3 years ago

Team's Response

The program is working as intended. Find, price, and sort are chained with one another as stated in the dialogue window. So after the find "milo gives an empty list, find milo is also ran based on this empty list. We also mentioned to get back the original menu the users will have to use menu command.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: There was no mention of this behaviour in the UserGuide hence I was confused as a first time user. In addition, the term "chain the current filter" is ambiguous and many users would know the behaviour of it. This term was not mentioned in the UserGuide nor in the DeveloperGuide.

From UserGuide: image.png


:question: Issue severity

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

Reason for disagreement: As a first time user, I was confused and this affected my operations when using find command. After the reading the UserGuide which describes "lists all menu items" containing the word, it occasionally works correctly whereas it changes behaviour to what I was not expecting. I understand that I can reset the menu using menu command however the first point about chaining was not clear to me. It introduces the idea of "current menu" or "displayed menu" which is briefly touched on in the Developer Guide. However there were other terms such as "selected menu", neither of which were defined in the Glossary or in the DeveloperGuide before it was first used. Perhaps, severity of high is harsh and can be argues for low/medium. For expert users who figured out this behaviour, this will not be an issue.

From DeveloperGuide:

image.png

image.png