fix(IArbitrator): change name to arbitration cost

Author: unknownunknown1

CHANGELOG.md

@@ -2,6 +2,7 @@
 
 - fix(Arbitrator): memory to calldata ([4770b1f](https://github.com/kleros/kleros-v2/commit/4770b1f))
 - fix(IArbitrator): appeals removed from the standard ([02c20ce](https://github.com/kleros/kleros-v2/commit/02c20ce))
+- fix(IArbitrator): interface simplification ([e81fb8b](https://github.com/kleros/kleros-v2/commit/e81fb8b))
 - fix(IArbitrator): replaced appealCost with fundingStatus ([f189dd9](https://github.com/kleros/kleros-v2/commit/f189dd9))
 - feat: modern toolchain setup and simple RNG smart contracts ([17f6a76](https://github.com/kleros/kleros-v2/commit/17f6a76))
 - feat(Arbitration): standard update ([ed930de](https://github.com/kleros/kleros-v2/commit/ed930de))

contracts/src/arbitration/CentralizedArbitrator.sol

@@ -50,7 +50,7 @@ contract CentralizedArbitrator is IArbitrator {
     address public owner = msg.sender; // Owner of the contract.
     uint256 public appealDuration; // The duration of the appeal period.
 
-    uint256 private arbitrationFee; // The cost to create a dispute. Made private because of the cost() getter.
+    uint256 private arbitrationFee; // The cost to create a dispute. Made private because of the arbitrationCost() getter.
     uint256 public appealFee; // The cost to fund one of the choices, not counting the additional fee stake amount.
 
     DisputeStruct[] public disputes; // Stores the dispute info. disputes[disputeID].
@@ -155,7 +155,7 @@ contract CentralizedArbitrator is IArbitrator {
     }
 
     /** @dev Create a dispute. Must be called by the arbitrable contract.
-     *  Must be paid at least cost().
+     *  Must be paid at least arbitrationCost().
      *  @param _choices Amount of choices the arbitrator can make in this dispute.
      *  @param _extraData Can be used to give additional info on the dispute to be created.
      *  @return disputeID ID of the dispute created.
@@ -166,8 +166,8 @@ contract CentralizedArbitrator is IArbitrator {
         override
         returns (uint256 disputeID)
     {
-        uint256 arbitrationCost = cost(_extraData);
-        require(msg.value >= arbitrationCost, "Not enough ETH to cover arbitration costs.");
+        uint256 localArbitrationCost = arbitrationCost(_extraData);
+        require(msg.value >= localArbitrationCost, "Not enough ETH to cover arbitration costs.");
         disputeID = disputes.length;
         disputes.push(
             DisputeStruct({
@@ -329,7 +329,7 @@ contract CentralizedArbitrator is IArbitrator {
     /** @dev Cost of arbitration.
      *  @return fee The required amount.
      */
-    function cost(
+    function arbitrationCost(
         bytes calldata /*_extraData*/
     ) public view override returns (uint256 fee) {
         return arbitrationFee;

contracts/src/arbitration/IArbitrator.sol

@@ -10,7 +10,7 @@ import "./IArbitrable.sol";
  * Unlike the ERC-792 this standard doesn't have anything related to appeals, so each arbitrator can implement an appeal system that suits it the most.
  * When developing arbitrator contracts we need to:
  * - Define the functions for dispute creation (createDispute). Don't forget to store the arbitrated contract and the disputeID (which should be unique, may nbDisputes).
- * - Define the functions for cost display (cost).
+ * - Define the functions for cost display (arbitrationCost).
  * - Allow giving rulings. For this a function must call arbitrable.rule(disputeID, ruling).
  */
 interface IArbitrator {
@@ -23,7 +23,7 @@ interface IArbitrator {
 
     /**
      * @dev Create a dispute. Must be called by the arbitrable contract.
-     * Must pay at least cost(_extraData).
+     * Must pay at least arbitrationCost(_extraData).
      * @param _choices Amount of choices the arbitrator can make in this dispute.
      * @param _extraData Can be used to give additional info on the dispute to be created.
      * @return disputeID ID of the dispute created.
@@ -35,5 +35,5 @@ interface IArbitrator {
      * @param _extraData Can be used to give additional info on the dispute to be created.
      * @return cost Required cost of arbitration.
      */
-    function cost(bytes calldata _extraData) external view returns (uint256 cost);
+    function arbitrationCost(bytes calldata _extraData) external view returns (uint256 cost);
 }