Fixed issue with insertId being returned for models with no auto-incrementing field. See http://stackoverflow.com/questions/22673127/returned-objects-id-is-zero-creating-a-record-in-database

sgress454 · sgress454 · commit 45a32e152567 · 2014-03-26T19:01:09.000-05:00
diff --git a/lib/adapter.js b/lib/adapter.js
@@ -300,15 +300,20 @@ module.exports = (function() {
           if (err) return cb( handleQueryError(err) );
 
           // Build model to return
-          var autoInc = 'id';
+          var autoInc = null;
 
           Object.keys(collection.definition).forEach(function(key) {
             if(!collection.definition[key].hasOwnProperty('autoIncrement')) return;
             autoInc = key;
           });
-
+            
           var autoIncData = {};
-          autoIncData[autoInc] = result.insertId;
+          
+          if (autoInc) {
+            
+            autoIncData[autoInc] = result.insertId;
+
+          }
 
           var model = _.extend({}, data, autoIncData);
 
