Stripping BOM if at the beginning of file

Closes #99
Leonidas-from-XIV · Sep 4, 2013 · 6e83733 · 6e83733
1 parent 36f2c5b
commit 6e83733
Show file tree

Hide file tree

Showing 5 changed files with 40 additions and 3 deletions.
diff --git a/lib/bom.js b/lib/bom.js
diff --git a/lib/xml2js.js b/lib/xml2js.js
diff --git a/src/bom.coffee b/src/bom.coffee
@@ -0,0 +1,8 @@
+xml2js = require '../lib/xml2js'
+
+exports.stripBOM = (str) ->
+  if str[0] == '\uFEFF'
+    str.substring(1)
+  else
+    str
+
diff --git a/src/xml2js.coffee b/src/xml2js.coffee
@@ -1,5 +1,6 @@
 sax = require 'sax'
 events = require 'events'
+bom = require './bom'
 
 # Underscore has a nice function for this, but we try to go without dependencies
 isEmpty = (thing) ->
@@ -232,7 +233,7 @@ class exports.Parser extends events.EventEmitter
       @emit "end", null
       return true
 
-    @saxParser.write str.toString()
+    @saxParser.write bom.stripBOM str.toString()
 
 exports.parseString = (str, a, b) ->
   # let's determine what we got as arguments

diff --git a/test/bom.test.coffee b/test/bom.test.coffee
@@ -0,0 +1,11 @@
+xml2js = require '../lib/xml2js'
+assert = require 'assert'
+equ = assert.equal
+
+module.exports =
+  'test decoded BOM': (test) ->
+    demo = '\uFEFF<xml><foo>bar</foo></xml>'
+    xml2js.parseString demo, (err, res) ->
+      equ err, undefined
+      equ res.xml.foo[0], 'bar'
+      test.done()