fix(Angular.js): toKeyValue is not serializing null values

Signed-off-by: Josh Kurz <jkurz25@gmail.com>
angular · Apr 3, 2014 · 0c5e1dd · 0c5e1dd
1 parent f9eb324
commit 0c5e1dd
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 9 deletions.
diff --git a/src/Angular.js b/src/Angular.js
@@ -1085,14 +1085,16 @@ function parseKeyValue(/**string*/keyValue) {
 function toKeyValue(obj) {
   var parts = [];
   forEach(obj, function(value, key) {
-    if (isArray(value)) {
-      forEach(value, function(arrayValue) {
-        parts.push(encodeUriQuery(key, true) +
-                   (arrayValue === true ? '' : '=' + encodeUriQuery(arrayValue, true)));
-      });
-    } else {
-    parts.push(encodeUriQuery(key, true) +
-               (value === true ? '' : '=' + encodeUriQuery(value, true)));
+    if(value !== null){
+      if (isArray(value)) {
+        forEach(value, function(arrayValue) {
+          parts.push(encodeUriQuery(key, true) +
+                     (arrayValue === true ? '' : '=' + encodeUriQuery(arrayValue, true)));
+        });
+      } else {
+      parts.push(encodeUriQuery(key, true) +
+                 (value === true ? '' : '=' + encodeUriQuery(value, true)));
+      }
     }
   });
   return parts.length ? parts.join('&') : '';

diff --git a/test/AngularSpec.js b/test/AngularSpec.js
@@ -457,7 +457,11 @@ describe('angular', function() {
       expect(toKeyValue({key: [323,'value',true]})).toEqual('key=323&key=value&key');
       expect(toKeyValue({key: [323,'value',true, 1234]})).
       toEqual('key=323&key=value&key&key=1234');
-  });
+    });
+
+    it('should not serialize null values', function() {
+      expect(toKeyValue({nullKey: null, key: 'value'})).toEqual('key=value');
+    });
   });