Merge pull request balderdashy#60 from Hiro-Nakamura/master

particlebanana · particlebanana · commit 2ddaa8964d75 · 2013-11-21T12:27:43.000-08:00
issue #59: verify result is valid before trying to use it.
diff --git a/lib/adapter.js b/lib/adapter.js
@@ -506,7 +506,8 @@ module.exports = (function() {
             return cb(null, resultArray);
           }
 
-          resultArray.push(result.insertId);
+          // issue #59: verify result.insertId is valid
+          if ((result) && (result.insertId))  resultArray.push(result.insertId);
           cb(err, resultArray);
         });
       }, dbs[collectionName].config, cb);

Original file line number	Diff line number	Diff line change
`@@ -506,7 +506,8 @@ module.exports = (function() {`
`506`	`506`	`return cb(null, resultArray);`
`507`	`507`	`}`
`508`	`508`
`509`		`- resultArray.push(result.insertId);`
	`509`	`+ // issue #59: verify result.insertId is valid`
	`510`	`+ if ((result) && (result.insertId)) resultArray.push(result.insertId);`
`510`	`511`	`cb(err, resultArray);`
`511`	`512`	`});`
`512`	`513`	`}, dbs[collectionName].config, cb);`