From ce50ea87fbd2bef7c9829a5134ea24a456eae318 Mon Sep 17 00:00:00 2001
From: agusduha <agusnduha@gmail.com>
Date: Fri, 9 Aug 2024 12:09:46 -0300
Subject: [PATCH] fix: use only a public function for create3

---
 .../OptimismMintableERC20Factory.sol          | 18 ++++++++++-----
 .../OptimismMintableERC20FactoryInterop.sol   | 22 ++-----------------
 2 files changed, 14 insertions(+), 26 deletions(-)

diff --git a/packages/contracts-bedrock/src/universal/OptimismMintableERC20Factory.sol b/packages/contracts-bedrock/src/universal/OptimismMintableERC20Factory.sol
index d0ec64eb56a6..8d62a4a1c4ec 100644
--- a/packages/contracts-bedrock/src/universal/OptimismMintableERC20Factory.sol
+++ b/packages/contracts-bedrock/src/universal/OptimismMintableERC20Factory.sol
@@ -12,7 +12,7 @@ import { Initializable } from "@openzeppelin/contracts/proxy/utils/Initializable
 ///         contracts on the network it's deployed to. Simplifies the deployment process for users
 ///         who may be less familiar with deploying smart contracts. Designed to be backwards
 ///         compatible with the older StandardL2ERC20Factory contract.
-contract OptimismMintableERC20Factory is ISemver, Initializable {
+abstract contract OptimismMintableERC20Factory is ISemver, Initializable {
     /// @custom:spacer OptimismMintableERC20Factory's initializer slot spacing
     /// @notice Spacer to avoid packing into the initializer slot
     bytes30 private spacer_0_2_30;
@@ -114,17 +114,23 @@ contract OptimismMintableERC20Factory is ISemver, Initializable {
         public
         returns (address)
     {
-        return _createWithCreate3(_remoteToken, _name, _symbol, _decimals);
+        return createWithCreate3(_remoteToken, _name, _symbol, _decimals);
     }
 
-    function _createWithCreate3(
+    /// @notice Creates an instance of the OptimismMintableERC20 contract, with specified decimals using CREATE3.
+    ///         To be implemented by the child contract.
+    /// @param _remoteToken Address of the token on the remote chain.
+    /// @param _name        ERC20 name.
+    /// @param _symbol      ERC20 symbol.
+    /// @param _decimals    ERC20 decimals.
+    /// @return Address of the newly created token.
+    function createWithCreate3(
         address _remoteToken,
         string memory _name,
         string memory _symbol,
         uint8 _decimals
     )
-        internal
+        public
         virtual
-        returns (address)
-    { }
+        returns (address);
 }
diff --git a/packages/contracts-bedrock/src/universal/OptimismMintableERC20FactoryInterop.sol b/packages/contracts-bedrock/src/universal/OptimismMintableERC20FactoryInterop.sol
index 04a8cefcda94..74751a57e9bb 100644
--- a/packages/contracts-bedrock/src/universal/OptimismMintableERC20FactoryInterop.sol
+++ b/packages/contracts-bedrock/src/universal/OptimismMintableERC20FactoryInterop.sol
@@ -33,37 +33,19 @@ contract OptimismMintableERC20FactoryInterop is OptimismMintableERC20Factory, IO
         }
     }
 
-    /// @notice Creates an instance of the OptimismMintableERC20 contract, with specified decimals using CREATE3.
-    /// @param _remoteToken Address of the token on the remote chain.
-    /// @param _name        ERC20 name.
-    /// @param _symbol      ERC20 symbol.
-    /// @param _decimals    ERC20 decimals.
-    /// @return Address of the newly created token.
-    function createWithCreate3(
-        address _remoteToken,
-        string memory _name,
-        string memory _symbol,
-        uint8 _decimals
-    )
-        external
-        returns (address)
-    {
-        return _createWithCreate3(_remoteToken, _name, _symbol, _decimals);
-    }
-
     /// @notice Creates an instance of the OptimismMintableERC20 contract, with specified decimals using CREATE3.
     /// @param _remoteToken Address of the token on the remote chain.
     /// @param _name        ERC20 name.
     /// @param _symbol      ERC20 symbol.
     /// @param _decimals    ERC20 decimals.
     /// @return _localToken Address of the newly created token.
-    function _createWithCreate3(
+    function createWithCreate3(
         address _remoteToken,
         string memory _name,
         string memory _symbol,
         uint8 _decimals
     )
-        internal
+        public
         override
         returns (address _localToken)
     {