fast-xml-parser 4.0.5 → 4.0.8

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 Note: If you find missing information about particular minor version, that version must have been changed without any functional change in this library.
 
+**4.0.8 / 2022-03-18**
+* Fix CDATA parsing returning empty string when value = 0 (#451) (By [ndelanou](https://github.com/ndelanou))
+* Fix stopNodes when same tag appears inside node (#456) (By [patrickshipe](https://github.com/patrickshipe))
+* fix #468: prettify own properties only
+
+**4.0.7 / 2022-03-18**
+* support CDATA even if tag order is not preserved
+* support Comments even if tag order is not preserved
+* fix #446: XMLbuilder should not indent XML declaration
+
+**4.0.6 / 2022-03-08**
+* fix: call tagValueProcessor only once for array items
+* fix: missing changed for #437
+
 **4.0.5 / 2022-03-06**
 * fix #437: call tagValueProcessor from XML builder
 

package/README.md

@@ -11,6 +11,8 @@
 
 Validate XML, Parse XML to JS Object, or Build XML from JS Object without C/C++ based libraries and no callback.
 
+> Looking for maintainers
+
 <a href="https://opencollective.com/fast-xml-parser/donate" target="_blank">
   <img src="https://opencollective.com/fast-xml-parser/donate/button@2x.png?color=blue" width=200 />
 </a>
@@ -18,6 +20,9 @@ Validate XML, Parse XML to JS Object, or Build XML from JS Object without C/C++
 
 Check [ThankYouBackers](https://github.com/NaturalIntelligence/ThankYouBackers) for our contributors
 
+
+
+[![](static/img/ni_ads_ads.gif)](https://github.com/NaturalIntelligence/ads/)
 ## Users
 
 <a href="https://github.com/renovatebot/renovate" title="renovate" ><img src="https://avatars1.githubusercontent.com/u/38656520" width="60px" ></a>
@@ -99,12 +104,12 @@ In a HTML page
 
 Check lib folder for different browser bundles
 
-| Bundle Name | Size |
-| -- | -- |
-| fxbuilder.min.js | 5.2K |
-| fxparser.js |  50K |
-| fxparser.min.js |  17K |
-| fxp.min.js |  22K |
+| Bundle Name        | Size |
+| ------------------ | ---- |
+| fxbuilder.min.js   | 5.2K |
+| fxparser.js        | 50K  |
+| fxparser.min.js    | 17K  |
+| fxp.min.js         | 22K  |
 | fxvalidator.min.js | 5.7K |
 
 ### Documents
@@ -139,7 +144,7 @@ Check lib folder for different browser bundles
 * **[BigBit standard](https://github.com/amitguptagwl/bigbit)** : 
   * Single text encoding to replace UTF-8, UTF-16, UTF-32 and more with less memory.
   * Single Numeric datatype alternative of integer, float, double, long, decimal and more without precision loss.
-* **[Cytorus](https://github.com/NaturalIntelligence/cytorus)**: Now be specific and flexible while running E2E tests.
+* **[Cytorus](https://github.com/NaturalIntelligence/cytorus)**:  Be specific and flexible while running E2E tests.
   * Run tests only for a particular User Story
   * Run tests for a route or from a route
   * Customizable reporting

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fast-xml-parser",
-  "version": "4.0.5",
+  "version": "4.0.8",
   "description": "Validate XML, Parse XML, Build XML without C/C++ based libraries",
   "main": "./src/fxp.js",
   "scripts": {

package/src/xmlbuilder/json2xml.js

@@ -156,7 +156,7 @@ function buildAttrPairStr(attrName, val){
 function processTextOrObjNode (object, key, level) {
   const result = this.j2x(object, level + 1);
   if (object[this.options.textNodeName] !== undefined && Object.keys(object).length === 1) {
-    return this.buildTextNode(result.val, key, result.attrStr, level);
+    return this.buildTextNode(object[this.options.textNodeName], key, result.attrStr, level);
   } else {
     return this.buildObjNode(result.val, key, result.attrStr, level);
   }
@@ -172,11 +172,10 @@ function buildObjectNode(val, key, attrStr, level) {
   }
 
   if (attrStr && val.indexOf('<') === -1) {
-    return (
-      this.indentate(level) + '<' +  key + attrStr + piClosingChar + '>' +
-      val +
-      tagEndExp    );
-  } else {
+    return ( this.indentate(level) + '<' +  key + attrStr + piClosingChar + '>' + val + tagEndExp );
+  } else if (this.options.commentPropName !== false && key === this.options.commentPropName && piClosingChar.length === 0) {
+    return this.indentate(level) + `<!--${val}-->` + this.newLine;
+  }else {
     return (
       this.indentate(level) + '<' + key + attrStr + piClosingChar + this.tagEndChar +
       val +
@@ -194,20 +193,27 @@ function buildEmptyObjNode(val, key, attrStr, level) {
 }
 
 function buildTextValNode(val, key, attrStr, level) {
-  let textValue = this.options.tagValueProcessor(key, val);
-  textValue = this.replaceEntitiesValue(val);
-
-  if( textValue === '' && this.options.unpairedTags.indexOf(key) !== -1){ //unpaired
-    if(this.options.suppressUnpairedNode){
-      return this.indentate(level) + '<' + key + this.tagEndChar;
-    }else{
-      return this.indentate(level) + '<' + key + "/" + this.tagEndChar;
-    }
+  if (this.options.cdataPropName !== false && key === this.options.cdataPropName) {
+    return this.indentate(level) + `<![CDATA[${val}]]>` +  this.newLine;
+  }else if (this.options.commentPropName !== false && key === this.options.commentPropName) {
+    return this.indentate(level) + `<!--${val}-->` +  this.newLine;
   }else{
-    return (
-      this.indentate(level) + '<' + key + attrStr + '>' +
-       textValue +
-      '</' + key + this.tagEndChar  );
+    let textValue = this.options.tagValueProcessor(key, val);
+    textValue = this.replaceEntitiesValue(textValue);
+  
+    if( textValue === '' && this.options.unpairedTags.indexOf(key) !== -1){ //unpaired
+      if(this.options.suppressUnpairedNode){
+        return this.indentate(level) + '<' + key + this.tagEndChar;
+      }else{
+        return this.indentate(level) + '<' + key + "/" + this.tagEndChar;
+      }
+    } else{
+      return (
+        this.indentate(level) + '<' + key + attrStr + '>' +
+         textValue +
+        '</' + key + this.tagEndChar  );
+    }
+
   }
 }
 

package/src/xmlbuilder/orderedJs2Xml.js

@@ -41,7 +41,10 @@ function arrToStr(arr, options, jPath, level){
             continue;
         }else if( tagName[0] === "?"){
             const attStr = attr_to_str(tagObj[":@"], options);
-            xmlStr += indentation + `<${tagName} ${tagObj[tagName][0][options.textNodeName]} ${attStr}?>`;
+            const tempInd = tagName === "?xml" ? "" : indentation;
+            let piTextNodeName = tagObj[tagName][0][options.textNodeName];
+            piTextNodeName = piTextNodeName.length !== 0 ? " " + piTextNodeName : ""; //remove extra spacing
+            xmlStr += tempInd + `<${tagName}${piTextNodeName}${attStr}?>`;
             continue;
         }
         const attStr = attr_to_str(tagObj[":@"], options);

package/src/xmlparser/OrderedObjParser.js

@@ -251,7 +251,7 @@ const parseXml = function(xmlData) {
           currentNode.add(this.options.cdataPropName, [ { [this.options.textNodeName] : tagExp } ]);
         }else{
           let val = this.parseTextData(tagExp, currentNode.tagname, jPath, true, false, true);
-          if(!val) val = "";
+          if(val == undefined) val = "";
           currentNode.add(this.options.textNodeName, val);
         }
         
@@ -486,17 +486,35 @@ function readTagExp(xmlData,i, removeNSPrefix, closingChar = ">"){
  */
 function readStopNodeData(xmlData, tagName, i){
   const startIndex = i;
+  // Starting at 1 since we already have an open tag
+  let openTagCount = 1;
+
   for (; i < xmlData.length; i++) {
-    if( xmlData[i] === "<" && xmlData[i+1] === "/"){ 
-        const closeIndex = findClosingIndex(xmlData, ">", i, `${tagName} is not closed`);
-        let closeTagName = xmlData.substring(i+2,closeIndex).trim();
-        if(closeTagName === tagName){
-          return {
-            tagContent: xmlData.substring(startIndex, i),
-            i : closeIndex
+    if( xmlData[i] === "<"){ 
+      if (xmlData[i+1] === "/") {
+          const closeIndex = findClosingIndex(xmlData, ">", i, `${tagName} is not closed`);
+          let closeTagName = xmlData.substring(i+2,closeIndex).trim();
+          if(closeTagName === tagName){
+            openTagCount--;
+            if (openTagCount === 0) {
+              return {
+                tagContent: xmlData.substring(startIndex, i),
+                i : closeIndex
+              }
+            }
+          }
+          i=closeIndex;
+        } else {
+          const tagData = readTagExp(xmlData, i, '>')
+
+          if (tagData) {
+            const openTagName = tagData && tagData.tagName;
+            if (openTagName === tagName) {
+              openTagCount++;
+            }
+            i=tagData.closeIndex;
           }
         }
-        i=closeIndex;
       }
   }//end for loop
 }

package/src/xmlparser/node2json.js

@@ -46,9 +46,9 @@ function compress(arr, options, jPath){
         else val = "";
       }
 
-      if(compressedObj[property] !== undefined) {
+      if(compressedObj[property] !== undefined && compressedObj.hasOwnProperty(property)) {
         if(!Array.isArray(compressedObj[property])) {
-          compressedObj[property] = [ compressedObj[property] ];
+            compressedObj[property] = [ compressedObj[property] ];
         }
         compressedObj[property].push(val);
       }else{