Merge branch 'master' into updated-oneListGroup-plus-attributes

NaturalIntelligence · Jul 9, 2024 · 52241a4 · 52241a4
2 parents 2360468 + 931e910
commit 52241a4
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 2 deletions.
diff --git a/spec/j2x_spec.js b/spec/j2x_spec.js
@@ -483,6 +483,19 @@ describe("XMLBuilder", function() {
         expect(result).toEqual(expected);
     });
 
+    it("should correctly handle values with oneListGroup", function() {
+        const jObj = {
+            "a": [
+                "(first)",
+                "(second)"
+            ],
+        };
+        const builder = new XMLBuilder({oneListGroup:"true", attributesGroupName: "@"});
+        const result = builder.build(jObj);
+        const expected = `<a>(first)(second)</a>`;
+        expect(result).toEqual(expected);
+    });
+
     it("should handle attributes with oneListGroup", function() {
         const jObj = {
             "a": [
@@ -506,7 +519,7 @@ describe("XMLBuilder", function() {
         const expected = `<a foo="bar" baz="foo" bar="baz"><b>1</b><b>2</b></a>`;
         expect(result).toEqual(expected);
     });
-    
+
     it('should build tag with only text node', async () => {
         const schema_obj = {
           field: {

diff --git a/src/xmlbuilder/json2xml.js b/src/xmlbuilder/json2xml.js
@@ -135,7 +135,13 @@ Builder.prototype.j2x = function(jObj, level) {
             listTagVal += this.processTextOrObjNode(item, key, level)
           }
         } else {
-          listTagVal += this.buildTextValNode(item, key, '', level);
+          if (this.options.oneListGroup) {
+            let textValue = this.options.tagValueProcessor(key, item);
+            textValue = this.replaceEntitiesValue(textValue);
+            listTagVal += textValue;
+          } else {
+            listTagVal += this.buildTextValNode(item, key, '', level);
+          }
         }
       }
       if(this.options.oneListGroup){