From 1b98bf706edee3ca727e757d022065e48955dd13 Mon Sep 17 00:00:00 2001
From: Corey Johnson & Nathan Sobo <cj+nathan@github.com>
Date: Thu, 25 Apr 2013 17:52:29 -0700
Subject: [PATCH] Setting value to null/undefined removes key from config

---
 spec/app/config-spec.coffee             | 5 +++++
 src/stdlib/underscore-extensions.coffee | 5 ++++-
 2 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/spec/app/config-spec.coffee b/spec/app/config-spec.coffee
index f0393d0af..7bfe22e2f 100644
--- a/spec/app/config-spec.coffee
+++ b/spec/app/config-spec.coffee
@@ -100,6 +100,11 @@ describe "Config", ->
       config.set('foo.bar.baz', "value 1")
       expect(observeHandler).toHaveBeenCalledWith("value 1")
 
+    it "fires the callback when the observed value is deleted", ->
+      observeHandler.reset() # clear the initial call
+      config.set('foo.bar.baz', undefined)
+      expect(observeHandler).toHaveBeenCalledWith(undefined)
+
     it "fires the callback when the full key path goes into and out of existence", ->
       observeHandler.reset() # clear the initial call
       config.set("foo.bar", undefined)
diff --git a/src/stdlib/underscore-extensions.coffee b/src/stdlib/underscore-extensions.coffee
index defdb576f..d89bc99b7 100644
--- a/src/stdlib/underscore-extensions.coffee
+++ b/src/stdlib/underscore-extensions.coffee
@@ -139,7 +139,10 @@ _.mixin
       key = keys.shift()
       object[key] ?= {}
       object = object[key]
-    object[keys.shift()] = value
+    if value?
+      object[keys.shift()] = value
+    else
+      delete object[keys.shift()]
 
   compactObject: (object) ->
     newObject = {}