[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