mexon/friendica
1
0
Fork 0
Code Pull requests Activity

User DOMDocument object call instead of static in Content\OEmbed

Browse source
This commit is contained in:
Hypolite Petovan 2019-01-21 16:56:15 -05:00
parent ddad9721c0
commit df706484cd
1 changed files with 4 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
src/Content/OEmbed.php
View file

@ -86,7 +86,8 @@ class OEmbed
$redirects = 0; $redirects = 0;
$html_text = Network::fetchUrl($embedurl, false, $redirects, 15, 'text/*'); $html_text = Network::fetchUrl($embedurl, false, $redirects, 15, 'text/*');
if ($html_text) { if ($html_text) {
$dom = @DOMDocument::loadHTML($html_text); $dom = new DOMDocument();
$dom->loadHTML($html_text);
if ($dom) { if ($dom) {
$xpath = new DOMXPath($dom); $xpath = new DOMXPath($dom);
$entries = $xpath->query("//link[@type='application/json+oembed']"); $entries = $xpath->query("//link[@type='application/json+oembed']");
@ -274,7 +275,8 @@ class OEmbed
$html_text = mb_convert_encoding($text, 'HTML-ENTITIES', mb_detect_encoding($text)); $html_text = mb_convert_encoding($text, 'HTML-ENTITIES', mb_detect_encoding($text));
// If it doesn't parse at all, just return the text. // If it doesn't parse at all, just return the text.
$dom = @DOMDocument::loadHTML($html_text); $dom = new DOMDocument();
$dom->loadHTML($html_text);
if (!$dom) { if (!$dom) {
return $text; return $text;
} }