Req #54037 [Opn]: [PATCH] Adds the ability to pass options to loadHTML

Edit report at http://bugs.php.net/bug.php?id=54037&edit=1 ID: 54037 User updated by: fxmulder at gmail dot com Reported by: fxmulder at gmail dot com Summary: [PATCH] Adds the ability to pass options to loadHTML Status: Open Type: Feature/Change Request Package: DOM XML related PHP Version: trunk-SVN-2011-02-17 (SVN) Block user comment: N Private report: N New Comment: That works for me, this still good to be committed to the trunk? Previous Comments: ------------------------------------------------------------------------ [2011-03-03 08:12:00] chregu@php.net The following patch has been added/updated: Patch Name: patch-for-adding-loadhtml-options.patch Revision: 1299136320 URL: http://bugs.php.net/patch-display.php?bug=54037&patch=patch-for-adding-loadhtml-options.patch&revision=1299136320 ------------------------------------------------------------------------ [2011-03-03 08:11:04] chregu@php.net After talking with Rob, we removed LIBXML_RECOVER again. People don't like seeing that wildly used, there were and are always a lot of discussions, if that's a good thing at all. So we just remove the constant again for now. I'll attach the new patch. If noone complains, I will commit it soon ------------------------------------------------------------------------ [2011-02-22 17:11:08] chregu@php.net Looks fine to me, Will commit soon to trunk ------------------------------------------------------------------------ [2011-02-18 00:21:20] fxmulder at gmail dot com I've also included a patch to create new constants for use as options ------------------------------------------------------------------------ [2011-02-17 18:57:57] fxmulder at gmail dot com Description: ------------ This patch allows passing options directly to libxml for the loadHTML and loadHTMLFile DOMDocument functions ------------------------------------------------------------------------ -- Edit this bug report at http://bugs.php.net/bug.php?id=54037&edit=1