aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLeonid <logvinov.leon@gmail.com>2018-02-01 23:25:25 +0800
committerGitHub <noreply@github.com>2018-02-01 23:25:25 +0800
commit913930b561717e39ad02a689216d933d7c732945 (patch)
tree91b183826dcaff29e7bc25b00e3b5b4a9734879d
parent994c8db1f3f9049130aafefa727be7ad4d8e7e12 (diff)
parent6811bdec404fccb9dcf10e6802671317159b6533 (diff)
downloaddexon-sol-tools-913930b561717e39ad02a689216d933d7c732945.tar
dexon-sol-tools-913930b561717e39ad02a689216d933d7c732945.tar.gz
dexon-sol-tools-913930b561717e39ad02a689216d933d7c732945.tar.bz2
dexon-sol-tools-913930b561717e39ad02a689216d933d7c732945.tar.lz
dexon-sol-tools-913930b561717e39ad02a689216d933d7c732945.tar.xz
dexon-sol-tools-913930b561717e39ad02a689216d933d7c732945.tar.zst
dexon-sol-tools-913930b561717e39ad02a689216d933d7c732945.zip
Merge pull request #357 from ERCdEX/luke/zrx_order_watch_cache
BUGFIX: don't remove maker/zrx order from cache twice
-rw-r--r--packages/0x.js/src/order_watcher/order_state_watcher.ts6
1 files changed, 5 insertions, 1 deletions
diff --git a/packages/0x.js/src/order_watcher/order_state_watcher.ts b/packages/0x.js/src/order_watcher/order_state_watcher.ts
index 576be00c8..c425c98ee 100644
--- a/packages/0x.js/src/order_watcher/order_state_watcher.ts
+++ b/packages/0x.js/src/order_watcher/order_state_watcher.ts
@@ -133,8 +133,12 @@ export class OrderStateWatcher {
delete this._orderStateByOrderHashCache[orderHash];
const exchange = (this._orderFilledCancelledLazyStore as any)._exchange as ExchangeWrapper;
const zrxTokenAddress = exchange.getZRXTokenAddress();
+
this._removeFromDependentOrderHashes(signedOrder.maker, zrxTokenAddress, orderHash);
- this._removeFromDependentOrderHashes(signedOrder.maker, signedOrder.makerTokenAddress, orderHash);
+ if (zrxTokenAddress !== signedOrder.makerTokenAddress) {
+ this.removeFromDependentOrderHashes(signedOrder.maker, signedOrder.makerTokenAddress, orderHash);
+ }
+
this._expirationWatcher.removeOrder(orderHash);
}
/**