Merge pull request #1264 from Quix0r/fixes/empty-array-get-info-from-url

Images::getInfoFromURL[Cached]() will both return empty arrays ...
This commit is contained in:
Hypolite Petovan 2022-06-22 13:32:31 -04:00 committed by GitHub
commit 4603ca00e6
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
3 changed files with 298 additions and 277 deletions

View file

@ -450,10 +450,7 @@ function advancedcontentfilter_prepare_item_row(array $item_row): array
$item_row['tags'] = $tags['tags']; $item_row['tags'] = $tags['tags'];
$item_row['hashtags'] = $tags['hashtags']; $item_row['hashtags'] = $tags['hashtags'];
$item_row['mentions'] = $tags['mentions']; $item_row['mentions'] = $tags['mentions'];
$item_row['attachments'] = Post\Media::splitAttachments($item_row['uri-id'], $item_row['guid'] ?? '');
$attachments = Post\Media::splitAttachments($item_row['uri-id'], $item_row['guid'] ?? '');
$item_row['attachments'] = $attachments;
return $item_row; return $item_row;
} }

View file

@ -74,13 +74,13 @@ function statusnet_install()
Hook::register('cron', 'addon/statusnet/statusnet.php', 'statusnet_cron'); Hook::register('cron', 'addon/statusnet/statusnet.php', 'statusnet_cron');
Hook::register('prepare_body', 'addon/statusnet/statusnet.php', 'statusnet_prepare_body'); Hook::register('prepare_body', 'addon/statusnet/statusnet.php', 'statusnet_prepare_body');
Hook::register('check_item_notification', 'addon/statusnet/statusnet.php', 'statusnet_check_item_notification'); Hook::register('check_item_notification', 'addon/statusnet/statusnet.php', 'statusnet_check_item_notification');
Logger::notice("installed GNU Social"); Logger::notice('installed GNU Social');
} }
function statusnet_check_item_notification(App $a, &$notification_data) function statusnet_check_item_notification(App $a, &$notification_data)
{ {
if (DI::pConfig()->get($notification_data["uid"], 'statusnet', 'post')) { if (DI::pConfig()->get($notification_data['uid'], 'statusnet', 'post')) {
$notification_data["profiles"][] = DI::pConfig()->get($notification_data["uid"], 'statusnet', 'own_url'); $notification_data['profiles'][] = DI::pConfig()->get($notification_data['uid'], 'statusnet', 'own_url');
} }
} }
@ -412,20 +412,22 @@ function statusnet_action(App $a, $uid, $pid, $action)
$connection = new StatusNetOAuth($api, $ckey, $csecret, $otoken, $osecret); $connection = new StatusNetOAuth($api, $ckey, $csecret, $otoken, $osecret);
Logger::debug("statusnet_action '" . $action . "' ID: " . $pid); Logger::debug('statusnet_action "' . $action . '" ID: ' . $pid);
switch ($action) { switch ($action) {
case "delete": case 'delete':
$result = $connection->post("statuses/destroy/" . $pid); $result = $connection->post('statuses/destroy/' . $pid);
break; break;
case "like":
$result = $connection->post("favorites/create/" . $pid); case 'like':
$result = $connection->post('favorites/create/' . $pid);
break; break;
case "unlike":
$result = $connection->post("favorites/destroy/" . $pid); case 'unlike':
$result = $connection->post('favorites/destroy/' . $pid);
break; break;
} }
Logger::info("statusnet_action '" . $action . "' send, result: " . print_r($result, true)); Logger::info('statusnet_action "' . $action . '" send, result: ' . print_r($result, true));
} }
function statusnet_post_hook(App $a, &$b) function statusnet_post_hook(App $a, &$b)
@ -433,46 +435,46 @@ function statusnet_post_hook(App $a, &$b)
/** /**
* Post to GNU Social * Post to GNU Social
*/ */
if (!DI::pConfig()->get($b["uid"], 'statusnet', 'import')) { if (!DI::pConfig()->get($b['uid'], 'statusnet', 'import')) {
if ($b['deleted'] || $b['private'] || ($b['created'] !== $b['edited'])) if ($b['deleted'] || $b['private'] || ($b['created'] !== $b['edited']))
return; return;
} }
$b['body'] = Post\Media::addAttachmentsToBody($b['uri-id'], $b['body']); $b['body'] = Post\Media::addAttachmentsToBody($b['uri-id'], $b['body']);
$api = DI::pConfig()->get($b["uid"], 'statusnet', 'baseapi'); $api = DI::pConfig()->get($b['uid'], 'statusnet', 'baseapi');
$hostname = preg_replace("=https?://([\w\.]*)/.*=ism", "$1", $api); $hostname = preg_replace("=https?://([\w\.]*)/.*=ism", "$1", $api);
if ($b['parent'] != $b['id']) { if ($b['parent'] != $b['id']) {
Logger::debug("statusnet_post_hook: parameter " . print_r($b, true)); Logger::debug('statusnet_post_hook: parameter ', ['b' => $b]);
// Looking if its a reply to a GNU Social post // Looking if its a reply to a GNU Social post
$hostlength = strlen($hostname) + 2; $hostlength = strlen($hostname) + 2;
if ((substr($b["parent-uri"], 0, $hostlength) != $hostname . "::") && (substr($b["extid"], 0, $hostlength) != $hostname . "::") && (substr($b["thr-parent"], 0, $hostlength) != $hostname . "::")) { if ((substr($b['parent-uri'], 0, $hostlength) != $hostname . '::') && (substr($b['extid'], 0, $hostlength) != $hostname . '::') && (substr($b['thr-parent'], 0, $hostlength) != $hostname . '::')) {
Logger::notice("statusnet_post_hook: no GNU Social post " . $b["parent"]); Logger::notice('statusnet_post_hook: no GNU Social post ' . $b['parent']);
return; return;
} }
$condition = ['uri' => $b["thr-parent"], 'uid' => $b["uid"]]; $condition = ['uri' => $b['thr-parent'], 'uid' => $b['uid']];
$orig_post = Post::selectFirst(['author-link', 'uri'], $condition); $orig_post = Post::selectFirst(['author-link', 'uri'], $condition);
if (!DBA::isResult($orig_post)) { if (!DBA::isResult($orig_post)) {
Logger::notice("statusnet_post_hook: no parent found " . $b["thr-parent"]); Logger::notice('statusnet_post_hook: no parent found ' . $b['thr-parent']);
return; return;
} else { } else {
$iscomment = true; $iscomment = true;
} }
$nick = preg_replace("=https?://(.*)/(.*)=ism", "$2", $orig_post["author-link"]); $nick = preg_replace("=https?://(.*)/(.*)=ism", "$2", $orig_post['author-link']);
$nickname = "@[url=" . $orig_post["author-link"] . "]" . $nick . "[/url]"; $nickname = '@[url=' . $orig_post['author-link'] . ']' . $nick . '[/url]';
$nicknameplain = "@" . $nick; $nicknameplain = '@' . $nick;
Logger::info("statusnet_post_hook: comparing " . $nickname . " and " . $nicknameplain . " with " . $b["body"]); Logger::info('statusnet_post_hook: comparing ' . $nickname . ' and ' . $nicknameplain . ' with ' . $b['body']);
if ((strpos($b["body"], $nickname) === false) && (strpos($b["body"], $nicknameplain) === false)) { if ((strpos($b['body'], $nickname) === false) && (strpos($b['body'], $nicknameplain) === false)) {
$b["body"] = $nickname . " " . $b["body"]; $b['body'] = $nickname . ' ' . $b['body'];
} }
Logger::info("statusnet_post_hook: parent found " . print_r($orig_post, true)); Logger::info('statusnet_post_hook: parent found ', ['orig_post' => $orig_post]);
} else { } else {
$iscomment = false; $iscomment = false;
@ -489,15 +491,15 @@ function statusnet_post_hook(App $a, &$b)
} }
if (($b['verb'] == Activity::POST) && $b['deleted']) { if (($b['verb'] == Activity::POST) && $b['deleted']) {
statusnet_action($a, $b["uid"], substr($orig_post["uri"], $hostlength), "delete"); statusnet_action($a, $b['uid'], substr($orig_post['uri'], $hostlength), 'delete');
} }
if ($b['verb'] == Activity::LIKE) { if ($b['verb'] == Activity::LIKE) {
Logger::info("statusnet_post_hook: parameter 2 " . substr($b["thr-parent"], $hostlength)); Logger::info('statusnet_post_hook: parameter 2 ' . substr($b['thr-parent'], $hostlength));
if ($b['deleted']) if ($b['deleted'])
statusnet_action($a, $b["uid"], substr($b["thr-parent"], $hostlength), "unlike"); statusnet_action($a, $b['uid'], substr($b['thr-parent'], $hostlength), 'unlike');
else else
statusnet_action($a, $b["uid"], substr($b["thr-parent"], $hostlength), "like"); statusnet_action($a, $b['uid'], substr($b['thr-parent'], $hostlength), 'like');
return; return;
} }
@ -510,7 +512,7 @@ function statusnet_post_hook(App $a, &$b)
return; return;
} }
if ($b['app'] == "StatusNet") { if ($b['app'] == 'StatusNet') {
return; return;
} }
@ -535,34 +537,34 @@ function statusnet_post_hook(App $a, &$b)
DI::pConfig()->set($b['uid'], 'statusnet', 'max_char', $max_char); DI::pConfig()->set($b['uid'], 'statusnet', 'max_char', $max_char);
$tempfile = ""; $tempfile = '';
$msgarr = Plaintext::getPost($b, $max_char, true, 7); $msgarr = Plaintext::getPost($b, $max_char, true, 7);
$msg = $msgarr["text"]; $msg = $msgarr['text'];
if (($msg == "") && isset($msgarr["title"])) if (($msg == '') && isset($msgarr['title']))
$msg = Plaintext::shorten($msgarr["title"], $max_char - 50, $b['uid']); $msg = Plaintext::shorten($msgarr['title'], $max_char - 50, $b['uid']);
$image = ""; $image = '';
if (isset($msgarr["url"]) && ($msgarr["type"] != "photo")) { if (isset($msgarr['url']) && ($msgarr['type'] != 'photo')) {
$msg .= " \n" . $msgarr["url"]; $msg .= " \n" . $msgarr['url'];
} elseif (isset($msgarr["image"]) && ($msgarr["type"] != "video")) { } elseif (isset($msgarr['image']) && ($msgarr['type'] != 'video')) {
$image = $msgarr["image"]; $image = $msgarr['image'];
} }
if ($image != "") { if ($image != '') {
$img_str = DI::httpClient()->fetch($image); $img_str = DI::httpClient()->fetch($image);
$tempfile = tempnam(System::getTempPath(), "cache"); $tempfile = tempnam(System::getTempPath(), 'cache');
file_put_contents($tempfile, $img_str); file_put_contents($tempfile, $img_str);
$postdata = ["status" => $msg, "media[]" => $tempfile]; $postdata = ['status' => $msg, 'media[]' => $tempfile];
} else { } else {
$postdata = ["status" => $msg]; $postdata = ['status' => $msg];
} }
// and now send it :-) // and now send it :-)
if (strlen($msg)) { if (strlen($msg)) {
if ($iscomment) { if ($iscomment) {
$postdata["in_reply_to_status_id"] = substr($orig_post["uri"], $hostlength); $postdata['in_reply_to_status_id'] = substr($orig_post['uri'], $hostlength);
Logger::info('statusnet_post send reply ' . print_r($postdata, true)); Logger::info('statusnet_post send reply ' . print_r($postdata, true));
} }
@ -578,17 +580,17 @@ function statusnet_post_hook(App $a, &$b)
"\nmessage: " . $msg . "\nOriginal post: " . print_r($b, true) . "\nPost Data: " . print_r($postdata, true)); "\nmessage: " . $msg . "\nOriginal post: " . print_r($b, true) . "\nPost Data: " . print_r($postdata, true));
if (!empty($result->source)) { if (!empty($result->source)) {
DI::pConfig()->set($b["uid"], "statusnet", "application_name", strip_tags($result->source)); DI::pConfig()->set($b['uid'], 'statusnet', 'application_name', strip_tags($result->source));
} }
if (!empty($result->error)) { if (!empty($result->error)) {
Logger::notice('Send to GNU Social failed: "' . $result->error . '"'); Logger::notice('Send to GNU Social failed: "' . $result->error . '"');
} elseif ($iscomment) { } elseif ($iscomment) {
Logger::notice('statusnet_post: Update extid ' . $result->id . " for post id " . $b['id']); Logger::notice('statusnet_post: Update extid ' . $result->id . ' for post id ' . $b['id']);
Item::update(['extid' => $hostname . "::" . $result->id, 'body' => $result->text], ['id' => $b['id']]); Item::update(['extid' => $hostname . '::' . $result->id, 'body' => $result->text], ['id' => $b['id']]);
} }
} }
if ($tempfile != "") { if ($tempfile != '') {
unlink($tempfile); unlink($tempfile);
} }
} }
@ -607,10 +609,10 @@ function statusnet_addon_admin_post(App $a)
$secret = trim($_POST['secret'][$id]); $secret = trim($_POST['secret'][$id]);
$key = trim($_POST['key'][$id]); $key = trim($_POST['key'][$id]);
//$applicationname = (!empty($_POST['applicationname']) ? Strings::escapeTags(trim($_POST['applicationname'][$id])):''); //$applicationname = (!empty($_POST['applicationname']) ? Strings::escapeTags(trim($_POST['applicationname'][$id])):'');
if ($sitename != "" && if ($sitename != '' &&
$apiurl != "" && $apiurl != '' &&
$secret != "" && $secret != '' &&
$key != "" && $key != '' &&
empty($_POST['delete'][$id])) { empty($_POST['delete'][$id])) {
$sites[] = [ $sites[] = [
@ -652,7 +654,7 @@ function statusnet_addon_admin(App $a, &$o)
//'applicationname' => Array("applicationname[$id]", DI::l10n()->t("Application name"), "", ""), //'applicationname' => Array("applicationname[$id]", DI::l10n()->t("Application name"), "", ""),
]; ];
$t = Renderer::getMarkupTemplate("admin.tpl", "addon/statusnet/"); $t = Renderer::getMarkupTemplate('admin.tpl', 'addon/statusnet/');
$o = Renderer::replaceMacros($t, [ $o = Renderer::replaceMacros($t, [
'$submit' => DI::l10n()->t('Save Settings'), '$submit' => DI::l10n()->t('Save Settings'),
'$sites' => $sitesform, '$sites' => $sitesform,
@ -661,41 +663,41 @@ function statusnet_addon_admin(App $a, &$o)
function statusnet_prepare_body(App $a, &$b) function statusnet_prepare_body(App $a, &$b)
{ {
if ($b["item"]["network"] != Protocol::STATUSNET) { if ($b['item']['network'] != Protocol::STATUSNET) {
return; return;
} }
if ($b["preview"]) { if ($b['preview']) {
$max_char = DI::pConfig()->get(local_user(), 'statusnet', 'max_char'); $max_char = DI::pConfig()->get(local_user(), 'statusnet', 'max_char');
if (intval($max_char) == 0) { if (intval($max_char) == 0) {
$max_char = 140; $max_char = 140;
} }
$item = $b["item"]; $item = $b['item'];
$item["plink"] = DI::baseUrl()->get() . "/display/" . $item["guid"]; $item['plink'] = DI::baseUrl()->get() . '/display/' . $item['guid'];
$condition = ['uri' => $item["thr-parent"], 'uid' => local_user()]; $condition = ['uri' => $item['thr-parent'], 'uid' => local_user()];
$orig_post = Post::selectFirst(['author-link', 'uri'], $condition); $orig_post = Post::selectFirst(['author-link', 'uri'], $condition);
if (DBA::isResult($orig_post)) { if (DBA::isResult($orig_post)) {
$nick = preg_replace("=https?://(.*)/(.*)=ism", "$2", $orig_post["author-link"]); $nick = preg_replace("=https?://(.*)/(.*)=ism", "$2", $orig_post['author-link']);
$nickname = "@[url=" . $orig_post["author-link"] . "]" . $nick . "[/url]"; $nickname = '@[url=' . $orig_post['author-link'] . ']' . $nick . '[/url]';
$nicknameplain = "@" . $nick; $nicknameplain = '@' . $nick;
if ((strpos($item["body"], $nickname) === false) && (strpos($item["body"], $nicknameplain) === false)) { if ((strpos($item['body'], $nickname) === false) && (strpos($item['body'], $nicknameplain) === false)) {
$item["body"] = $nickname . " " . $item["body"]; $item['body'] = $nickname . ' ' . $item['body'];
} }
} }
$msgarr = Plaintext::getPost($item, $max_char, true, 7); $msgarr = Plaintext::getPost($item, $max_char, true, 7);
$msg = $msgarr["text"]; $msg = $msgarr['text'];
if (isset($msgarr["url"]) && ($msgarr["type"] != "photo")) { if (isset($msgarr['url']) && ($msgarr['type'] != 'photo')) {
$msg .= " " . $msgarr["url"]; $msg .= ' ' . $msgarr['url'];
} }
if (isset($msgarr["image"])) { if (isset($msgarr['image'])) {
$msg .= " " . $msgarr["image"]; $msg .= ' ' . $msgarr['image'];
} }
$b['html'] = nl2br(htmlspecialchars($msg)); $b['html'] = nl2br(htmlspecialchars($msg));
@ -743,7 +745,7 @@ function statusnet_cron(App $a, $b)
} }
Logger::notice('statusnet: importing timeline from user ' . $rr['uid']); Logger::notice('statusnet: importing timeline from user ' . $rr['uid']);
statusnet_fetchhometimeline($a, $rr["uid"], $rr["v"]); statusnet_fetchhometimeline($a, $rr['uid'], $rr['v']);
} }
Logger::notice('statusnet: cron_end'); Logger::notice('statusnet: cron_end');
@ -765,21 +767,26 @@ function statusnet_fetchtimeline(App $a, $uid)
// 1st try personal config, then system config and fallback to the // 1st try personal config, then system config and fallback to the
// hostname of the node if neither one is set. // hostname of the node if neither one is set.
$application_name = DI::pConfig()->get($uid, 'statusnet', 'application_name'); $application_name = DI::pConfig()->get($uid, 'statusnet', 'application_name');
if ($application_name == "") { if ($application_name == '') {
$application_name = DI::config()->get('statusnet', 'application_name'); $application_name = DI::config()->get('statusnet', 'application_name');
} }
if ($application_name == "") { if ($application_name == '') {
$application_name = DI::baseUrl()->getHostname(); $application_name = DI::baseUrl()->getHostname();
} }
$connection = new StatusNetOAuth($api, $ckey, $csecret, $otoken, $osecret); $connection = new StatusNetOAuth($api, $ckey, $csecret, $otoken, $osecret);
$parameters = ["exclude_replies" => true, "trim_user" => true, "contributor_details" => false, "include_rts" => false]; $parameters = [
'exclude_replies' => true,
'trim_user' => true,
'contributor_details' => false,
'include_rts' => false,
];
$first_time = ($lastid == ""); $first_time = ($lastid == '');
if ($lastid <> "") { if ($lastid != '') {
$parameters["since_id"] = $lastid; $parameters['since_id'] = $lastid;
} }
$items = $connection->get('statuses/user_timeline', $parameters); $items = $connection->get('statuses/user_timeline', $parameters);
@ -799,7 +806,7 @@ function statusnet_fetchtimeline(App $a, $uid)
continue; continue;
} }
if ($post->source == "activity") { if ($post->source == 'activity') {
continue; continue;
} }
@ -807,48 +814,48 @@ function statusnet_fetchtimeline(App $a, $uid)
continue; continue;
} }
if ($post->in_reply_to_status_id != "") { if ($post->in_reply_to_status_id != '') {
continue; continue;
} }
if (!stristr($post->source, $application_name)) { if (!stristr($post->source, $application_name)) {
$_SESSION["authenticated"] = true; $_SESSION['authenticated'] = true;
$_SESSION["uid"] = $uid; $_SESSION['uid'] = $uid;
unset($_REQUEST); unset($_REQUEST);
$_REQUEST["api_source"] = true; $_REQUEST['api_source'] = true;
$_REQUEST["profile_uid"] = $uid; $_REQUEST['profile_uid'] = $uid;
//$_REQUEST["source"] = "StatusNet"; //$_REQUEST['source'] = 'StatusNet';
$_REQUEST["source"] = $post->source; $_REQUEST['source'] = $post->source;
$_REQUEST["extid"] = Protocol::STATUSNET; $_REQUEST['extid'] = Protocol::STATUSNET;
if (isset($post->id)) { if (isset($post->id)) {
$_REQUEST['message_id'] = Item::newURI($uid, Protocol::STATUSNET . ":" . $post->id); $_REQUEST['message_id'] = Item::newURI($uid, Protocol::STATUSNET . ':' . $post->id);
} }
//$_REQUEST["date"] = $post->created_at; //$_REQUEST['date'] = $post->created_at;
$_REQUEST["title"] = ""; $_REQUEST['title'] = '';
$_REQUEST["body"] = $post->text; $_REQUEST['body'] = $post->text;
if (is_string($post->place->name)) { if (is_string($post->place->name)) {
$_REQUEST["location"] = $post->place->name; $_REQUEST['location'] = $post->place->name;
} }
if (is_string($post->place->full_name)) { if (is_string($post->place->full_name)) {
$_REQUEST["location"] = $post->place->full_name; $_REQUEST['location'] = $post->place->full_name;
} }
if (is_array($post->geo->coordinates)) { if (is_array($post->geo->coordinates)) {
$_REQUEST["coord"] = $post->geo->coordinates[0] . " " . $post->geo->coordinates[1]; $_REQUEST['coord'] = $post->geo->coordinates[0] . ' ' . $post->geo->coordinates[1];
} }
if (is_array($post->coordinates->coordinates)) { if (is_array($post->coordinates->coordinates)) {
$_REQUEST["coord"] = $post->coordinates->coordinates[1] . " " . $post->coordinates->coordinates[0]; $_REQUEST['coord'] = $post->coordinates->coordinates[1] . ' ' . $post->coordinates->coordinates[0];
} }
//print_r($_REQUEST); //print_r($_REQUEST);
if ($_REQUEST["body"] != "") { if ($_REQUEST['body'] != '') {
Logger::notice('statusnet: posting for user ' . $uid); Logger::notice('statusnet: posting for user ' . $uid);
item_post($a); item_post($a);
@ -866,7 +873,7 @@ function statusnet_address($contact)
$hostname = preg_replace("=https?://([\w\.]*)/.*=ism", "$1", $contact->statusnet_profile_url); $hostname = preg_replace("=https?://([\w\.]*)/.*=ism", "$1", $contact->statusnet_profile_url);
$address = $contact->screen_name . "@" . $hostname; $address = $contact->screen_name . '@' . $hostname;
return $address; return $address;
} }
@ -884,8 +891,8 @@ function statusnet_fetch_contact($uid, $contact, $create_user)
return 0; return 0;
} }
if (DBA::isResult($contact_record) && ($contact_record["readonly"] || $contact_record["blocked"])) { if (DBA::isResult($contact_record) && ($contact_record['readonly'] || $contact_record['blocked'])) {
Logger::info("statusnet_fetch_contact: Contact '" . $contact_record["nick"] . "' is blocked or readonly."); Logger::info('statusnet_fetch_contact: Contact "' . $contact_record['nick'] . '" is blocked or readonly.');
return -1; return -1;
} }
@ -938,7 +945,7 @@ function statusnet_fetch_contact($uid, $contact, $create_user)
// check that we have all the photos, this has been known to fail on occasion // check that we have all the photos, this has been known to fail on occasion
if ((!$contact_record['photo']) || (!$contact_record['thumb']) || (!$contact_record['micro']) || ($update_photo)) { if ((!$contact_record['photo']) || (!$contact_record['thumb']) || (!$contact_record['micro']) || ($update_photo)) {
Logger::info("statusnet_fetch_contact: Updating contact " . $contact->screen_name); Logger::info('statusnet_fetch_contact: Updating contact ' . $contact->screen_name);
$photos = Photo::importProfilePhoto($contact->profile_image_url, $uid, $contact_record['id']); $photos = Photo::importProfilePhoto($contact->profile_image_url, $uid, $contact_record['id']);
@ -960,10 +967,10 @@ function statusnet_fetch_contact($uid, $contact, $create_user)
} }
} }
return $contact_record["id"]; return $contact_record['id'];
} }
function statusnet_fetchuser(App $a, $uid, $screen_name = "", $user_id = "") function statusnet_fetchuser(App $a, $uid, $screen_name = '', $user_id = '')
{ {
$ckey = DI::pConfig()->get($uid, 'statusnet', 'consumerkey'); $ckey = DI::pConfig()->get($uid, 'statusnet', 'consumerkey');
$csecret = DI::pConfig()->get($uid, 'statusnet', 'consumersecret'); $csecret = DI::pConfig()->get($uid, 'statusnet', 'consumersecret');
@ -984,12 +991,12 @@ function statusnet_fetchuser(App $a, $uid, $screen_name = "", $user_id = "")
$parameters = []; $parameters = [];
if ($screen_name != "") { if ($screen_name != '') {
$parameters["screen_name"] = $screen_name; $parameters['screen_name'] = $screen_name;
} }
if ($user_id != "") { if ($user_id != '') {
$parameters["user_id"] = $user_id; $parameters['user_id'] = $user_id;
} }
// Fetching user data // Fetching user data
@ -1006,7 +1013,7 @@ function statusnet_fetchuser(App $a, $uid, $screen_name = "", $user_id = "")
function statusnet_createpost(App $a, $uid, $post, $self, $create_user, $only_existing_contact) function statusnet_createpost(App $a, $uid, $post, $self, $create_user, $only_existing_contact)
{ {
Logger::info("statusnet_createpost: start"); Logger::info('statusnet_createpost: start');
$api = DI::pConfig()->get($uid, 'statusnet', 'baseapi'); $api = DI::pConfig()->get($uid, 'statusnet', 'baseapi');
$hostname = preg_replace("=https?://([\w\.]*)/.*=ism", "$1", $api); $hostname = preg_replace("=https?://([\w\.]*)/.*=ism", "$1", $api);
@ -1023,16 +1030,16 @@ function statusnet_createpost(App $a, $uid, $post, $self, $create_user, $only_ex
$content = $post; $content = $post;
} }
$postarray['uri'] = $hostname . "::" . $content->id; $postarray['uri'] = $hostname . '::' . $content->id;
if (Post::exists(['extid' => $postarray['uri'], 'uid' => $uid])) { if (Post::exists(['extid' => $postarray['uri'], 'uid' => $uid])) {
return []; return [];
} }
$contactid = 0; $contactId = 0;
if (!empty($content->in_reply_to_status_id)) { if (!empty($content->in_reply_to_status_id)) {
$thr_parent = $hostname . "::" . $content->in_reply_to_status_id; $thr_parent = $hostname . '::' . $content->in_reply_to_status_id;
$item = Post::selectFirst(['uri'], ['uri' => $thr_parent, 'uid' => $uid]); $item = Post::selectFirst(['uri'], ['uri' => $thr_parent, 'uid' => $uid]);
if (!DBA::isResult($item)) { if (!DBA::isResult($item)) {
@ -1050,13 +1057,13 @@ function statusnet_createpost(App $a, $uid, $post, $self, $create_user, $only_ex
$own_url = DI::pConfig()->get($uid, 'statusnet', 'own_url'); $own_url = DI::pConfig()->get($uid, 'statusnet', 'own_url');
if ($content->user->id == $own_url) { if ($content->user->id == $own_url) {
$self = DBA::selectFirst([], ['self' => true, 'uid' => $uid]); $self = DBA::selectFirst('contact', [], ['self' => true, 'uid' => $uid]);
if (DBA::isResult($self)) { if (DBA::isResult($self)) {
$contactid = $self["id"]; $contactId = $self['id'];
$postarray['owner-name'] = $self["name"]; $postarray['owner-name'] = $self['name'];
$postarray['owner-link'] = $self["url"]; $postarray['owner-link'] = $self['url'];
$postarray['owner-avatar'] = $self["photo"]; $postarray['owner-avatar'] = $self['photo'];
} else { } else {
return []; return [];
} }
@ -1067,19 +1074,19 @@ function statusnet_createpost(App $a, $uid, $post, $self, $create_user, $only_ex
$postarray['object-type'] = Activity\ObjectType::NOTE; $postarray['object-type'] = Activity\ObjectType::NOTE;
} }
if ($contactid == 0) { if ($contactId == 0) {
$contactid = statusnet_fetch_contact($uid, $post->user, $create_user); $contactId = statusnet_fetch_contact($uid, $post->user, $create_user);
$postarray['owner-name'] = $post->user->name; $postarray['owner-name'] = $post->user->name;
$postarray['owner-link'] = $post->user->statusnet_profile_url; $postarray['owner-link'] = $post->user->statusnet_profile_url;
$postarray['owner-avatar'] = $post->user->profile_image_url; $postarray['owner-avatar'] = $post->user->profile_image_url;
} }
if (($contactid == 0) && !$only_existing_contact) { if (($contactId == 0) && !$only_existing_contact) {
$contactid = $self['id']; $contactId = $self['id'];
} elseif ($contactid <= 0) { } elseif ($contactId <= 0) {
return []; return [];
} }
$postarray['contact-id'] = $contactid; $postarray['contact-id'] = $contactId;
$postarray['verb'] = Activity::POST; $postarray['verb'] = Activity::POST;
@ -1088,7 +1095,7 @@ function statusnet_createpost(App $a, $uid, $post, $self, $create_user, $only_ex
$postarray['author-avatar'] = $content->user->profile_image_url; $postarray['author-avatar'] = $content->user->profile_image_url;
// To-Do: Maybe unreliable? Can the api be entered without trailing "/"? // To-Do: Maybe unreliable? Can the api be entered without trailing "/"?
$hostname = str_replace("/api/", "/notice/", DI::pConfig()->get($uid, 'statusnet', 'baseapi')); $hostname = str_replace('/api/', '/notice/', DI::pConfig()->get($uid, 'statusnet', 'baseapi'));
$postarray['plink'] = $hostname . $content->id; $postarray['plink'] = $hostname . $content->id;
$postarray['app'] = strip_tags($content->source); $postarray['app'] = strip_tags($content->source);
@ -1106,22 +1113,22 @@ function statusnet_createpost(App $a, $uid, $post, $self, $create_user, $only_ex
$postarray['edited'] = DateTimeFormat::utc($content->created_at); $postarray['edited'] = DateTimeFormat::utc($content->created_at);
if (!empty($content->place->name)) { if (!empty($content->place->name)) {
$postarray["location"] = $content->place->name; $postarray['location'] = $content->place->name;
} }
if (!empty($content->place->full_name)) { if (!empty($content->place->full_name)) {
$postarray["location"] = $content->place->full_name; $postarray['location'] = $content->place->full_name;
} }
if (!empty($content->geo->coordinates)) { if (!empty($content->geo->coordinates)) {
$postarray["coord"] = $content->geo->coordinates[0] . " " . $content->geo->coordinates[1]; $postarray['coord'] = $content->geo->coordinates[0] . ' ' . $content->geo->coordinates[1];
} }
if (!empty($content->coordinates->coordinates)) { if (!empty($content->coordinates->coordinates)) {
$postarray["coord"] = $content->coordinates->coordinates[1] . " " . $content->coordinates->coordinates[0]; $postarray['coord'] = $content->coordinates->coordinates[1] . ' ' . $content->coordinates->coordinates[0];
} }
Logger::info("statusnet_createpost: end"); Logger::info('statusnet_createpost: end');
return $postarray; return $postarray;
} }
@ -1140,7 +1147,7 @@ function statusnet_fetchhometimeline(App $a, $uid, $mode = 1)
// "create_user" is deactivated, since currently you cannot add users manually by now // "create_user" is deactivated, since currently you cannot add users manually by now
$create_user = true; $create_user = true;
Logger::info("statusnet_fetchhometimeline: Fetching for user " . $uid); Logger::info('statusnet_fetchhometimeline: Fetching for user ' . $uid);
$connection = new StatusNetOAuth($api, $ckey, $csecret, $otoken, $osecret); $connection = new StatusNetOAuth($api, $ckey, $csecret, $otoken, $osecret);
@ -1152,36 +1159,41 @@ function statusnet_fetchhometimeline(App $a, $uid, $mode = 1)
$contact = Contact::selectFirst([], ['id' => $own_contact, 'uid' => $uid]); $contact = Contact::selectFirst([], ['id' => $own_contact, 'uid' => $uid]);
if (DBA::isResult($contact)) { if (DBA::isResult($contact)) {
$nick = $contact["nick"]; $nick = $contact['nick'];
} else { } else {
Logger::info("statusnet_fetchhometimeline: Own GNU Social contact not found for user " . $uid); Logger::info('statusnet_fetchhometimeline: Own GNU Social contact not found for user ' . $uid);
return; return;
} }
$self = Contact::selectFirst([], ['self' => true, 'uid' => $uid]); $self = Contact::selectFirst([], ['self' => true, 'uid' => $uid]);
if (!DBA::isResult($self)) { if (!DBA::isResult($self)) {
Logger::info("statusnet_fetchhometimeline: Own contact not found for user " . $uid); Logger::info('statusnet_fetchhometimeline: Own contact not found for user ' . $uid);
return; return;
} }
$user = User::getById($uid); $user = User::getById($uid);
if (!DBA::isResult($user)) { if (!DBA::isResult($user)) {
Logger::info("statusnet_fetchhometimeline: Own user not found for user " . $uid); Logger::info('statusnet_fetchhometimeline: Own user not found for user ' . $uid);
return; return;
} }
$parameters = ["exclude_replies" => false, "trim_user" => false, "contributor_details" => true, "include_rts" => true]; $parameters = [
//$parameters["count"] = 200; 'exclude_replies' => false,
'trim_user' => false,
'contributor_details' => true,
'include_rts' => true,
//'count' => 200,
];
if ($mode == 1) { if ($mode == 1) {
// Fetching timeline // Fetching timeline
$lastid = DI::pConfig()->get($uid, 'statusnet', 'lasthometimelineid'); $lastid = DI::pConfig()->get($uid, 'statusnet', 'lasthometimelineid');
//$lastid = 1; //$lastid = 1;
$first_time = ($lastid == ""); $first_time = ($lastid == '');
if ($lastid != "") { if ($lastid != '') {
$parameters["since_id"] = $lastid; $parameters['since_id'] = $lastid;
} }
$items = $connection->get('statuses/home_timeline', $parameters); $items = $connection->get('statuses/home_timeline', $parameters);
@ -1194,16 +1206,16 @@ function statusnet_fetchhometimeline(App $a, $uid, $mode = 1)
} elseif (is_string($items) || is_float($items) || is_int($items)) { } elseif (is_string($items) || is_float($items) || is_int($items)) {
$errormsg = $items; $errormsg = $items;
} else { } else {
$errormsg = "Unknown error"; $errormsg = 'Unknown error';
} }
Logger::info("statusnet_fetchhometimeline: Error fetching home timeline: " . $errormsg); Logger::info('statusnet_fetchhometimeline: Error fetching home timeline: ' . $errormsg);
return; return;
} }
$posts = array_reverse($items); $posts = array_reverse($items);
Logger::info("statusnet_fetchhometimeline: Fetching timeline for user " . $uid . " " . sizeof($posts) . " items"); Logger::info('statusnet_fetchhometimeline: Fetching timeline for user ' . $uid . ' ' . sizeof($posts) . ' items');
if (count($posts)) { if (count($posts)) {
foreach ($posts as $post) { foreach ($posts as $post) {
@ -1224,14 +1236,14 @@ function statusnet_fetchhometimeline(App $a, $uid, $mode = 1)
} else { } else {
$postarray = statusnet_createpost($a, $uid, $post, $self, $create_user, true); $postarray = statusnet_createpost($a, $uid, $post, $self, $create_user, true);
if (trim($postarray['body']) == "") { if (trim($postarray['body']) == '') {
continue; continue;
} }
$item = Item::insert($postarray); $item = Item::insert($postarray);
$postarray["id"] = $item; $postarray['id'] = $item;
Logger::notice('statusnet_fetchhometimeline: User ' . $self["nick"] . ' posted home timeline item ' . $item); Logger::notice('statusnet_fetchhometimeline: User ' . $self['nick'] . ' posted home timeline item ' . $item);
} }
} }
} }
@ -1240,22 +1252,22 @@ function statusnet_fetchhometimeline(App $a, $uid, $mode = 1)
// Fetching mentions // Fetching mentions
$lastid = DI::pConfig()->get($uid, 'statusnet', 'lastmentionid'); $lastid = DI::pConfig()->get($uid, 'statusnet', 'lastmentionid');
$first_time = ($lastid == ""); $first_time = ($lastid == '');
if ($lastid != "") { if ($lastid != '') {
$parameters["since_id"] = $lastid; $parameters['since_id'] = $lastid;
} }
$items = $connection->get('statuses/mentions_timeline', $parameters); $items = $connection->get('statuses/mentions_timeline', $parameters);
if (!is_array($items)) { if (!is_array($items)) {
Logger::info("statusnet_fetchhometimeline: Error fetching mentions: " . print_r($items, true)); Logger::info('statusnet_fetchhometimeline: Error fetching mentions: ' . print_r($items, true));
return; return;
} }
$posts = array_reverse($items); $posts = array_reverse($items);
Logger::info("statusnet_fetchhometimeline: Fetching mentions for user " . $uid . " " . sizeof($posts) . " items"); Logger::info('statusnet_fetchhometimeline: Fetching mentions for user ' . $uid . ' ' . sizeof($posts) . ' items');
if (count($posts)) { if (count($posts)) {
foreach ($posts as $post) { foreach ($posts as $post) {
@ -1275,13 +1287,13 @@ function statusnet_fetchhometimeline(App $a, $uid, $mode = 1)
$conversations[$post->statusnet_conversation_id] = $post->statusnet_conversation_id; $conversations[$post->statusnet_conversation_id] = $post->statusnet_conversation_id;
} }
} else { } else {
if (trim($postarray['body']) == "") { if (trim($postarray['body']) == '') {
continue; continue;
} }
$item = Item::insert($postarray); $item = Item::insert($postarray);
Logger::notice('statusnet_fetchhometimeline: User ' . $self["nick"] . ' posted mention timeline item ' . $item); Logger::notice('statusnet_fetchhometimeline: User ' . $self['nick'] . ' posted mention timeline item ' . $item);
} }
} }
} }
@ -1300,7 +1312,7 @@ function statusnet_complete_conversation(App $a, $uid, $self, $create_user, $nic
$connection = new StatusNetOAuth($api, $ckey, $csecret, $otoken, $osecret); $connection = new StatusNetOAuth($api, $ckey, $csecret, $otoken, $osecret);
$parameters["count"] = 200; $parameters['count'] = 200;
$items = $connection->get('statusnet/conversation/' . $conversation, $parameters); $items = $connection->get('statusnet/conversation/' . $conversation, $parameters);
if (is_array($items)) { if (is_array($items)) {
@ -1314,9 +1326,9 @@ function statusnet_complete_conversation(App $a, $uid, $self, $create_user, $nic
} }
$item = Item::insert($postarray); $item = Item::insert($postarray);
$postarray["id"] = $item; $postarray['id'] = $item;
Logger::notice('statusnet_complete_conversation: User ' . $self["nick"] . ' posted home timeline item ' . $item); Logger::notice('statusnet_complete_conversation: User ' . $self['nick'] . ' posted home timeline item ' . $item);
} }
} }
} }
@ -1325,19 +1337,16 @@ function statusnet_convertmsg(App $a, $body)
{ {
$body = preg_replace("=\[url\=https?://([0-9]*).([0-9]*).([0-9]*).([0-9]*)/([0-9]*)\](.*?)\[\/url\]=ism", "$1.$2.$3.$4/$5", $body); $body = preg_replace("=\[url\=https?://([0-9]*).([0-9]*).([0-9]*).([0-9]*)/([0-9]*)\](.*?)\[\/url\]=ism", "$1.$2.$3.$4/$5", $body);
$URLSearchString = "^\[\]"; $URLSearchString = '^\[\]';
$links = preg_match_all("/[^!#@]\[url\=([$URLSearchString]*)\](.*?)\[\/url\]/ism", $body, $matches, PREG_SET_ORDER); $links = preg_match_all("/[^!#@]\[url\=([$URLSearchString]*)\](.*?)\[\/url\]/ism", $body, $matches, PREG_SET_ORDER);
$footer = ""; $footer = $footerurl = $footerlink = $type = '';
$footerurl = "";
$footerlink = "";
$type = "";
if ($links) { if ($links) {
foreach ($matches AS $match) { foreach ($matches AS $match) {
$search = "[url=" . $match[1] . "]" . $match[2] . "[/url]"; $search = '[url=' . $match[1] . ']' . $match[2] . '[/url]';
Logger::info("statusnet_convertmsg: expanding url " . $match[1]); Logger::info('statusnet_convertmsg: expanding url ' . $match[1]);
try { try {
$expanded_url = DI::httpClient()->finalUrl($match[1]); $expanded_url = DI::httpClient()->finalUrl($match[1]);
@ -1346,56 +1355,56 @@ function statusnet_convertmsg(App $a, $body)
$expanded_url = $match[1]; $expanded_url = $match[1];
} }
Logger::info("statusnet_convertmsg: fetching data for " . $expanded_url); Logger::info('statusnet_convertmsg: fetching data for ' . $expanded_url);
$oembed_data = OEmbed::fetchURL($expanded_url, true); $oembed_data = OEmbed::fetchURL($expanded_url, true);
Logger::info("statusnet_convertmsg: fetching data: done"); Logger::info('statusnet_convertmsg: fetching data: done');
if ($type == "") { if ($type == '') {
$type = $oembed_data->type; $type = $oembed_data->type;
} }
if ($oembed_data->type == "video") { if ($oembed_data->type == 'video') {
//$body = str_replace($search, "[video]".$expanded_url."[/video]", $body); //$body = str_replace($search, '[video]'.$expanded_url.'[/video]', $body);
$type = $oembed_data->type; $type = $oembed_data->type;
$footerurl = $expanded_url; $footerurl = $expanded_url;
$footerlink = "[url=" . $expanded_url . "]" . $expanded_url . "[/url]"; $footerlink = '[url=' . $expanded_url . ']' . $expanded_url . '[/url]';
$body = str_replace($search, $footerlink, $body); $body = str_replace($search, $footerlink, $body);
} elseif (($oembed_data->type == "photo") && isset($oembed_data->url)) { } elseif (($oembed_data->type == 'photo') && isset($oembed_data->url)) {
$body = str_replace($search, "[url=" . $expanded_url . "][img]" . $oembed_data->url . "[/img][/url]", $body); $body = str_replace($search, '[url=' . $expanded_url . '][img]' . $oembed_data->url . '[/img][/url]', $body);
} elseif ($oembed_data->type != "link") { } elseif ($oembed_data->type != 'link') {
$body = str_replace($search, "[url=" . $expanded_url . "]" . $expanded_url . "[/url]", $body); $body = str_replace($search, '[url=' . $expanded_url . ']' . $expanded_url . '[/url]', $body);
} else { } else {
$img_str = DI::httpClient()->fetch($expanded_url, HttpClientAccept::DEFAULT, 4); $img_str = DI::httpClient()->fetch($expanded_url, HttpClientAccept::DEFAULT, 4);
$tempfile = tempnam(System::getTempPath(), "cache"); $tempfile = tempnam(System::getTempPath(), 'cache');
file_put_contents($tempfile, $img_str); file_put_contents($tempfile, $img_str);
$mime = mime_content_type($tempfile); $mime = mime_content_type($tempfile);
unlink($tempfile); unlink($tempfile);
if (substr($mime, 0, 6) == "image/") { if (substr($mime, 0, 6) == 'image/') {
$type = "photo"; $type = 'photo';
$body = str_replace($search, "[img]" . $expanded_url . "[/img]", $body); $body = str_replace($search, '[img]' . $expanded_url . '[/img]', $body);
} else { } else {
$type = $oembed_data->type; $type = $oembed_data->type;
$footerurl = $expanded_url; $footerurl = $expanded_url;
$footerlink = "[url=" . $expanded_url . "]" . $expanded_url . "[/url]"; $footerlink = '[url=' . $expanded_url . ']' . $expanded_url . '[/url]';
$body = str_replace($search, $footerlink, $body); $body = str_replace($search, $footerlink, $body);
} }
} }
} }
if ($footerurl != "") { if ($footerurl != '') {
$footer = "\n" . PageInfo::getFooterFromUrl($footerurl); $footer = "\n" . PageInfo::getFooterFromUrl($footerurl);
} }
if (($footerlink != "") && (trim($footer) != "")) { if (($footerlink != '') && (trim($footer) != '')) {
$removedlink = trim(str_replace($footerlink, "", $body)); $removedlink = trim(str_replace($footerlink, '', $body));
if (($removedlink == "") || strstr($body, $removedlink)) { if (($removedlink == '') || strstr($body, $removedlink)) {
$body = $removedlink; $body = $removedlink;
} }
@ -1417,7 +1426,7 @@ function statusnet_fetch_own_contact(App $a, $uid)
$contact_id = 0; $contact_id = 0;
if ($own_url == "") { if ($own_url == '') {
$connection = new StatusNetOAuth($api, $ckey, $csecret, $otoken, $osecret); $connection = new StatusNetOAuth($api, $ckey, $csecret, $otoken, $osecret);
// Fetching user data // Fetching user data
@ -1433,7 +1442,7 @@ function statusnet_fetch_own_contact(App $a, $uid)
} else { } else {
$contact = Contact::selectFirst([], ['uid' => $uid, 'alias' => $own_url]); $contact = Contact::selectFirst([], ['uid' => $uid, 'alias' => $own_url]);
if (DBA::isResult($contact)) { if (DBA::isResult($contact)) {
$contact_id = $contact["id"]; $contact_id = $contact['id'];
} else { } else {
DI::pConfig()->delete($uid, 'statusnet', 'own_url'); DI::pConfig()->delete($uid, 'statusnet', 'own_url');
} }
@ -1447,12 +1456,12 @@ function statusnet_is_retweet(App $a, $uid, $body)
// Skip if it isn't a pure repeated messages // Skip if it isn't a pure repeated messages
// Does it start with a share? // Does it start with a share?
if (strpos($body, "[share") > 0) { if (strpos($body, '[share') > 0) {
return false; return false;
} }
// Does it end with a share? // Does it end with a share?
if (strlen($body) > (strrpos($body, "[/share]") + 8)) { if (strlen($body) > (strrpos($body, '[/share]') + 8)) {
return false; return false;
} }
@ -1462,7 +1471,7 @@ function statusnet_is_retweet(App $a, $uid, $body)
return false; return false;
} }
$link = ""; $link = '';
preg_match("/link='(.*?)'/ism", $attributes, $matches); preg_match("/link='(.*?)'/ism", $attributes, $matches);
if (!empty($matches[1])) { if (!empty($matches[1])) {
$link = $matches[1]; $link = $matches[1];

View file

@ -116,7 +116,7 @@ function twitter_install()
Hook::register('probe_detect' , __FILE__, 'twitter_probe_detect'); Hook::register('probe_detect' , __FILE__, 'twitter_probe_detect');
Hook::register('item_by_link' , __FILE__, 'twitter_item_by_link'); Hook::register('item_by_link' , __FILE__, 'twitter_item_by_link');
Hook::register('parse_link' , __FILE__, 'twitter_parse_link'); Hook::register('parse_link' , __FILE__, 'twitter_parse_link');
Logger::info("installed twitter"); Logger::info('installed twitter');
} }
// Hook functions // Hook functions
@ -145,15 +145,15 @@ function twitter_support_follow(App $a, array &$data)
function twitter_follow(App $a, array &$contact) function twitter_follow(App $a, array &$contact)
{ {
Logger::info('Check if contact is twitter contact', ['url' => $contact["url"]]); Logger::info('Check if contact is twitter contact', ['url' => $contact['url']]);
if (!strstr($contact["url"], "://twitter.com") && !strstr($contact["url"], "@twitter.com")) { if (!strstr($contact['url'], '://twitter.com') && !strstr($contact['url'], '@twitter.com')) {
return; return;
} }
// contact seems to be a twitter contact, so continue // contact seems to be a twitter contact, so continue
$nickname = preg_replace("=https?://twitter.com/(.*)=ism", "$1", $contact["url"]); $nickname = preg_replace("=https?://twitter.com/(.*)=ism", "$1", $contact['url']);
$nickname = str_replace("@twitter.com", "", $nickname); $nickname = str_replace('@twitter.com', '', $nickname);
$uid = $a->getLoggedInUserId(); $uid = $a->getLoggedInUserId();
@ -169,7 +169,7 @@ function twitter_follow(App $a, array &$contact)
$contact = Contact::getById($contact_id, ['name', 'nick', 'url', 'addr', 'batch', 'notify', 'poll', 'request', 'confirm', 'poco', 'photo', 'priority', 'network', 'alias', 'pubkey']); $contact = Contact::getById($contact_id, ['name', 'nick', 'url', 'addr', 'batch', 'notify', 'poll', 'request', 'confirm', 'poco', 'photo', 'priority', 'network', 'alias', 'pubkey']);
if (DBA::isResult($contact)) { if (DBA::isResult($contact)) {
$contact["contact"] = $contact; $contact['contact'] = $contact;
} }
} }
@ -262,7 +262,7 @@ function twitter_settings_post(App $a)
} }
$connection = new TwitterOAuth($ckey, $csecret, $_POST['twitter-token'], $_POST['twitter-token2']); $connection = new TwitterOAuth($ckey, $csecret, $_POST['twitter-token'], $_POST['twitter-token2']);
$token = $connection->oauth("oauth/access_token", ["oauth_verifier" => $_POST['twitter-pin']]); $token = $connection->oauth('oauth/access_token', ['oauth_verifier' => $_POST['twitter-pin']]);
// ok, now that we have the Access Token, save them in the user config // ok, now that we have the Access Token, save them in the user config
DI::pConfig()->set(local_user(), 'twitter', 'oauthtoken', $token['oauth_token']); DI::pConfig()->set(local_user(), 'twitter', 'oauthtoken', $token['oauth_token']);
DI::pConfig()->set(local_user(), 'twitter', 'oauthsecret', $token['oauth_token_secret']); DI::pConfig()->set(local_user(), 'twitter', 'oauthsecret', $token['oauth_token_secret']);
@ -634,7 +634,7 @@ function twitter_post_hook(App $a, array &$b)
} }
// Post to Twitter // Post to Twitter
if (!DI::pConfig()->get($b["uid"], 'twitter', 'import') if (!DI::pConfig()->get($b['uid'], 'twitter', 'import')
&& ($b['private'] || ($b['created'] !== $b['edited']))) { && ($b['private'] || ($b['created'] !== $b['edited']))) {
return; return;
} }
@ -647,17 +647,17 @@ function twitter_post_hook(App $a, array &$b)
Logger::debug('Got comment', ['item' => $b]); Logger::debug('Got comment', ['item' => $b]);
// Looking if its a reply to a twitter post // Looking if its a reply to a twitter post
if (!twitter_get_id($b["parent-uri"]) && if (!twitter_get_id($b['parent-uri']) &&
!twitter_get_id($b["extid"]) && !twitter_get_id($b['extid']) &&
!twitter_get_id($b["thr-parent"])) { !twitter_get_id($b['thr-parent'])) {
Logger::info('No twitter post', ['parent' => $b["parent"]]); Logger::info('No twitter post', ['parent' => $b['parent']]);
return; return;
} }
$condition = ['uri' => $b["thr-parent"], 'uid' => $b["uid"]]; $condition = ['uri' => $b['thr-parent'], 'uid' => $b['uid']];
$thr_parent = Post::selectFirst(['uri', 'extid', 'author-link', 'author-nick', 'author-network'], $condition); $thr_parent = Post::selectFirst(['uri', 'extid', 'author-link', 'author-nick', 'author-network'], $condition);
if (!DBA::isResult($thr_parent)) { if (!DBA::isResult($thr_parent)) {
Logger::warning('No parent found', ['thr-parent' => $b["thr-parent"]]); Logger::warning('No parent found', ['thr-parent' => $b['thr-parent']]);
return; return;
} }
@ -665,9 +665,9 @@ function twitter_post_hook(App $a, array &$b)
$nickname = '@[url=' . $thr_parent['author-link'] . ']' . $thr_parent['author-nick'] . '[/url]'; $nickname = '@[url=' . $thr_parent['author-link'] . ']' . $thr_parent['author-nick'] . '[/url]';
$nicknameplain = '@' . $thr_parent['author-nick']; $nicknameplain = '@' . $thr_parent['author-nick'];
Logger::info('Comparing', ['nickname' => $nickname, 'nicknameplain' => $nicknameplain, 'body' => $b["body"]]); Logger::info('Comparing', ['nickname' => $nickname, 'nicknameplain' => $nicknameplain, 'body' => $b['body']]);
if ((strpos($b["body"], $nickname) === false) && (strpos($b["body"], $nicknameplain) === false)) { if ((strpos($b['body'], $nickname) === false) && (strpos($b['body'], $nicknameplain) === false)) {
$b["body"] = $nickname . " " . $b["body"]; $b['body'] = $nickname . ' ' . $b['body'];
} }
} }
@ -686,7 +686,7 @@ function twitter_post_hook(App $a, array &$b)
} }
if ($b['verb'] == Activity::LIKE) { if ($b['verb'] == Activity::LIKE) {
Logger::info('Like', ['uid' => $b['uid'], 'id' => twitter_get_id($b["thr-parent"])]); Logger::info('Like', ['uid' => $b['uid'], 'id' => twitter_get_id($b['thr-parent'])]);
twitter_api_post('favorites/create', twitter_get_id($b['thr-parent']), $b['uid']); twitter_api_post('favorites/create', twitter_get_id($b['thr-parent']), $b['uid']);
@ -694,7 +694,7 @@ function twitter_post_hook(App $a, array &$b)
} }
if ($b['verb'] == Activity::ANNOUNCE) { if ($b['verb'] == Activity::ANNOUNCE) {
Logger::info('Retweet', ['uid' => $b['uid'], 'id' => twitter_get_id($b["thr-parent"])]); Logger::info('Retweet', ['uid' => $b['uid'], 'id' => twitter_get_id($b['thr-parent'])]);
twitter_retweet($b['uid'], twitter_get_id($b['thr-parent'])); twitter_retweet($b['uid'], twitter_get_id($b['thr-parent']));
return; return;
} }
@ -708,7 +708,7 @@ function twitter_post_hook(App $a, array &$b)
return; return;
} }
if ($b['app'] == "Twitter") { if ($b['app'] == 'Twitter') {
return; return;
} }
@ -752,10 +752,10 @@ function twitter_post_hook(App $a, array &$b)
$msgarr = Plaintext::getPost($b, $max_char, true, BBCode::TWITTER); $msgarr = Plaintext::getPost($b, $max_char, true, BBCode::TWITTER);
Logger::info('Got plaintext', ['id' => $b['id'], 'message' => $msgarr]); Logger::info('Got plaintext', ['id' => $b['id'], 'message' => $msgarr]);
$msg = $msgarr["text"]; $msg = $msgarr['text'];
if (($msg == "") && isset($msgarr["title"])) { if (($msg == '') && isset($msgarr['title'])) {
$msg = Plaintext::shorten($msgarr["title"], $max_char - 50, $b['uid']); $msg = Plaintext::shorten($msgarr['title'], $max_char - 50, $b['uid']);
} }
// Add the link to the body if the type isn't a photo or there are more than 4 images in the post // Add the link to the body if the type isn't a photo or there are more than 4 images in the post
@ -823,7 +823,7 @@ function twitter_post_hook(App $a, array &$b)
Logger::info('twitter_post send', ['id' => $b['id'], 'result' => $result]); Logger::info('twitter_post send', ['id' => $b['id'], 'result' => $result]);
if (!empty($result->source)) { if (!empty($result->source)) {
DI::config()->set("twitter", "application_name", strip_tags($result->source)); DI::config()->set('twitter', 'application_name', strip_tags($result->source));
} }
if (!empty($result->errors)) { if (!empty($result->errors)) {
@ -831,7 +831,7 @@ function twitter_post_hook(App $a, array &$b)
Worker::defer(); Worker::defer();
} elseif ($thr_parent) { } elseif ($thr_parent) {
Logger::notice('Post send, updating extid', ['id' => $b['id'], 'extid' => $result->id_str]); Logger::notice('Post send, updating extid', ['id' => $b['id'], 'extid' => $result->id_str]);
Item::update(['extid' => "twitter::" . $result->id_str], ['id' => $b['id']]); Item::update(['extid' => 'twitter::' . $result->id_str], ['id' => $b['id']]);
} }
} }
} }
@ -911,7 +911,7 @@ function twitter_addon_admin_post(App $a)
function twitter_addon_admin(App $a, &$o) function twitter_addon_admin(App $a, &$o)
{ {
$t = Renderer::getMarkupTemplate("admin.tpl", "addon/twitter/"); $t = Renderer::getMarkupTemplate('admin.tpl', 'addon/twitter/');
$o = Renderer::replaceMacros($t, [ $o = Renderer::replaceMacros($t, [
'$submit' => DI::l10n()->t('Save Settings'), '$submit' => DI::l10n()->t('Save Settings'),
@ -942,7 +942,7 @@ function twitter_cron(App $a)
$pconfigs = DBA::selectToArray('pconfig', [], ['cat' => 'twitter', 'k' => 'mirror_posts', 'v' => true]); $pconfigs = DBA::selectToArray('pconfig', [], ['cat' => 'twitter', 'k' => 'mirror_posts', 'v' => true]);
foreach ($pconfigs as $rr) { foreach ($pconfigs as $rr) {
Logger::notice('Fetching', ['user' => $rr['uid']]); Logger::notice('Fetching', ['user' => $rr['uid']]);
Worker::add(['priority' => PRIORITY_MEDIUM, 'force_priority' => true], "addon/twitter/twitter_sync.php", 1, (int) $rr['uid']); Worker::add(['priority' => PRIORITY_MEDIUM, 'force_priority' => true], 'addon/twitter/twitter_sync.php', 1, (int) $rr['uid']);
} }
$abandon_days = intval(DI::config()->get('system', 'account_abandon_days')); $abandon_days = intval(DI::config()->get('system', 'account_abandon_days'));
@ -962,7 +962,7 @@ function twitter_cron(App $a)
} }
Logger::notice('importing timeline', ['user' => $rr['uid']]); Logger::notice('importing timeline', ['user' => $rr['uid']]);
Worker::add(['priority' => PRIORITY_MEDIUM, 'force_priority' => true], "addon/twitter/twitter_sync.php", 2, (int) $rr['uid']); Worker::add(['priority' => PRIORITY_MEDIUM, 'force_priority' => true], 'addon/twitter/twitter_sync.php', 2, (int) $rr['uid']);
/* /*
// To-Do // To-Do
// check for new contacts once a day // check for new contacts once a day
@ -1016,36 +1016,36 @@ function twitter_expire(App $a)
function twitter_prepare_body(App $a, array &$b) function twitter_prepare_body(App $a, array &$b)
{ {
if ($b["item"]["network"] != Protocol::TWITTER) { if ($b['item']['network'] != Protocol::TWITTER) {
return; return;
} }
if ($b["preview"]) { if ($b['preview']) {
$max_char = 280; $max_char = 280;
$item = $b["item"]; $item = $b['item'];
$item["plink"] = DI::baseUrl()->get() . "/display/" . $item["guid"]; $item['plink'] = DI::baseUrl()->get() . '/display/' . $item['guid'];
$condition = ['uri' => $item["thr-parent"], 'uid' => local_user()]; $condition = ['uri' => $item['thr-parent'], 'uid' => local_user()];
$orig_post = Post::selectFirst(['author-link'], $condition); $orig_post = Post::selectFirst(['author-link'], $condition);
if (DBA::isResult($orig_post)) { if (DBA::isResult($orig_post)) {
$nicknameplain = preg_replace("=https?://twitter.com/(.*)=ism", "$1", $orig_post["author-link"]); $nicknameplain = preg_replace("=https?://twitter.com/(.*)=ism", "$1", $orig_post['author-link']);
$nickname = "@[url=" . $orig_post["author-link"] . "]" . $nicknameplain . "[/url]"; $nickname = '@[url=' . $orig_post['author-link'] . ']' . $nicknameplain . '[/url]';
$nicknameplain = "@" . $nicknameplain; $nicknameplain = '@' . $nicknameplain;
if ((strpos($item["body"], $nickname) === false) && (strpos($item["body"], $nicknameplain) === false)) { if ((strpos($item['body'], $nickname) === false) && (strpos($item['body'], $nicknameplain) === false)) {
$item["body"] = $nickname . " " . $item["body"]; $item['body'] = $nickname . ' ' . $item['body'];
} }
} }
$msgarr = Plaintext::getPost($item, $max_char, true, BBCode::TWITTER); $msgarr = Plaintext::getPost($item, $max_char, true, BBCode::TWITTER);
$msg = $msgarr["text"]; $msg = $msgarr['text'];
if (isset($msgarr["url"]) && ($msgarr["type"] != "photo")) { if (isset($msgarr['url']) && ($msgarr['type'] != 'photo')) {
$msg .= " " . $msgarr["url"]; $msg .= ' ' . $msgarr['url'];
} }
if (isset($msgarr["image"])) { if (isset($msgarr['image'])) {
$msg .= " " . $msgarr["image"]; $msg .= ' ' . $msgarr['image'];
} }
$b['html'] = nl2br(htmlspecialchars($msg)); $b['html'] = nl2br(htmlspecialchars($msg));
@ -1206,7 +1206,7 @@ function twitter_fetchtimeline(App $a, $uid)
$application_name = DI::config()->get('twitter', 'application_name'); $application_name = DI::config()->get('twitter', 'application_name');
if ($application_name == "") { if ($application_name == '') {
$application_name = DI::baseUrl()->getHostname(); $application_name = DI::baseUrl()->getHostname();
} }
@ -1220,12 +1220,19 @@ function twitter_fetchtimeline(App $a, $uid)
return; return;
} }
$parameters = ["exclude_replies" => true, "trim_user" => false, "contributor_details" => true, "include_rts" => true, "tweet_mode" => "extended", "include_ext_alt_text" => true]; $parameters = [
'exclude_replies' => true,
'trim_user' => false,
'contributor_details' => true,
'include_rts' => true,
'tweet_mode' => 'extended',
'include_ext_alt_text' => true,
];
$first_time = ($lastid == ""); $first_time = ($lastid == '');
if ($lastid != "") { if ($lastid != '') {
$parameters["since_id"] = $lastid; $parameters['since_id'] = $lastid;
} }
try { try {
@ -1276,7 +1283,7 @@ function twitter_fetchtimeline(App $a, $uid)
function twitter_fix_avatar($avatar) function twitter_fix_avatar($avatar)
{ {
$new_avatar = str_replace("_normal.", "_400x400.", $avatar); $new_avatar = str_replace('_normal.', '_400x400.', $avatar);
$info = Images::getInfoFromURLCached($new_avatar); $info = Images::getInfoFromURLCached($new_avatar);
if (!$info) { if (!$info) {
@ -1365,7 +1372,7 @@ function twitter_user_to_contact($data)
function twitter_get_contact($data, int $uid = 0) function twitter_get_contact($data, int $uid = 0)
{ {
$contact = DBA::selectFirst('contact', ['id'], ['uid' => $uid, 'alias' => "twitter::" . $data->id_str]); $contact = DBA::selectFirst('contact', ['id'], ['uid' => $uid, 'alias' => 'twitter::' . $data->id_str]);
if (DBA::isResult($contact)) { if (DBA::isResult($contact)) {
return $contact['id']; return $contact['id'];
} else { } else {
@ -1386,7 +1393,7 @@ function twitter_fetch_contact($uid, $data, $create_user)
unset($fields['photo']); unset($fields['photo']);
// Update the public contact // Update the public contact
$pcontact = DBA::selectFirst('contact', ['id'], ['uid' => 0, 'alias' => "twitter::" . $data->id_str]); $pcontact = DBA::selectFirst('contact', ['id'], ['uid' => 0, 'alias' => 'twitter::' . $data->id_str]);
if (DBA::isResult($pcontact)) { if (DBA::isResult($pcontact)) {
$cid = $pcontact['id']; $cid = $pcontact['id'];
} else { } else {
@ -1400,7 +1407,7 @@ function twitter_fetch_contact($uid, $data, $create_user)
Logger::warning('No contact found', ['fields' => $fields]); Logger::warning('No contact found', ['fields' => $fields]);
} }
$contact = DBA::selectFirst('contact', [], ['uid' => $uid, 'alias' => "twitter::" . $data->id_str]); $contact = DBA::selectFirst('contact', [], ['uid' => $uid, 'alias' => 'twitter::' . $data->id_str]);
if (!DBA::isResult($contact) && empty($cid)) { if (!DBA::isResult($contact) && empty($cid)) {
Logger::warning('User contact not found', ['uid' => $uid, 'twitter-id' => $data->id_str]); Logger::warning('User contact not found', ['uid' => $uid, 'twitter-id' => $data->id_str]);
return 0; return 0;
@ -1430,8 +1437,8 @@ function twitter_fetch_contact($uid, $data, $create_user)
Group::addMember(User::getDefaultGroup($uid), $contact_id); Group::addMember(User::getDefaultGroup($uid), $contact_id);
} else { } else {
if ($contact["readonly"] || $contact["blocked"]) { if ($contact['readonly'] || $contact['blocked']) {
Logger::notice('Contact is blocked or readonly.', ['nickname' => $contact["nick"]]); Logger::notice('Contact is blocked or readonly.', ['nickname' => $contact['nick']]);
return -1; return -1;
} }
@ -1739,7 +1746,7 @@ function twitter_createpost(App $a, $uid, $post, array $self, $create_user, $onl
$postarray['network'] = Protocol::TWITTER; $postarray['network'] = Protocol::TWITTER;
$postarray['uid'] = $uid; $postarray['uid'] = $uid;
$postarray['wall'] = 0; $postarray['wall'] = 0;
$postarray['uri'] = "twitter::" . $post->id_str; $postarray['uri'] = 'twitter::' . $post->id_str;
$postarray['protocol'] = Conversation::PARCEL_TWITTER; $postarray['protocol'] = Conversation::PARCEL_TWITTER;
$postarray['source'] = json_encode($post); $postarray['source'] = json_encode($post);
$postarray['direction'] = Conversation::PULL; $postarray['direction'] = Conversation::PULL;
@ -1756,8 +1763,8 @@ function twitter_createpost(App $a, $uid, $post, array $self, $create_user, $onl
$contactid = 0; $contactid = 0;
if ($post->in_reply_to_status_id_str != "") { if ($post->in_reply_to_status_id_str != '') {
$thr_parent = "twitter::" . $post->in_reply_to_status_id_str; $thr_parent = 'twitter::' . $post->in_reply_to_status_id_str;
$item = Post::selectFirst(['uri'], ['uri' => $thr_parent, 'uid' => $uid]); $item = Post::selectFirst(['uri'], ['uri' => $thr_parent, 'uid' => $uid]);
if (!DBA::isResult($item)) { if (!DBA::isResult($item)) {
@ -1798,7 +1805,7 @@ function twitter_createpost(App $a, $uid, $post, array $self, $create_user, $onl
$postarray['owner-id'] = twitter_get_contact($post->user); $postarray['owner-id'] = twitter_get_contact($post->user);
$postarray['owner-name'] = $post->user->name; $postarray['owner-name'] = $post->user->name;
$postarray['owner-link'] = "https://twitter.com/" . $post->user->screen_name; $postarray['owner-link'] = 'https://twitter.com/' . $post->user->screen_name;
$postarray['owner-avatar'] = twitter_fix_avatar($post->user->profile_image_url_https); $postarray['owner-avatar'] = twitter_fix_avatar($post->user->profile_image_url_https);
} }
@ -1815,7 +1822,7 @@ function twitter_createpost(App $a, $uid, $post, array $self, $create_user, $onl
$postarray['author-name'] = $postarray['owner-name']; $postarray['author-name'] = $postarray['owner-name'];
$postarray['author-link'] = $postarray['owner-link']; $postarray['author-link'] = $postarray['owner-link'];
$postarray['author-avatar'] = $postarray['owner-avatar']; $postarray['author-avatar'] = $postarray['owner-avatar'];
$postarray['plink'] = "https://twitter.com/" . $post->user->screen_name . "/status/" . $post->id_str; $postarray['plink'] = 'https://twitter.com/' . $post->user->screen_name . '/status/' . $post->id_str;
$postarray['app'] = strip_tags($post->source); $postarray['app'] = strip_tags($post->source);
if ($post->user->protected) { if ($post->user->protected) {
@ -1857,16 +1864,16 @@ function twitter_createpost(App $a, $uid, $post, array $self, $create_user, $onl
} }
if (!empty($post->place->name)) { if (!empty($post->place->name)) {
$postarray["location"] = $post->place->name; $postarray['location'] = $post->place->name;
} }
if (!empty($post->place->full_name)) { if (!empty($post->place->full_name)) {
$postarray["location"] = $post->place->full_name; $postarray['location'] = $post->place->full_name;
} }
if (!empty($post->geo->coordinates)) { if (!empty($post->geo->coordinates)) {
$postarray["coord"] = $post->geo->coordinates[0] . " " . $post->geo->coordinates[1]; $postarray['coord'] = $post->geo->coordinates[0] . ' ' . $post->geo->coordinates[1];
} }
if (!empty($post->coordinates->coordinates)) { if (!empty($post->coordinates->coordinates)) {
$postarray["coord"] = $post->coordinates->coordinates[1] . " " . $post->coordinates->coordinates[0]; $postarray['coord'] = $post->coordinates->coordinates[1] . ' ' . $post->coordinates->coordinates[0];
} }
if (!empty($post->retweeted_status)) { if (!empty($post->retweeted_status)) {
$retweet = twitter_createpost($a, $uid, $post->retweeted_status, $self, false, false, $noquote); $retweet = twitter_createpost($a, $uid, $post->retweeted_status, $self, false, false, $noquote);
@ -1923,7 +1930,7 @@ function twitter_createpost(App $a, $uid, $post, array $self, $create_user, $onl
$postarray['body'] .= $quoted['body'] . '[/share]'; $postarray['body'] .= $quoted['body'] . '[/share]';
} else { } else {
// Quoted post author is blocked/ignored, so we just provide the link to avoid removing quote context. // Quoted post author is blocked/ignored, so we just provide the link to avoid removing quote context.
$postarray['body'] .= "\n\nhttps://twitter.com/" . $post->quoted_status->user->screen_name . "/status/" . $post->quoted_status->id_str; $postarray['body'] .= "\n\nhttps://twitter.com/" . $post->quoted_status->user->screen_name . '/status/' . $post->quoted_status->id_str;
} }
} }
} }
@ -1964,7 +1971,7 @@ function twitter_fetchparentposts(App $a, $uid, $post, TwitterOAuth $connection,
} }
if (empty($post->id_str)) { if (empty($post->id_str)) {
Logger::info("twitter_fetchparentposts: This is not a post", ['post' => $post]); Logger::info('twitter_fetchparentposts: This is not a post', ['post' => $post]);
break; break;
} }
@ -1975,7 +1982,7 @@ function twitter_fetchparentposts(App $a, $uid, $post, TwitterOAuth $connection,
$posts[] = $post; $posts[] = $post;
} }
Logger::info("twitter_fetchparentposts: Fetching " . count($posts) . " parents"); Logger::info('twitter_fetchparentposts: Fetching ' . count($posts) . ' parents');
$posts = array_reverse($posts); $posts = array_reverse($posts);
@ -1989,9 +1996,9 @@ function twitter_fetchparentposts(App $a, $uid, $post, TwitterOAuth $connection,
$item = Item::insert($postarray); $item = Item::insert($postarray);
$postarray["id"] = $item; $postarray['id'] = $item;
Logger::notice('twitter_fetchparentpost: User ' . $self["nick"] . ' posted parent timeline item ' . $item); Logger::notice('twitter_fetchparentpost: User ' . $self['nick'] . ' posted parent timeline item ' . $item);
} }
} }
} }
@ -2009,7 +2016,7 @@ function twitter_fetchhometimeline(App $a, $uid)
$application_name = DI::config()->get('twitter', 'application_name'); $application_name = DI::config()->get('twitter', 'application_name');
if ($application_name == "") { if ($application_name == '') {
$application_name = DI::baseUrl()->getHostname(); $application_name = DI::baseUrl()->getHostname();
} }
@ -2036,15 +2043,23 @@ function twitter_fetchhometimeline(App $a, $uid)
return; return;
} }
$parameters = ["exclude_replies" => false, "trim_user" => false, "contributor_details" => true, "include_rts" => true, "tweet_mode" => "extended", "include_ext_alt_text" => true]; $parameters = [
//$parameters["count"] = 200; 'exclude_replies' => false,
'trim_user' => false,
'contributor_details' => true,
'include_rts' => true,
'tweet_mode' => 'extended',
'include_ext_alt_text' => true,
//'count' => 200,
];
// Fetching timeline // Fetching timeline
$lastid = DI::pConfig()->get($uid, 'twitter', 'lasthometimelineid'); $lastid = DI::pConfig()->get($uid, 'twitter', 'lasthometimelineid');
$first_time = ($lastid == ""); $first_time = ($lastid == '');
if ($lastid != "") { if ($lastid != '') {
$parameters["since_id"] = $lastid; $parameters['since_id'] = $lastid;
} }
try { try {
@ -2080,16 +2095,16 @@ function twitter_fetchhometimeline(App $a, $uid)
} }
if (stristr($post->source, $application_name) && $post->user->screen_name == $own_id) { if (stristr($post->source, $application_name) && $post->user->screen_name == $own_id) {
Logger::info("Skip previously sent post"); Logger::info('Skip previously sent post');
continue; continue;
} }
if ($mirror_posts && $post->user->screen_name == $own_id && $post->in_reply_to_status_id_str == "") { if ($mirror_posts && $post->user->screen_name == $own_id && $post->in_reply_to_status_id_str == '') {
Logger::info("Skip post that will be mirrored"); Logger::info('Skip post that will be mirrored');
continue; continue;
} }
if ($post->in_reply_to_status_id_str != "") { if ($post->in_reply_to_status_id_str != '') {
twitter_fetchparentposts($a, $uid, $post, $connection, $self); twitter_fetchparentposts($a, $uid, $post, $connection, $self);
} }
@ -2097,7 +2112,7 @@ function twitter_fetchhometimeline(App $a, $uid)
$postarray = twitter_createpost($a, $uid, $post, $self, $create_user, true, false); $postarray = twitter_createpost($a, $uid, $post, $self, $create_user, true, false);
if (empty($postarray['body']) || trim($postarray['body']) == "") { if (empty($postarray['body']) || trim($postarray['body']) == '') {
Logger::info('Empty body for post ' . $post->id_str . ' and user ' . $uid); Logger::info('Empty body for post ' . $post->id_str . ' and user ' . $uid);
continue; continue;
} }
@ -2112,7 +2127,7 @@ function twitter_fetchhometimeline(App $a, $uid)
} }
$item = Item::insert($postarray, $notify); $item = Item::insert($postarray, $notify);
$postarray["id"] = $item; $postarray['id'] = $item;
Logger::notice('User ' . $uid . ' posted home timeline item ' . $item); Logger::notice('User ' . $uid . ' posted home timeline item ' . $item);
} }
@ -2124,10 +2139,10 @@ function twitter_fetchhometimeline(App $a, $uid)
// Fetching mentions // Fetching mentions
$lastid = DI::pConfig()->get($uid, 'twitter', 'lastmentionid'); $lastid = DI::pConfig()->get($uid, 'twitter', 'lastmentionid');
$first_time = ($lastid == ""); $first_time = ($lastid == '');
if ($lastid != "") { if ($lastid != '') {
$parameters["since_id"] = $lastid; $parameters['since_id'] = $lastid;
} }
try { try {
@ -2138,13 +2153,13 @@ function twitter_fetchhometimeline(App $a, $uid)
} }
if (!is_array($items)) { if (!is_array($items)) {
Logger::warning("mentions are no arrays", ['items' => $items]); Logger::warning('mentions are no arrays', ['items' => $items]);
return; return;
} }
$posts = array_reverse($items); $posts = array_reverse($items);
Logger::info("Fetching mentions for user " . $uid . " " . sizeof($posts) . " items"); Logger::info('Fetching mentions for user ' . $uid . ' ' . sizeof($posts) . ' items');
if (count($posts)) { if (count($posts)) {
foreach ($posts as $post) { foreach ($posts as $post) {
@ -2156,7 +2171,7 @@ function twitter_fetchhometimeline(App $a, $uid)
continue; continue;
} }
if ($post->in_reply_to_status_id_str != "") { if ($post->in_reply_to_status_id_str != '') {
twitter_fetchparentposts($a, $uid, $post, $connection, $self); twitter_fetchparentposts($a, $uid, $post, $connection, $self);
} }
@ -2188,7 +2203,7 @@ function twitter_fetch_own_contact(App $a, $uid)
$contact_id = 0; $contact_id = 0;
if ($own_id == "") { if ($own_id == '') {
$connection = new TwitterOAuth($ckey, $csecret, $otoken, $osecret); $connection = new TwitterOAuth($ckey, $csecret, $otoken, $osecret);
// Fetching user data // Fetching user data
@ -2219,12 +2234,12 @@ function twitter_is_retweet(App $a, $uid, $body)
// Skip if it isn't a pure repeated messages // Skip if it isn't a pure repeated messages
// Does it start with a share? // Does it start with a share?
if (strpos($body, "[share") > 0) { if (strpos($body, '[share') > 0) {
return false; return false;
} }
// Does it end with a share? // Does it end with a share?
if (strlen($body) > (strrpos($body, "[/share]") + 8)) { if (strlen($body) > (strrpos($body, '[/share]') + 8)) {
return false; return false;
} }
@ -2234,7 +2249,7 @@ function twitter_is_retweet(App $a, $uid, $body)
return false; return false;
} }
$link = ""; $link = '';
preg_match("/link='(.*?)'/ism", $attributes, $matches); preg_match("/link='(.*?)'/ism", $attributes, $matches);
if (!empty($matches[1])) { if (!empty($matches[1])) {
$link = $matches[1]; $link = $matches[1];
@ -2262,7 +2277,7 @@ function twitter_retweet(int $uid, int $id, int $item_id = 0)
if (!empty($item_id) && !empty($result->id_str)) { if (!empty($item_id) && !empty($result->id_str)) {
Logger::notice('Update extid', ['id' => $item_id, 'extid' => $result->id_str]); Logger::notice('Update extid', ['id' => $item_id, 'extid' => $result->id_str]);
Item::update(['extid' => "twitter::" . $result->id_str], ['id' => $item_id]); Item::update(['extid' => 'twitter::' . $result->id_str], ['id' => $item_id]);
} }
return !isset($result->errors); return !isset($result->errors);
@ -2270,7 +2285,7 @@ function twitter_retweet(int $uid, int $id, int $item_id = 0)
function twitter_update_mentions($body) function twitter_update_mentions($body)
{ {
$URLSearchString = "^\[\]"; $URLSearchString = '^\[\]';
$return = preg_replace_callback( $return = preg_replace_callback(
"/@\[url\=([$URLSearchString]*)\](.*?)\[\/url\]/ism", "/@\[url\=([$URLSearchString]*)\](.*?)\[\/url\]/ism",
function ($matches) { function ($matches) {