Skip to content

Commit a967ad3

Browse files
authored
chore: add missing events (#145)
Closes BES-693
1 parent c486393 commit a967ad3

File tree

57 files changed

+364
-48
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

57 files changed

+364
-48
lines changed

src/contracts/EmergencyModeHook.sol

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,23 +40,35 @@ abstract contract EmergencyModeHook is IEmergencyModeHook, SafeManageable {
4040
function setEmergencyMode() external {
4141
if (msg.sender != emergencyTrigger) revert Unauthorized(msg.sender, emergencyTrigger);
4242
emergencyMode = true;
43+
44+
emit EmergencyModeSet();
4345
}
4446

4547
/// @inheritdoc IEmergencyModeHook
4648
function unsetEmergencyMode() external isSafe {
4749
emergencyMode = false;
50+
51+
emit EmergencyModeUnset();
4852
}
4953

5054
/// @inheritdoc IEmergencyModeHook
5155
function setEmergencyCaller(address _emergencyCaller) external isSafe {
5256
if (_emergencyCaller == address(0)) revert ZeroAddress();
57+
58+
address _oldCaller = emergencyCaller;
5359
emergencyCaller = _emergencyCaller;
60+
61+
emit EmergencyCallerSet(_oldCaller, _emergencyCaller);
5462
}
5563

5664
/// @inheritdoc IEmergencyModeHook
5765
function setEmergencyTrigger(address _emergencyTrigger) external isSafe {
5866
if (_emergencyTrigger == address(0)) revert ZeroAddress();
67+
68+
address _oldTrigger = emergencyTrigger;
5969
emergencyTrigger = _emergencyTrigger;
70+
71+
emit EmergencyTriggerSet(_oldTrigger, _emergencyTrigger);
6072
}
6173

6274
// ~~~ INTERNAL METHODS ~~~

src/contracts/action-hubs/ActionHub.sol

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,8 +31,6 @@ abstract contract ActionHub is IActionHub {
3131
*/
3232
function _saveNewActionsBuilder(address _actionsBuilder) internal {
3333
_actionsBuilders[_actionsBuilder] = true;
34-
35-
emit NewActionsBuilderSaved(_actionsBuilder);
3634
}
3735

3836
/**

src/contracts/action-hubs/CappedTokenTransfersHub.sol

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -74,6 +74,8 @@ contract CappedTokenTransfersHub is ActionHub, ICappedTokenTransfersHub, SafeMan
7474
_actionsBuilder = address(new CappedTokenTransfers(_token, _amount, RECIPIENT));
7575

7676
_saveNewActionsBuilder(_actionsBuilder);
77+
78+
emit CappedTokenTransfersCreated(_actionsBuilder, _token, _amount);
7779
}
7880

7981
/// @inheritdoc ICappedTokenTransfersHub

src/contracts/factories/AllowanceClaimorFactory.sol

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,5 +21,7 @@ contract AllowanceClaimorFactory is IAllowanceClaimorFactory, Factory {
2121
_allowanceClaimor = address(new AllowanceClaimor(address(this), _token, _tokenOwner, _tokenRecipient));
2222

2323
_children[_allowanceClaimor] = true;
24+
25+
emit AllowanceClaimorCreated(_allowanceClaimor, _token, _tokenOwner, _tokenRecipient);
2426
}
2527
}

src/contracts/factories/CanonGuardFactory.sol

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -59,5 +59,7 @@ contract CanonGuardFactory is ICanonGuardFactory, Factory {
5959
);
6060

6161
_children[_canonGuard] = true;
62+
63+
emit CanonGuardCreated(_canonGuard, _safe, _emergencyTrigger, _emergencyCaller);
6264
}
6365
}

src/contracts/factories/CappedTokenTransfersHubFactory.sol

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,5 +25,7 @@ contract CappedTokenTransfersHubFactory is ICappedTokenTransfersHubFactory, Fact
2525
);
2626

2727
_children[_cappedTokenTransfersHub] = true;
28+
29+
emit CappedTokenTransfersHubCreated(_cappedTokenTransfersHub, _safe, _recipient);
2830
}
2931
}

src/contracts/factories/ChangeSafeGuardActionFactory.sol

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,5 +17,7 @@ contract ChangeSafeGuardActionFactory is IChangeSafeGuardActionFactory, Factory
1717
_changeSafeGuardAction = address(new ChangeSafeGuardAction(address(this), _safeGuard));
1818

1919
_children[_changeSafeGuardAction] = true;
20+
21+
emit ChangeSafeGuardActionCreated(_changeSafeGuardAction, _safeGuard);
2022
}
2123
}

src/contracts/factories/EverclearTokenConversionFactory.sol

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,5 +20,7 @@ contract EverclearTokenConversionFactory is IEverclearTokenConversionFactory, Fa
2020
_everclearTokenConversion = address(new EverclearTokenConversion(address(this), _lockbox, _next));
2121

2222
_children[_everclearTokenConversion] = true;
23+
24+
emit EverclearTokenConversionCreated(_everclearTokenConversion, _lockbox, _next);
2325
}
2426
}

src/contracts/factories/OPxActionFactory.sol

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,5 +17,7 @@ contract OPxActionFactory is IOPxActionFactory, Factory {
1717
_opxAction = address(new OPxAction(address(this), _opx));
1818

1919
_children[_opxAction] = true;
20+
21+
emit OPxActionCreated(_opxAction, _opx);
2022
}
2123
}

src/contracts/factories/PreApproveActionFactory.sol

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,5 +20,7 @@ contract PreApproveActionFactory is IPreApproveActionFactory, Factory {
2020
_preApproveAction = address(new PreApproveAction(address(this), _actionsBuilder, _approvalDuration));
2121

2222
_children[_preApproveAction] = true;
23+
24+
emit PreApproveActionCreated(_preApproveAction, _actionsBuilder, _approvalDuration);
2325
}
2426
}

0 commit comments

Comments
 (0)