search

YANYAN-w / pe

0 stars 0 forks source link

Duplicate parameter in find command does not thrown exception #1

Open YANYAN-w opened 1 week ago

YANYAN-w commented 1 week ago

when executing the find command in the buyer view,

The following screenshot are: view followed by find n/alex n/david, and only the result of David(the later part) is returned. The use of the duplicated prefix 'n/' does not throw an error. If it is not allowed, it would be better to throw an exception saying incorrect command format. and if it is allowed, it would be better to return results that ats least match one keyword. In this case both key word should be taken into account.

image.png

image.png

soc-pe-bot commented 5 days 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

Accepted but severity is likely medium. In the User Guide, it is mentioned that it should be used in the format of n/KEYWORD [MORE_KEYWORDS] and examples are given as well.

image.png

It is not a high severity as it does not cause a major problem and does not make the product unusable for users.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: [replace this with your reason]