Add support for 'always' cdata builder option

.gitignore

@@ -1,4 +1,5 @@
 *.swp
+.idea
 node_modules
 npm-debug.log
 .nyc_output

README.md

@@ -347,9 +347,10 @@ Possible options are:
   * `headless` (default: `false`): omit the XML header. Added in 0.4.3.
   * `allowSurrogateChars` (default: `false`): allows using characters from the Unicode
     surrogate blocks.
-  * `cdata` (default: `false`): wrap text nodes in `<![CDATA[ ... ]]>` instead of
-    escaping when necessary. Does not add `<![CDATA[ ... ]]>` if it is not required.
+  * `cdata` (default: `false`): 
+    *  When `true` wrap text nodes in `<![CDATA[ ... ]]>` instead of escaping when necessary. Does not add `<![CDATA[ ... ]]>` if it is not required.
     Added in 0.4.5.
+    * When `'always'` all inner text wrapped in `<![CDATA[ ... ]]>` regardless of content.
 
 `renderOpts`, `xmldec`,`doctype` and `headless` pass through to
 [xmlbuilder-js](https://github.com/oozcitak/xmlbuilder-js).

src/builder.coffee

@@ -44,7 +44,7 @@ class exports.Builder
     render = (element, obj) =>
       if typeof obj isnt 'object'
         # single element, just append it as text
-        if @options.cdata && requiresCDATA obj
+        if @options.cdata is 'always' or (@options.cdata && requiresCDATA obj)
           element.raw wrapCDATA obj
         else
           element.txt obj
@@ -64,7 +64,7 @@ class exports.Builder
 
           # Case #2 Char data (CDATA, etc.)
           else if key is charkey
-            if @options.cdata && requiresCDATA child
+            if @options.cdata is 'always' or (@options.cdata && requiresCDATA child)
               element = element.raw wrapCDATA child
             else
               element = element.txt child
@@ -73,7 +73,7 @@ class exports.Builder
           else if Array.isArray child
             for own index, entry of child
               if typeof entry is 'string'
-                if @options.cdata && requiresCDATA entry
+                if @options.cdata is 'always' or (@options.cdata && requiresCDATA entry)
                   element = element.ele(key).raw(wrapCDATA entry).up()
                 else
                   element = element.ele(key, entry).up()
@@ -86,7 +86,7 @@ class exports.Builder
 
           # Case #5 String and remaining types
           else
-            if typeof child is 'string' && @options.cdata && requiresCDATA child
+            if typeof child is 'string' && @options.cdata is 'always' or (@options.cdata && requiresCDATA child)
               element = element.ele(key).raw(wrapCDATA child).up()
             else
               if not child?

src/parser.coffee

@@ -126,7 +126,7 @@ class exports.Parser extends events.EventEmitter
       nodeName = obj["#name"]
       delete obj["#name"] if not @options.explicitChildren or not @options.preserveChildrenOrder
 
-      if obj.cdata == true
+      if obj.cdata is 'always' or obj.cdata == true
         cdata = obj.cdata
         delete obj.cdata
 

test/builder.test.coffee

@@ -220,6 +220,22 @@ module.exports =
     diffeq expected, actual
     test.finish()
 
+  "test uses cdata for all chars when cdata is 'always'": (test) ->
+    expected = """
+      <?xml version="1.0" encoding="UTF-8" standalone="yes"?>
+      <xml>
+        <MsgId><![CDATA[& <<]]></MsgId>
+        <Message><![CDATA[Hello]]></Message>
+      </xml>
+
+    """
+    opts = cdata: 'always'
+    builder = new xml2js.Builder opts
+    obj = {"xml":{"MsgId":["& <<"],"Message":["Hello"]}}
+    actual = builder.buildObject obj
+    diffeq expected, actual
+    test.finish()
+
   'test uses cdata for string values of objects': (test) ->
     expected = """
       <?xml version="1.0" encoding="UTF-8" standalone="yes"?>