X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=src%2FContent%2FOEmbed.php;h=df2cafea45059b878fa05cd41e7fc84fc2697c82;hb=ddc3c0db03267d29ace272b504dc6f803fa02990;hp=65b450f51e84f0b52fdcad9ce401b40cc338d7f6;hpb=2bda6980bbd36ef011e6c19502ea2a0fe3376d02;p=friendica.git
diff --git a/src/Content/OEmbed.php b/src/Content/OEmbed.php
index 65b450f51e..df2cafea45 100644
--- a/src/Content/OEmbed.php
+++ b/src/Content/OEmbed.php
@@ -1,6 +1,6 @@
videowidth . ':' . $embedurl;
+ $cache_key = 'oembed:' . $a->getThemeInfoValue('videowidth') . ':' . $embedurl;
- $condition = ['url' => Strings::normaliseLink($embedurl), 'maxwidth' => $a->videowidth];
+ $condition = ['url' => Strings::normaliseLink($embedurl), 'maxwidth' => $a->getThemeInfoValue('videowidth')];
$oembed_record = DBA::selectFirst('oembed', ['content'], $condition);
if (DBA::isResult($oembed_record)) {
$json_string = $oembed_record['content'];
@@ -97,7 +104,7 @@ class OEmbed
if (!in_array($ext, $noexts)) {
// try oembed autodiscovery
- $html_text = DI::httpRequest()->fetch($embedurl, 15, 'text/*');
+ $html_text = DI::httpClient()->fetch($embedurl, HttpClientAccept::HTML, 15);
if (!empty($html_text)) {
$dom = new DOMDocument();
if (@$dom->loadHTML($html_text)) {
@@ -111,7 +118,7 @@ class OEmbed
// but their OEmbed endpoint is only accessible by HTTPS ¯\_(ã)_/¯
$href = str_replace(['http://www.youtube.com/', 'http://player.vimeo.com/'],
['https://www.youtube.com/', 'https://player.vimeo.com/'], $href);
- $result = DI::httpRequest()->fetchFull($href . '&maxwidth=' . $a->videowidth);
+ $result = DI::httpClient()->fetchFull($href . '&maxwidth=' . $a->getThemeInfoValue('videowidth'));
if ($result->getReturnCode() === 200) {
$json_string = $result->getBody();
break;
@@ -132,7 +139,7 @@ class OEmbed
if (!empty($oembed->type) && $oembed->type != 'error') {
DBA::insert('oembed', [
'url' => Strings::normaliseLink($embedurl),
- 'maxwidth' => $a->videowidth,
+ 'maxwidth' => $a->getThemeInfoValue('videowidth'),
'content' => $json_string,
'created' => DateTimeFormat::utcNow()
], Database::INSERT_UPDATE);
@@ -208,12 +215,18 @@ class OEmbed
return $oembed;
}
- private static function formatObject(\Friendica\Object\OEmbed $oembed)
+ /**
+ * Returns a formatted string from OEmbed object
+ *
+ * @param \Friendica\Object\OEmbed $oembed
+ * @return string
+ */
+ private static function formatObject(\Friendica\Object\OEmbed $oembed): string
{
$ret = '
';
switch ($oembed->type) {
- case "video":
+ case 'video':
if ($oembed->thumbnail_url) {
$tw = (isset($oembed->thumbnail_width) && intval($oembed->thumbnail_width)) ? $oembed->thumbnail_width : 200;
$th = (isset($oembed->thumbnail_height) && intval($oembed->thumbnail_height)) ? $oembed->thumbnail_height : 180;
@@ -235,15 +248,15 @@ class OEmbed
}
break;
- case "photo":
- $ret .= '
![](' . ProxyUtils::proxifyUrl($oembed->url) . ')
';
+ case 'photo':
+ $ret .= '
![](' . Proxy::proxifyUrl($oembed->url) . ')
';
break;
- case "link":
+ case 'link':
break;
- case "rich":
- $ret .= ProxyUtils::proxifyHtml($oembed->html);
+ case 'rich':
+ $ret .= Proxy::proxifyHtml($oembed->html);
break;
}
@@ -291,48 +304,52 @@ class OEmbed
return str_replace("\n", "", $ret);
}
- public static function BBCode2HTML($text)
+ /**
+ * Converts BBCode to HTML code
+ *
+ * @param string $text
+ * @return string
+ */
+ public static function BBCode2HTML(string $text): string
{
- $stopoembed = DI::config()->get("system", "no_oembed");
- if ($stopoembed == true) {
+ if (DI::config()->get('system', 'no_oembed')) {
return preg_replace("/\[embed\](.+?)\[\/embed\]/is", "
" . DI::l10n()->t('Embedding disabled') . " : $1", $text);
}
- return preg_replace_callback("/\[embed\](.+?)\[\/embed\]/is", ['self', 'replaceCallback'], $text);
+ return preg_replace_callback("/\[embed\](.+?)\[\/embed\]/is", [self::class, 'replaceCallback'], $text);
}
/**
* Find
....
* and replace it with [embed]url[/embed]
*
- * @param $text
+ * @param string $text
* @return string
*/
- public static function HTML2BBCode($text)
+ public static function HTML2BBCode(string $text): string
{
// start parser only if 'oembed' is in text
- if (strpos($text, "oembed")) {
-
+ if (strpos($text, 'oembed')) {
// convert non ascii chars to html entities
$html_text = mb_convert_encoding($text, 'HTML-ENTITIES', mb_detect_encoding($text));
// If it doesn't parse at all, just return the text.
- $dom = @DOMDocument::loadHTML($html_text);
- if (!$dom) {
+ $dom = new DOMDocument();
+ if (!@$dom->loadHTML($html_text)) {
return $text;
}
$xpath = new DOMXPath($dom);
- $xattr = self::buildXPath("class", "oembed");
+ $xattr = self::buildXPath('class', 'oembed');
$entries = $xpath->query("//div[$xattr]");
$xattr = "@rel='oembed'"; //oe_build_xpath("rel","oembed");
foreach ($entries as $e) {
$href = $xpath->evaluate("a[$xattr]/@href", $e)->item(0)->nodeValue;
if (!is_null($href)) {
- $e->parentNode->replaceChild(new DOMText("[embed]" . $href . "[/embed]"), $e);
+ $e->parentNode->replaceChild(new DOMText('[embed]' . $href . '[/embed]'), $e);
}
}
- return self::getInnerHTML($dom->getElementsByTagName("body")->item(0));
+ return self::getInnerHTML($dom->getElementsByTagName('body')->item(0));
} else {
return $text;
}
@@ -345,7 +362,7 @@ class OEmbed
* @return boolean
* @throws \Friendica\Network\HTTPException\InternalServerErrorException
*/
- public static function isAllowedURL($url)
+ public static function isAllowedURL(string $url): bool
{
if (!DI::config()->get('system', 'no_oembed_rich_content')) {
return true;
@@ -366,7 +383,14 @@ class OEmbed
return Network::isDomainAllowed($domain, $allowed);
}
- public static function getHTML($url, $title = null)
+ /**
+ * Returns a formatted HTML code from given URL and sets optional title
+ *
+ * @param string $url URL to fetch
+ * @param string $title Optional title (default: what comes from OEmbed object)
+ * @return string Formatted HTML
+ */
+ public static function getHTML(string $url, string $title = ''): string
{
$o = self::fetchURL($url, !self::isAllowedURL($url));
@@ -400,12 +424,12 @@ class OEmbed
* @param string $src Original remote URL to embed
* @param string $width
* @param string $height
- * @return string formatted HTML
+ * @return string Formatted HTML
*
* @throws \Friendica\Network\HTTPException\InternalServerErrorException
* @see oembed_format_object()
*/
- private static function iframe($src, $width, $height)
+ private static function iframe(string $src, string $width, string $height): string
{
if (!$height || strstr($height, '%')) {
$height = '200';
@@ -422,11 +446,11 @@ class OEmbed
* Generates an XPath query to select elements whose provided attribute contains
* the provided value in a space-separated list.
*
- * @param string $attr Name of the attribute to seach
+ * @param string $attr Name of the attribute to search
* @param string $value Value to search in a space-separated list
* @return string
*/
- private static function buildXPath($attr, $value)
+ private static function buildXPath(string $attr, $value): string
{
// https://www.westhoffswelt.de/blog/2009/6/9/select-html-elements-with-more-than-one-css-class-using-xpath
return "contains(normalize-space(@$attr), ' $value ') or substring(normalize-space(@$attr), 1, string-length('$value') + 1) = '$value ' or substring(normalize-space(@$attr), string-length(@$attr) - string-length('$value')) = ' $value' or @$attr = '$value'";
@@ -438,7 +462,7 @@ class OEmbed
* @param DOMNode $node
* @return string
*/
- private static function getInnerHTML(DOMNode $node)
+ private static function getInnerHTML(DOMNode $node): string
{
$innerHTML = '';
$children = $node->childNodes;