fixes #178 ensure package.json cache is expired immediately when new …

…version is published to npm
alanshaw · Feb 26, 2016 · 973e2ff · 973e2ff
1 parent 9cb2156
commit 973e2ff
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/lib/manifest.js b/lib/manifest.js
@@ -38,9 +38,7 @@ exports.getManifest = function (user, repo, path, ref, authToken, cb) {
     manifestKey += "/" + path
   }
 
-  if (ref) {
-    manifestKey += "/#" + ref
-  }
+  manifestKey += "/#" + (ref || '')
 
   db.get(manifestKey, function (er, manifest) {
     if (er && !er.notFound) return cb(er)
@@ -206,7 +204,7 @@ registry.on("change", function (change) {
     var keys = batch.map(function (b) { return b.key })
 
     db.batch(batch, function (er) {
-      if (er) console.error("Failed to expire cached manifest", keys, er)
+      if (er) return console.error("Failed to expire cached manifest", keys, er)
       console.log("Expired cached manifest", keys)
     })
   })