Merge pull request #197 from kleros/fix-multiple-arbitrable-transaction

fix(multiple-arbitrable-transaction): fix incompatible events

Author: n1c01a5

contracts/standard/arbitration/MultipleArbitrableTransaction.sol

@@ -66,7 +66,7 @@ contract MultipleArbitrableTransaction {
      *  @param _party The address of the party submiting the evidence. Note that 0 is kept for evidences not submitted by any party.
      *  @param _evidence A link to evidence or if it is short the evidence itself. Can be web link ("http://X"), IPFS ("ipfs:/X") or another storing service (using the URI, see https://en.wikipedia.org/wiki/Uniform_Resource_Identifier ). One usecase of short evidence is to include the hash of the plain English contract.
      */
-    event Evidence(Arbitrator indexed _arbitrator, uint indexed _disputeID, address _party, string _evidence);
+    event Evidence(Arbitrator indexed _arbitrator, uint indexed _disputeID, address indexed _party, string _evidence);
 
     /** @dev To be emmited when a dispute is created to link the correct meta-evidence to the disputeID.
      *  @param _arbitrator The arbitrator of the contract.
@@ -76,12 +76,11 @@ contract MultipleArbitrableTransaction {
     event Dispute(Arbitrator indexed _arbitrator, uint indexed _disputeID, uint _metaEvidenceID);
 
     /** @dev To be raised when a ruling is given.
-     *  @param _transactionID The index of the transaction in dispute.
      *  @param _arbitrator The arbitrator giving the ruling.
      *  @param _disputeID ID of the dispute in the Arbitrator contract.
      *  @param _ruling The ruling which was given.
      */
-    event Ruling(uint indexed _transactionID, Arbitrator indexed _arbitrator, uint indexed _disputeID, uint _ruling);
+    event Ruling(Arbitrator indexed _arbitrator, uint indexed _disputeID, uint _ruling);
 
     // **************************** //
     // *    Arbitrable functions  * //
@@ -292,7 +291,7 @@ contract MultipleArbitrableTransaction {
         require(msg.sender == address(arbitrator), "The caller must be the arbitrator.");
         require(transaction.status == Status.DisputeCreated, "The dispute has already been resolved.");
 
-        emit Ruling(transactionID, Arbitrator(msg.sender), _disputeID, _ruling);
+        emit Ruling(Arbitrator(msg.sender), _disputeID, _ruling);
 
         executeRuling(transactionID, _ruling);
     }