Implemented emptyTag option. If set, empty tags will not be {} but th…

…e value of the option. Closes GH Leonidas-from-XIV#20
tkpage · Aug 14, 2011 · 5f54d69 · 5f54d69
1 parent 0313fd0
commit 5f54d69
Show file tree

Hide file tree

Showing 3 changed files with 20 additions and 3 deletions.
diff --git a/lib/xml2js.js b/lib/xml2js.js
diff --git a/src/xml2js.coffee b/src/xml2js.coffee
@@ -1,6 +1,10 @@
 sax = require 'sax'
 events = require 'events'
 
+# Underscore has a nice function for this, but we try to go without dependencies
+isEmpty = (thing) ->
+  return typeof thing == 'object' && thing? && Object.keys(thing).length == 0
+
 class exports.Parser extends events.EventEmitter
   constructor: (opts) ->
 
@@ -65,6 +69,10 @@ class exports.Parser extends events.EventEmitter
         if Object.keys(obj).length == 1 and "#" of obj and not @EXPLICIT_CHARKEY
           obj = obj["#"]
 
+      if options.emptyTag != undefined && isEmpty obj
+        obj = options.emptyTag
+
+      # check whether we closed all the open tags
       if stack.length > 0
         if nodeName not of s
           s[nodeName] = obj

diff --git a/test/xml2js.test.coffee b/test/xml2js.test.coffee
@@ -71,3 +71,6 @@ module.exports =
 
   'test default empty tag result': skeleton(undefined, (r) ->
     assert.deepEqual r['emptytest'], {})
+
+  'test empty tag result specified null': skeleton({emptyTag: null}, (r) ->
+    assert.equal r['emptytest'], null)