search

theodorekwok / pe

0 stars 0 forks source link

Parser sequence diagram for adding calories wrong method call #25

Open theodorekwok opened 2 years ago

theodorekwok commented 2 years ago

ss9.PNG From the code, it is setParse that calls the SetCaloriesCommand constructor method not parseMaster()

nus-pe-bot commented 2 years ago

Team's Response

This is indeed a bug, however, this is a very low severity bug since SetCaloriesCommand is called from the :Parser class, and therefore doesn't affect the implementation of SetCaloriesCommand at all.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: Severity VeryLow are for purely cosmetic issues. This issue pointed out relates to the correctness of the UML diagram. This will interfere with the reader when the reader is reading this diagram along with the reference to the codebase. Therefore as it pertains to the correctness and accuracy of the seq diagram, its error has nothing to do with it being a cosmetic.