Skip to content

Commit

Permalink
fix: tailing renaming
Browse files Browse the repository at this point in the history
  • Loading branch information
failingtwice committed Feb 11, 2025
1 parent fce5c97 commit 7e1f108
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 6 deletions.
2 changes: 1 addition & 1 deletion contracts/0.8.25/utils/AccessControlConfirmable.sol
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ abstract contract AccessControlConfirmable is AccessControlEnumerable {
* @notice The order of confirmations does not matter
*
*/
modifier onlyMutuallyConfirmed(bytes32[] memory _roles) {
modifier onlyConfirmed(bytes32[] memory _roles) {
if (_roles.length == 0) revert ZeroConfirmingRoles();
if (confirmLifetime == 0) revert ConfirmLifetimeNotSet();

Expand Down
4 changes: 2 additions & 2 deletions contracts/0.8.25/vaults/Delegation.sol
Original file line number Diff line number Diff line change
Expand Up @@ -157,7 +157,7 @@ contract Delegation is Dashboard {
* the confirm is considered expired, no longer counts and must be recasted.
* @param _newConfirmLifetime The new confirm lifetime in seconds.
*/
function setConfirmLifetime(uint256 _newConfirmLifetime) external onlyMutuallyConfirmed(_confirmingRoles()) {
function setConfirmLifetime(uint256 _newConfirmLifetime) external onlyConfirmed(_confirmingRoles()) {
_setConfirmLifetime(_newConfirmLifetime);
}

Expand Down Expand Up @@ -185,7 +185,7 @@ contract Delegation is Dashboard {
* which is why the deciding confirm must make sure that `nodeOperatorUnclaimedFee()` is 0 before calling this function.
* @param _newNodeOperatorFeeBP The new node operator fee in basis points.
*/
function setNodeOperatorFeeBP(uint256 _newNodeOperatorFeeBP) external onlyMutuallyConfirmed(_confirmingRoles()) {
function setNodeOperatorFeeBP(uint256 _newNodeOperatorFeeBP) external onlyConfirmed(_confirmingRoles()) {
if (_newNodeOperatorFeeBP + curatorFeeBP > MAX_FEE_BP) revert CombinedFeesExceed100Percent();
if (nodeOperatorUnclaimedFee() > 0) revert NodeOperatorFeeUnclaimed();
uint256 oldNodeOperatorFeeBP = nodeOperatorFeeBP;
Expand Down
6 changes: 3 additions & 3 deletions contracts/0.8.25/vaults/Permissions.sol
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
pragma solidity 0.8.25;

import {Clones} from "@openzeppelin/contracts-v5.2/proxy/Clones.sol";
import {AccessControlMutuallyConfirmable} from "contracts/0.8.25/utils/AccessControlMutuallyConfirmable.sol";
import {AccessControlConfirmable} from "contracts/0.8.25/utils/AccessControlConfirmable.sol";
import {OwnableUpgradeable} from "contracts/openzeppelin/5.2/upgradeable/access/OwnableUpgradeable.sol";

import {IStakingVault} from "./interfaces/IStakingVault.sol";
Expand All @@ -16,7 +16,7 @@ import {VaultHub} from "./VaultHub.sol";
* @author Lido
* @notice Provides granular permissions for StakingVault operations.
*/
abstract contract Permissions is AccessControlMutuallyConfirmable {
abstract contract Permissions is AccessControlConfirmable {
/**
* @notice Struct containing an account and a role for granting/revoking roles.
*/
Expand Down Expand Up @@ -179,7 +179,7 @@ abstract contract Permissions is AccessControlMutuallyConfirmable {
vaultHub.voluntaryDisconnect(address(stakingVault()));
}

function _transferStakingVaultOwnership(address _newOwner) internal onlyMutuallyConfirmed(_confirmingRoles()) {
function _transferStakingVaultOwnership(address _newOwner) internal onlyConfirmed(_confirmingRoles()) {
OwnableUpgradeable(address(stakingVault())).transferOwnership(_newOwner);
}

Expand Down

0 comments on commit 7e1f108

Please sign in to comment.