workaround for issue #1

Author: c4milo

lib/index.js

@@ -1,13 +1,12 @@
 var expat = require('node-expat');
 var fs = require('fs');
-var parser = new expat.Parser('UTF-8');
 
 // This object will hold the final result.
-var obj = {};
-var currentObject = obj;
+var obj = currentObject = {};
 var ancestors = [];
 
 var options = {}; //configuration options
+
 function startElement(name, attrs) {
     if (! (name in currentObject)) {
         currentObject[name] = attrs;
@@ -66,26 +65,35 @@ function endElement(name) {
     currentObject = ancestor;
 }
 
-parser.on('startElement', startElement);
-parser.on('text', text);
-parser.on('endElement', endElement);
 
 module.exports.toJson = function(xml, _options) {
-    options = null;
+    var parser = new expat.Parser('UTF-8');
+
+    parser.on('startElement', startElement);
+    parser.on('text', text);
+    parser.on('endElement', endElement);
+
+    obj = currentObject = {};
+    ancestors = [];
+    
     options = {
         object: false,
         reversible: false
-    }
+    };
 
     for (var opt in _options) {
         options[opt] = _options[opt];
     }
 
-    if (parser.parse(xml)) {
-        if (options.object) {
-            return obj;
-        }
-        return JSON.stringify(obj);
+    if (!parser.parse(xml)) {
+        console.log('-->'+ xml + '<--');
+        throw new Error('There are errors in your xml file: ' + parser.getError()); 
+    }
+
+    if (options.object) {
+        return obj;
     }
+        
+    return JSON.stringify(obj);
 };
 

test/test.js

@@ -15,6 +15,8 @@ fs.readdir(fixturesPath, function(err, files) {
 
             var data = fs.readFileSync(fixturesPath + '/' + file);
             var result = parser.toJson(data);
+            var  data2 =  fs.readFileSync(fixturesPath + '/' + file);
+            result = parser.toJson(data2);
 
             var jsonFile = basename + '.json'
             var expected = fs.readFileSync(fixturesPath + '/' + jsonFile) + '';