PHx + CDATA, problems

361 Posts

The Man Can! Reply #1, 17 years, 10 months ago

Every couple of weeks I run into PHx just outputting the actual PHx call and not what it should be doing. I usually spend 30 minutes making sure my syntax is correct and such, then I remember that PHx barfs when it sees a CDATA tag (specifically, the ]]> sequence).

Once I figure it out I remove the CDATA tags and proceed, but this time I figured I’d bring it up in the forums. Has anybody else noticed this, and if so, any fixes? I’ve searched but I only see one reference to it with no answers.

Need MODx Ecommerce? Try FoxyCart!

3,975 Posts

Zaigham (aka zi) Reply #2, 17 years, 10 months ago

May be this is related: http://wiki.modxcms.com/index.php/PHx#Known_issues ?

Zaigham R - MODX Professional | Skype | Email | Twitter

Digging the interwebs for #MODX gems and bringing it to you. modx.link

361 Posts

The Man Can! Reply #3, 17 years, 10 months ago

Yeah. It’s not in that list but the ]] is definitely the culprit. I’ve updated the wiki page.

In case anybody wants a workaround, a quickie is to write your ]]> as ].]> and then add a plugin to replace them OnWebPagePrerender:

//<?php
// Begin plugin code
$e = &$modx->event;

$id = $modx->event->params['id'];

if ($e->name == 'OnWebPagePrerender') {
	$o = &$modx->documentOutput; // get a reference of the output
	$o = str_replace('/* ].]> */', '/* ]]> */',$o);
}
$e->output($o);
return '';

Need MODx Ecommerce? Try FoxyCart!

2,877 Posts

mrhaw Reply #4, 17 years, 10 months ago

THANK YOU! cool

@hawproductions | http://mrhaw.com/

Infograph: MODX Advanced Install in 7 steps:
http://forums.modx.com/thread/96954/infograph-modx-advanced-install-in-7-steps

Recap: Portland, OR (PDX) MODX CMS Meetup, Oct 6, 2015. US Bancorp Tower
http://mrhaw.com/modx_portland_oregon_pdx_modx_cms_meetup_oct_2015_us_bancorp_tower

3,975 Posts

Zaigham (aka zi) Reply #5, 17 years, 10 months ago

Nice work Brett laugh

Zaigham R - MODX Professional | Skype | Email | Twitter

Digging the interwebs for #MODX gems and bringing it to you. modx.link

19 Posts

withincontext Reply #6, 16 years, 8 months ago

Thanks for the plug-in tip. Unfortunately, the plug-in doesn’t seem to be executing OnWebPagePrerender. I’m getting this error:

XML Parsing Error: unclosed CDATA section

The XML fragment looks like this: <section title="What I Did"><![CDATA[ Foo ].]></section>

94 Posts

digitalbart2 Reply #7, 16 years, 8 months ago

First of thanks for the plugin. But can you elaborate on getting the plugin working? I tried this code and it does not seem to work?

I checked OnWebPagePrerender, did variants of the spacing of ’/* ].]> */’ also and it still does not work.

Any suggestions?

//<?php
// Begin plugin code
$e = &$modx->event;

$id = $modx->event->params['id'];

if ($e->name == 'OnWebPagePrerender') {
	$o = &$modx->documentOutput; // get a reference of the output
	$o = str_replace('/* ].]> */', '/* ]]> */',$o);
}
$e->output($o);
return '';

361 Posts

The Man Can! Reply #8, 16 years, 8 months ago

What error are you getting? Same as above? And what v of MODx? I haven’t used this code in a while so I’m not sure if perhaps something has changed lately.

Need MODx Ecommerce? Try FoxyCart!

94 Posts

digitalbart2 Reply #9, 16 years, 8 months ago

In my logs, I am getting "Parse error: syntax error, unexpected ’<’ in ...httpdocs/manager/includes/document.parser.class.inc.php(746) : eval()’d code on line 1"

361 Posts

The Man Can! Reply #10, 16 years, 8 months ago

Do you have

//<?php

in your plugin? Try removing that? Odd.

Need MODx Ecommerce? Try FoxyCart!