nus-cs2103-AY2122S2 / pe-dev-response

0 stars 0 forks source link

Application allows adding empty Medical information field #3189

Open nus-pe-bot opened 2 years ago

nus-pe-bot commented 2 years ago

All of the fields in Add medical information are optional, as a result, it is possible to "add" empty fields. The application does not detect this.

image.png


[original: nus-cs2103-AY2122S2/pe-interim#3142] [original labels: severity.Low type.FeatureFlaw]

ghost commented 2 years ago

Team's Response

The application does detect omitted parameters and automatically adds an empty placeholder NIL for omitted parameters.

This behavior was not explicitly included in the UG in line with UG Deliverables instructions "It is not necessary for the UG to contain every nitty-gritty detail about the product behavior." It is quite obvious that an empty placeholder NIL is used when optional parameters are omitted.

Screenshot 2022-04-18 at 1.33.07 PM.png

Duplicate status (if any):

--