php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login

go to bug id or search bugs for

Doc Bug #54037 [PATCH] Adds the ability to pass options to loadHTML
Submitted: 2011-02-17 18:57 UTC Modified: 2013-12-22 17:15 UTC
Votes:2
Avg. Score:5.0 ± 0.0
Reproduced:1 of 1 (100.0%)
Same Version:0 (0.0%)
Same OS:0 (0.0%)
From: fxmulder at gmail dot com Assigned: salathe (profile)
Status: Closed Package: DOM XML related
PHP Version: trunk-SVN-2011-02-17 (SVN) OS:
Private report: No CVE-ID: None
View Add Comment Developer Edit
 [2011-02-17 18:57 UTC] fxmulder at gmail dot com 
Description:
------------
This patch allows passing options directly to libxml for the loadHTML and 
loadHTMLFile DOMDocument functions

Patches

patch-for-new-loadhtml-options-documentation (last revision 2011-08-12 21:06 UTC by fxmulder at gmail dot com)
patch-for-adding-loadhtml-options.patch (last revision 2011-03-03 07:12 UTC by chregu@php.net)
libxml-html-constants.patch (last revision 2011-02-17 23:19 UTC by fxmulder at gmail dot com)
domdocument-loadhtml-options.patch (last revision 2011-02-17 18:31 UTC by fxmulder at gmail dot com)

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2011-02-18 00:21 UTC] fxmulder at gmail dot com 
I've also included a patch to create new constants for use as options
 [2011-02-22 17:11 UTC] chregu@php.net 
Looks fine to me, Will commit soon to trunk
 [2011-03-03 08:11 UTC] 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-03-03 08:12 UTC] 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-29 22:27 UTC] fxmulder at gmail dot com 
That works for me, this still good to be committed to the trunk?
 [2011-07-11 11:30 UTC] chregu@php.net
-Status: Open +Status: To be documented
 [2011-07-11 11:30 UTC] chregu@php.net 
Committed to trunk and PHP_5_4 branch
 [2011-08-12 21:07 UTC] fxmulder at gmail dot com 
I've included a patch for proposed documentation changes for the new options
 [2012-07-22 23:39 UTC] tyrael@php.net 
This bug has been fixed in SVN.

Snapshots of the sources are packaged every three hours; this change
will be in the next snapshot. You can grab the snapshot at
http://snaps.php.net/.

 For Windows:

http://windows.php.net/snapshots/
 
Thank you for the report, and for helping us make PHP better.

I've opened a separate bug for the documentation issue: 
https://bugs.php.net/bug.php?id=62635
 [2012-07-22 23:39 UTC] tyrael@php.net
-Status: Open +Status: Closed -Assigned To: +Assigned To: tyrael
 [2012-08-07 23:04 UTC] philip@php.net 
The documentation patch was not committed.
 [2012-08-07 23:04 UTC] philip@php.net
-Status: Closed +Status: Re-Opened
 [2012-09-16 20:51 UTC] tyrael@php.net
-Assigned To: tyrael +Assigned To:
 [2013-12-02 14:59 UTC] mike@php.net
-Type: Bug +Type: Documentation Problem
 [2013-12-22 17:15 UTC] salathe@php.net
-Status: Re-Opened +Status: Closed -Assigned To: +Assigned To: salathe
 [2013-12-22 17:15 UTC] salathe@php.net 
The fix for this bug has been committed.

Snapshots of the sources are packaged every three hours; this change
will be in the next snapshot. You can grab the snapshot at
http://snaps.php.net/.

 For Windows:

http://windows.php.net/snapshots/
 
Thank you for the report, and for helping us make PHP better.
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved. 		Last updated: Fri Apr 26 15:01:56 2024 UTC