From: David A. <web...@av...> - 2011-06-30 10:27:15
|
On Jun 30, 2011, at 5:31 AM, Paul Hoadley <pa...@lo...> wrote: > On 30/06/2011, at 4:34 PM, Q wrote: > >>> After clearing through a number of issues with the component generating the page content, I've hit a hurdle that I'm not sure how to tackle. I'm using (don't laugh) GoogleChart.framework to show some charts. The src attributes of those components end up containing a lot of query parameters, and the ampersands are bare '&' characters rather than the equivalent '&' entity. I don't immediately see any options (in GCAbstractChart or the ERXMutableURL that it's using to construct the URL) to escape the ampersands as entities to make ERPDFWrapper's parser happy. The only thing I can think of is using ERXResponseRewriter. replaceAllInResponse(), but that (a) seems pretty brute force, and (b) would take me the rest of the afternoon to get the Pattern argument right. >> >> You could try setting the "er.extensions.ERXPatcher.cleanupXHTML" property, but that might be a bit of a sledge hammer to swat a fly. > > That didn't seem to work. It looks like it should, but the '&'s were passed through untouched. > > Neither did ERXResponseRewriter. If I removed the ERPDFWrapper, the HTML output looked good (bare '&'s replaced with '&'), but the XHTML parser still made the same complaint. (Not sure how/why it was seeing a different response to what I got when I removed the wrapper.) If they look like they are all fixed, maybe the problem is now something else entirely. What is the error you are seeing? "Look! A red herring! Let's follow it and see where it takes us." - Phineas Dave > >> The ideal solution would be to just fix ERXMutableURL or GCAbstractChart to properly escape the & values when required. > > A two line patch to ERXMutableURL.queryParametersAsString(StringBuffer) did work: replace the two calls to sb.append('&') with sb.append("&"). I'm fairly sure that's the correct syntax for all flavours of HTML, but is that likely to make any browsers and/or existing code explode? That is, should I submit this patch to ERXMutableURL, or should I be looking at creating my own subclass just for this particular purpose? > > > -- > Paul. > > http://logicsquad.net/ > > > > ------------------------------------------------------------------------------ > All of the data generated in your IT infrastructure is seriously valuable. > Why? It contains a definitive record of application performance, security > threats, fraudulent activity, and more. Splunk takes this data and makes > sense of it. IT sense. And common sense. > http://p.sf.net/sfu/splunk-d2d-c2 > _______________________________________________ > Wonder-disc mailing list > Won...@li... > https://lists.sourceforge.net/lists/listinfo/wonder-disc > > |