From 4fd25e1e492e5732284fd4889efad422c0c6f982 Mon Sep 17 00:00:00 2001
From: Isaac Abadi <IsaacMGrynsztein@gmail.com>
Date: Fri, 1 Dec 2023 15:43:56 -0500
Subject: [PATCH] Fixed issue where sub check would crash server if no subs
 existed

---
 backend/subscriptions.js | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/backend/subscriptions.js b/backend/subscriptions.js
index 4e7f7f4..c777435 100644
--- a/backend/subscriptions.js
+++ b/backend/subscriptions.js
@@ -214,6 +214,7 @@ exports.watchSubscriptionsInterval = async () => {
         if (!change) return;
         if (change['key'] === 'ytdl_subscriptions_check_interval' || change['key'] === 'ytdl_multi_user_mode') {
             current_sub_index = 0; // TODO: start after the last sub check
+            logger.verbose('Resetting sub check schedule due to config change');
             clearInterval(parent_interval);
             const new_interval = config_api.getConfigItem('ytdl_subscriptions_check_interval');
             parent_interval = setInterval(() => watchSubscriptions(), new_interval*1000);
@@ -224,6 +225,10 @@ exports.watchSubscriptionsInterval = async () => {
 
 async function watchSubscriptions() {
     const subscription_ids = await getValidSubscriptionsToCheck();
+    if (subscription_ids.length === 0) {
+        logger.info('Skipping subscription check as no valid subscriptions exist.');
+        return;
+    }
     checkSubscription(subscription_ids[current_sub_index]);
     current_sub_index = (current_sub_index + 1) % subscription_ids.length;
 }
@@ -243,7 +248,6 @@ async function checkSubscription(sub_id) {
     }
 
     await exports.getVideosForSub(sub.id);
-    // TODO: make sure if multi user mode changes we are also cancelling this
 }
 
 async function getValidSubscriptionsToCheck() {