Note from the teaching team: This bug was reported during the Part II (Evaluating Documents) stage of the PE. You may reject this bug if it is not related to the quality of documentation.
In the main UML diagram of the model class, Person class does not contain Role as an attribute. However, the UML diagram below shows that addressbook contains UniqueRoleList as an attribute with Person class having reference to any number of Roles. As a developer, I feel that this might be a bit confusing since the developer would not be sure which UML diagram is correct (i.e. developer would not be sure whether addressbook contains uniquerolelist, and whether person contains role due to the discrepancy bewteen two diagrams). THe main model UML could add these extra classes.
Note from the teaching team: This bug was reported during the Part II (Evaluating Documents) stage of the PE. You may reject this bug if it is not related to the quality of documentation.
In the main UML diagram of the model class, Person class does not contain Role as an attribute. However, the UML diagram below shows that addressbook contains UniqueRoleList as an attribute with Person class having reference to any number of Roles. As a developer, I feel that this might be a bit confusing since the developer would not be sure which UML diagram is correct (i.e. developer would not be sure whether addressbook contains uniquerolelist, and whether person contains role due to the discrepancy bewteen two diagrams). THe main model UML could add these extra classes.
[original: nus-cs2103-AY2324S2/pe-interim#1491] [original labels: severity.Low type.DocumentationBug]