[fix] Fix regression introduced by 36e061c

indexzero · Jul 10, 2012 · d96d254 · d96d254
1 parent 7e8d9d6
commit d96d254
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 4 deletions.
diff --git a/lib/nconf/provider.js b/lib/nconf/provider.js
@@ -460,15 +460,22 @@ Provider.prototype.save = function (value, callback) {
     //
 
     if (store.save) {
-      store.save(function (err, data) {
-        if (err) return next(err);
+      return store.save(function (err, data) {
+        if (err) {
+          return next(err);
+        }
+
         if (typeof data == 'object' && data !== null) {
           memo.push(data);
         }
+
+        next(null, memo);
       });
-    } else if (store.saveSync) {
+    } 
+    else if (store.saveSync) {
       memo.push(store.saveSync());
     }
+
     next(null, memo);
   }
 

diff --git a/test/provider-save-test.js b/test/provider-save-test.js
@@ -14,7 +14,7 @@ var assert = require('assert'),
 //
 require('./mocks/mock-store');
 
-vows.describe('nconf/provider').addBatch({
+vows.describe('nconf/provider/save').addBatch({
   "When using nconf": {
     "an instance of 'nconf.Provider'": {
       "with a Mock store": {