search

Denniszedead / pe

0 stars 0 forks source link

Multipliers for each client model. #16

Open Denniszedead opened 2 years ago

Denniszedead commented 2 years ago

Screenshot 2022-04-16 at 3.30.47 PM.png

If the commands forces the user to enter only one name, phone, address, email and plan, the multiplier should be 1 for these composites.

nus-pe-script commented 2 years ago

Team's Response

Multiplicity is optional in a class diagram.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I agree that multipliers is optional for class diagrams.

However, I have to consider it as a bug for the following reasons:

  1. If the other objects have multipliers, how come the objects I highlighted do not have any multipliers.
  2. By observing the code, as well as through developer testing on your code, the objects that are highlighted above are compulsory, as such, there should be a 1 multiplier not to confuse future developers looking at them.

Base on what you said and the reasons stated above, as well as the reasons stated, i think this should be a serverity.veryLow bug instead as this is mainly cosmetic.