diff --git a/src/Content/Text/BBCode.php b/src/Content/Text/BBCode.php
index 3054170fc0..2dc13ae5e0 100644
--- a/src/Content/Text/BBCode.php
+++ b/src/Content/Text/BBCode.php
@@ -572,17 +572,17 @@ class BBCode extends BaseObject
$return = sprintf('
', $data["type"]);
}
- if (!empty($data["image"])) {
- $return .= sprintf('

', $data["url"], self::proxyUrl($data["image"], $simplehtml), $data["title"]);
- } elseif (!empty($data["preview"])) {
- $return .= sprintf('

', $data["url"], self::proxyUrl($data["preview"], $simplehtml), $data["title"]);
- }
-
- // Show a picture only when the BBCode is meant for posts to connector networks
- if (($simplehtml != 0) && ($data["type"] == "photo") && !empty($data["url"]) && !empty($data["image"])) {
- $return .= sprintf('

', $data["url"], self::proxyUrl($data["image"], $simplehtml), $data["title"]);
- } elseif (!empty($data['title']) && !empty($data['url'])) {
- $return .= sprintf('
', $data['url'], $data['title']);
+ if (!empty($data['title']) && !empty($data['url'])) {
+ if (!empty($data["image"]) && empty($data["text"]) && ($data["type"] == "photo")) {
+ $return .= sprintf('

', $data["url"], self::proxyUrl($data["image"], $simplehtml), $data["title"]);
+ } else {
+ if (!empty($data["image"])) {
+ $return .= sprintf('

', $data["url"], self::proxyUrl($data["image"], $simplehtml), $data["title"]);
+ } elseif (!empty($data["preview"])) {
+ $return .= sprintf('

', $data["url"], self::proxyUrl($data["preview"], $simplehtml), $data["title"]);
+ }
+ $return .= sprintf('
', $data['url'], $data['title']);
+ }
}
if (!empty($data["description"]) && $data["description"] != $data["title"]) {
diff --git a/src/Util/ParseUrl.php b/src/Util/ParseUrl.php
index 520100b6d7..4443da7564 100644
--- a/src/Util/ParseUrl.php
+++ b/src/Util/ParseUrl.php
@@ -271,11 +271,9 @@ class ParseUrl
$siteinfo['image'] = $meta_tag['content'];
break;
case 'twitter:card':
- // Convert Twitter types in our own
+ // Detect photo pages
if ($meta_tag['content'] == 'summary_large_image') {
$siteinfo['type'] = 'photo';
- } else {
- $siteinfo['type'] = $meta_tag['content'];
}
break;
case 'twitter:description':
@@ -334,6 +332,11 @@ class ParseUrl
}
}
+ // Prevent to have a photo type without an image
+ if (empty($siteinfo['image']) && (siteinfo['type'] == 'photo')) {
+ $siteinfo['type'] = 'link';
+ }
+
if ((@$siteinfo['image'] == '') && !$no_guessing) {
$list = $xpath->query('//img[@src]');
foreach ($list as $node) {