Avoid several notices in ParseUrl

This commit is contained in:
Michael 2021-03-23 20:03:08 +00:00
parent e6f4ed47e2
commit fb663f0668
1 changed files with 19 additions and 14 deletions

View File

@ -512,8 +512,10 @@ class ParseUrl
{
if (!empty($siteinfo['images'])) {
array_walk($siteinfo['images'], function (&$image) use ($page_url) {
$image = [];
// According to the specifications someone could place a picture url into the content field as well.
// But this doesn't seem to happen in the wild, so we don't cover it here.
if (!empty($image['url'])) {
$image['url'] = self::completeUrl($image['url'], $page_url);
$photodata = Images::getInfoFromURLCached($image['url']);
if (!empty($photodata) && ($photodata[0] > 50) && ($photodata[1] > 50)) {
@ -523,8 +525,7 @@ class ParseUrl
$image['contenttype'] = $photodata['mime'];
unset($image['url']);
ksort($image);
} else {
$image = [];
}
}
});
@ -542,7 +543,7 @@ class ParseUrl
if (!empty($media[$field])) {
$media[$field] = self::completeUrl($media[$field], $page_url);
$type = self::getContentType($media[$field]);
if ($type[0] == 'text') {
if (($type[0] ?? '') == 'text') {
if ($field == 'embed') {
$embed = $media[$field];
} else {
@ -685,8 +686,10 @@ class ParseUrl
{
if (!empty($jsonld['@graph']) && is_array($jsonld['@graph'])) {
foreach ($jsonld['@graph'] as $part) {
if (!empty($part)) {
$siteinfo = self::parseParts($siteinfo, $part);
}
}
} elseif (!empty($jsonld['@type'])) {
$siteinfo = self::parseJsonLd($siteinfo, $jsonld);
} elseif (!empty($jsonld)) {
@ -699,10 +702,12 @@ class ParseUrl
}
if ($numeric_keys) {
foreach ($jsonld as $part) {
if (!empty($part)) {
$siteinfo = self::parseParts($siteinfo, $part);
}
}
}
}
array_walk_recursive($siteinfo, function (&$element) {
if (is_string($element)) {