Merge pull request #370 from flexi-xu/master

Fix issue #119.
Leonidas-from-XIV · Mar 28, 2017 · af83e31 · af83e31
2 parents 6821d3d + 6852241
commit af83e31
Show file tree

Hide file tree

Showing 3 changed files with 30 additions and 0 deletions.
diff --git a/lib/builder.js b/lib/builder.js
diff --git a/src/builder.coffee b/src/builder.coffee
@@ -48,6 +48,11 @@ class exports.Builder
           element.raw wrapCDATA obj
         else
           element.txt obj
+      else if Array.isArray obj
+        # fix issue #119
+        for own index, child of obj
+          for key, entry of child
+            element = render(element.ele(key), entry).up()
       else
         for own key, child of obj
           # Case #1 Attribute

diff --git a/test/builder.test.coffee b/test/builder.test.coffee
@@ -265,3 +265,19 @@ module.exports =
     actual = builder.buildObject obj
     diffeq expected, actual
     test.finish()
+
+  'test building obj with array': (test) ->
+    expected = """
+      <?xml version="1.0" encoding="UTF-8" standalone="yes"?>
+      <root>
+        <MsgId>10</MsgId>
+        <MsgId2>12</MsgId2>
+      </root>
+
+    """
+    opts = cdata: true
+    builder = new xml2js.Builder opts
+    obj = [{"MsgId": 10}, {"MsgId2": 12}]
+    actual = builder.buildObject obj
+    diffeq expected, actual
+    test.finish()