Merge pull request #1062 from MrPetovan/bug/fatal-errors
[showmore] Use DOMDocument object instead of static call to loadHTML()pull/1063/head
commit
6439a51a19
|
@ -81,8 +81,9 @@ function get_body_length($body)
|
|||
// We need to get rid of hidden tags (display: none)
|
||||
|
||||
// Get rid of the warning. It would be better to have some valid html as input
|
||||
$dom = @DomDocument::loadHTML($body);
|
||||
$xpath = new DOMXPath($dom);
|
||||
$doc = new DOMDocument();
|
||||
@$doc->loadHTML($body);
|
||||
$xpath = new DOMXPath($doc);
|
||||
|
||||
/*
|
||||
* Checking any possible syntax of the style attribute with xpath is impossible
|
||||
|
@ -96,7 +97,7 @@ function get_body_length($body)
|
|||
}
|
||||
}
|
||||
// Now we can get the body of our HTML DomDocument, it contains only what is visible
|
||||
$string = $dom->saveHTML();
|
||||
$string = $doc->saveHTML();
|
||||
|
||||
$string = strip_tags($string);
|
||||
return strlen($string);
|
||||
|
|
Loading…
Reference in New Issue