This module uses CDATA for all tags' values. In fact, they are completely unnecessary except for highly unusual circumstances. Please see http://en.wikipedia.org/wiki/CDATA#CDATA_sections_in_XML for an explanation.

CommentFileSizeAuthor
#6 6.x-1.1_add_cdata_wrapper_support.patch2.46 KBagalitsyn
#3 no_cdata.patch936 bytesaren cambre
#1 no_cdata.patch892 bytesaren cambre

Comments

aren cambre’s picture

aren cambre commented
Status: Active » Needs review
StatusFileSize
new no_cdata.patch892 bytes

Patch attached.

aren cambre’s picture

aren cambre commented

This also appears to fix #494846: XML export pads all values with spaces. I don't know why!

aren cambre’s picture

aren cambre commented
StatusFileSize
new no_cdata.patch936 bytes

Mistake in original patch caused incorrect entity references. Fixed version attached.

neclimdul’s picture

neclimdul
He/Him
commented
Status: Needs review » Fixed

Committed a modified version of this with the logic in the preprocess function. Thanks.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

agalitsyn’s picture

agalitsyn commented
Version: 6.x-1.0-beta2 » 6.x-1.1
Status: Closed (fixed) » Needs review
StatusFileSize
new 6.x-1.1_add_cdata_wrapper_support.patch2.46 KB

CDATA can be needed for specific fields.

So I use a standart field option called "Element wrapper type" to add CDATA wrapper. Views have variable "views_field_rewrite_elements" which can be overriden with yours additional wrappers.

I add cdata using drush

drush eval "variable_set('views_field_rewrite_elements', array('' => t('- Use default -'), '0' => t('- None -'), 'div' => t('DIV'), 'span' => t('SPAN'), 'h1' => t('H1'), 'h2' => t('H2'), 'h3' => t('H3'), 'h4' => t('H4'), 'h5' => t('H5'), 'h6' => t('H6'), 'p' => t('P'), 'strong' => t('STRONG'), 'em' => t('EM'), 'cdata' => t('CDATA')));"

And write come code to support this wrapper. (see patch)

neclimdul’s picture

neclimdul
He/Him
commented
Issue summary: View changes
Status: Needs review » Closed (outdated)