aboutsummaryrefslogtreecommitdiffstats
path: root/packages/contracts
diff options
context:
space:
mode:
authorGreg Hysen <greg.hysen@gmail.com>2018-07-16 23:11:14 +0800
committerGreg Hysen <greg.hysen@gmail.com>2018-07-18 21:52:14 +0800
commite1a9f3435f2527fcf38e68c5749bfddf1ba3b37d (patch)
treeb75b219da2f95246ecc8719d226b2ad90883f579 /packages/contracts
parent9f74feb3475b14ebdf2a0b312856fc927f91b6ab (diff)
downloaddexon-0x-contracts-e1a9f3435f2527fcf38e68c5749bfddf1ba3b37d.tar
dexon-0x-contracts-e1a9f3435f2527fcf38e68c5749bfddf1ba3b37d.tar.gz
dexon-0x-contracts-e1a9f3435f2527fcf38e68c5749bfddf1ba3b37d.tar.bz2
dexon-0x-contracts-e1a9f3435f2527fcf38e68c5749bfddf1ba3b37d.tar.lz
dexon-0x-contracts-e1a9f3435f2527fcf38e68c5749bfddf1ba3b37d.tar.xz
dexon-0x-contracts-e1a9f3435f2527fcf38e68c5749bfddf1ba3b37d.tar.zst
dexon-0x-contracts-e1a9f3435f2527fcf38e68c5749bfddf1ba3b37d.zip
Removed receiverData from ERC721 Proxy Id generation
Diffstat (limited to 'packages/contracts')
-rw-r--r--packages/contracts/src/2.0.0/forwarder/MixinForwarderCore.sol2
-rw-r--r--packages/contracts/src/2.0.0/forwarder/MixinTransfer.sol6
-rw-r--r--packages/contracts/src/2.0.0/protocol/AssetProxy/ERC721Proxy.sol2
-rw-r--r--packages/contracts/test/asset_proxy/proxies.ts4
4 files changed, 6 insertions, 8 deletions
diff --git a/packages/contracts/src/2.0.0/forwarder/MixinForwarderCore.sol b/packages/contracts/src/2.0.0/forwarder/MixinForwarderCore.sol
index eadeaf5ba..cb0ed5422 100644
--- a/packages/contracts/src/2.0.0/forwarder/MixinForwarderCore.sol
+++ b/packages/contracts/src/2.0.0/forwarder/MixinForwarderCore.sol
@@ -40,7 +40,7 @@ contract MixinForwarderCore is
MForwarderCore
{
bytes4 constant internal ERC20_DATA_ID = bytes4(keccak256("ERC20Token(address)"));
- bytes4 constant internal ERC721_DATA_ID = bytes4(keccak256("ERC721Token(address,uint256,bytes)"));
+ bytes4 constant internal ERC721_DATA_ID = bytes4(keccak256("ERC721Token(address,uint256)"));
uint256 constant internal MAX_UINT = 2**256 - 1;
constructor ()
diff --git a/packages/contracts/src/2.0.0/forwarder/MixinTransfer.sol b/packages/contracts/src/2.0.0/forwarder/MixinTransfer.sol
index 6c49330f2..bebfc976b 100644
--- a/packages/contracts/src/2.0.0/forwarder/MixinTransfer.sol
+++ b/packages/contracts/src/2.0.0/forwarder/MixinTransfer.sol
@@ -109,12 +109,10 @@ contract MixinTransfer is
// Decode asset data.
address token = assetData.readAddress(16);
uint256 tokenId = assetData.readUint256(36);
- bytes memory receiverData = assetData.readBytesWithLength(100);
- IERC721Token(token).safeTransferFrom(
+ IERC721Token(token).transferFrom(
address(this),
to,
- tokenId,
- receiverData
+ tokenId
);
}
}
diff --git a/packages/contracts/src/2.0.0/protocol/AssetProxy/ERC721Proxy.sol b/packages/contracts/src/2.0.0/protocol/AssetProxy/ERC721Proxy.sol
index a56c41adb..53f5a14e0 100644
--- a/packages/contracts/src/2.0.0/protocol/AssetProxy/ERC721Proxy.sol
+++ b/packages/contracts/src/2.0.0/protocol/AssetProxy/ERC721Proxy.sol
@@ -26,7 +26,7 @@ contract ERC721Proxy is
MixinAuthorizable
{
// Id of this proxy.
- bytes4 constant internal PROXY_ID = bytes4(keccak256("ERC721Token(address,uint256,bytes)"));
+ bytes4 constant internal PROXY_ID = bytes4(keccak256("ERC721Token(address,uint256)"));
// solhint-disable-next-line payable-fallback
function ()
diff --git a/packages/contracts/test/asset_proxy/proxies.ts b/packages/contracts/test/asset_proxy/proxies.ts
index 281f3bf27..1a24ee0b6 100644
--- a/packages/contracts/test/asset_proxy/proxies.ts
+++ b/packages/contracts/test/asset_proxy/proxies.ts
@@ -377,9 +377,9 @@ describe('Asset Transfer Proxies', () => {
});
});
- it('should have an id of 0x08e937fa', async () => {
+ it('should have an id of 0x02571792', async () => {
const proxyId = await erc721Proxy.getProxyId.callAsync();
- const expectedProxyId = '0x08e937fa';
+ const expectedProxyId = '0x02571792';
expect(proxyId).to.equal(expectedProxyId);
});
});