clean up to previous release version

naszam · May 15, 2020 · 343298b · 343298b
1 parent 0d979c8
commit 343298b
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/contracts/AntsReview.sol b/contracts/AntsReview.sol
@@ -91,13 +91,13 @@ contract AntsReview is Ownable, AccessControl, Pausable {
     _;
   }
 
-  modifier onlyIssuer() {
+  modifier onlyIssuer(uint256 _antReviewId) {
       require(hasRole(ISSUER_ROLE, msg.sender), "Caller is not an issuer");
       _;
   }
 
-  modifier notIssuer() {
-      require(!hasRole(ISSUER_ROLE, msg.sender), "Caller is an issuer");
+  modifier notIssuer(uint256 _antReviewId) {
+      require(msg.sender!= antreviews[_antReviewId].issuer);
       _;
   }
 
@@ -148,7 +148,7 @@ contract AntsReview is Ownable, AccessControl, Pausable {
   function fulfillAntReview(uint256 _antReviewId, string memory _data)
     public
     antReviewExists(_antReviewId)
-    notIssuer
+    notIssuer(_antReviewId)
     hasStatus(_antReviewId, AntReviewStatus.CREATED)
     isBeforeDeadline(_antReviewId)
     whenNotPaused()
@@ -166,7 +166,7 @@ contract AntsReview is Ownable, AccessControl, Pausable {
       public
       antReviewExists(_antReviewId)
       fulfillmentExists(_antReviewId,_fulfillmentId)
-      onlyIssuer
+      onlyIssuer(_antReviewId)
       hasStatus(_antReviewId, AntReviewStatus.CREATED)
       fulfillmentNotYetAccepted(_antReviewId, _fulfillmentId)
       whenNotPaused()
@@ -188,7 +188,7 @@ contract AntsReview is Ownable, AccessControl, Pausable {
   function cancelAntReview(uint256 _antReviewId)
       public
       antReviewExists(_antReviewId)
-      onlyIssuer
+      onlyIssuer(_antReviewId)
       hasStatus(_antReviewId, AntReviewStatus.CREATED)
       whenNotPaused()
   {