feat(BlacklistValidator): Remove Double entrypoint on role Granting.

monerium · Jun 7, 2024 · 1b64e79 · 1b64e79
1 parent 5a62067
commit 1b64e79
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/script/configureToken.s.sol b/script/configureToken.s.sol
@@ -36,6 +36,7 @@ contract All is Script {
 
         token.addAdminAccount(devKey);
         token.setMintAllowance(system, allowance);
+        console.log("mint allowance set successfully.");
         token.removeAdminAccount(devKey);
         vm.stopBroadcast();
     }

diff --git a/src/BlacklistValidatorUpgradeable.sol b/src/BlacklistValidatorUpgradeable.sol
@@ -165,7 +165,7 @@ contract BlacklistValidatorUpgradeable is
      * @param account The address of the account.
      */
     function addAdminAccount(address account) public virtual onlyOwner {
-        grantRole(ADMIN_ROLE, account);
+        _grantRole(ADMIN_ROLE, account);
         emit AdminAccountAdded(account);
     }
 
@@ -174,7 +174,7 @@ contract BlacklistValidatorUpgradeable is
      * @param account The address of the account.
      */
     function removeAdminAccount(address account) public virtual onlyOwner {
-        revokeRole(ADMIN_ROLE, account);
+        _revokeRole(ADMIN_ROLE, account);
         emit AdminAccountRemoved(account);
     }