search

nus-cs2113-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Hydration List bug #935

Open nus-pe-bot opened 5 months ago

nus-pe-bot commented 5 months ago

image.png If user directly uses input from CLI, the hydration in function is able to restrict user's input keep to a max of 10000, however if I access to modify the data from the txt file, and through command hydration list the "corrupted" data of 100001 volume is able to show in the program.

In contrast, you guys did a filter for calories function, the calories list is able to filter out corrupted data.

[original: nus-cs2113-AY2324S2/pe-interim#859] [original labels: type.FunctionalityBug severity.Medium]

rexyyong commented 5 months ago

Team's Response

Hi there, thank you for the bug report!

Our team believes that this is a case of deliberate sabotage because consuming more than 10,000ml of liquids at once is unrealistic. However, this does not cause harm to our program even if this input was entered as a user mistake. As such we rejected this as it is a deliberate sabotage.

image.png

Thank you!

Duplicate status (if any):

--