Re: Patch regarding TagSoup and JTidy

Posted by Erik Bruchez on
URL: https://discuss.orbeon.com/RfE-Tagsoup-configuration-in-properties-local-xml-tp2550129p3005023.html

Florian,

Thanks for this patch.

Just one concern: this means that the Tidy properties file will be
loaded every time htmlStringToDocument() is called, right? And this
can be pretty often as this is used by all controls that output HTML.

Somehow, this should be cached. What do you think?

-Erik

On Wed, Sep 29, 2010 at 1:24 AM, Florian Schmitt
<[hidden email]> wrote:

> Hi all,
>
> to make the TagSoup and JTidy configuration accessible without the need
> to recompile XFormsUtils.java, i'd like to propose the attached patch.
> It handles two issues as follows:
>
> - TagSoup: there's a new boolean config property names
> "oxf.xforms.tagsoup.ignoreBogonsFeature" with default value "true".
> Changing this to false should make TagSoup accept unknown (non-html)
> elements.
>
> - JTidy: a new set of tidy config options with priority over the
> hard-coded ones in XFormsUtils.java is defined using the new config
> property "oxf.xforms.tidy.propertiesFile" (anyURI). That URI by default
> points to oxf:/config/tidy.properties, making the complete Tidy config
> accessible using the Java properties syntax. The proposed
> tidy.properties defines the canvas tag as additional, valid element that
> would otherwise get stripped from the content parsed by JTidy.
>
> Maybe some additional hints would be useful which tidy properties are
> available (could be placed in tidy.properties or in the wiki).
>
> I would be glad to hear your opinions!
>
>
> florian
>
>
>
>
> --
> You receive this message as a subscriber of the [hidden email] mailing list.
> To unsubscribe: mailto:[hidden email]
> For general help: mailto:[hidden email]?subject=help
> OW2 mailing lists service home page: http://www.ow2.org/wws
>
>


--
You receive this message as a subscriber of the [hidden email] mailing list.
To unsubscribe: mailto:[hidden email]
For general help: mailto:[hidden email]?subject=help
OW2 mailing lists service home page: http://www.ow2.org/wws