From a85d4460b1426629391159d41945057da8e842b0 Mon Sep 17 00:00:00 2001
From: Adria Navarro <adria@budibase.com>
Date: Thu, 21 Mar 2024 14:18:45 +0100
Subject: [PATCH] Clean code

---
 packages/server/src/api/controllers/table/utils.ts | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/packages/server/src/api/controllers/table/utils.ts b/packages/server/src/api/controllers/table/utils.ts
index ac13617869..0c9933a4cf 100644
--- a/packages/server/src/api/controllers/table/utils.ts
+++ b/packages/server/src/api/controllers/table/utils.ts
@@ -31,6 +31,7 @@ import {
   RelationshipFieldMetadata,
   FieldType,
   FieldTypeSubtypes,
+  AttachmentFieldMetadata,
 } from "@budibase/types"
 
 export async function clearColumns(table: Table, columnNames: string[]) {
@@ -90,11 +91,14 @@ export async function checkForColumnUpdates(
     await checkForViewUpdates(updatedTable, deletedColumns, columnRename)
   }
 
-  for (const attachmentColumn of Object.values(updatedTable.schema).filter(
-    column =>
+  const changedAttachmentSubtypeColumns = Object.values(
+    updatedTable.schema
+  ).filter(
+    (column): column is AttachmentFieldMetadata =>
       column.type === FieldType.ATTACHMENT &&
       column.subtype !== oldTable?.schema[column.name]?.subtype
-  )) {
+  )
+  for (const attachmentColumn of changedAttachmentSubtypeColumns) {
     if (attachmentColumn.subtype === FieldTypeSubtypes.ATTACHMENT.SINGLE) {
       attachmentColumn.constraints ??= { length: {} }
       attachmentColumn.constraints.length ??= {}