]> granicus.if.org Git - php/commitdiff
Revert "Merge branch 'pull-request/2344' into PHP-7.0"
authorNikita Popov <nikita.ppv@gmail.com>
Fri, 3 Feb 2017 17:41:36 +0000 (18:41 +0100)
committerNikita Popov <nikita.ppv@gmail.com>
Fri, 3 Feb 2017 17:42:10 +0000 (18:42 +0100)
This reverts commit 6988d070ea8c7d44452d8c98e594d65ea722a1d8, reversing
changes made to 75ad2b301a047209e7b5a6b5912eb589cfbc773d.

NEWS
ext/dom/document.c
ext/dom/tests/bug74004.phpt [deleted file]

diff --git a/NEWS b/NEWS
index e2878e0e5963bf5e1c2be8c542233db534d1a899..102c9eae005ff28b79b4d4299a0ccf099c48fcfd 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -8,10 +8,6 @@ PHP                                                                        NEWS
   . Fixed bug #73998 (array_key_exists fails on arrays created by
     get_object_vars). (mhagstrand)
 
-- DOM:
-  . Fixed bug #74004 (LIBXML_NOWARNING (etc) ignored by DOMDocument::loadHTML).
-    (somedaysummer)
-
 - GD:
   . Fixed bug #74031 (ReflectionFunction for imagepng is missing last two
     parameters). (finwe)
index a884087f5f0019da1e64d433985ba31bf7341e2c..cab0aa55ce43a8d1983741b407e7232add83e5cc 100644 (file)
@@ -2038,16 +2038,16 @@ static void dom_load_html(INTERNAL_FUNCTION_PARAMETERS, int mode) /* {{{ */
                RETURN_FALSE;
        }
 
-       
+       if (options) {
+               htmlCtxtUseOptions(ctxt, (int)options);
+       }
+
        ctxt->vctxt.error = php_libxml_ctx_error;
        ctxt->vctxt.warning = php_libxml_ctx_warning;
        if (ctxt->sax != NULL) {
                ctxt->sax->error = php_libxml_ctx_error;
                ctxt->sax->warning = php_libxml_ctx_warning;
        }
-       if (options) {
-               htmlCtxtUseOptions(ctxt, (int)options);
-       }
        htmlParseDocument(ctxt);
        newdoc = ctxt->myDoc;
        htmlFreeParserCtxt(ctxt);
diff --git a/ext/dom/tests/bug74004.phpt b/ext/dom/tests/bug74004.phpt
deleted file mode 100644 (file)
index 853dfa5..0000000
+++ /dev/null
@@ -1,15 +0,0 @@
---TEST--
-Bug #74004 (DOMDocument->loadHTML and ->loadHTMLFile do not heed LIBXML_NOWARNING and LIBXML_NOERROR options)
---SKIPIF--
-<?php require_once('skipif.inc'); ?>
---FILE--
-<?php
-
-$doc=new DOMDocument();
-libxml_use_internal_errors(true);
-$doc->loadHTML("<tag-throw-warning></tag-throw-warning>",LIBXML_NOWARNING|LIBXML_NOERROR);
-print count(libxml_get_errors());
-
-?>
---EXPECT--
-0