aboutsummaryrefslogtreecommitdiffstats
path: root/src/contract_wrappers/token_wrapper.ts
diff options
context:
space:
mode:
authorLeonid <logvinov.leon@gmail.com>2017-05-31 17:18:42 +0800
committerGitHub <noreply@github.com>2017-05-31 17:18:42 +0800
commit1958b28b96e64af74f989ecaf537356308334aad (patch)
tree9e3be161b0a741048c9d66ff1ae6de34ca065dbe /src/contract_wrappers/token_wrapper.ts
parent9409e0aba1d1fa62a60c25149ae1b7115de9e2c9 (diff)
parent9364c5647ff5518aaa2b3fcd133a9e6c47ec0eda (diff)
downloaddexon-sol-tools-1958b28b96e64af74f989ecaf537356308334aad.tar
dexon-sol-tools-1958b28b96e64af74f989ecaf537356308334aad.tar.gz
dexon-sol-tools-1958b28b96e64af74f989ecaf537356308334aad.tar.bz2
dexon-sol-tools-1958b28b96e64af74f989ecaf537356308334aad.tar.lz
dexon-sol-tools-1958b28b96e64af74f989ecaf537356308334aad.tar.xz
dexon-sol-tools-1958b28b96e64af74f989ecaf537356308334aad.tar.zst
dexon-sol-tools-1958b28b96e64af74f989ecaf537356308334aad.zip
Merge branch 'master' into erc20transfer
Diffstat (limited to 'src/contract_wrappers/token_wrapper.ts')
-rw-r--r--src/contract_wrappers/token_wrapper.ts14
1 files changed, 6 insertions, 8 deletions
diff --git a/src/contract_wrappers/token_wrapper.ts b/src/contract_wrappers/token_wrapper.ts
index e93b6b7c2..cedbfbdae 100644
--- a/src/contract_wrappers/token_wrapper.ts
+++ b/src/contract_wrappers/token_wrapper.ts
@@ -6,7 +6,7 @@ import {constants} from '../utils/constants';
import {ContractWrapper} from './contract_wrapper';
import * as TokenArtifacts from '../artifacts/Token.json';
import * as ProxyArtifacts from '../artifacts/Proxy.json';
-import {TokenContract, InternalError} from '../types';
+import {TokenContract, ZeroExError} from '../types';
const ALLOWANCE_TO_ZERO_GAS_AMOUNT = 45730;
@@ -20,7 +20,7 @@ export class TokenWrapper extends ContractWrapper {
this.tokenContractsByAddress = {};
}
/**
- * Returns an owner's ERC20 token balance
+ * Returns an owner's ERC20 token balance.
*/
public async getBalanceAsync(tokenAddress: string, ownerAddress: string): Promise<BigNumber.BigNumber> {
assert.isETHAddressHex('ownerAddress', ownerAddress);
@@ -30,12 +30,12 @@ export class TokenWrapper extends ContractWrapper {
let balance = await tokenContract.balanceOf.call(ownerAddress);
// The BigNumber instance returned by Web3 is of a much older version then our own, we therefore
// should always re-instantiate the returned BigNumber after retrieval.
- balance = _.isUndefined(balance) ? new BigNumber(0) : new BigNumber(balance);
+ balance = new BigNumber(balance);
return balance;
}
/**
* Retrieves the allowance in baseUnits of the ERC20 token set to the 0x proxy contract
- * by an owner address
+ * by an owner address.
*/
public async getProxyAllowanceAsync(tokenAddress: string, ownerAddress: string) {
assert.isETHAddressHex('ownerAddress', ownerAddress);
@@ -44,9 +44,7 @@ export class TokenWrapper extends ContractWrapper {
const tokenContract = await this.getTokenContractAsync(tokenAddress);
const proxyAddress = await this.getProxyAddressAsync();
let allowanceInBaseUnits = await tokenContract.allowance.call(ownerAddress, proxyAddress);
- allowanceInBaseUnits = _.isUndefined(allowanceInBaseUnits) ?
- new BigNumber(0) :
- new BigNumber(allowanceInBaseUnits);
+ allowanceInBaseUnits = new BigNumber(allowanceInBaseUnits);
return allowanceInBaseUnits;
}
/**
@@ -102,7 +100,7 @@ export class TokenWrapper extends ContractWrapper {
undefined :
(ProxyArtifacts as any).networks[networkIdIfExists];
if (_.isUndefined(proxyNetworkConfigsIfExists)) {
- throw new Error(InternalError.PROXY_ADDRESS_NOT_FOUND);
+ throw new Error(ZeroExError.CONTRACT_NOT_DEPLOYED_ON_NETWORK);
}
const proxyAddress = proxyNetworkConfigsIfExists.address;
return proxyAddress;