From eb09318fadec48082b880ba7c881c552ed42015c Mon Sep 17 00:00:00 2001
From: Lyubov Voloshko <lyubov@voloshko.com>
Date: Mon, 28 Jul 2025 11:59:31 +0300
Subject: [PATCH] add/edit record: add checking that primary key with
 column_default is non modifing key

---
 .../db-table-row-edit.component.html              |  2 +-
 .../db-table-row-edit.component.ts                | 15 +++++++++++----
 2 files changed, 12 insertions(+), 5 deletions(-)

diff --git a/frontend/src/app/components/db-table-row-edit/db-table-row-edit.component.html b/frontend/src/app/components/db-table-row-edit/db-table-row-edit.component.html
index 1123b6281..21debaee0 100644
--- a/frontend/src/app/components/db-table-row-edit/db-table-row-edit.component.html
+++ b/frontend/src/app/components/db-table-row-edit/db-table-row-edit.component.html
@@ -170,7 +170,7 @@ <h3>
                         Back
                     </a>
                     <button type="button" mat-button color="primary"
-                        *ngIf="(keyAttributesFromStructure.length || hasKeyAttributesFromURL) && permissions.edit"
+                        *ngIf="(keyAttributesListFromStructure.length || hasKeyAttributesFromURL) && permissions.edit"
                         class="actions__continue"
                         data-testid="record-save-and-continue-editing-button"
                         [disabled]="submitting || editRowForm.form.invalid"
diff --git a/frontend/src/app/components/db-table-row-edit/db-table-row-edit.component.ts b/frontend/src/app/components/db-table-row-edit/db-table-row-edit.component.ts
index 488e2d4a1..be6396baa 100644
--- a/frontend/src/app/components/db-table-row-edit/db-table-row-edit.component.ts
+++ b/frontend/src/app/components/db-table-row-edit/db-table-row-edit.component.ts
@@ -86,7 +86,7 @@ export class DbTableRowEditComponent implements OnInit {
   public keyAttributesFromURL: object = {};
   public hasKeyAttributesFromURL: boolean;
   public dubURLParams: object;
-  public keyAttributesFromStructure: [] = [];
+  public keyAttributesListFromStructure: string[] = [];
   public isPrimaryKeyUpdated: boolean;
   public tableTypes: object;
   public tableWidgets: object;
@@ -174,7 +174,7 @@ export class DbTableRowEditComponent implements OnInit {
               edit: true
             };
 
-            this.keyAttributesFromStructure = res.primaryColumns;
+            this.keyAttributesListFromStructure = res.primaryColumns.map((field: TableField) => field.column_name);
             this.readonlyFields = res.readonly_fields;
             this.tableForeignKeys = res.foreignKeys;
             this.setRowStructure(res.structure);
@@ -214,8 +214,11 @@ export class DbTableRowEditComponent implements OnInit {
             this.dispalyTableName = res.display_name || normalizeTableName(this.tableName);
             this.title.setTitle(`${this.dispalyTableName} - Edit record | Rocketadmin`);
             this.permissions = res.table_access_level;
+            this.keyAttributesListFromStructure = res.primaryColumns.map((field: TableField) => field.column_name);
 
-            this.nonModifyingFields = res.structure.filter((field: TableField) => !this.getModifyingFields(res.structure).some(modifyingField => field.column_name === modifyingField.column_name)).map((field: TableField) => field.column_name);
+            this.nonModifyingFields = res.structure
+              .filter((field: TableField) => !this.getModifyingFields(res.structure).some(modifyingField => field.column_name === modifyingField.column_name))
+              .map((field: TableField) => field.column_name);
             this.readonlyFields = [...res.readonly_fields, ...this.nonModifyingFields];
             if (this.connectionType === DBtype.Dynamo) {
               this.readonlyFields = [...this.readonlyFields, ...res.primaryColumns.map((field: TableField) => field.column_name)];
@@ -521,7 +524,11 @@ export class DbTableRowEditComponent implements OnInit {
   }
 
   getModifyingFields(fields) {
-    return fields.filter((field: TableField) => !field.auto_increment && !(timestampTypes.includes(field.data_type) && field.column_default && defaultTimestampValues[this.connectionType].includes(field.column_default.toLowerCase().replace(/\(.*\)/, ""))));
+    return fields.filter((field: TableField) =>
+      !field.auto_increment &&
+      !(this.keyAttributesListFromStructure.includes(field.column_name) && field.column_default) &&
+      !(timestampTypes.includes(field.data_type) && field.column_default && defaultTimestampValues[this.connectionType].includes(field.column_default.toLowerCase().replace(/\(.*\)/, "")))
+    );
   }
 
   updateField = (updatedValue: any, field: string) => {