fix DOM insertion operators for null values

Closes madrobby#670
buddydvd · Apr 24, 2013 · c3a3aa9 · c3a3aa9
1 parent 7ea25f5
commit c3a3aa9
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/src/zepto.js b/src/zepto.js
@@ -733,7 +733,8 @@ var Zepto = (function() {
       // arguments can be nodes, arrays of nodes, Zepto objects and HTML strings
       var argType, nodes = $.map(arguments, function(arg) {
             argType = type(arg)
-            return argType == "object" || argType == "array" ? arg : zepto.fragment(arg)
+            return argType == "object" || argType == "array" || arg == null ?
+              arg : zepto.fragment(arg)
           }),
           parent, copyByClone = this.length > 1
       if (nodes.length < 1) return this

diff --git a/test/zepto.html b/test/zepto.html
@@ -1785,6 +1785,11 @@ <h1>Zepto DOM unit tests</h1>
         t.assertEqual('<span>Test</span>,<span>Test</span>', spans.join(','))
       },
 
+      testAppendNull: function(t){
+        var el = $(document.body)
+        t.assertIdentical(el, el.append(null))
+      },
+
       testBeforeAfterFragment: function(t){
         var fragment = $('<div class=fragment />')
         fragment.before('before').after('after')