rubyforgood / pdx_diaper

Inventory system for Portland Diaper Bank, created at Ruby For Good 2016
MIT License
6 stars 3 forks source link

Fixes and edge case with the transfer/ticket functionality #78

Closed seanmarcia closed 8 years ago

seanmarcia commented 8 years ago

There was an error with an edge case in both tickets and transfers if a user attempted to transfer the the same item twice in the same ticket or transfer it would only record the last container value.

So attempting to transfer two batches of diapers(newborn) with quantities 100 and 200 would result in only 200 being transfered/ticketed.

This was an error in the original ticket logic that I inadvertently brought into the transfer logic.