See https://issues.apache.org/jira/browse/WICKET-4425
On Tue, Feb 21, 2012, at 14:01, Martin Grigorov wrote: > Hi, > > Agree. Wicket should not do this in this case. > File a ticket with a quickstart. > > On Tue, Feb 21, 2012 at 1:42 PM, Pointbreak > <[email protected]> wrote: > > I have recently upgraded from Wicket 1.4.14 to 1.5.4. One issue that I > > encountered is that <script> tags in panel templates are rewritten by > > Wicket, even when the <script> tags in question have no wicket handlers > > attached to them. I.e. the following panel template (notice that there > > are no wicket:id attributes whatsoever): > > > > <wicket:panel> > > <script id="template-upload" type="text/x-jquery-tmpl"> > > <span>${name}</span> > > </script> > > </wicket:panel> > > > > Would render in the panel as: > > > > <script id="template-upload" type="text/x-jquery-tmpl"> > > /*<![CDATA[*/ > > <span>${name}</span> > > /*]]>*/ > > </script> > > > > Imho this is unwanted behavior that is a regression from the behavior in > > Wicket 1.4.x (or at least 1.4.14). Wicket should not add content to the > > body of the script tags (or any other tags in a template, unless their > > content is provided programmatically), as it does not have the knowledge > > how that affects the functionality of the page. Moreover, the content > > that Wicket adds to these script tags is only correct for Javascript > > (hence incorrect for the scripts in the example as they are not > > javascript). In the above example adding /*, */ and even the CDATA part > > will change the functionality of the script tag. If the "/*<![CDATA[*/" > > part was necessary in the script tags above, they should be added by the > > person that provides the template, not magically added by Wicket. > > > > Comments? > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: [email protected] > > For additional commands, e-mail: [email protected] > > > > > > -- > Martin Grigorov > jWeekend > Training, Consulting, Development > http://jWeekend.com > > --------------------------------------------------------------------- > 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]
