Merge pull request #175 from rhinestonewtf/fix/module-already-initial…

…ized changing error for AlreadyInitialized to not conflict with ownable
rhinestonewtf · Jan 8, 2025 · 8c221e0 · 8c221e0
2 parents 8172971 + 2cf4704
commit 8c221e0
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/src/accounts/common/interfaces/IERC7579Module.sol b/src/accounts/common/interfaces/IERC7579Module.sol
@@ -14,7 +14,7 @@ uint256 constant MODULE_TYPE_FALLBACK = 3;
 uint256 constant MODULE_TYPE_HOOK = 4;
 
 interface IModule {
-    error AlreadyInitialized(address smartAccount);
+    error ModuleAlreadyInitialized(address smartAccount);
     error NotInitialized(address smartAccount);
 
     /**

diff --git a/src/accounts/kernel/interfaces/IERC7579Modules.sol b/src/accounts/kernel/interfaces/IERC7579Modules.sol
@@ -6,7 +6,7 @@ import { PackedUserOperation } from
     "@ERC4337/account-abstraction/contracts/core/UserOperationLib.sol";
 
 interface IModule {
-    error AlreadyInitialized(address smartAccount);
+    error ModuleAlreadyInitialized(address smartAccount);
     error NotInitialized(address smartAccount);
 
     /**

diff --git a/src/module-bases/SchedulingBase.sol b/src/module-bases/SchedulingBase.sol
@@ -40,7 +40,7 @@ abstract contract SchedulingBase is ERC7579ExecutorBase {
     function _onInstall(bytes calldata packedSchedulingData) internal {
         address account = msg.sender;
         if (isInitialized(account)) {
-            revert AlreadyInitialized(account);
+            revert ModuleAlreadyInitialized(account);
         }
 
         _createExecution({ orderData: packedSchedulingData });