Ensure only matching deps update with update --save*

Issue #4438

1 files changed, 17 insertions, 0 deletions

diff --git a/lib/outdated.js b/lib/outdated.js
index fd409bd59..c053ddeea 100644
--- a/lib/outdated.js
+++ b/lib/outdated.js
@@ -155,12 +155,29 @@ function outdated_ (args, dir, parentHas, depth, cb) {
   }
   var deps = null
   readJson(path.resolve(dir, "package.json"), function (er, d) {
+    d = d || {}
     if (er && er.code !== "ENOENT" && er.code !== "ENOTDIR") return cb(er)
     deps = (er) ? true : (d.dependencies || {})
+
+    if (npm.config.get("save-dev")) {
+      deps = d.devDependencies || {}
+      return next()
+    }
+
+    if (npm.config.get("save")) {
+      return next()
+    }
+
+    if (npm.config.get("save-optional")) {
+      deps = d.optionalDependencies || {}
+      return next()
+    }
+
     var doUpdate = npm.config.get("dev") ||
                     (!npm.config.get("production") &&
                     !Object.keys(parentHas).length &&
                     !npm.config.get("global"))
+
     if (!er && d && doUpdate) {
       Object.keys(d.devDependencies || {}).forEach(function (k) {
         if (!(k in parentHas)) {