fixed PR comments

packages/nocodb/src/db/BaseModelSqlv2.ts

@@ -2178,20 +2178,6 @@ class BaseModelSqlv2 {
 
   async insert(data, trx?, cookie?, _disableOptimization = false) {
     try {
-      await populatePk(this.model, data);
-
-      // todo: filter based on view
-      const insertObj = await this.model.mapAliasToColumn(
-        data,
-        this.clientMeta,
-        this.dbDriver,
-      );
-
-      await this.validate(insertObj);
-
-      if ('beforeInsert' in this) {
-        await this.beforeInsert(insertObj, trx, cookie);
-      }
 
       const columns = await this.model.getColumns();
 
@@ -2204,9 +2190,24 @@ class BaseModelSqlv2 {
           if (data[keyName]) {
             delete data[keyName];
           }
+
         }
       }
 
+      await populatePk(this.model, data);
+
+      // todo: filter based on view
+      const insertObj = await this.model.mapAliasToColumn(
+        data,
+        this.clientMeta,
+        this.dbDriver,
+      );
+
+      await this.validate(insertObj);
+
+      if ('beforeInsert' in this) {
+        await this.beforeInsert(insertObj, trx, cookie);
+      }
       await this.prepareAttachmentData(insertObj);
 
       let response;