search

tandeshao / pe

0 stars 0 forks source link

Incorrect sequence diagram for execute() method. #20

Open tandeshao opened 2 years ago

tandeshao commented 2 years ago

Steps to reproduce:

  1. Go to the sequence diagram shown in the image below (under Add temporary/recurring lesson section).

image.png

The execute method from LogicManager to AddLessonCommand is execute() as shown in the diagram but in the github code repo, execute takes in a string.

image.png

soc-pe-bot commented 2 years ago

Team's Response

The execute method from LogicManager to AddLessonCommand is calling the execute method in the AddLessonCommand class, and not the execute method in the LogicManager class (as reported). Thus, it is not relevant that the execute method in the LogicManager class takes in a String.

For this reason, we are rejecting this bug report!

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]