fix: add support for database server id generation

Author: Astaldos

adminforth/dataConnectors/baseConnector.ts

@@ -187,7 +187,12 @@ export default class AdminForthBaseConnector implements IAdminForthDataSourceCon
     }
 
     process.env.HEAVY_DEBUG && console.log('🪲🆕 creating record',JSON.stringify(recordWithOriginalValues));
-    const pkValue = await this.createRecordOriginalValues({ resource, record: recordWithOriginalValues });
+    let pkValue = await this.createRecordOriginalValues({ resource, record: recordWithOriginalValues });
+    if (recordWithOriginalValues[this.getPrimaryKey(resource)] !== undefined) {
+      // some data sources always return some value for pk, even if it is was not auto generated
+      // this check prevents wrong value from being used later in get request
+      pkValue = recordWithOriginalValues[this.getPrimaryKey(resource)];
+    }
 
     let createdRecord = recordWithOriginalValues;
     if (pkValue) {

adminforth/dataConnectors/clickhouse.ts

@@ -341,13 +341,13 @@ class ClickhouseConnector extends AdminForthBaseConnector implements IAdminForth
     async createRecordOriginalValues({ resource, record }: { resource: AdminForthResource, record: any }): Promise<string> {
       const tableName = resource.table;
       const columns = Object.keys(record);
-      await this.client.insert({
+      const ret = await this.client.insert({
         database: this.dbName,
         table: tableName,
         columns: columns,
         values: [Object.values(record)],
       });
-      return ''; // todo
+      return ret.query_id;
     }
 
     async updateRecordOriginalValues({ resource, recordId, newValues }: { resource: AdminForthResource, recordId: any, newValues: any }) {

adminforth/dataConnectors/mongo.ts

@@ -186,8 +186,8 @@ class MongoConnector extends AdminForthBaseConnector implements IAdminForthDataS
         for (const colName of columns) {
             newRecord[colName] = record[colName];
         }
-        await collection.insertOne(newRecord);
-        return ''; // todo
+        const ret = await collection.insertOne(newRecord);
+        return ret.insertedId;
     }
 
     async updateRecordOriginalValues({ resource, recordId, newValues }) {

adminforth/dataConnectors/mysql.ts

@@ -287,9 +287,8 @@ class MysqlConnector extends AdminForthBaseConnector implements IAdminForthDataS
     if (process.env.HEAVY_DEBUG_QUERY) {
       console.log('🪲📜 MySQL Q:', q, 'values:', values);
     }
-    await this.client.execute(q, values);
-
-    return ''; // todo
+    const ret = await this.client.execute(q, values);
+    return ret.insertId;
   }
 
   async updateRecordOriginalValues({ resource, recordId,  newValues }) {

adminforth/dataConnectors/sqlite.ts

@@ -269,8 +269,8 @@ class SQLiteConnector extends AdminForthBaseConnector implements IAdminForthData
       const placeholders = columns.map(() => '?').join(', ');
       const values = columns.map((colName) => record[colName]);
       const q = this.client.prepare(`INSERT INTO ${tableName} (${columns.join(', ')}) VALUES (${placeholders})`);
-      await q.run(values);
-      return ''; // todo q.lastInsertRowid;
+      const ret = await q.run(values);
+      return ret.lastInsertRowid;
     }
 
     async updateRecordOriginalValues({ resource, recordId, newValues }: { resource: AdminForthResource, recordId: any, newValues: any }) {