M-3 removing payable

Synthetixio · Aug 16, 2023 · e783387 · e783387
1 parent ad52224
commit e783387
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/protocol/synthetix/contracts/interfaces/IMulticallModule.sol b/protocol/synthetix/contracts/interfaces/IMulticallModule.sol
@@ -11,7 +11,7 @@ interface IMulticallModule {
      * @param data Array of calldata objects, one for each function that is to be called in the system.
      * @return results Array of each `delegatecall`'s response corresponding to the incoming calldata array.
      */
-    function multicall(bytes[] calldata data) external payable returns (bytes[] memory results);
+    function multicall(bytes[] calldata data) external returns (bytes[] memory results);
 
     /**
      * @notice Similar to `multicall`, but allows for transactions to be executed
@@ -24,7 +24,7 @@ interface IMulticallModule {
     /*function multicallThrough(
         address[] calldata to,
         bytes[] calldata data
-    ) external payable returns (bytes[] memory results);
+    ) external returns (bytes[] memory results);
 
     function getMessageSender() external view returns (address);*/
 }
diff --git a/protocol/synthetix/contracts/modules/core/MulticallModule.sol b/protocol/synthetix/contracts/modules/core/MulticallModule.sol
@@ -18,9 +18,7 @@ contract MulticallModule is IMulticallModule {
     /**
      * @inheritdoc IMulticallModule
      */
-    function multicall(
-        bytes[] calldata data
-    ) public payable override returns (bytes[] memory results) {
+    function multicall(bytes[] calldata data) public override returns (bytes[] memory results) {
         results = new bytes[](data.length);
         for (uint256 i = 0; i < data.length; i++) {
             (bool success, bytes memory result) = address(this).delegatecall(data[i]);
@@ -40,7 +38,7 @@ contract MulticallModule is IMulticallModule {
     /*function multicallThrough(
         address[] calldata to,
         bytes[] calldata data
-    ) public payable override returns (bytes[] memory results) {
+    ) public override returns (bytes[] memory results) {
         if (Config.read(_CONFIG_MESSAGE_SENDER, 0) != 0) {
             revert RecursiveMulticall(msg.sender);
         }