fix: make sure rowNode exists before trying to loop on it

6pac · Oct 24, 2023 · 244870d · 244870d
1 parent 3e261a4
commit 244870d
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/slick.grid.js b/slick.grid.js
@@ -3543,6 +3543,10 @@ if (typeof Slick === "undefined") {
           });
         }
       }
+
+      if (!cacheEntry.rowNode) {
+        cacheEntry.rowNode = [];
+      }
       postProcessedCleanupQueue.push({
         actionType: 'R',
         groupId: postProcessgroupId,
@@ -3566,7 +3570,7 @@ if (typeof Slick === "undefined") {
 
     function removeRowFromCache(row) {
       var cacheEntry = rowsCache[row];
-      if (!cacheEntry) {
+      if (!cacheEntry || !cacheEntry.rowNode) {
         return;
       }
 
@@ -5113,7 +5117,7 @@ if (typeof Slick === "undefined") {
       if (activeCellNode !== null) {
         makeActiveCellNormal();
         activeCellNode.classList.remove("active");
-        if (rowsCache[activeRow]) {
+        if (rowsCache[activeRow] && rowsCache[activeRow].rowNode) {
           rowsCache[activeRow].rowNode.forEach(function (node) {
             node.classList.remove("active");
           });
@@ -5146,7 +5150,7 @@ if (typeof Slick === "undefined") {
 
         if (options.showCellSelection) {
           activeCellNode.classList.add("active");
-          if (rowsCache[activeRow]) {
+          if (rowsCache[activeRow] && rowsCache[activeRow].rowNode) {
             rowsCache[activeRow].rowNode.forEach(function (node) {
               node.classList.add("active");
             });