I don't think it is - or a least not in this case. The changes.jsl script is the one that is producing the invalid XML.
I'm about to throw in the towel and convert it to plain old XSLT. > -----Original Message----- > From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] > Sent: Friday, 11 July 2003 4:48 PM > To: Maven Developers List > Subject: RE: [changes plugin] Support for html in change description - > bugs in jelly? > > > This is site.jsl being busted..... > -- > dIon Gillard, Multitask Consulting > Blog: http://blogs.codehaus.org/people/dion/ > Work: http://www.multitask.com.au > > > "Tim Anderson" <[EMAIL PROTECTED]> wrote on 11/07/2003 04:13:23 PM: > > > That fixes the whitespace issue thanks, but unfortunately > > CDATA sections are still screwed. > > > > Eg: > > <source><![CDATA[ > > <Connectors> > > <Connector scheme="rmi"/> > > </Connectors>]]> > > </source> > > > > Becomes: > > <source><![CDATA[ > > <Connectors> > > <Connector scheme="rmi"> > > </Connectors>]]> > > </source> > > > > And the HTML produced is: > > <div id="source"> > > <pre> > > &lt;Connectors&gt; > > .... > > > > > > > -----Original Message----- > > > From: Emmanuel Venisse [mailto:[EMAIL PROTECTED] > > > Sent: Friday, 11 July 2003 1:22 AM > > > To: Maven Developers List; [EMAIL PROTECTED] > > > Subject: Re: [changes plugin] Support for html in change description - > > > bugs in jelly? > > > > > > > > > The problem is due to prettyPrint in plugin.jelly. > > > I think that if you pass prettyPrint to false, the result will be ok. > > > > > > Emmanuel > > > > > > ----- Original Message ----- > > > From: "Tim Anderson" <[EMAIL PROTECTED]> > > > To: "Maven Developers List" <[EMAIL PROTECTED]> > > > Sent: Thursday, July 10, 2003 7:04 AM > > > Subject: [changes plugin] Support for html in change description - > bugs in > > > jelly? > > > > > > > > > > Problem (short version): > > > > > > > > I'm trying to modify the changes plugin to support > > > > html in the changes description field - but > > > > I'm having issues with jelly:xml. > > > > Its stripping out whitespace, and screwing up elements > > > > embedded in CDATA sections. Has anyone seen this > > > > before and know of a workaround? > > > > > > > > For the exercise (and to make sure I wasn't going nuts), I > > > > rewrote the plugins' changes.jsl in XSLT, and ran > > > > it via Ant's style task. It works beautifully... > > > > > > > > Problem (long version): > > > > > > > > Currently, the changes plugin doesn't support html in > > > > change descriptions, e.g: > > > > <action type="update"> > > > > Tinkered with foobars.<p/> > > > > To construct a foobar: > > > > <source><![CDATA[ > > > > Hashtable properties = new Hashtable(); > > > > ... > > > > ]]> > > > > </source> > > > > </action> > > > > > > > > Gets transformed to: > > > > ... > > > > <td>Tinkered with foobars.To construct a foobar:Hastable > > > > properties...</td> > > > > ... > > > > > > > > This is because it transforms the text using: > > > > <x:expr select="string(.)"/> > > > > > > > > If the above is changed to: > > > > <x:copyOf select="node()" trim="false"/> > > > > > > > > ..things improve slightly, but all whitespace is lost. > > > > > > > > Worse still, CDATA sections containing elements aren't processed > > > > properly, e.g: > > > > <source><![CDATA[ > > > > <Connectors> > > > > <Connector scheme="rmi"/> > > > > </Connections>]]> > > > > </source> > > > > > > > > Becomes: > > > > <![CDATA[<Connectors> <Connector scheme="rmi"> > > > > </Connector> ... > > > > > > > > > > > > > > > > > --------------------------------------------------------------------- > > > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > > > For additional commands, e-mail: [EMAIL PROTECTED] > > > > > > > > > _____________________________________________________________________ > > > > Envie de discuter en "live" avec vos amis ? T�l�charger MSN > Messenger > > > > http://www.ifrance.com/_reloc/m la 1�re messagerie instantan�e de > France > > > > > > _____________________________________________________________________ > > > Envie de discuter en "live" avec vos amis ? T�l�charger MSN Messenger > > > http://www.ifrance.com/_reloc/m la 1�re messagerie instantan�e de > France > > > > > > > > > --------------------------------------------------------------------- > > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > > For additional commands, e-mail: [EMAIL PROTECTED] > > > > > > > > > > > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > For additional commands, e-mail: [EMAIL PROTECTED] > > > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
