diff options
author | Steve Klebanoff <steve.klebanoff@gmail.com> | 2018-12-22 08:29:53 +0800 |
---|---|---|
committer | Steve Klebanoff <steve.klebanoff@gmail.com> | 2018-12-22 08:29:53 +0800 |
commit | 26977f6408442883c20001a57a3b9001f4ab9b25 (patch) | |
tree | 7890fbbee04b5f02dc4ef258690870628a4e0b73 /packages/asset-buyer | |
parent | c8c8219c055cc5798cf5cdc71199ee7ae505cd5a (diff) | |
download | dexon-0x-contracts-26977f6408442883c20001a57a3b9001f4ab9b25.tar dexon-0x-contracts-26977f6408442883c20001a57a3b9001f4ab9b25.tar.gz dexon-0x-contracts-26977f6408442883c20001a57a3b9001f4ab9b25.tar.bz2 dexon-0x-contracts-26977f6408442883c20001a57a3b9001f4ab9b25.tar.lz dexon-0x-contracts-26977f6408442883c20001a57a3b9001f4ab9b25.tar.xz dexon-0x-contracts-26977f6408442883c20001a57a3b9001f4ab9b25.tar.zst dexon-0x-contracts-26977f6408442883c20001a57a3b9001f4ab9b25.zip |
Fix var name and use floor instead of .round(0, ROUND_DOWN)
Diffstat (limited to 'packages/asset-buyer')
-rw-r--r-- | packages/asset-buyer/src/utils/buy_quote_calculator.ts | 16 |
1 files changed, 7 insertions, 9 deletions
diff --git a/packages/asset-buyer/src/utils/buy_quote_calculator.ts b/packages/asset-buyer/src/utils/buy_quote_calculator.ts index ceeee93d3..fcded6ab1 100644 --- a/packages/asset-buyer/src/utils/buy_quote_calculator.ts +++ b/packages/asset-buyer/src/utils/buy_quote_calculator.ts @@ -36,16 +36,14 @@ export const buyQuoteCalculator = { if (remainingFillAmount.gt(constants.ZERO_AMOUNT)) { // We needed the amount they requested to buy, plus the amount for slippage const totalAmountRequested = assetBuyAmount.plus(slippageBufferAmount); - const amountUnableToFill = totalAmountRequested.minus(remainingFillAmount); - // multiplerNeededWithSlippage represents what we need to multiply the assetBuyAmount by + const amountAbleToFill = totalAmountRequested.minus(remainingFillAmount); + // multiplierNeededWithSlippage represents what we need to multiply the assetBuyAmount by // in order to get the total amount needed considering slippage - // i.e. if slippagePercent was 0.2 (20%), multiplerNeededWithSlippage would be 1.2 - const multiplerNeededWithSlippage = new BigNumber(1).plus(slippagePercentage); - // Given amountAvailableToFillConsideringSlippage * multiplerNeededWithSlippage = amountUnableToFill - // We divide amountUnableToFill by multiplerNeededWithSlippage to determine amountAvailableToFillConsideringSlippage - const amountAvailableToFillConsideringSlippage = amountUnableToFill - .div(multiplerNeededWithSlippage) - .round(0, BigNumber.ROUND_DOWN); + // i.e. if slippagePercent was 0.2 (20%), multiplierNeededWithSlippage would be 1.2 + const multiplierNeededWithSlippage = new BigNumber(1).plus(slippagePercentage); + // Given amountAvailableToFillConsideringSlippage * multiplierNeededWithSlippage = amountAbleToFill + // We divide amountUnableToFill by multiplierNeededWithSlippage to determine amountAvailableToFillConsideringSlippage + const amountAvailableToFillConsideringSlippage = amountAbleToFill.div(multiplierNeededWithSlippage).floor(); throw new InsufficientAssetLiquidityError(amountAvailableToFillConsideringSlippage); } |