Browse Source

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

- Address https://github.com/friendica/friendica/issues/9498#issuecomment-751191039
pull/1062/head
Hypolite Petovan 6 months ago
parent
commit
368a0d3f6c
1 changed files with 4 additions and 3 deletions
  1. +4
    -3
      showmore/showmore.php

+ 4
- 3
showmore/showmore.php View File

@ -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…
Cancel
Save