Refactor heartbeat event name and extract internal function for setti…

…ng heartbeat
overlay-market · Mar 18, 2024 · b440e00 · b440e00
1 parent 1a875d9
commit b440e00
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/contracts/feeds/chainlink/OverlayV1ChainlinkFeed.sol b/contracts/feeds/chainlink/OverlayV1ChainlinkFeed.sol
@@ -12,7 +12,7 @@ contract OverlayV1ChainlinkFeed is OverlayV1Feed {
     uint256 public heartbeat;
     string public description;
 
-    event HeartbeatUpdated(uint256 heartbeat);
+    event HeartbeatSet(uint256 heartbeat);
 
     modifier onlyGovernor() {
         require(ov.hasRole(GOVERNOR_ROLE, msg.sender), "OVV1: !governor");
@@ -31,7 +31,7 @@ contract OverlayV1ChainlinkFeed is OverlayV1Feed {
         aggregator = AggregatorV3Interface(_aggregator);
         decimals = aggregator.decimals();
         description = aggregator.description();
-        heartbeat = _heartbeat;
+        _setHeartbeat(_heartbeat);
         ov = IOverlayV1Token(_ov);
     }
 
@@ -125,7 +125,11 @@ contract OverlayV1ChainlinkFeed is OverlayV1Feed {
     }
 
     function setHeartbeat(uint256 _heartbeat) external onlyGovernor {
+        _setHeartbeat(_heartbeat);
+    }
+
+    function _setHeartbeat(uint256 _heartbeat) internal {
         heartbeat = _heartbeat;
-        emit HeartbeatUpdated(_heartbeat);
+        emit HeartbeatSet(_heartbeat);
     }
 }