friendica/friendica-addons
7
2
Fork 14
Code Issues 3 Pull requests 3 Releases 16 Wiki Activity

[showmore] Use DOMDocument object instead of static call to loadHTML() #1062

Merged
MrPetovan merged 1 commit from bug/fatal-errors into 2020.12-rc 2020-12-27 06:58:44 +01:00
Conversation 0 Commits 1 Files changed 1 +4 -3
1 changed files with 4 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
showmore/showmore.php
View file

@ -81,8 +81,9 @@ function get_body_length($body)
// We need to get rid of hidden tags (display: none) // 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 // Get rid of the warning. It would be better to have some valid html as input
$dom = @DomDocument::loadHTML($body); $doc = new DOMDocument();
$xpath = new DOMXPath($dom); @$doc->loadHTML($body);
$xpath = new DOMXPath($doc);
/* /*
* Checking any possible syntax of the style attribute with xpath is impossible * 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 // 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); $string = strip_tags($string);
return strlen($string); return strlen($string);