Сonsistent Order Within a Contract

Author: zZoMROT

contracts/extensions/FeeTaker.sol

@@ -161,14 +161,6 @@ contract FeeTaker is IPostInteraction, AmountGetterWithFee, Ownable {
         }
     }
 
-    /**
-     * @dev Parses fee data from `extraData`.
-     * Override this function if whitelist structure in postInteraction is different from getters.
-     */
-    function _isWhitelistedPostInteractionImpl(bytes calldata whitelistData, address taker) internal view virtual returns (bool isWhitelisted, bytes calldata tail) {
-        return _isWhitelistedGetterImpl(whitelistData, taker);
-    }
-
     /**
      * @dev Calculates fee amounts depending on whether the taker is in the whitelist and whether they have an _ACCESS_TOKEN.
      * `extraData` consists of:
@@ -189,6 +181,14 @@ contract FeeTaker is IPostInteraction, AmountGetterWithFee, Ownable {
         protocolFeeAmount = takingAmount.mulDiv(resolverFee, denominator) + integratorFeeTotal - integratorFeeAmount;
     }
 
+    /**
+     * @dev Parses fee data from `extraData`.
+     * Override this function if whitelist structure in postInteraction is different from getters.
+     */
+    function _isWhitelistedPostInteractionImpl(bytes calldata whitelistData, address taker) internal view virtual returns (bool isWhitelisted, bytes calldata tail) {
+        return _isWhitelistedGetterImpl(whitelistData, taker);
+    }
+
     function _sendEth(address target, uint256 amount) private {
         (bool success, ) = target.call{value: amount}("");
         if (!success) {

contracts/interfaces/IOrderMixin.sol

@@ -96,6 +96,21 @@ interface IOrderMixin {
      */
     function rawRemainingInvalidatorForOrder(address maker, bytes32 orderHash) external view returns(uint256 remainingRaw);
 
+    /**
+     * @notice Returns order hash, hashed with limit order protocol contract EIP712
+     * @param order Order
+     * @return orderHash Hash of the order
+     */
+    function hashOrder(IOrderMixin.Order calldata order) external view returns(bytes32 orderHash);
+
+    /**
+     * @notice Delegates execution to custom implementation. Could be used to validate if `transferFrom` works properly
+     * @dev The function always reverts and returns the simulation results in revert data.
+     * @param target Addresses that will be delegated
+     * @param data Data that will be passed to delegatee
+     */
+    function simulate(address target, bytes calldata data) external;
+
     /**
      * @notice Cancels order's quote
      * @param makerTraits Order makerTraits
@@ -117,21 +132,6 @@ interface IOrderMixin {
      */
     function bitsInvalidateForOrder(MakerTraits makerTraits, uint256 additionalMask) external;
 
-    /**
-     * @notice Returns order hash, hashed with limit order protocol contract EIP712
-     * @param order Order
-     * @return orderHash Hash of the order
-     */
-    function hashOrder(IOrderMixin.Order calldata order) external view returns(bytes32 orderHash);
-
-    /**
-     * @notice Delegates execution to custom implementation. Could be used to validate if `transferFrom` works properly
-     * @dev The function always reverts and returns the simulation results in revert data.
-     * @param target Addresses that will be delegated
-     * @param data Data that will be passed to delegatee
-     */
-    function simulate(address target, bytes calldata data) external;
-
     /**
      * @notice Fills order's quote, fully or partially (whichever is possible).
      * @param order Order quote to fill