(#46) - Add the id on the return of the updated:false response

…to align with the docs and the `updated:true` flow. fixes #46
pouchdb · Mar 21, 2017 · ff2a6e6 · ff2a6e6
1 parent a5db122
commit ff2a6e6
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 3 deletions.
diff --git a/dist/pouchdb.upsert.js b/dist/pouchdb.upsert.js
@@ -25,7 +25,7 @@ function upsertInner(db, docId, diffFun) {
     if (!newDoc) {
       // if the diffFun returns falsy, we short-circuit as
       // an optimization
-      return { updated: false, rev: docRev };
+      return { updated: false, rev: docRev, id: docId };
     }
 
     // users aren't allowed to modify these values,

diff --git a/dist/pouchdb.upsert.min.js b/dist/pouchdb.upsert.min.js
diff --git a/index.js b/index.js
@@ -24,7 +24,7 @@ function upsertInner(db, docId, diffFun) {
     if (!newDoc) {
       // if the diffFun returns falsy, we short-circuit as
       // an optimization
-      return { updated: false, rev: docRev };
+      return { updated: false, rev: docRev, id: docId };
     }
 
     // users aren't allowed to modify these values,

diff --git a/test/test.js b/test/test.js
@@ -189,6 +189,7 @@ function tests(dbName, dbType) {
       }).then(function (res) {
         res.updated.should.equal(false);
         res.rev.should.match(/1-/);
+        res.id.should.equal('myid');
         return db.get('myid');
       }).then(function (doc) {
         should.exist(doc._rev);