Merge pull request #35 from wikimedia/overwrites

Support overwrites if the tid matches

Author: gwicke

lib/dbutils.js

@@ -480,7 +480,7 @@ dbu.extractGetParams = function(query, schema, includePreparedForDelete) {
 
 function buildUpdateQuery(req, tableName, schema, dataKVMap, primaryKeyKVMap, ignore) {
     var dataParams = [];
-    var condition = buildCondition(Object.assign(primaryKeyKVMap, req.if), schema, false, true);
+    var condition = buildCondition(Object.assign(primaryKeyKVMap, req.if), schema, true, true);
     var sql = (ignore ? 'update or ignore ' : 'update ') + '[' + tableName + '_data] set ';
     sql += Object.keys(dataKVMap).filter(function(column) {
         return schema.iKeys.indexOf(column) < 0;
@@ -569,7 +569,7 @@ dbu.buildPutQuery = function(req, tableName, schema, ignoreStatic) {
     } else {
         if (Object.keys(dataKVMap).length > Object.keys(primaryKeyKVMap).length) {
             queries.push(buildUpdateQuery(req, tableName, schema,
-                dataKVMap, primaryKeyKVMap, true));
+                dataKVMap, primaryKeyKVMap, false));
         }
         queries.push(buildInsertQuery(tableName, dataKVMap));
     }

package.json

@@ -1,7 +1,7 @@
 {
   "name": "restbase-mod-table-sqlite",
   "description": "RESTBase table storage using sqlite for testing purposes",
-  "version": "0.1.17",
+  "version": "0.1.18",
   "main": "index.js",
   "repository": {
     "type": "git",