From: Mikael Nordfeldth Date: Thu, 25 Feb 2016 18:46:17 +0000 (+0100) Subject: $metadata->thumbnail_url is not guaranteed to be set X-Git-Url: https://git.mxchange.org/?a=commitdiff_plain;h=80f7a5f025654c88bde16acfce40bda2af8c4526;p=quix0rs-gnu-social.git $metadata->thumbnail_url is not guaranteed to be set We should probably have a separate class for this, so we can more easily combine different technologies similar to oEmbed/OpenGraph. --- diff --git a/plugins/Oembed/OembedPlugin.php b/plugins/Oembed/OembedPlugin.php index c90f57439b..44e4ac9318 100644 --- a/plugins/Oembed/OembedPlugin.php +++ b/plugins/Oembed/OembedPlugin.php @@ -73,18 +73,20 @@ class OembedPlugin extends Plugin $metadata = OpenGraphHelper::ogFromHtml($dom); } - // sometimes sites serve the path, not the full URL, for images - // let's "be liberal in what you accept from others"! - // add protocol and host if the thumbnail_url starts with / - if(substr($metadata->thumbnail_url,0,1) == '/') { - $thumbnail_url_parsed = parse_url($metadata->url); - $metadata->thumbnail_url = $thumbnail_url_parsed['scheme']."://".$thumbnail_url_parsed['host'].$metadata->thumbnail_url; - } + if (isset($metadata->thumbnail_url)) { + // sometimes sites serve the path, not the full URL, for images + // let's "be liberal in what you accept from others"! + // add protocol and host if the thumbnail_url starts with / + if(substr($metadata->thumbnail_url,0,1) == '/') { + $thumbnail_url_parsed = parse_url($metadata->url); + $metadata->thumbnail_url = $thumbnail_url_parsed['scheme']."://".$thumbnail_url_parsed['host'].$metadata->thumbnail_url; + } - // some wordpress opengraph implementations sometimes return a white blank image - // no need for us to save that! - if($metadata->thumbnail_url == 'https://s0.wp.com/i/blank.jpg') { - unset($metadata->thumbnail_url); + // some wordpress opengraph implementations sometimes return a white blank image + // no need for us to save that! + if($metadata->thumbnail_url == 'https://s0.wp.com/i/blank.jpg') { + unset($metadata->thumbnail_url); + } } }