Alex Browne
68f150eff2
Remove as any hack from artifacts in contracts package
2018-10-15 13:38:33 -07:00
Alex Browne
1e9ea09f08
Introduce new contract-addresses package and use it everywhere
2018-10-15 13:38:33 -07:00
Alex Browne
17c6661e2d
Cleanup migrations and contracts package. Make contracts private again.
2018-10-15 13:37:57 -07:00
Alex Browne
026ad1f9a1
Update contract-wrappers package to use new contracts package for generated files
2018-10-15 13:36:09 -07:00
Alex Browne
81c4887241
Move generated contract wrappers and artifacts into contracts package
2018-10-15 13:36:09 -07:00
Amir Bandeali
e34b1f2f8b
Remove TODOs that will not be addressed
2018-09-04 07:57:47 -07:00
Amir Bandeali
7a8ab6fbe8
Update MultiSigWalletWithTimeLock to 0.4.24
2018-09-04 07:57:47 -07:00
Amir Bandeali
ada5563b1f
Update to most recent multisig
2018-09-04 07:57:47 -07:00
Amir Bandeali
99fbf384fd
Apply styling fixes
2018-09-03 20:55:37 -07:00
Amir Bandeali
cb4fcf4de7
Revert in Forwarder constructor if ERC20 proxy isn't registered
2018-09-03 20:35:00 -07:00
Amir Bandeali
81ba2a8411
Merge pull request #1045 from 0xProject/feature/contracts/cancelOrderInternal
...
Add cancelOrderInternal, use within batchCancelOrders
2018-08-30 13:07:51 -07:00
Amir Bandeali
09b4d5e0e4
Add cancelOrderInternal, use within batchCancelOrders
2018-08-30 12:10:24 -07:00
Amir Bandeali
d0f6933980
Add tests for token that returns >32 bytes
2018-08-30 09:27:28 -07:00
Amir Bandeali
d80701c277
Fix geth tests
2018-08-29 15:35:09 -07:00
Amir Bandeali
bf3ab1127d
Remove unused imports
2018-08-29 14:42:42 -07:00
Amir Bandeali
dcb12b6ad6
Update AssetProxyOwner to 0.4.24
2018-08-29 14:12:10 -07:00
Greg Hysen
5f1c9cfee5
Reverted syntax used by readBytes4 in AssetProxyOwner to be compatible with Solidity v0.4.10
2018-08-29 13:15:40 -07:00
Greg Hysen
8c803ab232
Updated comments for ERC20 proxy to clarify how we load the token address from calldata (3.18 from audit)
2018-08-29 12:25:06 -07:00
Greg Hysen
1c3b2b7141
Updated readBytes4 to match spec + added unit tests. These are 3.5/3.6 from audit
2018-08-29 11:39:52 -07:00
Amir Bandeali
f225f9e7c8
Making rounding consistent in calculateFillResults
2018-08-28 13:25:05 -07:00
Amir Bandeali
14fdb71a71
safeGetPartialAmount ( #1035 )
...
* Added Test "Should transfer correct amounts when left order is fully filled and values pass isRoundingErrorCeil but fail isRoundingErrorFloor"
* Added RoundingError exception to reference function for getPartialAmount
* Added RoundingError exception to reference function for getPartialAmount
* Added isRoundingErrorCeil to getPartialAmountCeil reference funtion
* Computed new values for "Should give right maker a better buy price when correct price is not integral" that does not have a rounding error
* Almost all tests for match orders are passing after adding isRoundingErrorCeil check
* WIP commit: Added rounding error checks to getPartialAmount
* WIP commit: Added rounding error checks to getPartialAmount
* Use safe versions of getPartialAmount
* Update Exchange internals tests
* Run linter
* Found new values for "Should transfer correct amounts when right order fill amount deviates from amount derived by `Exchange.fillOrder`"
* Fixed merge conflicts
* Run all tests
* Cleaned up some comments on match Orders tests
* Fix tests for geth
2018-08-28 13:00:49 -07:00
Amir Bandeali
6a99bfa68e
Add clarifying comments
2018-08-27 12:01:58 -07:00
Amir Bandeali
f60adbdd72
Remove redundant mstores from fillOrderNoThrow
2018-08-27 11:54:20 -07:00
Amir Bandeali
00e7c70b4d
Merge pull request #986 from 0xProject/feature/contracts/assertions
...
Add more assertions to assertValidFill
2018-08-24 20:03:28 -07:00
Remco Bloemen
d652deea23
Merge branch 'fix/contracts/robustMatching' of github.com:0xProject/0x.js into fix/contracts/robustMatching
2018-08-24 18:54:15 -07:00
Greg Hysen
878db3b849
Added comments to order matching
2018-08-24 18:48:29 -07:00
Greg Hysen
ec2e726be0
Rephrased some of the math in MixinMatchOrders to improve readability
2018-08-24 18:20:26 -07:00
Greg Hysen
0a6f107243
Added temporary @todo to MixinMatchOrders
2018-08-24 18:20:25 -07:00
Greg Hysen
a93f95c55e
Wording in MixinMatchOrders
2018-08-24 18:20:25 -07:00
Greg Hysen
81dc893d1d
Removed a redundant comment from matchOrders
2018-08-24 18:20:25 -07:00
Greg Hysen
a32b201afe
Rounding for fees in match orders addressed, plus example
2018-08-24 18:17:27 -07:00
Greg Hysen
057891b342
Added fees to matchOrders (previously in calculateFillResults
2018-08-24 18:17:26 -07:00
Greg Hysen
407f63ef20
Removed calculateFillResults from matchOrders workflow. Eliminates compounded rounding errors.
2018-08-24 18:17:26 -07:00
Amir Bandeali
c28c3db63f
Only use one nonReentrant modifier, remove modifier from fillOrderNoThrow variations
2018-08-24 17:30:56 -07:00
Amir Bandeali
56c3c29feb
Update ReentrantERC20Token with new functions and check that revert is occuring for correct reason
2018-08-24 17:30:56 -07:00
Amir Bandeali
c75212bef0
Add nonReentrant modifiers on functions that use getCurrentContextAddress only, add lockMutex modifier on functions that make external calls
2018-08-24 17:30:56 -07:00
Amir Bandeali
6d0dedc62c
Split modifiers into check only and check, lock, unlock
2018-08-24 17:30:56 -07:00
Amir Bandeali
cf12daea2f
Add ReentrantToken
2018-08-24 17:30:56 -07:00
Amir Bandeali
6f88e9bdbd
Add internal fill functions, add reentrancy guard to public functions that make external calls
2018-08-24 17:30:56 -07:00
Amir Bandeali
d8cb56caa3
Add ReentrancyGuard contract
2018-08-24 17:30:56 -07:00
Amir Bandeali
044415e23d
Remove redundant sload from getCurrentContextAddress
2018-08-24 17:30:56 -07:00
Remco Bloemen
6b866d6053
Revert maker not equal taker check
2018-08-24 17:29:52 -07:00
Amir Bandeali
74ce893f52
Merge pull request #1003 from 0xProject/feature/contracts/roundup
...
[contracts] Add getPartialAmountCeil and isRoundingErrorCeil
2018-08-24 17:29:09 -07:00
Remco Bloemen
e215992859
Fix mixin api
2018-08-24 16:46:24 -07:00
Remco Bloemen
29971f36cf
Split into assertFillable and assertValidFill
2018-08-24 16:46:24 -07:00
Remco Bloemen
3e4493b389
Disallow self filling
2018-08-24 16:46:24 -07:00
Remco Bloemen
e6e7bae445
Remove BUG_ from revert reasons
2018-08-24 16:46:23 -07:00
Remco Bloemen
a1d8943552
Document accetable price check
2018-08-24 16:46:23 -07:00
Remco Bloemen
07e56b3cc7
Fix taker overpay check
2018-08-24 16:46:23 -07:00
Remco Bloemen
b16f5f55fb
Check fillable early
2018-08-24 16:46:23 -07:00