fix: apply consistent mapping order ProvisionManagerNotAuthorized (OZ…

…_N-05)
graphprotocol · Oct 4, 2024 · addcfbc · addcfbc
1 parent c7bc66a
commit addcfbc
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/packages/horizon/contracts/data-service/utilities/ProvisionManager.sol b/packages/horizon/contracts/data-service/utilities/ProvisionManager.sol
@@ -71,7 +71,7 @@ abstract contract ProvisionManager is Initializable, GraphDirectory, ProvisionMa
      * @param caller The address of the caller.
      * @param serviceProvider The address of the service provider.
      */
-    error ProvisionManagerNotAuthorized(address caller, address serviceProvider);
+    error ProvisionManagerNotAuthorized(address serviceProvider, address caller);
 
     /**
      * @notice Thrown when a provision is not found.
@@ -84,8 +84,8 @@ abstract contract ProvisionManager is Initializable, GraphDirectory, ProvisionMa
      */
     modifier onlyProvisionAuthorized(address serviceProvider) {
         require(
-            _graphStaking().isAuthorized(msg.sender, serviceProvider, address(this)),
-            ProvisionManagerNotAuthorized(msg.sender, serviceProvider)
+            _graphStaking().isAuthorized(serviceProvider, msg.sender, address(this)),
+            ProvisionManagerNotAuthorized(serviceProvider, msg.sender)
         );
         _;
     }