search

ZHANGTIANYAO1 / pe

0 stars 0 forks source link

UML incorrect in Model component #8

Open ZHANGTIANYAO1 opened 6 months ago

ZHANGTIANYAO1 commented 6 months ago

UML for Phone is incorrect in Model component. Based on UML, we can have more than one Phone object but in the program it only allow one phone number for each person.

2024-04-19 171718.png

nus-se-script commented 6 months ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

DG wrong multiplicity for phone attribute

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.

Phone is an optional in Person, should be 0..1 multiplicity.

image.png

[original: nus-cs2103-AY2324S2/pe-interim#56] [original labels: type.DocumentationBug severity.VeryLow]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

Okay. (introduced by https://github.com/AY2324S2-CS2103T-T11-2/tp/commit/fdfa93a80abb82f430e3396dd3c25048c4322541 )

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]