Merge pull request #17 from SaltwaterC/master

Emits an error even for the sax event onerror
Leonidas-from-XIV · Jun 23, 2011 · c0e8a01 · c0e8a01
2 parents 726397c + a48e4a4
commit c0e8a01
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 1 deletion.
diff --git a/lib/xml2js.js b/lib/xml2js.js
diff --git a/src/xml2js.coffee b/src/xml2js.coffee
@@ -20,6 +20,15 @@ class exports.Parser extends events.EventEmitter
       trim: false,
       normalize: false
     }
+
+    # emit one error event if the sax parser fails. this is mostly a hack, but
+    # the sax parser isn't state of the art either.
+    err = false
+    @saxParser.onerror = (error) =>
+      if ! err
+        err = true
+        @emit "error", error
+
     # always use the '#' key, even if there are no subkeys
     # setting this property by and is deprecated, yet still supported.
     # better pass it as explicitCharkey option to the constructor