fixed adding column

packages/tools/src/structureTools.ts

@@ -1,4 +1,4 @@
-import { DatabaseInfo } from 'dbgate-types';
+import { DatabaseInfo, TableInfo } from 'dbgate-types';
 import _flatten from 'lodash/flatten';
 
 export function addTableDependencies(db: DatabaseInfo): DatabaseInfo {
@@ -12,50 +12,54 @@ export function addTableDependencies(db: DatabaseInfo): DatabaseInfo {
   };
 }
 
-function fillTableExtendedInfo(db: DatabaseInfo): DatabaseInfo {
+export function extendTableInfo(table: TableInfo): TableInfo {
   return {
-    ...db,
+    ...table,
-    tables: (db.tables || []).map(obj => ({
-      ...obj,
     objectTypeField: 'tables',
-      columns: (obj.columns || []).map(column => ({
+    columns: (table.columns || []).map(column => ({
-        pureName: obj.pureName,
+      pureName: table.pureName,
-        schemaName: obj.schemaName,
+      schemaName: table.schemaName,
       ...column,
     })),
-      primaryKey: obj.primaryKey
+    primaryKey: table.primaryKey
       ? {
-            ...obj.primaryKey,
+          ...table.primaryKey,
-            pureName: obj.pureName,
+          pureName: table.pureName,
-            schemaName: obj.schemaName,
+          schemaName: table.schemaName,
           constraintType: 'primaryKey',
         }
       : undefined,
-      foreignKeys: (obj.foreignKeys || []).map(cnt => ({
+    foreignKeys: (table.foreignKeys || []).map(cnt => ({
       ...cnt,
-        pureName: obj.pureName,
+      pureName: table.pureName,
-        schemaName: obj.schemaName,
+      schemaName: table.schemaName,
       constraintType: 'foreignKey',
     })),
-      indexes: (obj.indexes || []).map(cnt => ({
+    indexes: (table.indexes || []).map(cnt => ({
       ...cnt,
-        pureName: obj.pureName,
+      pureName: table.pureName,
-        schemaName: obj.schemaName,
+      schemaName: table.schemaName,
       constraintType: 'index',
     })),
-      checks: (obj.checks || []).map(cnt => ({
+    checks: (table.checks || []).map(cnt => ({
       ...cnt,
-        pureName: obj.pureName,
+      pureName: table.pureName,
-        schemaName: obj.schemaName,
+      schemaName: table.schemaName,
       constraintType: 'check',
     })),
-      uniques: (obj.uniques || []).map(cnt => ({
+    uniques: (table.uniques || []).map(cnt => ({
       ...cnt,
-        pureName: obj.pureName,
+      pureName: table.pureName,
-        schemaName: obj.schemaName,
+      schemaName: table.schemaName,
       constraintType: 'unique',
     })),
-    })),
+  };
+}
+
+function fillDatabaseExtendedInfo(db: DatabaseInfo): DatabaseInfo {
+  return {
+    ...db,
+    tables: (db.tables || []).map(extendTableInfo),
     collections: (db.collections || []).map(obj => ({
       ...obj,
       objectTypeField: 'collections',
@@ -84,5 +88,5 @@ function fillTableExtendedInfo(db: DatabaseInfo): DatabaseInfo {
 }
 
 export function extendDatabaseInfo(db: DatabaseInfo): DatabaseInfo {
-  return fillTableExtendedInfo(addTableDependencies(db));
+  return fillDatabaseExtendedInfo(addTableDependencies(db));
 }

packages/types/dbinfo.d.ts

@@ -62,12 +62,12 @@ export interface DatabaseObjectInfo extends NamedObjectInfo {
   createDate?: string;
   modifyDate?: string;
   hashCode?: string;
+  objectTypeField?: string;
 }
 
 export interface SqlObjectInfo extends DatabaseObjectInfo {
   createSql?: string;
   requiresFormat?: boolean; // SQL is human unreadable, requires formatting (eg. MySQL views)
-  objectTypeField?: string;
 }
 
 export interface TableInfo extends DatabaseObjectInfo {

packages/web/src/tabs/TableStructureTab.svelte

@@ -28,7 +28,13 @@
 </script>
 
 <script lang="ts">
-  import { fillConstraintNames, findEngineDriver, generateTablePairingId, getAlterTableScript } from 'dbgate-tools';
+  import {
+    fillConstraintNames,
+    extendTableInfo,
+    findEngineDriver,
+    generateTablePairingId,
+    getAlterTableScript,
+  } from 'dbgate-tools';
 
   import _ from 'lodash';
   import registerCommand from '../commands/registerCommand';
@@ -38,7 +44,6 @@
   import ForeignKeyObjectListControl from '../elements/ForeignKeyObjectListControl.svelte';
 
   import { extensions } from '../stores';
-  import ObjectListControl from '../elements/ObjectListControl.svelte';
   import useEditorData from '../query/useEditorData';
   import TableEditor from '../tableeditor/TableEditor.svelte';
   import createActivator, { getActiveComponent } from '../utility/createActivator';
@@ -102,9 +107,10 @@
 
   function doSave(createTableName) {
     const driver = findEngineDriver($connection, $extensions);
+
     const { sql, recreates } = getAlterTableScript(
       $editorValue.base,
-      fillConstraintNames($editorValue.current, driver.dialect),
+      extendTableInfo(fillConstraintNames($editorValue.current, driver.dialect)),
       {},
       $dbInfo,
       driver