From 06396b8874f2cff1333d3f19c536015502f69d28 Mon Sep 17 00:00:00 2001 From: Amir Bandeali Date: Mon, 23 Jul 2018 09:34:08 -0500 Subject: Remove ERC721 callback functions --- .../contracts/src/2.0.0/forwarder/MixinAssets.sol | 27 ---------------------- .../src/2.0.0/forwarder/MixinExchangeWrapper.sol | 2 +- .../src/2.0.0/forwarder/interfaces/IAssets.sol | 21 +---------------- .../2.0.0/forwarder/mixins/MExchangeWrapper.sol | 2 +- 4 files changed, 3 insertions(+), 49 deletions(-) (limited to 'packages') diff --git a/packages/contracts/src/2.0.0/forwarder/MixinAssets.sol b/packages/contracts/src/2.0.0/forwarder/MixinAssets.sol index 084a28550..5cf5f831b 100644 --- a/packages/contracts/src/2.0.0/forwarder/MixinAssets.sol +++ b/packages/contracts/src/2.0.0/forwarder/MixinAssets.sol @@ -35,8 +35,6 @@ contract MixinAssets is using LibBytes for bytes; bytes4 constant internal ERC20_TRANSFER_SELECTOR = bytes4(keccak256("transfer(address,uint256)")); - bytes4 constant internal ERC721_RECEIVED = bytes4(keccak256("onERC721Received(address,uint256,bytes)")); - bytes4 constant internal ERC721_RECEIVED_OPERATOR = bytes4(keccak256("onERC721Received(address,address,uint256,bytes)")); /// @dev Withdraws ERC20 tokens from this contract. The contract requires a ZRX balance in order to /// function optimally, and this function allows the ZRX to be withdrawn by owner. It may also be @@ -56,31 +54,6 @@ contract MixinAssets is ); } - function onERC721Received( - address, - uint256, - bytes memory - ) - public - pure - returns(bytes4) - { - return ERC721_RECEIVED; - } - - function onERC721Received( - address, - address, - uint256, - bytes memory - ) - public - pure - returns(bytes4) - { - return ERC721_RECEIVED_OPERATOR; - } - /// @dev Transfers given amount of asset to sender. /// @param assetData Byte array encoded for the respective asset proxy. /// @param amount Amount of asset to transfer to sender. diff --git a/packages/contracts/src/2.0.0/forwarder/MixinExchangeWrapper.sol b/packages/contracts/src/2.0.0/forwarder/MixinExchangeWrapper.sol index e150791da..f3aa483c5 100644 --- a/packages/contracts/src/2.0.0/forwarder/MixinExchangeWrapper.sol +++ b/packages/contracts/src/2.0.0/forwarder/MixinExchangeWrapper.sol @@ -250,4 +250,4 @@ contract MixinExchangeWrapper is return totalFillResults; } -} \ No newline at end of file +} diff --git a/packages/contracts/src/2.0.0/forwarder/interfaces/IAssets.sol b/packages/contracts/src/2.0.0/forwarder/interfaces/IAssets.sol index 27adb1221..9b0d995eb 100644 --- a/packages/contracts/src/2.0.0/forwarder/interfaces/IAssets.sol +++ b/packages/contracts/src/2.0.0/forwarder/interfaces/IAssets.sol @@ -31,23 +31,4 @@ contract IAssets { uint256 amount ) external; - - function onERC721Received( - address, - uint256, - bytes memory - ) - public - pure - returns(bytes4); - - function onERC721Received( - address, - address, - uint256, - bytes memory - ) - public - pure - returns(bytes4); -} \ No newline at end of file +} diff --git a/packages/contracts/src/2.0.0/forwarder/mixins/MExchangeWrapper.sol b/packages/contracts/src/2.0.0/forwarder/mixins/MExchangeWrapper.sol index 0fa90fd82..5a2def7e5 100644 --- a/packages/contracts/src/2.0.0/forwarder/mixins/MExchangeWrapper.sol +++ b/packages/contracts/src/2.0.0/forwarder/mixins/MExchangeWrapper.sol @@ -84,4 +84,4 @@ contract MExchangeWrapper { ) internal returns (LibFillResults.FillResults memory totalFillResults); -} \ No newline at end of file +} -- cgit v1.2.3