Merge pull request #5823 from annando/ap-attach

Changed attachment handling to improve Mastodon compatibility
This commit is contained in:
Hypolite Petovan 2018-10-05 00:21:41 -04:00 committed by GitHub
commit 253ac95ca0
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -27,16 +27,16 @@ use Friendica\Protocol\ActivityPub;
* @brief ActivityPub Transmitter Protocol class * @brief ActivityPub Transmitter Protocol class
* *
* To-Do: * To-Do:
*
* Missing object types:
* - Event * - Event
* *
* Complicated: * Complicated object types:
* - Announce * - Announce
* - Undo Announce * - Undo Announce
* *
* General: * General:
* - Queueing unsucessful deliveries * - Queueing unsucessful deliveries
* - Type "note": Remove inline images and add them as attachments
* - Type "article": Leave imaged embedded and don't add them as attachments
*/ */
class Transmitter class Transmitter
{ {
@ -521,15 +521,9 @@ class Transmitter
$data['id'] = $item['uri'] . '#' . $type; $data['id'] = $item['uri'] . '#' . $type;
$data['type'] = $type; $data['type'] = $type;
$data['actor'] = $item['author-link']; $data['actor'] = $item['owner-link'];
$data['published'] = DateTimeFormat::utc($item["created"]."+00:00", DateTimeFormat::ATOM); $data['published'] = DateTimeFormat::utc($item['created'] . '+00:00', DateTimeFormat::ATOM);
if ($item["created"] != $item["edited"]) {
$data['updated'] = DateTimeFormat::utc($item["edited"]."+00:00", DateTimeFormat::ATOM);
}
$data['context'] = self::fetchContextURLForItem($item);
$data = array_merge($data, self::createPermissionBlockForItem($item)); $data = array_merge($data, self::createPermissionBlockForItem($item));
@ -607,9 +601,9 @@ class Transmitter
* *
* @param array $item Data of the item that is to be posted * @param array $item Data of the item that is to be posted
* @param text $type Object type * @param text $type Object type
*
* @return array with attachment data * @return array with attachment data
*/ */
private static function createAttachmentList($item, $type) private static function createAttachmentList($item, $type)
{ {
$attachments = []; $attachments = [];
@ -638,24 +632,43 @@ class Transmitter
return $attachments; return $attachments;
} }
/// @todo Replace this with a function that takes all pictures from the post // Simplify image codes
$siteinfo = BBCode::getAttachedData($item['body']); $body = preg_replace("/\[img\=([0-9]*)x([0-9]*)\](.*?)\[\/img\]/ism", '[img]$3[/img]', $item['body']);
if (!empty($siteinfo['image']) && // Grab all pictures and create attachments out of them
(($siteinfo['type'] == 'photo') || if (preg_match_all("/\[img\]([^\[\]]*)\[\/img\]/Usi", $body, $pictures)) {
!Config::get('system', 'ostatus_not_attach_preview'))) { foreach ($pictures[1] as $picture) {
$imgdata = Image::getInfoFromURL($siteinfo['image']); $imgdata = Image::getInfoFromURL($picture);
if ($imgdata) { if ($imgdata) {
$attachments[] = ['type' => 'Document', $attachments[] = ['type' => 'Document',
'mediaType' => $imgdata['mime'], 'mediaType' => $imgdata['mime'],
'url' => $siteinfo['image'], 'url' => $picture,
'name' => null]; 'name' => null];
} }
}
} }
return $attachments; return $attachments;
} }
/**
* @brief Remove image elements and replaces them with links to the image
*
* @param string $body
*
* @return string with replaced elements
*/
private static function removePictures($body)
{
// Simplify image codes
$body = preg_replace("/\[img\=([0-9]*)x([0-9]*)\](.*?)\[\/img\]/ism", '[img]$3[/img]', $body);
$body = preg_replace("/\[url=([^\[\]]*)\]\[img\](.*)\[\/img\]\[\/url\]/Usi", '[url]$1[/url]', $body);
$body = preg_replace("/\[img\]([^\[\]]*)\[\/img\]/Usi", '[url]$1[/url]', $body);
return $body;
}
/** /**
* @brief Fetches the "context" value for a givem item array from the "conversation" table * @brief Fetches the "context" value for a givem item array from the "conversation" table
* *
@ -676,6 +689,13 @@ class Transmitter
return $context_uri; return $context_uri;
} }
/**
* @brief Returns if the post contains sensitive content ("nsfw")
*
* @param integer $item_id
*
* @return boolean
*/
private static function isSensitive($item_id) private static function isSensitive($item_id)
{ {
$condition = ['otype' => TERM_OBJ_POST, 'oid' => $item_id, 'type' => TERM_HASHTAG, 'term' => 'nsfw']; $condition = ['otype' => TERM_OBJ_POST, 'oid' => $item_id, 'type' => TERM_HASHTAG, 'term' => 'nsfw'];
@ -718,10 +738,10 @@ class Transmitter
} }
$data['diaspora:guid'] = $item['guid']; $data['diaspora:guid'] = $item['guid'];
$data['published'] = DateTimeFormat::utc($item["created"]."+00:00", DateTimeFormat::ATOM); $data['published'] = DateTimeFormat::utc($item['created'] . '+00:00', DateTimeFormat::ATOM);
if ($item["created"] != $item["edited"]) { if ($item['created'] != $item['edited']) {
$data['updated'] = DateTimeFormat::utc($item["edited"]."+00:00", DateTimeFormat::ATOM); $data['updated'] = DateTimeFormat::utc($item['edited'] . '+00:00', DateTimeFormat::ATOM);
} }
$data['url'] = $item['plink']; $data['url'] = $item['plink'];
@ -734,7 +754,13 @@ class Transmitter
$data['name'] = BBCode::convert($item['title'], false, 7); $data['name'] = BBCode::convert($item['title'], false, 7);
} }
$data['content'] = BBCode::convert($item['body'], false, 7); $body = $item['body'];
if ($type == 'Note') {
$body = self::removePictures($body);
}
$data['content'] = BBCode::convert($body, false, 7);
$data['source'] = ['content' => $item['body'], 'mediaType' => "text/bbcode"]; $data['source'] = ['content' => $item['body'], 'mediaType' => "text/bbcode"];
if (!empty($item['signed_text']) && ($item['uri'] != $item['thr-parent'])) { if (!empty($item['signed_text']) && ($item['uri'] != $item['thr-parent'])) {
@ -770,7 +796,7 @@ class Transmitter
$signed = LDSignature::sign($data, $owner); $signed = LDSignature::sign($data, $owner);
logger('Deliver profile deletion for user ' . $uid . ' to ' . $inbox .' via ActivityPub', LOGGER_DEBUG); logger('Deliver profile deletion for user ' . $uid . ' to ' . $inbox . ' via ActivityPub', LOGGER_DEBUG);
HTTPSignature::transmit($signed, $inbox, $uid); HTTPSignature::transmit($signed, $inbox, $uid);
} }
@ -796,7 +822,7 @@ class Transmitter
$signed = LDSignature::sign($data, $owner); $signed = LDSignature::sign($data, $owner);
logger('Deliver profile update for user ' . $uid . ' to ' . $inbox .' via ActivityPub', LOGGER_DEBUG); logger('Deliver profile update for user ' . $uid . ' to ' . $inbox . ' via ActivityPub', LOGGER_DEBUG);
HTTPSignature::transmit($signed, $inbox, $uid); HTTPSignature::transmit($signed, $inbox, $uid);
} }
@ -854,7 +880,7 @@ class Transmitter
} }
/** /**
* @brief * @brief Reject a contact request or terminates the contact relation
* *
* @param string $target Target profile * @param string $target Target profile
* @param $id * @param $id
@ -881,7 +907,7 @@ class Transmitter
} }
/** /**
* @brief * @brief Transmits a message that we don't want to follow this contact anymore
* *
* @param string $target Target profile * @param string $target Target profile
* @param integer $uid User ID * @param integer $uid User ID