eugenetyc / pe

0 stars 0 forks source link

Incorrect index given for findc not corresponding for listc. #7

Open eugenetyc opened 4 years ago

eugenetyc commented 4 years ago

Performing findc n/jackson gives this:

image.png

But if I want to edit a particular entry, the index shown is not the same as if I were to view jackson in the original listc for customers.

nus-pe-bot commented 4 years ago

Team's Response

The product is working as expected.

If you use findc, then the list of customers shown will be different. You're supposed to use the index of the customer in that list (which will be different from the original list of customers). E.g. findc n/jackson -> editc INDEX where INDEX is the index shown on screen (after findc), NOT on the original list (after listc)

This is further emphasised in the UG, under editc: Edits the customer at the specified INDEX. The index refers to the index number shown in the displayed customer list. The index must be a positive integer 1, 2, 3, …

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]


:question: Issue severity

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

Reason for disagreement: [replace this with your reason]