WANG-Yuchen-Alice / CodeReview2

0 stars 0 forks source link

Bug detected in Edit person. #11

Open WANG-Yuchen-Alice opened 4 years ago

WANG-Yuchen-Alice commented 4 years ago

When the edit is executed after a find command, the index is wrongly matched.

pre-condition: the 3rd person is called null in the person list

command : find null - edit 3 n/non

image.png

nus-pe-bot commented 4 years ago

Team's Response

The command is working as expected. To edit the person with the name null, the index 1 should have been entered instead; index 3 is invalid as there is only 1 person on the currently displayed person list. It is specified in the relevant user guide section that the index entered should be based on the currently displayed list, and not the overall person list.

Screenshot 2020-04-18 at 6.30.15 AM.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]