fix: remove redundant access

Author: tejasbadadare

target_chains/ethereum/contracts/contracts/pulse/Scheduler.sol

@@ -524,16 +524,17 @@ abstract contract Scheduler is IScheduler, SchedulerState {
     /// BALANCE MANAGEMENT
 
     function addFunds(uint256 subscriptionId) external payable override {
-        if (!_state.subscriptionParams[subscriptionId].isActive) {
-            revert InactiveSubscription();
-        }
-
         SubscriptionParams storage params = _state.subscriptionParams[
             subscriptionId
         ];
         SubscriptionStatus storage status = _state.subscriptionStatuses[
             subscriptionId
         ];
+        
+        if (!status.isActive) {
+            revert InactiveSubscription();
+        }
+
 
         // Check deposit limit for permanent subscriptions
         if (params.isPermanent && msg.value > MAX_DEPOSIT_LIMIT) {

target_chains/ethereum/contracts/contracts/pulse/SchedulerState.sol

@@ -9,7 +9,7 @@ contract SchedulerState {
     uint8 public constant MAX_PRICE_IDS_PER_SUBSCRIPTION = 255;
     /// Maximum number of addresses in the reader whitelist
     uint8 public constant MAX_READER_WHITELIST_SIZE = 255;
-    /// Maximum deposit limit for permanent subscriptions in wei (100 ETH)
+    /// Maximum deposit limit for permanent subscriptions in wei
     uint256 public constant MAX_DEPOSIT_LIMIT = 100 ether;
 
     /// Maximum time in the past (relative to current block timestamp)