[dev] [commits] Horde branch master updated. 6149c84e973f3fb5c61760834c148bae4cbf04b8
Chuck Hagenbuch
chuck at horde.org
Mon Aug 23 03:03:54 UTC 2010
Quoting Michael M Slusarz <slusarz at horde.org>:
> Bug #9190: Add Horde_Support_Domhtml.
>
> Charset handling with libxml is less than ideal. Abstract the
> loadHTML() code out into a single place so that we can experiment and
> fix things all at once instead of piecemeal.
>
> Suggestions for where else this could reside? It does require
> Horde_String, so maybe horde/Util would be better.
I think if it stays as just this loader method that Util is a good
place, but if it becomes a more full-featured html lib a some point a
new Horde_Html package (with a Horde_Html_Document class) might be a
good place for it.
-chuck
More information about the dev
mailing list