refactor: Update deploy status to false and use constant for divisor

OasisDEX · Feb 28, 2024 · ad02740 · ad02740
1 parent b49b008
commit ad02740
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git 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
@@ -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
@@ -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);