fix(api): if cache(list) is not present skip pushing it into the list to avoid unexpected behaviour

Signed-off-by: Pranav C <pranavxc@gmail.com>

packages/nocodb/src/lib/models/FormViewColumn.ts

@@ -76,11 +76,19 @@ export default class FormViewColumn implements FormColumnType {
 
     await NocoCache.set(`${CacheScope.FORM_VIEW_COLUMN}:${fk_column_id}`, id);
 
-    await NocoCache.appendToList(
-      CacheScope.FORM_VIEW_COLUMN,
-      [column.fk_view_id],
-      `${CacheScope.FORM_VIEW_COLUMN}:${id}`
-    );
+    // if cache is not present skip pushing it into the list to avoid unexpected behaviour
+    if (
+      (
+        await NocoCache.getList(CacheScope.FORM_VIEW_COLUMN, [
+          column.fk_view_id,
+        ])
+      )?.length
+    )
+      await NocoCache.appendToList(
+        CacheScope.FORM_VIEW_COLUMN,
+        [column.fk_view_id],
+        `${CacheScope.FORM_VIEW_COLUMN}:${id}`
+      );
     return this.get(id, ncMeta);
   }
 

packages/nocodb/src/lib/models/GalleryViewColumn.ts

@@ -72,11 +72,19 @@ export default class GalleryViewColumn {
       id
     );
 
-    await NocoCache.appendToList(
-      CacheScope.GALLERY_VIEW_COLUMN,
-      [column.fk_view_id],
-      `${CacheScope.GALLERY_VIEW_COLUMN}:${id}`
-    );
+    // if cache is not present skip pushing it into the list to avoid unexpected behaviour
+    if (
+      (
+        await NocoCache.getList(CacheScope.GALLERY_VIEW_COLUMN, [
+          column.fk_view_id,
+        ])
+      )?.length
+    )
+      await NocoCache.appendToList(
+        CacheScope.GALLERY_VIEW_COLUMN,
+        [column.fk_view_id],
+        `${CacheScope.GALLERY_VIEW_COLUMN}:${id}`
+      );
 
     return this.get(id, ncMeta);
   }

packages/nocodb/src/lib/models/GridViewColumn.ts

@@ -93,11 +93,19 @@ export default class GridViewColumn implements GridColumnType {
 
     await NocoCache.set(`${CacheScope.GRID_VIEW_COLUMN}:${fk_column_id}`, id);
 
-    await NocoCache.appendToList(
-      CacheScope.GRID_VIEW_COLUMN,
-      [column.fk_view_id],
-      `${CacheScope.GRID_VIEW_COLUMN}:${id}`
-    );
+    // if cache is not present skip pushing it into the list to avoid unexpected behaviour
+    if (
+      (
+        await NocoCache.getList(CacheScope.GRID_VIEW_COLUMN, [
+          column.fk_view_id,
+        ])
+      )?.length
+    )
+      await NocoCache.appendToList(
+        CacheScope.GRID_VIEW_COLUMN,
+        [column.fk_view_id],
+        `${CacheScope.GRID_VIEW_COLUMN}:${id}`
+      );
 
     return this.get(id, ncMeta);
   }