f: inline release and lock stake

Author: matiasedgeandnode

packages/subgraph-service/contracts/SubgraphService.sol

@@ -775,7 +775,15 @@ contract SubgraphService is
             })
         );
 
-        _releaseAndLockStake(indexer, tokensCollected);
+        _releaseStake(indexer, 0);
+        if (tokensCollected > 0) {
+            // lock stake as economic security for fees
+            _lockStake(
+                indexer,
+                tokensCollected * stakeToFeesRatio,
+                block.timestamp + _disputeManager().getDisputePeriod()
+            );
+        }
 
         return tokensCollected;
     }
@@ -790,25 +798,6 @@ contract SubgraphService is
         emit StakeToFeesRatioSet(_stakeToFeesRatio);
     }
 
-    /**
-     * @notice Release stake claims and lock new stake as economic security for fees
-     * @dev This function releases all expired stake claims and locks new stake as economic security for fees.
-     * It is called after collecting query fees or indexing fees.
-     * @param _indexer The address of the indexer
-     * @param _tokensCollected The amount of tokens collected from fees
-     */
-    function _releaseAndLockStake(address _indexer, uint256 _tokensCollected) private {
-        _releaseStake(_indexer, 0);
-        if (_tokensCollected > 0) {
-            // lock stake as economic security for fees
-            _lockStake(
-                _indexer,
-                _tokensCollected * stakeToFeesRatio,
-                block.timestamp + _disputeManager().getDisputePeriod()
-            );
-        }
-    }
-
     /**
      * @notice Encodes the data for the GraphTallyCollector
      * @dev The purpose of this function is just to avoid stack too deep errors