From 3abb400ba85b9d5d586f99a8f85f1af53e0da1a5 Mon Sep 17 00:00:00 2001
From: Diego Imbert <diego@windmill.dev>
Date: Fri, 27 Mar 2026 17:27:33 +0100
Subject: [PATCH] Revert "refactor: reuse columnDefToTableEditorValuesColumn
 for default value handling"

This reverts commit bd8f071d9f224215672f25d1187bfec05ac63d2f.
---
 .../lib/components/DatatableSchemaDiff.svelte | 28 ++++++++-----------
 1 file changed, 12 insertions(+), 16 deletions(-)

diff --git a/frontend/src/lib/components/DatatableSchemaDiff.svelte b/frontend/src/lib/components/DatatableSchemaDiff.svelte
index fc8a9e855e..1ab67330de 100644
--- a/frontend/src/lib/components/DatatableSchemaDiff.svelte
+++ b/frontend/src/lib/components/DatatableSchemaDiff.svelte
@@ -1,11 +1,9 @@
 <script module lang="ts">
-	import {
-		type TableEditorValues,
-		type TableEditorValuesColumn,
-		type TableEditorForeignKey,
-		columnDefToTableEditorValuesColumn
+	import type {
+		TableEditorValues,
+		TableEditorValuesColumn,
+		TableEditorForeignKey
 	} from '$lib/components/apps/components/display/dbtable/tableEditor'
-	import type { ColumnMetadata } from '$lib/components/apps/components/display/dbtable/utils'
 	import {
 		diffTableEditorValues,
 		type AlterTableValues,
@@ -25,16 +23,14 @@
 				if (!table || typeof table !== 'object') continue
 				result[schemaName][tableName] = {
 					name: table.name ?? tableName,
-					columns: (table.columns ?? []).map((c: any) =>
-						columnDefToTableEditorValuesColumn({
-							field: c.name,
-							datatype: c.datatype ?? '',
-							defaultvalue: c.default_value ?? c.defaultValue ?? '',
-							isprimarykey: c.primary_key ?? c.primaryKey ?? false,
-							isidentity: 'No',
-							isnullable: c.nullable === false ? 'NO' : 'YES',
-							isenum: false
-						} as ColumnMetadata)
+					columns: (table.columns ?? []).map(
+						(c: any): TableEditorValuesColumn => ({
+							name: c.name,
+							datatype: c.datatype,
+							primaryKey: c.primary_key ?? c.primaryKey,
+							defaultValue: c.default_value ?? c.defaultValue,
+							nullable: c.nullable
+						})
 					),
 					foreignKeys: (table.foreign_keys ?? table.foreignKeys ?? []).map(
 						(fk: any): TableEditorForeignKey => ({