From ad02740fa4d796fc576670594ecc4f185e5aa746 Mon Sep 17 00:00:00 2001 From: James Tuckett Date: Wed, 28 Feb 2024 12:51:47 +0000 Subject: [PATCH] refactor: Update deploy status to false and use constant for divisor --- packages/deploy-configurations/configs/arbitrum.conf.ts | 2 +- packages/deploy-configurations/configs/base.conf.ts | 2 +- packages/dma-contracts/contracts/actions/common/CollectFee.sol | 3 ++- 3 files changed, 4 insertions(+), 3 deletions(-) diff --git a/packages/deploy-configurations/configs/arbitrum.conf.ts b/packages/deploy-configurations/configs/arbitrum.conf.ts index 897f9f4a..6c7a0f2e 100644 --- a/packages/deploy-configurations/configs/arbitrum.conf.ts +++ b/packages/deploy-configurations/configs/arbitrum.conf.ts @@ -223,7 +223,7 @@ export const config: SystemConfig = { }, CollectFee: { name: 'CollectFee', - deploy: true, + deploy: false, address: '', serviceRegistryName: SERVICE_REGISTRY_NAMES.common.COLLECT_FEE, history: [], diff --git a/packages/deploy-configurations/configs/base.conf.ts b/packages/deploy-configurations/configs/base.conf.ts index 57fd1af9..72ed7858 100644 --- a/packages/deploy-configurations/configs/base.conf.ts +++ b/packages/deploy-configurations/configs/base.conf.ts @@ -188,7 +188,7 @@ export const config: SystemConfig = { }, CollectFee: { name: 'CollectFee', - deploy: true, + deploy: false, address: '', serviceRegistryName: SERVICE_REGISTRY_NAMES.common.COLLECT_FEE, history: [], diff --git a/packages/dma-contracts/contracts/actions/common/CollectFee.sol b/packages/dma-contracts/contracts/actions/common/CollectFee.sol index f6eeccc0..a1bce3ba 100644 --- a/packages/dma-contracts/contracts/actions/common/CollectFee.sol +++ b/packages/dma-contracts/contracts/actions/common/CollectFee.sol @@ -19,6 +19,7 @@ contract CollectFee is Executable, UseStorageSlot, UseRegistry { // Fee percentage (e.g., 1% = 100, 0.5% = 50) uint256 public feePercentage; address public feeRecipient; + uint256 constant DIVISOR = 10000; constructor(address _registry, uint256 _feePercentage, address _feeRecipient) UseRegistry(ServiceRegistry(_registry)) { feePercentage = _feePercentage; @@ -33,7 +34,7 @@ contract CollectFee is Executable, UseStorageSlot, UseRegistry { function execute(bytes calldata data, uint8[] memory paramsMap) external payable override { address asset = parseInputs(data); uint256 transactionAmount = store().readUint(bytes32(0), paramsMap[0]); - uint256 feeAmount = (transactionAmount * feePercentage) / 10000; + uint256 feeAmount = (transactionAmount * feePercentage) / DIVISOR; // Transfer fee from the user's proxy to the feeRecipient IERC20(asset).safeTransferFrom(msg.sender, feeRecipient, feeAmount);