ARPspoofing / pe

0 stars 0 forks source link

Prefix chaining does not work beyond 2 prefixes #14

Open ARPspoofing opened 1 year ago

ARPspoofing commented 1 year ago

As stated in this UG:

If a parameter is expected only once in the command but you specified it multiple times, only the last occurrence of the parameter will be taken. (eg. if you specify p/12341234 p/56785678 , only p/56785678 will be taken)

So, if the parameter for find is specified more than once, it should take the last one. However, it returns nothing even if the name is valid

Steps to reproduce (1 is the working command, 2 is the not working command) :

  1. find n/thirdName

Screenshot:

Screenshot 2023-04-14 at 3.04.11 PM.png

  1. find n/thirdName n/firstName n/thirdName Screenshot 2023-04-14 at 3.05.05 PM.png
nus-se-bot commented 1 year ago

Team's Response

This is the intended as clearly stated in the UG.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Teams response: "This is the intended as clearly stated in the UG"

What is stated in the UG:

Screenshot 2023-04-18 at 12.35.24 PM.png

My response: From the screenshot from their UG, it means that "find n/thirdName" should return the same output as find n/thirdName n/firstName n/thirdName

But from the screenshot in the issue, clearly they do not return the same output.

Hence, I disagree and this should be a bug.