Missing dependency in class diagram of Parser classes

[kwokieee]

Bug inherited from AB3:

Under the class diagram for the Logic component's Parser classes, ArgumentMultiMap has a missing dependency on Prefix

[nus-pe-bot]

Team's Response

Even though this is true, the dependency is not very important, and it is a very small detail that is left out. The overall purpose of the diagram is still achieve and does not inconvenience the reader.

Items for the Tester to Verify

❓ Issue severity

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

• I disagree

Reason for disagreement: I don't think the dependency is not very important. The ArgumentMultiMap class heavily depends on the Prefix class in its underlying implementation, as Prefix is used in all its methods and as the key to the underlying HashMap data structure. It might misinform and hence cause a little bit of confusion to the reader if they were trying to understand or utilize the class.

---