From d88af4dfa6b15d8e1d6d3fc4ec75b67f6ee0e652 Mon Sep 17 00:00:00 2001 From: Greg Hysen Date: Thu, 14 Mar 2019 21:30:56 -0700 Subject: [PATCH] Bumped version of ERC1155Proxy to 0.5.5 + merged ERC20Proxy/ERC721Proxy/MultiAssetProxy dependencies into base contract files --- contracts/asset-proxy/CHANGELOG.json | 4 ++++ contracts/asset-proxy/contracts/src/ERC1155Proxy.sol | 2 +- contracts/asset-proxy/contracts/src/ERC20Proxy.sol | 2 +- contracts/asset-proxy/contracts/src/ERC721Proxy.sol | 2 +- .../contracts/src/MixinAssetProxyDispatcher.sol | 8 ++++---- contracts/asset-proxy/contracts/src/MixinAuthorizable.sol | 2 +- contracts/asset-proxy/contracts/src/MultiAssetProxy.sol | 2 +- .../asset-proxy/contracts/src/interfaces/IAssetData.sol | 6 +++--- .../asset-proxy/contracts/src/interfaces/IAssetProxy.sol | 4 ++-- .../contracts/src/interfaces/IAssetProxyDispatcher.sol | 2 +- .../contracts/src/interfaces/IAuthorizable.sol | 2 +- .../contracts/src/mixins/MAssetProxyDispatcher.sol | 2 +- .../asset-proxy/contracts/src/mixins/MAuthorizable.sol | 2 +- contracts/asset-proxy/package.json | 2 +- 14 files changed, 23 insertions(+), 19 deletions(-) diff --git a/contracts/asset-proxy/CHANGELOG.json b/contracts/asset-proxy/CHANGELOG.json index 40f67bad8e..d4a8083933 100644 --- a/contracts/asset-proxy/CHANGELOG.json +++ b/contracts/asset-proxy/CHANGELOG.json @@ -13,6 +13,10 @@ { "note": "Add ERC1155Proxy", "pr": 1661 + }, + { + "note": "Bumped solidity version to ^0.5.5", + "pr": 1701 } ] }, diff --git a/contracts/asset-proxy/contracts/src/ERC1155Proxy.sol b/contracts/asset-proxy/contracts/src/ERC1155Proxy.sol index b11f0d8d7d..1249616714 100644 --- a/contracts/asset-proxy/contracts/src/ERC1155Proxy.sol +++ b/contracts/asset-proxy/contracts/src/ERC1155Proxy.sol @@ -16,7 +16,7 @@ */ -pragma solidity 0.4.24; +pragma solidity ^0.5.5; import "./MixinAuthorizable.sol"; diff --git a/contracts/asset-proxy/contracts/src/ERC20Proxy.sol b/contracts/asset-proxy/contracts/src/ERC20Proxy.sol index 258443bca9..446d879c71 100644 --- a/contracts/asset-proxy/contracts/src/ERC20Proxy.sol +++ b/contracts/asset-proxy/contracts/src/ERC20Proxy.sol @@ -16,7 +16,7 @@ */ -pragma solidity 0.4.24; +pragma solidity ^0.5.5; import "./MixinAuthorizable.sol"; diff --git a/contracts/asset-proxy/contracts/src/ERC721Proxy.sol b/contracts/asset-proxy/contracts/src/ERC721Proxy.sol index 65b664b8b3..7730eda532 100644 --- a/contracts/asset-proxy/contracts/src/ERC721Proxy.sol +++ b/contracts/asset-proxy/contracts/src/ERC721Proxy.sol @@ -16,7 +16,7 @@ */ -pragma solidity 0.4.24; +pragma solidity ^0.5.5; import "./MixinAuthorizable.sol"; diff --git a/contracts/asset-proxy/contracts/src/MixinAssetProxyDispatcher.sol b/contracts/asset-proxy/contracts/src/MixinAssetProxyDispatcher.sol index 36c287ea34..8ac521e384 100644 --- a/contracts/asset-proxy/contracts/src/MixinAssetProxyDispatcher.sol +++ b/contracts/asset-proxy/contracts/src/MixinAssetProxyDispatcher.sol @@ -16,7 +16,7 @@ */ -pragma solidity ^0.4.24; +pragma solidity ^0.5.5; import "@0x/contracts-utils/contracts/src/Ownable.sol"; import "./mixins/MAssetProxyDispatcher.sol"; @@ -41,7 +41,7 @@ contract MixinAssetProxyDispatcher is // Ensure that no asset proxy exists with current id. bytes4 assetProxyId = assetProxyContract.getProxyId(); - address currentAssetProxy = assetProxies[assetProxyId]; + address currentAssetProxy = address(assetProxies[assetProxyId]); require( currentAssetProxy == address(0), "ASSET_PROXY_ALREADY_EXISTS" @@ -63,7 +63,7 @@ contract MixinAssetProxyDispatcher is view returns (address) { - return assetProxies[assetProxyId]; + return address(assetProxies[assetProxyId]); } /// @dev Forwards arguments to assetProxy and calls `transferFrom`. Either succeeds or throws. @@ -95,7 +95,7 @@ contract MixinAssetProxyDispatcher is 0xFFFFFFFF00000000000000000000000000000000000000000000000000000000 ) } - address assetProxy = assetProxies[assetProxyId]; + address assetProxy = address(assetProxies[assetProxyId]); // Ensure that assetProxy exists require( diff --git a/contracts/asset-proxy/contracts/src/MixinAuthorizable.sol b/contracts/asset-proxy/contracts/src/MixinAuthorizable.sol index ace820625c..a7fc9d155d 100644 --- a/contracts/asset-proxy/contracts/src/MixinAuthorizable.sol +++ b/contracts/asset-proxy/contracts/src/MixinAuthorizable.sol @@ -16,7 +16,7 @@ */ -pragma solidity ^0.4.24; +pragma solidity ^0.5.5; import "@0x/contracts-utils/contracts/src/Ownable.sol"; import "./mixins/MAuthorizable.sol"; diff --git a/contracts/asset-proxy/contracts/src/MultiAssetProxy.sol b/contracts/asset-proxy/contracts/src/MultiAssetProxy.sol index 0b2cb41348..b26d8c0138 100644 --- a/contracts/asset-proxy/contracts/src/MultiAssetProxy.sol +++ b/contracts/asset-proxy/contracts/src/MultiAssetProxy.sol @@ -16,7 +16,7 @@ */ -pragma solidity 0.4.24; +pragma solidity ^0.5.5; import "./MixinAssetProxyDispatcher.sol"; import "./MixinAuthorizable.sol"; diff --git a/contracts/asset-proxy/contracts/src/interfaces/IAssetData.sol b/contracts/asset-proxy/contracts/src/interfaces/IAssetData.sol index a130e615fc..2f5e49bd93 100644 --- a/contracts/asset-proxy/contracts/src/interfaces/IAssetData.sol +++ b/contracts/asset-proxy/contracts/src/interfaces/IAssetData.sol @@ -17,7 +17,7 @@ */ // solhint-disable -pragma solidity ^0.4.24; +pragma solidity ^0.5.5; pragma experimental ABIEncoderV2; @@ -36,8 +36,8 @@ interface IAssetData { external; function MultiAsset( - uint256[] amounts, - bytes[] nestedAssetData + uint256[] calldata amounts, + bytes[] calldata nestedAssetData ) external; diff --git a/contracts/asset-proxy/contracts/src/interfaces/IAssetProxy.sol b/contracts/asset-proxy/contracts/src/interfaces/IAssetProxy.sol index 706412dd0b..d176b57ade 100644 --- a/contracts/asset-proxy/contracts/src/interfaces/IAssetProxy.sol +++ b/contracts/asset-proxy/contracts/src/interfaces/IAssetProxy.sol @@ -16,7 +16,7 @@ */ -pragma solidity ^0.4.24; +pragma solidity ^0.5.5; import "./IAuthorizable.sol"; @@ -30,7 +30,7 @@ contract IAssetProxy is /// @param to Address to transfer asset to. /// @param amount Amount of asset to transfer. function transferFrom( - bytes assetData, + bytes calldata assetData, address from, address to, uint256 amount diff --git a/contracts/asset-proxy/contracts/src/interfaces/IAssetProxyDispatcher.sol b/contracts/asset-proxy/contracts/src/interfaces/IAssetProxyDispatcher.sol index b73881c077..6ae3c1cbe2 100644 --- a/contracts/asset-proxy/contracts/src/interfaces/IAssetProxyDispatcher.sol +++ b/contracts/asset-proxy/contracts/src/interfaces/IAssetProxyDispatcher.sol @@ -16,7 +16,7 @@ */ -pragma solidity ^0.4.24; +pragma solidity ^0.5.5; contract IAssetProxyDispatcher { diff --git a/contracts/asset-proxy/contracts/src/interfaces/IAuthorizable.sol b/contracts/asset-proxy/contracts/src/interfaces/IAuthorizable.sol index 0df6547117..2fcf3e5110 100644 --- a/contracts/asset-proxy/contracts/src/interfaces/IAuthorizable.sol +++ b/contracts/asset-proxy/contracts/src/interfaces/IAuthorizable.sol @@ -16,7 +16,7 @@ */ -pragma solidity ^0.4.24; +pragma solidity ^0.5.5; import "@0x/contracts-utils/contracts/src/interfaces/IOwnable.sol"; diff --git a/contracts/asset-proxy/contracts/src/mixins/MAssetProxyDispatcher.sol b/contracts/asset-proxy/contracts/src/mixins/MAssetProxyDispatcher.sol index 0ae555ddad..b53e7166e6 100644 --- a/contracts/asset-proxy/contracts/src/mixins/MAssetProxyDispatcher.sol +++ b/contracts/asset-proxy/contracts/src/mixins/MAssetProxyDispatcher.sol @@ -16,7 +16,7 @@ */ -pragma solidity ^0.4.24; +pragma solidity ^0.5.5; import "../interfaces/IAssetProxyDispatcher.sol"; diff --git a/contracts/asset-proxy/contracts/src/mixins/MAuthorizable.sol b/contracts/asset-proxy/contracts/src/mixins/MAuthorizable.sol index c9c07e788b..6b3d0b7d9f 100644 --- a/contracts/asset-proxy/contracts/src/mixins/MAuthorizable.sol +++ b/contracts/asset-proxy/contracts/src/mixins/MAuthorizable.sol @@ -16,7 +16,7 @@ */ -pragma solidity ^0.4.24; +pragma solidity ^0.5.5; import "../interfaces/IAuthorizable.sol"; diff --git a/contracts/asset-proxy/package.json b/contracts/asset-proxy/package.json index 138485472a..36bda3404c 100644 --- a/contracts/asset-proxy/package.json +++ b/contracts/asset-proxy/package.json @@ -71,7 +71,7 @@ "@0x/contracts-erc20": "^1.0.9", "@0x/contracts-erc721": "^1.0.9", "@0x/contracts-erc1155": "^1.0.0", - "@0x/contracts-utils": "2.0.1", + "@0x/contracts-utils": "^2.0.8", "@0x/order-utils": "^7.0.2", "@0x/types": "^2.1.1", "@0x/typescript-typings": "^4.1.0",