parse html entities in text values, fixes #40

Author: Delagen

bin/parser.js

@@ -1,3 +1,4 @@
+var he = require("he");
 var getAllMatches = require("./util").getAllMatches;
 
 var xmlNode = function(tagname,parent,val){
@@ -129,10 +130,10 @@ function resolveNameSpace(tagname,ignore){
     return tagname;
 }
 
-function parseValue(val,conversion){
+function parseValue(val,conversion,isAttribute){
     if(val){
         if(!conversion || isNaN(val)){
-            val = "" + val ;
+            val = "" + he.decode(val, {isAttributeValue:isAttribute, strict:true});
         }else{
             if(val.indexOf(".") !== -1){
                 if(parseFloat){
@@ -165,7 +166,7 @@ function buildAttributesArr(attrStr,ignore,prefix,attrNodeName,ignoreNS,conversi
         for (var i = 0; i < matches.length; i++) {
             var attrName = resolveNameSpace(matches[i][1],ignoreNS);
             if(attrName.length && attrName !== "xmlns") {
-                attrs[prefix + attrName] = parseValue(matches[i][3], conversion);
+                attrs[prefix + attrName] = parseValue(matches[i][3], conversion, true);
             }
         }
         if(!Object.keys(attrs).length){

lib/parser.js

@@ -5,7 +5,7 @@
   "main": "./bin/parser.js",
   "scripts": {
     "test": "jasmine spec/*spec.js",
-    "bundle": "browserify bin/parser.js --s parser > lib/parser.js",
+    "bundle": "browserify bin/parser.js -s parser > lib/parser.js",
     "coverage": "node ./benchmark/perfTest.js; istanbul cover -x \"spec/*spec.js\" jasmine spec/*spec.js;",
     "coverage:check": "istanbul check-coverage --branch 90 --statement 90"
   },
@@ -66,5 +66,8 @@
     "jasmine-core": "^2.8.0",
     "portfinder": "^1.0.13",
     "zombie": "^5.0.7"
+  },
+  "dependencies": {
+    "he": "~1.1.1"
   }
 }

package.json

@@ -33,13 +33,14 @@ describe("XMLParser", function () {
     });
 
     it("should parse all values of attributes as string", function () {
-        var xmlData = "<rootNode><tag int='045' float='65.34'>value</tag></rootNode>";
+        var xmlData = "<rootNode><tag int='045' float='65.34' text='foo&ampbar&apos;'>value&amp;\r\n&apos;</tag></rootNode>";
         var expected = {
             "rootNode": {
                 "tag": {
-                    "#text": "value",
+                    "#text": "value&\r\n'",
                     "@_int": "045",
-                    "@_float": "65.34"
+                    "@_float": "65.34",
+                    "@_text": "foo&ampbar'"
                 }
             }
         };