fix: remove redundant getter.

graphprotocol · Oct 1, 2024 · c8f499e · c8f499e
1 parent 0dce797
commit c8f499e
Showing 1 changed file with 5 additions and 13 deletions.
diff --git a/packages/subgraph-service/contracts/DisputeManager.sol b/packages/subgraph-service/contracts/DisputeManager.sol
@@ -101,14 +101,14 @@ contract DisputeManager is
     /**
      * @notice Initialize this contract.
      * @param arbitrator Arbitrator role
-     * @param disputePeriod Dispute period in seconds
+     * @param _disputePeriod Dispute period in seconds
      * @param disputeDeposit Deposit required to create a Dispute
      * @param fishermanRewardCut_ Percent of slashed funds for fisherman (ppm)
      * @param maxSlashingCut_ Maximum percentage of indexer stake that can be slashed (ppm)
      */
     function initialize(
         address arbitrator,
-        uint64 disputePeriod,
+        uint64 _disputePeriod,
         uint256 disputeDeposit,
         uint32 fishermanRewardCut_,
         uint32 maxSlashingCut_
@@ -117,7 +117,7 @@ contract DisputeManager is
         __AttestationManager_init();
 
         _setArbitrator(arbitrator);
-        _setDisputePeriod(disputePeriod);
+        _setDisputePeriod(_disputePeriod);
         _setDisputeDeposit(disputeDeposit);
         _setFishermanRewardCut(fishermanRewardCut_);
         _setMaxSlashingCut(maxSlashingCut_);
@@ -168,14 +168,6 @@ contract DisputeManager is
             );
     }
 
-    /**
-     * @notice Returns the current dispute period.
-     * @return The dispute period in seconds as a uint64.
-     */
-    function disputePeriod() external view override returns (uint64) {
-        return disputePeriod;
-    }
-
     /**
      * @notice Create query disputes for two conflicting attestations.
      * A conflicting attestation is a proof presented by two different indexers
@@ -319,8 +311,8 @@ contract DisputeManager is
      * @dev Update the dispute period to `_disputePeriod` in seconds
      * @param disputePeriod Dispute period in seconds
      */
-    function setDisputePeriod(uint64 disputePeriod) external override onlyOwner {
-        _setDisputePeriod(disputePeriod);
+    function setDisputePeriod(uint64 _disputePeriod) external override onlyOwner {
+        _setDisputePeriod(_disputePeriod);
     }
 
     /**