search

nus-cs2113-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Bug in restock-all command #435

Open nus-pe-bot opened 1 week ago

nus-pe-bot commented 1 week ago

Bug Description For items without cost yet, restock-all command will restock it while ignoring the cost price instead of asking for these items to have their cost added

Actual output:

image.png

Expected output: Please add cost to items missing cost

[original: nus-cs2113-AY2425S1/pe-interim#695] [original labels: type.FeatureFlaw severity.Low]

cxc0418 commented 1 week ago

Team's Response

Thank you for highlighting the issue. In v2.1 of the PILL system, all items have their default prices/costs as shown in the picture (also stated in UG), which is $0.00. If the user does not specify the cost/price of an item, they are taken as $0.00.

In a real-world scenario, it is possible for items to have no cost (e.g. donations) which would be considered a zero-cost restock.

Thus, the application restocks all items, including those with $0.00 cost, as intended. Therefore, there should not be a prompt to add cost to an item.

Duplicate status (if any):

--