search

nus-cs2113-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Print message seems to be different from quantity removed for removing flowers #397

Open nus-pe-bot opened 7 months ago

nus-pe-bot commented 7 months ago

image.png

I tried removing 70 red roses from a bouquet with only 20 red roses. This caused the program to remove only 20 roses however, it prints the message that I have removed 70 red roses.

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

samuelory commented 7 months ago

Team's Response

We felt that because of the line "tried to remove more than the quantity available, quantity set to", it makes it clear enough to the user that instead of going to a negative value, the quantity is set to 0. Thus, the user was able to "successfully" remove more than the quantity, albeit with a warning.

However we understand that the "You have successfully removed the following: ... " might sound misleading to the user and it is an easy fix we can implement in the future.

Duplicate status (if any):

--