]> granicus.if.org Git - php/commitdiff
revert r319254 (fix bug #54682) since it is not correct
authorNuno Lopes <nlopess@php.net>
Sat, 21 Jan 2012 11:32:56 +0000 (11:32 +0000)
committerNuno Lopes <nlopess@php.net>
Sat, 21 Jan 2012 11:32:56 +0000 (11:32 +0000)
add test that was broken with that patch.
this bug is probably in libtidy itself and not in PHP. I didn't investigate it enough to confirm, though

ext/tidy/tests/004.phpt
ext/tidy/tidy.c

index e941de452bbc4201e79e73a2394bf847982bc881..7ca17c6d95f3ec72bda9d41b6e5b5101567b2a45 100644 (file)
@@ -4,14 +4,28 @@ tidy_diagnose()
 <?php if (!extension_loaded("tidy")) print "skip"; ?>
 --FILE--
 <?php 
-       $a = tidy_parse_string("<HTML></HTML>");
-       tidy_diagnose($a);
-       echo tidy_get_error_buffer($a);
+$a = tidy_parse_string('<HTML></HTML>');
+var_dump(tidy_diagnose($a));
+echo tidy_get_error_buffer($a);
+
+$html = <<< HTML
+<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 3.2//EN">
+<html>
+<head><title>foo</title></head>
+<body><p>hello</p></body>
+</html>
+HTML;
+$a = tidy_parse_string($html);
+var_dump(tidy_diagnose($a));
+echo tidy_get_error_buffer($a);
 ?>
 --EXPECT--
-
+bool(true)
 line 1 column 1 - Warning: missing <!DOCTYPE> declaration
 line 1 column 7 - Warning: discarding unexpected </html>
 line 1 column 14 - Warning: inserting missing 'title' element
 Info: Document content looks like HTML 3.2
-3 warnings, 0 errors were found!
\ No newline at end of file
+3 warnings, 0 errors were found!
+bool(true)
+Info: Document content looks like HTML 3.2
+No warnings or errors were found.
index 50a0f3f2b9e8a83b1539d6d18f176357e3254586..55c33825d453fff64af962ef635d48a98082adb1 100644 (file)
@@ -1336,7 +1336,7 @@ static PHP_FUNCTION(tidy_diagnose)
 {
        TIDY_FETCH_OBJECT;
 
-       if (tidyStatus(obj->ptdoc->doc) != 0 && tidyRunDiagnostics(obj->ptdoc->doc) >= 0) {
+       if (tidyRunDiagnostics(obj->ptdoc->doc) >= 0) {
                tidy_doc_update_properties(obj TSRMLS_CC);
                RETURN_TRUE;
        }