oceanprotocol/ocean.js
1
0
Fork 0
mirror of https://github.com/oceanprotocol/ocean.js.git synced 2024-11-26 20:39:05 +01:00
Code Releases Activity

Fix/pool_approve (#1207)

Browse Source 
* fix

* fixes

* fix estimate

* fix approve

* convert to wei

* final fix
This commit is contained in:
mihaisc 2022-01-11 05:29:19 -08:00 committed by GitHub
parent aa1c1fcfbf
commit 33ca7f43c1
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 9 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
src/pools/balancer/Pool.ts
View File

@ -58,11 +58,11 @@ export class Pool {
let estGas
try {
estGas = await tokenContract.methods
.approve(spender, new BigNumber(amount))
.approve(spender, amount)
.estimateGas({ from: account }, (err, estGas) => (err ? gasLimitDefault : estGas))
} catch (e) {
estGas = gasLimitDefault
LoggerInstance.error('estimage gas failed for approve!', e)
LoggerInstance.error('estimate gas failed for approve!', e)
}
return estGas
}
@ -128,7 +128,7 @@ export class Pool {
const token = new this.web3.eth.Contract(minABI, tokenAddress)
if (!force) {
const currentAllowence = await this.allowance(tokenAddress, account, spender)
if (new Decimal(currentAllowence).greaterThanOrEqualTo(amount)) {
if (new Decimal(currentAllowence).greaterThanOrEqualTo(new Decimal(amount))) {
return currentAllowence
}
}
@ -137,13 +137,11 @@ export class Pool {
const estGas = await this.estApprove(account, tokenAddress, spender, amountFormatted)
try {
result = await token.methods
.approve(spender, new BigNumber(await this.amountToUnits(tokenAddress, amount)))
.send({
from: account,
gas: estGas + 1,
gasPrice: await getFairGasPrice(this.web3)
})
result = await token.methods.approve(spender, amountFormatted).send({
from: account,
gas: estGas + 1,
gasPrice: await getFairGasPrice(this.web3)
})
} catch (e) {
this.logger.error(`ERRPR: Failed to approve spender to spend tokens : ${e.message}`)
}
@ -747,6 +745,7 @@ export class Pool {
decimals = 18
}
const amountFormatted = new BigNumber(parseInt(amount) * 10 ** decimals)
BigNumber.config({ EXPONENTIAL_AT: 50 })
return amountFormatted.toString()
} catch (e) {
this.logger.error('ERROR: FAILED TO CALL DECIMALS(), USING 18')
@ -1056,7 +1055,6 @@ export class Pool {
const amount = await this.amountToUnits(tokens[i], maxAmountsIn[i])
weiMaxAmountsIn.push(amount)
}
// console.log(weiMaxAmountsIn)
let result = null
@ -1524,14 +1522,11 @@ export class Pool {
if (decimalsTokenIn > decimalsTokenOut) {
decimalsDiff = decimalsTokenIn - decimalsTokenOut
price = new BigNumber(price / 10 ** decimalsDiff)
// console.log(price.toString())
price = price / 10 ** decimalsTokenOut
// console.log('dtIn')
} else {
decimalsDiff = decimalsTokenOut - decimalsTokenIn
price = new BigNumber(price * 10 ** (2 * decimalsDiff))
price = price / 10 ** decimalsTokenOut
// console.log('usdcIn')
}
return price.toString()