Hi Felix, On Mon, May 23, 2011 at 11:50 PM, Felix Meschberger <[email protected]> wrote: > I am not sure about the new public static escapeHtmlText method, though: > > * Do we really need to add new methods ? > * Is HtmlResponse the correct location ? > * Isn't such a method already available elsewhere in Sling ?...
I dug deeper and yes, we have org.apache.sling.api.request.ResponseUtil which does the required escaping, so I'll remove the new method I added and use that class instead. > * IMHO we should also deprecate the Sling API HtmlResponse since > we have a better public mechanism now in the Sling POST Servlet.... Ok, created SLING-2087 for that. -Bertrand
