Regression in dev/optional --save stuff

chrisdew · Apr 18, 2012 · e5d7817 · e5d7817
1 parent de80b21
commit e5d7817
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/lib/install.js b/lib/install.js
@@ -260,14 +260,18 @@ function save (where, installed, tree, pretty, cb) {
     } catch (ex) {
       er = ex
     }
-    if (er) return cb(null, installed, tree, pretty)
+    if (er) {
+      return cb(null, installed, tree, pretty)
+
+    }
 
     var deps = npm.config.get("save-optional") ? "optionalDependencies"
              : npm.config.get("save-dev") ? "devDependencies"
              : "dependencies"
 
+    data[deps] = data[deps] || {}
     Object.keys(things).forEach(function (t) {
-      deps[t] = things[t]
+      data[deps][t] = things[t]
     })
     data = JSON.stringify(data, null, 2) + "\n"
     fs.writeFile(saveTarget, data, function (er) {