lamassu/lamassu-server
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge pull request #1044 from chaotixkilla/fix-undefined-denomination-same-cassettes

Browse source 
Initialize possible empty cassette values
This commit is contained in:
Rafael Taranto 2022-01-14 17:55:04 +00:00 • committed by GitHub
commit af8eaad61a
1 changed files with 3 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

5
lib/bill-math.js
View file

@ -73,8 +73,9 @@ function unmergeCassettes(cassettes, output) {
}
function makeChangeDuo(cassettes, amount) {
const small = cassettes[0]
const large = cassettes[1]
// Initialize empty cassettes in case of undefined, due to same denomination across all cassettes results in a single merged cassette
const small = cassettes[0] ?? { denomination: 0, count: 0 }
const large = cassettes[1] ?? { denomination: 0, count: 0 }
const largeDenom = large.denomination
const smallDenom = small.denomination