From bc37afd65fcd4083679f8b5881e04efb830d9e88 Mon Sep 17 00:00:00 2001
From: tadscottsmith <78445808+tadscottsmith@users.noreply.github.com>
Date: Wed, 12 Mar 2025 15:25:43 -0500
Subject: [PATCH] Update patch saving logic.

---
 backend/controllers/uploads.js | 23 ++++++++++++-----------
 1 file changed, 12 insertions(+), 11 deletions(-)

diff --git a/backend/controllers/uploads.js b/backend/controllers/uploads.js
index 3f051f6..8399227 100644
--- a/backend/controllers/uploads.js
+++ b/backend/controllers/uploads.js
@@ -108,17 +108,18 @@ exports.upload = async function (req, res, next) {
           return;
         }
 
-        patches = [];
+        let patches = [];
+        try {
+          let req_patches = req.body.patch_list || "[]"; 
 
-        let req_patches;
-        req_patches = req.body.patch_list;
-      
-        if(typeof req_patches != "undefined"){
-          var split_patches = req_patches.replace("[","").replace("]","").split(",");
-      
-          for (var patch in split_patches){
-            patches.push(split_patches[patch]);
-          } 
+          if (typeof req_patches === 'string') {
+            req_patches = req_patches.replace(/\[|\]/g, "").split(",").map(num => num.trim());
+            patches = req_patches.map(num => Number(num)).filter(num => !isNaN(num));
+          }
+        } catch (err) {
+          console.warn(`[${req.params.shortName}] Error /:shortName/upload Parsing Patches List - Error: ${err}`);
+          res.status(500).send("Error parsing patch_list " + err);
+          return;
         }
 
         let item = null;
@@ -279,4 +280,4 @@ exports.upload = async function (req, res, next) {
 
     });
   });
-};
\ No newline at end of file
+};