[fix] Do not change lexigraphic ordering of keys in this.stores.

indexzero · Aug 16, 2017 · bb07f1d · bb07f1d
1 parent 4d9364d
commit bb07f1d
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/lib/nconf/provider.js b/lib/nconf/provider.js
@@ -102,9 +102,11 @@ Provider.prototype.use = function (name, options) {
   var store = this.stores[name],
       update = store && !sameOptions(store);
 
-  if (!store) {
-    this.add(name, options);
-  } else if (store && update) {
+  if (!store || update) {
+    if (update) {
+      this.remove(name);
+    }
+
     this.add(name, options);
   }
 
@@ -146,7 +148,7 @@ Provider.prototype.add = function (name, options, usage) {
 // this was used in the call to `.add()`.
 //
 Provider.prototype.remove = function (name) {
-  delete this.stores[name];
+  this.stores[name] = null;
   return this;
 };