Removing unused functions, added (deactivated) attachments

This commit is contained in:
Michael 2021-11-23 21:54:19 +00:00
parent 0c3e491410
commit 07c2f36946
6 changed files with 264 additions and 752 deletions

View file

@ -24,7 +24,6 @@
*/ */
use Friendica\App; use Friendica\App;
use Friendica\Content\ContactSelector;
use Friendica\Content\Text\BBCode; use Friendica\Content\Text\BBCode;
use Friendica\Content\Text\HTML; use Friendica\Content\Text\HTML;
use Friendica\Core\Logger; use Friendica\Core\Logger;
@ -262,14 +261,6 @@ function api_account_verify_credentials($type)
// "verified" isn't used here in the standard // "verified" isn't used here in the standard
unset($user_info["verified"]); unset($user_info["verified"]);
// - Adding last status
if (!$skip_status) {
$item = api_get_last_status($user_info['pid'], 0);
if (!empty($item)) {
$user_info['status'] = api_format_item($item, $type);
}
}
// "uid" is only needed for some internal stuff, so remove it from here // "uid" is only needed for some internal stuff, so remove it from here
unset($user_info['uid']); unset($user_info['uid']);
@ -335,7 +326,8 @@ function api_statuses_mediap($type)
$item_id = item_post($a); $item_id = item_post($a);
// output the post that we just posted. // output the post that we just posted.
return api_status_show($type, $item_id); $status_info = DI::twitterStatus()->createFromItemId($item_id)->toArray();
return DI::apiResponse()->formatData('statuses', $type, ['status' => $status_info]);
} }
/// @TODO move this to top of file or somewhere better! /// @TODO move this to top of file or somewhere better!
@ -516,7 +508,8 @@ function api_statuses_update($type)
} }
// output the post that we just posted. // output the post that we just posted.
return api_status_show($type, $item_id); $status_info = DI::twitterStatus()->createFromItemId($item_id)->toArray();
return DI::apiResponse()->formatData('statuses', $type, ['status' => $status_info]);
} }
/// @TODO move to top of file or somewhere better /// @TODO move to top of file or somewhere better
@ -619,64 +612,6 @@ function api_media_metadata_create($type)
api_register_func('api/media/metadata/create', 'api_media_metadata_create', true, API_METHOD_POST); api_register_func('api/media/metadata/create', 'api_media_metadata_create', true, API_METHOD_POST);
/**
* @param string $type Return format (atom, rss, xml, json)
* @param int $item_id
* @return array|string
* @throws Exception
*/
function api_status_show($type, $item_id)
{
Logger::info(API_LOG_PREFIX . 'Start', ['action' => 'status_show', 'type' => $type, 'item_id' => $item_id]);
$status_info = [];
$item = api_get_item(['id' => $item_id]);
if (!empty($item)) {
$status_info = api_format_item($item, $type);
}
Logger::info(API_LOG_PREFIX . 'End', ['action' => 'get_status', 'status_info' => $status_info]);
return DI::apiResponse()->formatData('statuses', $type, ['status' => $status_info]);
}
/**
* Retrieves the last public status of the provided user info
*
* @param int $ownerId Public contact Id
* @param int $uid User Id
* @return array
* @throws Exception
*/
function api_get_last_status($ownerId, $uid)
{
$condition = [
'author-id'=> $ownerId,
'uid' => $uid,
'gravity' => [GRAVITY_PARENT, GRAVITY_COMMENT],
'private' => [Item::PUBLIC, Item::UNLISTED]
];
$item = api_get_item($condition);
return $item;
}
/**
* Retrieves a single item record based on the provided condition and converts it for API use.
*
* @param array $condition Item table condition array
* @return array
* @throws Exception
*/
function api_get_item(array $condition)
{
$item = Post::selectFirst(Item::DISPLAY_FIELDLIST, $condition, ['order' => ['id' => true]]);
return $item;
}
/** /**
* Returns extended information of a given user, specified by ID or screen name as per the required id parameter. * Returns extended information of a given user, specified by ID or screen name as per the required id parameter.
* The author's most recent status will be returned inline. * The author's most recent status will be returned inline.
@ -696,9 +631,16 @@ function api_users_show($type)
$user_info = DI::twitterUser()->createFromUserId($uid)->toArray(); $user_info = DI::twitterUser()->createFromUserId($uid)->toArray();
$item = api_get_last_status($user_info['pid'], 0); $condition = [
'author-id'=> $user_info['pid'],
'uid' => $uid,
'gravity' => [GRAVITY_PARENT, GRAVITY_COMMENT],
'private' => [Item::PUBLIC, Item::UNLISTED]
];
$item = Post::selectFirst(['uri-id', 'id'], $condition);
if (!empty($item)) { if (!empty($item)) {
$user_info['status'] = api_format_item($item, $type); $user_info['status'] = DI::twitterStatus()->createFromUriId($item['uri-id'], $item['uid'])->toArray();
} }
// "uid" is only needed for some internal stuff, so remove it from here // "uid" is only needed for some internal stuff, so remove it from here
@ -898,7 +840,7 @@ function api_search($type)
$ret = []; $ret = [];
while ($status = DBA::fetch($statuses)) { while ($status = DBA::fetch($statuses)) {
$ret[] = api_format_item($status, $type); $ret[] = DI::twitterStatus()->createFromUriId($status['uri-id'], $status['uid'])->toArray();
} }
DBA::close($statuses); DBA::close($statuses);
@ -972,7 +914,7 @@ function api_statuses_home_timeline($type)
$ret = []; $ret = [];
$idarray = []; $idarray = [];
while ($status = DBA::fetch($statuses)) { while ($status = DBA::fetch($statuses)) {
$ret[] = api_format_item($status, $type); $ret[] = DI::twitterStatus()->createFromUriId($status['uri-id'], $status['uid'])->toArray();
$idarray[] = intval($status['id']); $idarray[] = intval($status['id']);
} }
DBA::close($statuses); DBA::close($statuses);
@ -1051,7 +993,7 @@ function api_statuses_public_timeline($type)
$ret = []; $ret = [];
while ($status = DBA::fetch($statuses)) { while ($status = DBA::fetch($statuses)) {
$ret[] = api_format_item($status, $type); $ret[] = DI::twitterStatus()->createFromUriId($status['uri-id'], $status['uid'])->toArray();
} }
DBA::close($statuses); DBA::close($statuses);
@ -1099,7 +1041,7 @@ function api_statuses_networkpublic_timeline($type)
$ret = []; $ret = [];
while ($status = DBA::fetch($statuses)) { while ($status = DBA::fetch($statuses)) {
$ret[] = api_format_item($status, $type); $ret[] = DI::twitterStatus()->createFromUriId($status['uri-id'], $status['uid'])->toArray();
} }
DBA::close($statuses); DBA::close($statuses);
@ -1173,7 +1115,7 @@ function api_statuses_show($type)
$ret = []; $ret = [];
while ($status = DBA::fetch($statuses)) { while ($status = DBA::fetch($statuses)) {
$ret[] = api_format_item($status, $type); $ret[] = DI::twitterStatus()->createFromUriId($status['uri-id'], $status['uid'])->toArray();
} }
DBA::close($statuses); DBA::close($statuses);
@ -1256,7 +1198,7 @@ function api_conversation_show($type)
$ret = []; $ret = [];
while ($status = DBA::fetch($statuses)) { while ($status = DBA::fetch($statuses)) {
$ret[] = api_format_item($status, $type); $ret[] = DI::twitterStatus()->createFromUriId($status['uri-id'], $status['uid'])->toArray();
} }
DBA::close($statuses); DBA::close($statuses);
@ -1339,7 +1281,8 @@ function api_statuses_repeat($type)
} }
// output the post that we just posted. // output the post that we just posted.
return api_status_show($type, $item_id); $status_info = DI::twitterStatus()->createFromItemId($item_id)->toArray();
return DI::apiResponse()->formatData('statuses', $type, ['status' => $status_info]);
} }
/// @TODO move to top of file or somewhere better /// @TODO move to top of file or somewhere better
@ -1446,7 +1389,7 @@ function api_statuses_mentions($type)
$ret = []; $ret = [];
while ($status = DBA::fetch($statuses)) { while ($status = DBA::fetch($statuses)) {
$ret[] = api_format_item($status, $type); $ret[] = DI::twitterStatus()->createFromUriId($status['uri-id'], $status['uid'])->toArray();
} }
DBA::close($statuses); DBA::close($statuses);
@ -1510,7 +1453,7 @@ function api_statuses_user_timeline($type)
$ret = []; $ret = [];
while ($status = DBA::fetch($statuses)) { while ($status = DBA::fetch($statuses)) {
$ret[] = api_format_item($status, $type); $ret[] = DI::twitterStatus()->createFromUriId($status['uri-id'], $status['uid'])->toArray();
} }
DBA::close($statuses); DBA::close($statuses);
@ -1579,7 +1522,7 @@ function api_favorites_create_destroy($type)
throw new InternalServerErrorException("DB error"); throw new InternalServerErrorException("DB error");
} }
$ret = api_format_item($item, $type); $ret = DI::twitterStatus()->createFromUriId($item['uri-id'], $item['uid'])->toArray();
return DI::apiResponse()->formatData("status", $type, ['status' => $ret], Contact::getPublicIdByUserId($uid)); return DI::apiResponse()->formatData("status", $type, ['status' => $ret], Contact::getPublicIdByUserId($uid));
} }
@ -1631,7 +1574,7 @@ function api_favorites($type)
$ret = []; $ret = [];
while ($status = DBA::fetch($statuses)) { while ($status = DBA::fetch($statuses)) {
$ret[] = api_format_item($status, $type); $ret[] = DI::twitterStatus()->createFromUriId($status['uri-id'], $status['uid'])->toArray();
} }
DBA::close($statuses); DBA::close($statuses);
@ -1695,346 +1638,6 @@ function api_format_messages($item, $recipient, $sender)
return $ret; return $ret;
} }
/**
*
* @param array $item
*
* @return array
* @throws InternalServerErrorException
*/
function api_convert_item($item)
{
$body = api_add_attachments_to_body($item);
$entities = api_get_entitities($statustext, $body, $item['uri-id']);
// Add pictures to the attachment array and remove them from the body
$attachments = api_get_attachments($body, $item['uri-id']);
// Workaround for ostatus messages where the title is identically to the body
$html = BBCode::convertForUriId($item['uri-id'], api_clean_plain_items($body), BBCode::API);
$statusbody = trim(HTML::toPlaintext($html, 0));
// handle data: images
$statusbody = api_format_items_embeded_images($item, $statusbody);
$statustitle = trim($item['title']);
if (($statustitle != '') && (strpos($statusbody, $statustitle) !== false)) {
$statustext = trim($statusbody);
} else {
$statustext = trim($statustitle."\n\n".$statusbody);
}
if ((($item['network'] ?? Protocol::PHANTOM) == Protocol::FEED) && (mb_strlen($statustext)> 1000)) {
$statustext = mb_substr($statustext, 0, 1000) . "... \n" . ($item['plink'] ?? '');
}
$statushtml = BBCode::convertForUriId($item['uri-id'], BBCode::removeAttachment($body), BBCode::API);
// Workaround for clients with limited HTML parser functionality
$search = ["<br>", "<blockquote>", "</blockquote>",
"<h1>", "</h1>", "<h2>", "</h2>",
"<h3>", "</h3>", "<h4>", "</h4>",
"<h5>", "</h5>", "<h6>", "</h6>"];
$replace = ["<br>", "<br><blockquote>", "</blockquote><br>",
"<br><h1>", "</h1><br>", "<br><h2>", "</h2><br>",
"<br><h3>", "</h3><br>", "<br><h4>", "</h4><br>",
"<br><h5>", "</h5><br>", "<br><h6>", "</h6><br>"];
$statushtml = str_replace($search, $replace, $statushtml);
if ($item['title'] != "") {
$statushtml = "<br><h4>" . BBCode::convertForUriId($item['uri-id'], $item['title']) . "</h4><br>" . $statushtml;
}
do {
$oldtext = $statushtml;
$statushtml = str_replace("<br><br>", "<br>", $statushtml);
} while ($oldtext != $statushtml);
if (substr($statushtml, 0, 4) == '<br>') {
$statushtml = substr($statushtml, 4);
}
if (substr($statushtml, 0, -4) == '<br>') {
$statushtml = substr($statushtml, -4);
}
// feeds without body should contain the link
if ((($item['network'] ?? Protocol::PHANTOM) == Protocol::FEED) && (strlen($item['body']) == 0)) {
$statushtml .= BBCode::convertForUriId($item['uri-id'], $item['plink']);
}
return [
"text" => $statustext,
"html" => $statushtml,
"attachments" => $attachments,
"entities" => $entities
];
}
/**
* Add media attachments to the body
*
* @param array $item
* @return string body with added media
*/
function api_add_attachments_to_body(array $item)
{
$body = Post\Media::addAttachmentsToBody($item['uri-id'], $item['body']);
if (strpos($body, '[/img]') !== false) {
return $body;
}
foreach (Post\Media::getByURIId($item['uri-id'], [Post\Media::HTML]) as $media) {
if (!empty($media['preview'])) {
$description = $media['description'] ?: $media['name'];
if (!empty($description)) {
$body .= "\n[img=" . $media['preview'] . ']' . $description .'[/img]';
} else {
$body .= "\n[img]" . $media['preview'] .'[/img]';
}
}
}
return $body;
}
/**
*
* @param string $body
* @param int $uriid
*
* @return array
* @throws InternalServerErrorException
*/
function api_get_attachments(&$body, $uriid)
{
$body = preg_replace("/\[img\=([0-9]*)x([0-9]*)\](.*?)\[\/img\]/ism", '[img]$3[/img]', $body);
$body = preg_replace("/\[img\=(.*?)\](.*?)\[\/img\]/ism", '[img]$1[/img]', $body);
$URLSearchString = "^\[\]";
if (!preg_match_all("/\[img\]([$URLSearchString]*)\[\/img\]/ism", $body, $images)) {
return [];
}
// Remove all embedded pictures, since they are added as attachments
foreach ($images[0] as $orig) {
$body = str_replace($orig, '', $body);
}
$attachments = [];
foreach ($images[1] as $image) {
$imagedata = Images::getInfoFromURLCached($image);
if ($imagedata) {
$attachments[] = ["url" => Post\Link::getByLink($uriid, $image), "mimetype" => $imagedata["mime"], "size" => $imagedata["size"]];
}
}
return $attachments;
}
/**
*
* @param string $text
* @param string $bbcode
*
* @return array
* @throws InternalServerErrorException
* @todo Links at the first character of the post
*/
function api_get_entitities(&$text, $bbcode, $uriid)
{
$include_entities = strtolower($_REQUEST['include_entities'] ?? 'false');
if ($include_entities != "true") {
preg_match_all("/\[img](.*?)\[\/img\]/ism", $bbcode, $images);
foreach ($images[1] as $image) {
$replace = Post\Link::getByLink($uriid, $image);
$text = str_replace($image, $replace, $text);
}
return [];
}
$bbcode = BBCode::cleanPictureLinks($bbcode);
// Change pure links in text to bbcode uris
$bbcode = preg_replace("/([^\]\='".'"'."]|^)(https?\:\/\/[a-zA-Z0-9\:\/\-\?\&\;\.\=\_\~\#\%\$\!\+\,]+)/ism", '$1[url=$2]$2[/url]', $bbcode);
$entities = [];
$entities["hashtags"] = [];
$entities["symbols"] = [];
$entities["urls"] = [];
$entities["user_mentions"] = [];
$URLSearchString = "^\[\]";
$bbcode = preg_replace("/#\[url\=([$URLSearchString]*)\](.*?)\[\/url\]/ism", '#$2', $bbcode);
$bbcode = preg_replace("/\[bookmark\=([$URLSearchString]*)\](.*?)\[\/bookmark\]/ism", '[url=$1]$2[/url]', $bbcode);
$bbcode = preg_replace("/\[video\](.*?)\[\/video\]/ism", '[url=$1]$1[/url]', $bbcode);
$bbcode = preg_replace(
"/\[youtube\]([A-Za-z0-9\-_=]+)(.*?)\[\/youtube\]/ism",
'[url=https://www.youtube.com/watch?v=$1]https://www.youtube.com/watch?v=$1[/url]',
$bbcode
);
$bbcode = preg_replace("/\[youtube\](.*?)\[\/youtube\]/ism", '[url=$1]$1[/url]', $bbcode);
$bbcode = preg_replace(
"/\[vimeo\]([0-9]+)(.*?)\[\/vimeo\]/ism",
'[url=https://vimeo.com/$1]https://vimeo.com/$1[/url]',
$bbcode
);
$bbcode = preg_replace("/\[vimeo\](.*?)\[\/vimeo\]/ism", '[url=$1]$1[/url]', $bbcode);
$bbcode = preg_replace("/\[img\=([0-9]*)x([0-9]*)\](.*?)\[\/img\]/ism", '[img]$3[/img]', $bbcode);
preg_match_all("/\[url\=([$URLSearchString]*)\](.*?)\[\/url\]/ism", $bbcode, $urls);
$ordered_urls = [];
foreach ($urls[1] as $id => $url) {
$start = iconv_strpos($text, $url, 0, "UTF-8");
if (!($start === false)) {
$ordered_urls[$start] = ["url" => $url, "title" => $urls[2][$id]];
}
}
ksort($ordered_urls);
$offset = 0;
foreach ($ordered_urls as $url) {
if ((substr($url["title"], 0, 7) != "http://") && (substr($url["title"], 0, 8) != "https://")
&& !strpos($url["title"], "http://") && !strpos($url["title"], "https://")
) {
$display_url = $url["title"];
} else {
$display_url = str_replace(["http://www.", "https://www."], ["", ""], $url["url"]);
$display_url = str_replace(["http://", "https://"], ["", ""], $display_url);
if (strlen($display_url) > 26) {
$display_url = substr($display_url, 0, 25)."";
}
}
$start = iconv_strpos($text, $url["url"], $offset, "UTF-8");
if (!($start === false)) {
$entities["urls"][] = ["url" => $url["url"],
"expanded_url" => $url["url"],
"display_url" => $display_url,
"indices" => [$start, $start+strlen($url["url"])]];
$offset = $start + 1;
}
}
preg_match_all("/\[img\=(.*?)\](.*?)\[\/img\]/ism", $bbcode, $images, PREG_SET_ORDER);
$ordered_images = [];
foreach ($images as $image) {
$start = iconv_strpos($text, $image[1], 0, "UTF-8");
if (!($start === false)) {
$ordered_images[$start] = ['url' => $image[1], 'alt' => $image[2]];
}
}
preg_match_all("/\[img](.*?)\[\/img\]/ism", $bbcode, $images);
foreach ($images[1] as $image) {
$start = iconv_strpos($text, $image, 0, "UTF-8");
if (!($start === false)) {
$ordered_images[$start] = ['url' => $image, 'alt' => ''];
}
}
$offset = 0;
foreach ($ordered_images as $image) {
$url = $image['url'];
$ext_alt_text = $image['alt'];
$display_url = str_replace(["http://www.", "https://www."], ["", ""], $url);
$display_url = str_replace(["http://", "https://"], ["", ""], $display_url);
if (strlen($display_url) > 26) {
$display_url = substr($display_url, 0, 25)."";
}
$start = iconv_strpos($text, $url, $offset, "UTF-8");
if (!($start === false)) {
$image = Images::getInfoFromURLCached($url);
if ($image) {
$media_url = Post\Link::getByLink($uriid, $url);
$sizes["medium"] = ["w" => $image[0], "h" => $image[1], "resize" => "fit"];
$entities["media"][] = [
"id" => $start+1,
"id_str" => (string) ($start + 1),
"indices" => [$start, $start+strlen($url)],
"media_url" => Strings::normaliseLink($media_url),
"media_url_https" => $media_url,
"url" => $url,
"display_url" => $display_url,
"expanded_url" => $url,
"ext_alt_text" => $ext_alt_text,
"type" => "photo",
"sizes" => $sizes];
}
$offset = $start + 1;
}
}
return $entities;
}
/**
*
* @param array $item
* @param string $text
*
* @return string
*/
function api_format_items_embeded_images($item, $text)
{
$text = preg_replace_callback(
'|data:image/([^;]+)[^=]+=*|m',
function () use ($item) {
return DI::baseUrl() . '/display/' . $item['guid'];
},
$text
);
return $text;
}
/**
* return <a href='url'>name</a> as array
*
* @param string $txt text
* @return array
* 'name' => 'name',
* 'url => 'url'
*/
function api_contactlink_to_array($txt)
{
$match = [];
$r = preg_match_all('|<a href="([^"]*)">([^<]*)</a>|', $txt, $match);
if ($r && count($match)==3) {
$res = [
'name' => $match[2],
'url' => $match[1]
];
} else {
$res = [
'name' => $txt,
'url' => ""
];
}
return $res;
}
/** /**
* return likes, dislikes and attend status for item * return likes, dislikes and attend status for item
* *
@ -2112,175 +1715,6 @@ function api_format_items_activities($item, $type = "json")
return $activities; return $activities;
} }
/**
* @param array $item Item record
* @param string $type Return format (atom, rss, xml, json)
* @param array $status_user User record of the item author, can be provided by api_item_get_user()
* @param array $author_user User record of the item author, can be provided by api_item_get_user()
* @param array $owner_user User record of the item owner, can be provided by api_item_get_user()
* @return array API-formatted status
* @throws BadRequestException
* @throws ImagickException
* @throws InternalServerErrorException
* @throws UnauthorizedException
*/
function api_format_item($item, $type = "json")
{
return DI::twitterStatus()->createFromUriId($item['uri-id'], $item['uid'])->toArray();
$author_user = DI::twitterUser()->createFromContactId($item['author-id'], $item['uid'])->toArray();
$owner_user = DI::twitterUser()->createFromContactId($item['owner-id'], $item['uid'])->toArray();
DI::contentItem()->localize($item);
$in_reply_to = api_in_reply_to($item);
$converted = api_convert_item($item);
if ($type == "xml") {
$geo = "georss:point";
} else {
$geo = "geo";
}
$status = [
'text' => $converted["text"],
'truncated' => false,
'created_at'=> DateTimeFormat::utc($item['created'], DateTimeFormat::API),
'in_reply_to_status_id' => $in_reply_to['status_id'],
'in_reply_to_status_id_str' => $in_reply_to['status_id_str'],
'source' => (($item['app']) ? $item['app'] : 'web'),
'id' => intval($item['id']),
'id_str' => (string) intval($item['id']),
'in_reply_to_user_id' => $in_reply_to['user_id'],
'in_reply_to_user_id_str' => $in_reply_to['user_id_str'],
'in_reply_to_screen_name' => $in_reply_to['screen_name'],
$geo => null,
'favorited' => $item['starred'] ? true : false,
'user' => $author_user,
'friendica_author' => $author_user,
'friendica_owner' => $owner_user,
'friendica_private' => $item['private'] == Item::PRIVATE,
//'entities' => NULL,
'statusnet_html' => $converted["html"],
'statusnet_conversation_id' => $item['parent'],
'external_url' => DI::baseUrl() . "/display/" . $item['guid'],
'friendica_activities' => api_format_items_activities($item, $type),
'friendica_title' => $item['title'],
'friendica_html' => BBCode::convertForUriId($item['uri-id'], $item['body'], BBCode::EXTERNAL),
'friendica_comments' => Post::countPosts(['thr-parent-id' => $item['uri-id'], 'deleted' => false, 'gravity' => GRAVITY_COMMENT])
];
if (count($converted["attachments"]) > 0) {
$status["attachments"] = $converted["attachments"];
}
if (count($converted["entities"]) > 0) {
$status["entities"] = $converted["entities"];
}
if ($status["source"] == 'web') {
$status["source"] = ContactSelector::networkToName($item['author-network'], $item['author-link'], $item['network']);
} elseif (ContactSelector::networkToName($item['author-network'], $item['author-link'], $item['network']) != $status["source"]) {
$status["source"] = trim($status["source"].' ('.ContactSelector::networkToName($item['author-network'], $item['author-link'], $item['network']).')');
}
$retweeted_item = [];
$quoted_item = [];
if (empty($retweeted_item) && ($item['owner-id'] == $item['author-id'])) {
$announce = api_get_announce($item);
if (!empty($announce)) {
$retweeted_item = $item;
$item = $announce;
$status['friendica_owner'] = DI::twitterUser()->createFromContactId($announce['author-id'], $item['uid'])->toArray();
}
}
if (!empty($quoted_item)) {
if ($quoted_item['id'] != $item['id']) {
$quoted_status = api_format_item($quoted_item);
/// @todo Only remove the attachments that are also contained in the quotes status
unset($status['attachments']);
unset($status['entities']);
} else {
$conv_quoted = api_convert_item($quoted_item);
$quoted_status = $status;
unset($quoted_status['attachments']);
unset($quoted_status['entities']);
unset($quoted_status['statusnet_conversation_id']);
$quoted_status['text'] = $conv_quoted['text'];
$quoted_status['statusnet_html'] = $conv_quoted['html'];
try {
$quoted_status["user"] = DI::twitterUser()->createFromContactId($quoted_item['author-id'], $item['uid'])->toArray();
} catch (BadRequestException $e) {
// user not found. should be found?
/// @todo check if the user should be always found
$quoted_status["user"] = [];
}
}
unset($quoted_status['friendica_author']);
unset($quoted_status['friendica_owner']);
unset($quoted_status['friendica_activities']);
unset($quoted_status['friendica_private']);
}
if (!empty($retweeted_item)) {
$retweeted_status = $status;
unset($retweeted_status['friendica_author']);
unset($retweeted_status['friendica_owner']);
unset($retweeted_status['friendica_activities']);
unset($retweeted_status['friendica_private']);
unset($retweeted_status['statusnet_conversation_id']);
$status['user'] = $status['friendica_owner'];
try {
$retweeted_status["user"] = DI::twitterUser()->createFromContactId($retweeted_item['author-id'], $item['uid'])->toArray();
} catch (BadRequestException $e) {
// user not found. should be found?
/// @todo check if the user should be always found
$retweeted_status["user"] = [];
}
$rt_converted = api_convert_item($retweeted_item);
$retweeted_status['text'] = $rt_converted["text"];
$retweeted_status['statusnet_html'] = $rt_converted["html"];
$retweeted_status['friendica_html'] = $rt_converted["html"];
$retweeted_status['created_at'] = DateTimeFormat::utc($retweeted_item['created'], DateTimeFormat::API);
if (!empty($quoted_status)) {
$retweeted_status['quoted_status'] = $quoted_status;
}
$status['friendica_author'] = $retweeted_status['user'];
$status['retweeted_status'] = $retweeted_status;
} elseif (!empty($quoted_status)) {
$root_status = api_convert_item($item);
$status['text'] = $root_status["text"];
$status['statusnet_html'] = $root_status["html"];
$status['quoted_status'] = $quoted_status;
}
// "uid" is only needed for some internal stuff, so remove it from here
unset($status["user"]['uid']);
if ($item["coord"] != "") {
$coords = explode(' ', $item["coord"]);
if (count($coords) == 2) {
if ($type == "json") {
$status["geo"] = ['type' => 'Point',
'coordinates' => [(float) $coords[0],
(float) $coords[1]]];
} else {// Not sure if this is the official format - if someone founds a documentation we can check
$status["georss:point"] = $item["coord"];
}
}
}
return $status;
}
/** /**
* Returns all lists the user subscribes to. * Returns all lists the user subscribes to.
* *
@ -2406,7 +1840,7 @@ function api_lists_statuses($type)
$items = []; $items = [];
while ($status = DBA::fetch($statuses)) { while ($status = DBA::fetch($statuses)) {
$items[] = api_format_item($status, $type); $items[] = DI::twitterStatus()->createFromUriId($status['uri-id'], $status['uid'])->toArray();
} }
DBA::close($statuses); DBA::close($statuses);
@ -3641,7 +3075,7 @@ function prepare_photo_data($type, $scale, $photo_id)
// prepare output of comments // prepare output of comments
$commentData = []; $commentData = [];
while ($status = DBA::fetch($statuses)) { while ($status = DBA::fetch($statuses)) {
$commentData[] = api_format_item($status, $type); $commentData[] = DI::twitterStatus()->createFromUriId($status['uri-id'], $status['uid'])->toArray();
} }
DBA::close($statuses); DBA::close($statuses);
@ -3701,58 +3135,6 @@ function api_get_announce($item)
return array_merge($item, $announce); return array_merge($item, $announce);
} }
/**
*
* @param array $item
*
* @return array
* @throws Exception
*/
function api_in_reply_to($item)
{
$in_reply_to = [];
$in_reply_to['status_id'] = null;
$in_reply_to['user_id'] = null;
$in_reply_to['status_id_str'] = null;
$in_reply_to['user_id_str'] = null;
$in_reply_to['screen_name'] = null;
if (!empty($item['thr-parent']) && ($item['thr-parent'] != $item['uri']) && ($item['gravity'] != GRAVITY_PARENT)) {
$parent = Post::selectFirst(['id'], ['uid' => $item['uid'], 'uri' => $item['thr-parent']]);
if (DBA::isResult($parent)) {
$in_reply_to['status_id'] = intval($parent['id']);
} else {
$in_reply_to['status_id'] = intval($item['parent']);
}
$in_reply_to['status_id_str'] = (string) intval($in_reply_to['status_id']);
$fields = ['author-nick', 'author-name', 'author-id', 'author-link'];
$parent = Post::selectFirst($fields, ['id' => $in_reply_to['status_id']]);
if (DBA::isResult($parent)) {
$in_reply_to['screen_name'] = (($parent['author-nick']) ? $parent['author-nick'] : $parent['author-name']);
$in_reply_to['user_id'] = intval($parent['author-id']);
$in_reply_to['user_id_str'] = (string) intval($parent['author-id']);
}
// There seems to be situation, where both fields are identical:
// https://github.com/friendica/friendica/issues/1010
// This is a bugfix for that.
if (intval($in_reply_to['status_id']) == intval($item['id'])) {
Logger::warning(API_LOG_PREFIX . 'ID {id} is similar to reply-to {reply-to}', ['module' => 'api', 'action' => 'in_reply_to', 'id' => $item['id'], 'reply-to' => $in_reply_to['status_id']]);
$in_reply_to['status_id'] = null;
$in_reply_to['user_id'] = null;
$in_reply_to['status_id_str'] = null;
$in_reply_to['user_id_str'] = null;
$in_reply_to['screen_name'] = null;
}
}
return $in_reply_to;
}
/** /**
* *
* @param string $text * @param string $text
@ -4161,7 +3543,7 @@ function api_friendica_notification_seen($type)
$item = Post::selectFirstForUser($uid, [], ['id' => $Notify->iid, 'uid' => $uid]); $item = Post::selectFirstForUser($uid, [], ['id' => $Notify->iid, 'uid' => $uid]);
if (DBA::isResult($item)) { if (DBA::isResult($item)) {
// we found the item, return it to the user // we found the item, return it to the user
$ret = [api_format_item($item, $type)]; $ret = [DI::twitterStatus()->createFromUriId($item['uri-id'], $item['uid'])->toArray()];
$data = ['status' => $ret]; $data = ['status' => $ret];
return DI::apiResponse()->formatData('status', $type, $data); return DI::apiResponse()->formatData('status', $type, $data);
} }

View file

@ -0,0 +1,51 @@
<?php
/**
* @copyright Copyright (C) 2010-2021, the Friendica project
*
* @license GNU AGPL version 3 or any later version
*
* This program is free software: you can redistribute it and/or modify
* it under the terms of the GNU Affero General Public License as
* published by the Free Software Foundation, either version 3 of the
* License, or (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU Affero General Public License for more details.
*
* You should have received a copy of the GNU Affero General Public License
* along with this program. If not, see <https://www.gnu.org/licenses/>.
*
*/
namespace Friendica\Factory\Api\Twitter;
use Friendica\BaseFactory;
use Friendica\Network\HTTPException;
use Friendica\Model\Post;
use Psr\Log\LoggerInterface;
class Attachment extends BaseFactory
{
public function __construct(LoggerInterface $logger)
{
parent::__construct($logger);
}
/**
* @param int $uriId Uri-ID of the attachments
* @return array
* @throws HTTPException\InternalServerErrorException
*/
public function createFromUriId(int $uriId): array
{
$attachments = [];
foreach (Post\Media::getByURIId($uriId, [Post\Media::AUDIO, Post\Media::VIDEO, Post\Media::IMAGE]) as $attachment) {
$object = new \Friendica\Object\Api\Twitter\Attachment($attachment);
$attachments[] = $object->toArray();
}
return $attachments;
}
}

View file

@ -50,8 +50,10 @@ class Status extends BaseFactory
private $mention; private $mention;
/** @var Activities entity */ /** @var Activities entity */
private $activities; private $activities;
/** @var Activities entity */
private $attachment;
public function __construct(LoggerInterface $logger, Database $dba, TwitterUser $twitteruser, Hashtag $hashtag, Media $media, Url $url, Mention $mention, Activities $activities) public function __construct(LoggerInterface $logger, Database $dba, TwitterUser $twitteruser, Hashtag $hashtag, Media $media, Url $url, Mention $mention, Activities $activities, Attachment $attachment)
{ {
parent::__construct($logger); parent::__construct($logger);
$this->dba = $dba; $this->dba = $dba;
@ -61,6 +63,7 @@ class Status extends BaseFactory
$this->url = $url; $this->url = $url;
$this->mention = $mention; $this->mention = $mention;
$this->activities = $activities; $this->activities = $activities;
$this->attachment = $attachment;
} }
/** /**
@ -71,15 +74,42 @@ class Status extends BaseFactory
* @throws HTTPException\InternalServerErrorException * @throws HTTPException\InternalServerErrorException
* @throws ImagickException|HTTPException\NotFoundException * @throws ImagickException|HTTPException\NotFoundException
*/ */
public function createFromItemId(int $id): \Friendica\Object\Api\Twitter\Status
{
$item = Post::selectFirst([], ['id' => $id], ['order' => ['uid' => true]]);
if (!$item) {
throw new HTTPException\NotFoundException('Item with ID ' . $id . ' not found.');
}
return $this->createFromArray($item);
}
/**
* @param int $uriId Uri-ID of the item
* @param int $uid Item user
*
* @return \Friendica\Object\Api\Mastodon\Status
* @throws HTTPException\InternalServerErrorException
* @throws ImagickException|HTTPException\NotFoundException
*/
public function createFromUriId(int $uriId, $uid = 0): \Friendica\Object\Api\Twitter\Status public function createFromUriId(int $uriId, $uid = 0): \Friendica\Object\Api\Twitter\Status
{ {
$fields = ['id', 'parent', 'uri-id', 'uid', 'author-id', 'author-link', 'author-network', 'owner-id', 'starred', 'app', 'title', 'body', 'raw-body', 'created', 'network', $item = Post::selectFirst([], ['uri-id' => $uriId, 'uid' => [0, $uid]], ['order' => ['uid' => true]]);
'thr-parent-id', 'parent-author-id', 'parent-author-nick', 'language', 'uri', 'plink', 'private', 'vid', 'gravity', 'coord'];
$item = Post::selectFirst($fields, ['uri-id' => $uriId, 'uid' => [0, $uid]], ['order' => ['uid' => true]]);
if (!$item) { if (!$item) {
throw new HTTPException\NotFoundException('Item with URI ID ' . $uriId . ' not found' . ($uid ? ' for user ' . $uid : '.')); throw new HTTPException\NotFoundException('Item with URI ID ' . $uriId . ' not found' . ($uid ? ' for user ' . $uid : '.'));
} }
return $this->createFromArray($item);
}
/**
* @param array $item item array
* @param int $uid Item user
*
* @return \Friendica\Object\Api\Mastodon\Status
* @throws HTTPException\InternalServerErrorException
* @throws ImagickException|HTTPException\NotFoundException
*/
public function createFromArray(array $item, $uid = 0): \Friendica\Object\Api\Twitter\Status
{
$author = $this->twitterUser->createFromContactId($item['author-id'], $item['uid']); $author = $this->twitterUser->createFromContactId($item['author-id'], $item['uid']);
$owner = $this->twitterUser->createFromContactId($item['owner-id'], $item['uid']); $owner = $this->twitterUser->createFromContactId($item['owner-id'], $item['uid']);
@ -99,12 +129,13 @@ class Status extends BaseFactory
} }
} }
$hashtags = $this->hashtag->createFromUriId($uriId, $text); $hashtags = $this->hashtag->createFromUriId($item['uri-id'], $text);
$medias = $this->media->createFromUriId($uriId, $text); $medias = $this->media->createFromUriId($item['uri-id'], $text);
$urls = $this->url->createFromUriId($uriId, $text); $urls = $this->url->createFromUriId($item['uri-id'], $text);
$mentions = $this->mention->createFromUriId($uriId, $text); $mentions = $this->mention->createFromUriId($item['uri-id'], $text);
$friendica_activities = $this->activities->createFromUriId($uriId, $uid); $friendica_activities = $this->activities->createFromUriId($item['uri-id'], $uid);
$attachments = $this->attachment->createFromUriId($item['uri-id'], $text);
$shared = BBCode::fetchShareAttributes($item['body']); $shared = BBCode::fetchShareAttributes($item['body']);
if (!empty($shared['guid'])) { if (!empty($shared['guid'])) {
@ -112,12 +143,11 @@ class Status extends BaseFactory
$shared_uri_id = $shared_item['uri-id'] ?? 0; $shared_uri_id = $shared_item['uri-id'] ?? 0;
$hashtags = array_merge($hashtags, $this->hashtag->createFromUriId($shared_uri_id, $text)); $hashtags = array_merge($hashtags, $this->hashtag->createFromUriId($shared_uri_id, $text));
$medias = array_merge($medias, $this->media->createFromUriId($shared_uri_id, $text)); $medias = array_merge($medias, $this->media->createFromUriId($shared_uri_id, $text));
$urls = array_merge($urls, $this->url->createFromUriId($shared_uri_id, $text)); $urls = array_merge($urls, $this->url->createFromUriId($shared_uri_id, $text));
$mentions = array_merge($mentions, $this->mention->createFromUriId($shared_uri_id, $text)); $mentions = array_merge($mentions, $this->mention->createFromUriId($shared_uri_id, $text));
$attachments = array_merge($attachments, $this->attachment->createFromUriId($item['uri-id'], $text));
$friendica_activities = [];
} }
if ($item['vid'] == Verb::getID(Activity::ANNOUNCE)) { if ($item['vid'] == Verb::getID(Activity::ANNOUNCE)) {
@ -134,6 +164,9 @@ class Status extends BaseFactory
$entities = ['hashtags' => $hashtags, 'media' => $medias, 'urls' => $urls, 'user_mentions' => $mentions]; $entities = ['hashtags' => $hashtags, 'media' => $medias, 'urls' => $urls, 'user_mentions' => $mentions];
return new \Friendica\Object\Api\Twitter\Status($text, $item, $author, $owner, $retweeted, $quoted, $geo, $friendica_activities, $entities, $friendica_comments); // Attachments are currently deactivated for testing purposes
$attachments = [];
return new \Friendica\Object\Api\Twitter\Status($text, $item, $author, $owner, $retweeted, $quoted, $geo, $friendica_activities, $entities, $attachments, $friendica_comments);
} }
} }

View file

@ -0,0 +1,72 @@
<?php
/**
* @copyright Copyright (C) 2010-2021, the Friendica project
*
* @license GNU AGPL version 3 or any later version
*
* This program is free software: you can redistribute it and/or modify
* it under the terms of the GNU Affero General Public License as
* published by the Free Software Foundation, either version 3 of the
* License, or (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU Affero General Public License for more details.
*
* You should have received a copy of the GNU Affero General Public License
* along with this program. If not, see <https://www.gnu.org/licenses/>.
*
*/
namespace Friendica\Object\Api\Twitter;
use Friendica\BaseDataTransferObject;
/**
* Class Attachment
*
*
*/
class Attachment extends BaseDataTransferObject
{
/** @var string */
protected $url;
/** @var string */
protected $mimetype;
/** @var int */
protected $size;
/**
* Creates an Attachment entity array
*
* @param array $attachment
* @throws \Friendica\Network\HTTPException\InternalServerErrorException
*/
public function __construct(array $media)
{
$this->url = $media['url'];
$this->mimetype = $media['mimetype'];
$this->size = $media['size'];
}
/**
* Returns the current entity as an array
*
* @return array
*/
public function toArray(): array
{
$status = parent::toArray();
if (empty($status['mimetype'])) {
unset($status['mimetype']);
}
if (empty($status['size'])) {
unset($status['size']);
}
return $status;
}
}

View file

@ -34,22 +34,32 @@ use Friendica\Util\DateTimeFormat;
*/ */
class Status extends BaseDataTransferObject class Status extends BaseDataTransferObject
{ {
/** @var int */
protected $id;
/** @var string */ /** @var string */
protected $id_str; protected $text;
/** @var bool */
protected $truncated;
/** @var string (Datetime) */ /** @var string (Datetime) */
protected $created_at; protected $created_at;
/** @var int|null */ /** @var int|null */
protected $in_reply_to_status_id = null; protected $in_reply_to_status_id = null;
/** @var string|null */ /** @var string|null */
protected $in_reply_to_status_id_str = null; protected $in_reply_to_status_id_str = null;
/** @var string */
protected $source;
/** @var int */
protected $id;
/** @var string */
protected $id_str;
/** @var int|null */ /** @var int|null */
protected $in_reply_to_user_id = null; protected $in_reply_to_user_id = null;
/** @var string|null */ /** @var string|null */
protected $in_reply_to_user_id_str = null; protected $in_reply_to_user_id_str = null;
/** @var string|null */ /** @var string|null */
protected $in_reply_to_screen_name = null; protected $in_reply_to_screen_name = null;
/** @var array|null */
protected $geo;
/** @var bool */
protected $favorited = false;
/** @var User */ /** @var User */
protected $user; protected $user;
/** @var User */ /** @var User */
@ -57,34 +67,27 @@ class Status extends BaseDataTransferObject
/** @var User */ /** @var User */
protected $friendica_owner; protected $friendica_owner;
/** @var bool */ /** @var bool */
protected $favorited = false; protected $friendica_private;
/** @var string */
protected $statusnet_html;
/** @var int */
protected $statusnet_conversation_id;
/** @var string */
protected $external_url;
/** @var array */
protected $friendica_activities;
/** @var string */
protected $friendica_title;
/** @var string */
protected $friendica_html;
/** @var int */
protected $friendica_comments;
/** @var Status|null */ /** @var Status|null */
protected $retweeted_status = null; protected $retweeted_status = null;
/** @var Status|null */ /** @var Status|null */
protected $quoted_status = null; protected $quoted_status = null;
/** @var string */
protected $text;
/** @var string */
protected $statusnet_html;
/** @var string */
protected $friendica_html;
/** @var string */
protected $friendica_title;
/** @var bool */
protected $truncated;
/** @var int */
protected $friendica_comments;
/** @var string */
protected $source;
/** @var string */
protected $external_url;
/** @var int */
protected $statusnet_conversation_id;
/** @var bool */
protected $friendica_private;
protected $geo;
/** @var array */ /** @var array */
protected $friendica_activities; protected $attachments;
/** @var array */ /** @var array */
protected $entities; protected $entities;
/** @var array */ /** @var array */
@ -96,7 +99,7 @@ class Status extends BaseDataTransferObject
* @param array $item * @param array $item
* @throws \Friendica\Network\HTTPException\InternalServerErrorException * @throws \Friendica\Network\HTTPException\InternalServerErrorException
*/ */
public function __construct(string $text, array $item, User $author, User $owner, array $retweeted, array $quoted, array $geo, array $friendica_activities, array $entities, int $friendica_comments) public function __construct(string $text, array $item, User $author, User $owner, array $retweeted, array $quoted, array $geo, array $friendica_activities, array $entities, array $attachments, int $friendica_comments)
{ {
$this->id = (int)$item['id']; $this->id = (int)$item['id'];
$this->id_str = (string)$item['id']; $this->id_str = (string)$item['id'];
@ -129,6 +132,7 @@ class Status extends BaseDataTransferObject
$this->source = $item['app'] ?: 'web'; $this->source = $item['app'] ?: 'web';
$this->geo = $geo; $this->geo = $geo;
$this->friendica_activities = $friendica_activities; $this->friendica_activities = $friendica_activities;
$this->attachments = $attachments;
$this->entities = $entities; $this->entities = $entities;
$this->extended_entities = $entities; $this->extended_entities = $entities;
@ -156,6 +160,10 @@ class Status extends BaseDataTransferObject
unset($status['quoted_status']); unset($status['quoted_status']);
} }
if (empty($status['geo'])) {
$status['geo'] = null;
}
return $status; return $status;
} }
} }

View file

@ -1130,8 +1130,8 @@ class ApiTest extends FixtureTest
*/ */
public function testApiStatusShowWithJson() public function testApiStatusShowWithJson()
{ {
$result = api_status_show('json', 1); // $result = api_status_show('json', 1);
self::assertStatus($result['status']); // self::assertStatus($result['status']);
} }
/** /**
@ -1139,8 +1139,8 @@ class ApiTest extends FixtureTest
*/ */
public function testApiStatusShowWithXml() public function testApiStatusShowWithXml()
{ {
$result = api_status_show('xml', 1); // $result = api_status_show('xml', 1);
self::assertXml($result, 'statuses'); // self::assertXml($result, 'statuses');
} }
/** /**
@ -1148,9 +1148,8 @@ class ApiTest extends FixtureTest
*/ */
public function testApiGetLastStatus() public function testApiGetLastStatus()
{ {
$item = api_get_last_status($this->selfUser['id'], $this->selfUser['id']); // $item = api_get_last_status($this->selfUser['id'], $this->selfUser['id']);
// self::assertNotNull($item);
self::assertNotNull($item);
} }
/** /**
@ -1998,6 +1997,7 @@ class ApiTest extends FixtureTest
*/ */
public function testApiConvertItem() public function testApiConvertItem()
{ {
/*
$result = api_convert_item( $result = api_convert_item(
[ [
'network' => 'feed', 'network' => 'feed',
@ -2032,6 +2032,7 @@ class ApiTest extends FixtureTest
); );
self::assertStringStartsWith('item_title', $result['text']); self::assertStringStartsWith('item_title', $result['text']);
self::assertStringStartsWith('<h4>item_title</h4><br>perspiciatis impedit voluptatem', $result['html']); self::assertStringStartsWith('<h4>item_title</h4><br>perspiciatis impedit voluptatem', $result['html']);
*/
} }
/** /**
@ -2041,6 +2042,7 @@ class ApiTest extends FixtureTest
*/ */
public function testApiConvertItemWithoutBody() public function testApiConvertItemWithoutBody()
{ {
/*
$result = api_convert_item( $result = api_convert_item(
[ [
'network' => 'feed', 'network' => 'feed',
@ -2052,6 +2054,7 @@ class ApiTest extends FixtureTest
); );
self::assertEquals("item_title", $result['text']); self::assertEquals("item_title", $result['text']);
self::assertEquals('<h4>item_title</h4><br>item_plink', $result['html']); self::assertEquals('<h4>item_title</h4><br>item_plink', $result['html']);
*/
} }
/** /**
@ -2061,6 +2064,7 @@ class ApiTest extends FixtureTest
*/ */
public function testApiConvertItemWithTitleInBody() public function testApiConvertItemWithTitleInBody()
{ {
/*
$result = api_convert_item( $result = api_convert_item(
[ [
'title' => 'item_title', 'title' => 'item_title',
@ -2070,6 +2074,7 @@ class ApiTest extends FixtureTest
); );
self::assertEquals('item_title item_body', $result['text']); self::assertEquals('item_title item_body', $result['text']);
self::assertEquals('<h4>item_title</h4><br>item_title item_body', $result['html']); self::assertEquals('<h4>item_title</h4><br>item_title item_body', $result['html']);
*/
} }
/** /**
@ -2079,8 +2084,8 @@ class ApiTest extends FixtureTest
*/ */
public function testApiGetAttachments() public function testApiGetAttachments()
{ {
$body = 'body'; // $body = 'body';
self::assertEmpty(api_get_attachments($body, 0)); // self::assertEmpty(api_get_attachments($body, 0));
} }
/** /**
@ -2090,8 +2095,8 @@ class ApiTest extends FixtureTest
*/ */
public function testApiGetAttachmentsWithImage() public function testApiGetAttachmentsWithImage()
{ {
$body = '[img]http://via.placeholder.com/1x1.png[/img]'; // $body = '[img]http://via.placeholder.com/1x1.png[/img]';
self::assertIsArray(api_get_attachments($body, 0)); // self::assertIsArray(api_get_attachments($body, 0));
} }
/** /**
@ -2101,9 +2106,9 @@ class ApiTest extends FixtureTest
*/ */
public function testApiGetAttachmentsWithImageAndAndStatus() public function testApiGetAttachmentsWithImageAndAndStatus()
{ {
$_SERVER['HTTP_USER_AGENT'] = 'AndStatus'; // $_SERVER['HTTP_USER_AGENT'] = 'AndStatus';
$body = '[img]http://via.placeholder.com/1x1.png[/img]'; // $body = '[img]http://via.placeholder.com/1x1.png[/img]';
self::assertIsArray(api_get_attachments($body, 0)); // self::assertIsArray(api_get_attachments($body, 0));
} }
/** /**
@ -2113,8 +2118,8 @@ class ApiTest extends FixtureTest
*/ */
public function testApiGetEntitities() public function testApiGetEntitities()
{ {
$text = 'text'; // $text = 'text';
self::assertIsArray(api_get_entitities($text, 'bbcode', 0)); // self::assertIsArray(api_get_entitities($text, 'bbcode', 0));
} }
/** /**
@ -2124,6 +2129,7 @@ class ApiTest extends FixtureTest
*/ */
public function testApiGetEntititiesWithIncludeEntities() public function testApiGetEntititiesWithIncludeEntities()
{ {
/*
$_REQUEST['include_entities'] = 'true'; $_REQUEST['include_entities'] = 'true';
$text = 'text'; $text = 'text';
$result = api_get_entitities($text, 'bbcode', 0); $result = api_get_entitities($text, 'bbcode', 0);
@ -2131,6 +2137,7 @@ class ApiTest extends FixtureTest
self::assertIsArray($result['symbols']); self::assertIsArray($result['symbols']);
self::assertIsArray($result['urls']); self::assertIsArray($result['urls']);
self::assertIsArray($result['user_mentions']); self::assertIsArray($result['user_mentions']);
*/
} }
/** /**
@ -2140,42 +2147,12 @@ class ApiTest extends FixtureTest
*/ */
public function testApiFormatItemsEmbededImages() public function testApiFormatItemsEmbededImages()
{ {
/*
self::assertEquals( self::assertEquals(
'text ' . DI::baseUrl() . '/display/item_guid', 'text ' . DI::baseUrl() . '/display/item_guid',
api_format_items_embeded_images(['guid' => 'item_guid'], 'text data:image/foo') api_format_items_embeded_images(['guid' => 'item_guid'], 'text data:image/foo')
); );
} */
/**
* Test the api_contactlink_to_array() function.
*
* @return void
*/
public function testApiContactlinkToArray()
{
self::assertEquals(
[
'name' => 'text',
'url' => '',
],
api_contactlink_to_array('text')
);
}
/**
* Test the api_contactlink_to_array() function with an URL.
*
* @return void
*/
public function testApiContactlinkToArrayWithUrl()
{
self::assertEquals(
[
'name' => ['link_text'],
'url' => ['url'],
],
api_contactlink_to_array('text <a href="url">link_text</a>')
);
} }
/** /**
@ -2216,11 +2193,13 @@ class ApiTest extends FixtureTest
*/ */
public function testApiFormatItems() public function testApiFormatItems()
{ {
/*
$items = Post::selectToArray([], ['uid' => 42]); $items = Post::selectToArray([], ['uid' => 42]);
foreach ($items as $item) { foreach ($items as $item) {
$status = api_format_item($item); $status = api_format_item($item);
self::assertStatus($status); self::assertStatus($status);
} }
*/
} }
/** /**
@ -2229,11 +2208,13 @@ class ApiTest extends FixtureTest
*/ */
public function testApiFormatItemsWithXml() public function testApiFormatItemsWithXml()
{ {
/*
$items = Post::selectToArray([], ['uid' => 42]); $items = Post::selectToArray([], ['uid' => 42]);
foreach ($items as $item) { foreach ($items as $item) {
$status = api_format_item($item, 'xml'); $status = api_format_item($item, 'xml');
self::assertStatus($status); self::assertStatus($status);
} }
*/
} }
/** /**
@ -3055,21 +3036,6 @@ class ApiTest extends FixtureTest
$this->markTestIncomplete(); $this->markTestIncomplete();
} }
/**
* Test the api_in_reply_to() function.
*
* @return void
*/
public function testApiInReplyTo()
{
$result = api_in_reply_to(['id' => 0, 'parent' => 0, 'uri' => '', 'thr-parent' => '']);
self::assertArrayHasKey('status_id', $result);
self::assertArrayHasKey('user_id', $result);
self::assertArrayHasKey('status_id_str', $result);
self::assertArrayHasKey('user_id_str', $result);
self::assertArrayHasKey('screen_name', $result);
}
/** /**
* Test the api_in_reply_to() function with a valid item. * Test the api_in_reply_to() function with a valid item.
* *