php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
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
Welcome back! If you're the original bug submitter, here's where you can edit the bug or add additional notes.
If you forgot your password, you can retrieve your password here.
Password:
Status:
Package:
Bug Type:
Summary:
From: fxmulder at gmail dot com
New email:
PHP Version: OS:

 

 [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)

Pull Requests

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-2025 The PHP Group
All rights reserved.
Last updated: Mon Apr 28 18:01:30 2025 UTC