@naturalcycles/mysql-lib 3.13.0 → 3.14.0

package/dist/mysql.db.js

@@ -190,32 +190,31 @@ class MysqlDB extends db_lib_1.BaseCommonDB {
             }
             return v;
         }));
-        const verb = opt.saveMethod === 'insert' ? 'INSERT' : 'REPLACE';
-        if (opt.assignGeneratedIds && opt.saveMethod !== 'update') {
+        if (opt.assignGeneratedIds) {
             // Insert rows one-by-one, to get their auto-generated id
             let i = -1;
             for await (const row of rows) {
                 i++;
-                const sql = (0, query_util_1.insertSQL)(table, [row], verb, this.cfg.logger)[0];
-                const { insertId } = await this.runSQL({ sql });
-                // Mutate the input row with insertIt
-                rowsInput[i].id = insertId;
+                if (row.id) {
+                    // Update already existing
+                    const query = new db_lib_1.DBQuery(table).filterEq('id', row.id);
+                    await this.updateByQuery(query, (0, js_lib_1._omit)(row, ['id']));
+                }
+                else {
+                    // Create new
+                    const sql = (0, query_util_1.insertSQL)(table, [row], 'INSERT', this.cfg.logger)[0];
+                    const { insertId } = await this.runSQL({ sql });
+                    // Mutate the input row with insertIt
+                    rowsInput[i].id = insertId;
+                }
             }
             return;
         }
+        const verb = opt.saveMethod === 'insert' ? 'INSERT' : 'REPLACE';
         // inserts are split into multiple sentenses to respect the max_packet_size (1Mb usually)
-        if (opt.saveMethod === 'update') {
-            for await (const row of rows) {
-                (0, js_lib_1._assert)(row.id, 'Cannot update without providing an id');
-                const query = new db_lib_1.DBQuery(table).filterEq('id', row.id);
-                await this.updateByQuery(query, (0, js_lib_1._omit)(row, ['id']));
-            }
-        }
-        else {
-            const sqls = (0, query_util_1.insertSQL)(table, rows, verb, this.cfg.logger);
-            for await (const sql of sqls) {
-                await this.runSQL({ sql });
-            }
+        const sqls = (0, query_util_1.insertSQL)(table, rows, verb, this.cfg.logger);
+        for await (const sql of sqls) {
+            await this.runSQL({ sql });
         }
     }
     // DELETE

package/package.json

@@ -36,7 +36,7 @@
   "engines": {
     "node": ">=16.15.0"
   },
-  "version": "3.13.0",
+  "version": "3.14.0",
   "description": "MySQL client implementing CommonDB interface",
   "keywords": [
     "mysql",

package/src/mysql.db.ts

@@ -306,37 +306,36 @@ export class MysqlDB extends BaseCommonDB implements CommonDB {
       }),
     )
 
-    const verb = opt.saveMethod === 'insert' ? 'INSERT' : 'REPLACE'
-
-    if (opt.assignGeneratedIds && opt.saveMethod !== 'update') {
+    if (opt.assignGeneratedIds) {
       // Insert rows one-by-one, to get their auto-generated id
 
       let i = -1
       for await (const row of rows) {
         i++
-        const sql = insertSQL(table, [row], verb, this.cfg.logger)[0]!
-        const { insertId } = await this.runSQL<OkPacket>({ sql })
-
-        // Mutate the input row with insertIt
-        rowsInput[i]!.id = insertId
+        if (row.id) {
+          // Update already existing
+          const query = new DBQuery(table).filterEq('id', row.id)
+          await this.updateByQuery(query, _omit(row, ['id']))
+        } else {
+          // Create new
+          const sql = insertSQL(table, [row], 'INSERT', this.cfg.logger)[0]!
+          const { insertId } = await this.runSQL<OkPacket>({ sql })
+
+          // Mutate the input row with insertIt
+          rowsInput[i]!.id = insertId
+        }
       }
 
       return
     }
 
+    const verb = opt.saveMethod === 'insert' ? 'INSERT' : 'REPLACE'
+
     // inserts are split into multiple sentenses to respect the max_packet_size (1Mb usually)
-    if (opt.saveMethod === 'update') {
-      for await (const row of rows) {
-        _assert(row.id, 'Cannot update without providing an id')
-        const query = new DBQuery(table).filterEq('id', row.id)
-        await this.updateByQuery(query, _omit(row, ['id']))
-      }
-    } else {
-      const sqls = insertSQL(table, rows, verb, this.cfg.logger)
+    const sqls = insertSQL(table, rows, verb, this.cfg.logger)
 
-      for await (const sql of sqls) {
-        await this.runSQL({ sql })
-      }
+    for await (const sql of sqls) {
+      await this.runSQL({ sql })
     }
   }