Object builder preserves items with null values. Closes #44

Author: oozcitak

src/XMLNode.coffee

@@ -40,9 +40,9 @@ module.exports = class XMLNode
     else if _.isFunction name
       lastChild = @element name.apply()
 
-    # expand if object but skip null values
+    # expand if object
     else if _.isObject name
-      for own key, val of name when val?
+      for own key, val of name
          # evaluate if function
         val = val.apply() if _.isFunction val
 

test/objectconsistency.coffee

@@ -0,0 +1,75 @@
+vows = require 'vows'
+assert = require 'assert'
+
+xmlbuilder = require '../src/index.coffee'
+
+vows
+    .describe('Creating XML')
+    .addBatch
+        'Object builder consistency':
+            topic: () ->
+                obj =
+                    ele: "simple element"
+                    person:
+                        name: "John"
+                        '@age': 35
+                        '?pi': 'mypi'
+                        '#comment': 'Good guy'
+                        '#cdata': 'well formed!'
+                        unescaped:
+                          '#raw': '&<>&'
+                        address:
+                            city: "Istanbul"
+                            street: "End of long and winding road"
+                        contact: [
+                            { phone: "555-1234" }
+                            { phone: "555-1235" }
+                        ]
+                        id: () -> return 42
+                        details:
+                          '#text': 'classified'
+                        nullval: null
+    
+                t1 = xmlbuilder.create('root', { headless: true })
+                    .ele(obj)
+
+                t2 = xmlbuilder.create('root', { headless: true })
+                    .ele('ele', 'simple element')
+                      .up()
+                    .ele('person')
+                      .ele('name', 'John')
+                        .up()
+                    .att('age', 35)
+                    .ins('pi', 'mypi')
+                    .com('Good guy')
+                    .dat('well formed!')
+                    .ele('unescaped')
+                      .raw('&<>&')
+                        .up()
+                    .ele('address')
+                      .ele('city', 'Istanbul')
+                        .up()
+                      .ele('street', 'End of long and winding road')
+                        .up()
+                      .up()
+                    .ele('contact')
+                      .ele('phone', '555-1234')
+                        .up()
+                      .ele('phone', '555-1235')
+                        .up()
+                      .up()
+                    .ele('id', 42)
+                      .up()
+                    .ele('details')
+                      .text('classified')
+                      .up()
+                    .ele('nullval', null)
+
+                return [t1, t2]
+
+            'resulting XML': (topic) ->
+                assert.strictEqual topic[0].end(), topic[1].end()
+
+
+    .export(module)
+