fix(multi-party-agreements): record ruling

kleros · Oct 16, 2018 · 4158802 · 4158802
1 parent a0259a0
commit 4158802
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/contracts/standard/arbitration/composed-arbitrable/agreement/MultiPartyAgreements.sol b/contracts/standard/arbitration/composed-arbitrable/agreement/MultiPartyAgreements.sol
@@ -98,6 +98,7 @@ contract MultiPartyAgreements is Arbitrable {
         Agreement storage agreement = agreements[_agreementID];
         require(agreement.creator != address(0), "The specified agreement does not exist.");
         require(!agreement.executed, "The specified agreement has already been executed.");
+        agreement.ruling = _ruling;
         agreement.executed = true;
     }
 
@@ -106,10 +107,7 @@ contract MultiPartyAgreements is Arbitrable {
      *  @param _ruling The ruling.
      */
     function executeRuling(uint _disputeID, uint _ruling) internal {
-        bytes32 _agreementID = arbitratorAndDisputeIDToAgreementID[msg.sender][_disputeID];
-        Agreement storage agreement = agreements[_agreementID];
-        agreement.ruling = _ruling;
-        executeAgreementRuling(_agreementID, _ruling);
+        executeAgreementRuling(arbitratorAndDisputeIDToAgreementID[msg.sender][_disputeID], _ruling);
     }
 
     /* External Views */