And some warnings removed
This commit is contained in:
parent
3874e491f9
commit
1613cba915
|
@ -455,7 +455,7 @@ function fromgplus_fetch($a, $uid) {
|
|||
|
||||
$lastdate = 0;
|
||||
|
||||
if (!is_array($activities->items))
|
||||
if (empty($activities->items))
|
||||
return;
|
||||
|
||||
$reversed = array_reverse($activities->items);
|
||||
|
|
|
@ -654,8 +654,7 @@ function pumpio_action(&$a, $uid, $uri, $action, $content = "")
|
|||
|
||||
function pumpio_sync(&$a)
|
||||
{
|
||||
$r = q("SELECT * FROM `addon` WHERE `installed` = 1 AND `name` = 'pumpio'",
|
||||
$plugin);
|
||||
$r = q("SELECT * FROM `addon` WHERE `installed` = 1 AND `name` = 'pumpio'");
|
||||
|
||||
if (!DBM::is_result($r)) {
|
||||
return;
|
||||
|
@ -765,7 +764,7 @@ function pumpio_fetchtimeline(&$a, $uid)
|
|||
|
||||
logger('pumpio: fetching for user '.$uid.' '.$url.' C:'.$client->client_id.' CS:'.$client->client_secret.' T:'.$client->access_token.' TS:'.$client->access_token_secret);
|
||||
|
||||
$username = $user.'@'.$host;
|
||||
$useraddr = $username.'@'.$hostname;
|
||||
|
||||
if (pumpio_reachable($url)) {
|
||||
$success = $client->CallAPI($url, 'GET', [], ['FailOnAccessError'=>true], $user);
|
||||
|
@ -774,7 +773,7 @@ function pumpio_fetchtimeline(&$a, $uid)
|
|||
}
|
||||
|
||||
if (!$success) {
|
||||
logger('pumpio: error fetching posts for user '.$uid." ".$username." ".print_r($user, true));
|
||||
logger('pumpio: error fetching posts for user '.$uid." ".$useraddr." ".print_r($user, true));
|
||||
return;
|
||||
}
|
||||
|
||||
|
@ -903,7 +902,7 @@ function pumpio_dolike(&$a, $uid, $self, $post, $own_id, $threadcompletion = tru
|
|||
{
|
||||
require_once('include/items.php');
|
||||
|
||||
if ($post->object->id == "") {
|
||||
if (empty($post->object->id)) {
|
||||
logger('Got empty like: '.print_r($post, true), LOGGER_DEBUG);
|
||||
return;
|
||||
}
|
||||
|
@ -1187,7 +1186,7 @@ function pumpio_dopost(&$a, $client, $uid, $self, $post, $own_id, $threadcomplet
|
|||
$postarray['parent-uri'] = $post->object->inReplyTo->id;
|
||||
}
|
||||
|
||||
if ($post->object->pump_io->proxyURL) {
|
||||
if (!empty($post->object->pump_io->proxyURL)) {
|
||||
$postarray['extid'] = $post->object->pump_io->proxyURL;
|
||||
}
|
||||
|
||||
|
@ -1204,7 +1203,7 @@ function pumpio_dopost(&$a, $client, $uid, $self, $post, $own_id, $threadcomplet
|
|||
$postarray['body'] = HTML::toBBCode($post->object->content);
|
||||
$postarray['object'] = json_encode($post);
|
||||
|
||||
if ($post->object->fullImage->url != "") {
|
||||
if (!empty($post->object->fullImage->url)) {
|
||||
$postarray["body"] = "[url=".$post->object->fullImage->url."][img]".$post->object->image->url."[/img][/url]\n".$postarray["body"];
|
||||
}
|
||||
|
||||
|
@ -1588,9 +1587,9 @@ function pumpio_fetchallcomments(&$a, $uid, $id)
|
|||
$like->object->id = $item->id;
|
||||
$like->actor = new stdClass;
|
||||
$like->actor->displayName = $item->displayName;
|
||||
$like->actor->preferredUsername = $item->preferredUsername;
|
||||
//$like->actor->preferredUsername = $item->preferredUsername;
|
||||
//$like->actor->image = $item->image;
|
||||
$like->actor->url = $item->url;
|
||||
$like->actor->image = $item->image;
|
||||
$like->generator = new stdClass;
|
||||
$like->generator->displayName = "pumpio";
|
||||
pumpio_dolike($a, $uid, $self, $post, $own_id, false);
|
||||
|
|
|
@ -1065,7 +1065,7 @@ function statusnet_createpost(App $a, $uid, $post, $self, $create_user, $only_ex
|
|||
$postarray['uid'] = $uid;
|
||||
$postarray['wall'] = 0;
|
||||
|
||||
if (is_object($post->retweeted_status)) {
|
||||
if (!empty($post->retweeted_status)) {
|
||||
$content = $post->retweeted_status;
|
||||
statusnet_fetch_contact($uid, $content->user, false);
|
||||
} else {
|
||||
|
@ -1080,7 +1080,7 @@ function statusnet_createpost(App $a, $uid, $post, $self, $create_user, $only_ex
|
|||
|
||||
$contactid = 0;
|
||||
|
||||
if ($content->in_reply_to_status_id != "") {
|
||||
if (!empty($content->in_reply_to_status_id)) {
|
||||
|
||||
$parent = $hostname . "::" . $content->in_reply_to_status_id;
|
||||
|
||||
|
@ -1166,19 +1166,19 @@ function statusnet_createpost(App $a, $uid, $post, $self, $create_user, $only_ex
|
|||
$postarray['created'] = DateTimeFormat::utc($content->created_at);
|
||||
$postarray['edited'] = DateTimeFormat::utc($content->created_at);
|
||||
|
||||
if (is_string($content->place->name)) {
|
||||
if (!empty($content->place->name)) {
|
||||
$postarray["location"] = $content->place->name;
|
||||
}
|
||||
|
||||
if (is_string($content->place->full_name)) {
|
||||
if (!empty($content->place->full_name)) {
|
||||
$postarray["location"] = $content->place->full_name;
|
||||
}
|
||||
|
||||
if (is_array($content->geo->coordinates)) {
|
||||
if (!empty($content->geo->coordinates)) {
|
||||
$postarray["coord"] = $content->geo->coordinates[0] . " " . $content->geo->coordinates[1];
|
||||
}
|
||||
|
||||
if (is_array($content->coordinates->coordinates)) {
|
||||
if (!empty($content->coordinates->coordinates)) {
|
||||
$postarray["coord"] = $content->coordinates->coordinates[1] . " " . $content->coordinates->coordinates[0];
|
||||
}
|
||||
|
||||
|
|
|
@ -799,7 +799,7 @@ function twitter_do_mirrorpost(App $a, $uid, $post)
|
|||
// $datarray['object'] = json_encode($post); // Activate for debugging
|
||||
$datarray["title"] = "";
|
||||
|
||||
if (is_object($post->retweeted_status)) {
|
||||
if (!empty($post->retweeted_status)) {
|
||||
// We don't support nested shares, so we mustn't show quotes as shares on retweets
|
||||
$item = twitter_createpost($a, $uid, $post->retweeted_status, ['id' => 0], false, false, true);
|
||||
|
||||
|
@ -1254,7 +1254,7 @@ function twitter_expand_entities(App $a, $body, $item, $picture)
|
|||
function twitter_media_entities($post, &$postarray)
|
||||
{
|
||||
// There are no media entities? So we quit.
|
||||
if (!is_array($post->extended_entities->media)) {
|
||||
if (empty($post->extended_entities->media)) {
|
||||
return "";
|
||||
}
|
||||
|
||||
|
@ -1274,6 +1274,9 @@ function twitter_media_entities($post, &$postarray)
|
|||
// This is a pure media post, first search for all media urls
|
||||
$media = [];
|
||||
foreach ($post->extended_entities->media AS $medium) {
|
||||
if (!isset($media[$medium->url])) {
|
||||
$media[$medium->url] = '';
|
||||
}
|
||||
switch ($medium->type) {
|
||||
case 'photo':
|
||||
$media[$medium->url] .= "\n[img]" . $medium->media_url_https . "[/img]";
|
||||
|
@ -1396,6 +1399,9 @@ function twitter_createpost(App $a, $uid, $post, $self, $create_user, $only_exis
|
|||
if ($post->user->protected) {
|
||||
$postarray['private'] = 1;
|
||||
$postarray['allow_cid'] = '<' . $self['id'] . '>';
|
||||
} else {
|
||||
$postarray['private'] = 0;
|
||||
$postarray['allow_cid'] = '';
|
||||
}
|
||||
|
||||
if (is_string($post->full_text)) {
|
||||
|
@ -1420,22 +1426,22 @@ function twitter_createpost(App $a, $uid, $post, $self, $create_user, $only_exis
|
|||
|
||||
$statustext = $converted["plain"];
|
||||
|
||||
if (is_string($post->place->name)) {
|
||||
if (!empty($post->place->name)) {
|
||||
$postarray["location"] = $post->place->name;
|
||||
}
|
||||
if (is_string($post->place->full_name)) {
|
||||
if (!empty($post->place->full_name)) {
|
||||
$postarray["location"] = $post->place->full_name;
|
||||
}
|
||||
if (is_array($post->geo->coordinates)) {
|
||||
if (!empty($post->geo->coordinates)) {
|
||||
$postarray["coord"] = $post->geo->coordinates[0] . " " . $post->geo->coordinates[1];
|
||||
}
|
||||
if (is_array($post->coordinates->coordinates)) {
|
||||
if (!empty($post->coordinates->coordinates)) {
|
||||
$postarray["coord"] = $post->coordinates->coordinates[1] . " " . $post->coordinates->coordinates[0];
|
||||
}
|
||||
if (is_object($post->retweeted_status)) {
|
||||
if (!empty($post->retweeted_status)) {
|
||||
$retweet = twitter_createpost($a, $uid, $post->retweeted_status, $self, false, false, $noquote);
|
||||
|
||||
$retweet['object'] = $postarray['object'];
|
||||
//$retweet['object'] = $postarray['object'];
|
||||
$retweet['private'] = $postarray['private'];
|
||||
$retweet['allow_cid'] = $postarray['allow_cid'];
|
||||
$retweet['contact-id'] = $postarray['contact-id'];
|
||||
|
@ -1446,7 +1452,7 @@ function twitter_createpost(App $a, $uid, $post, $self, $create_user, $only_exis
|
|||
$postarray = $retweet;
|
||||
}
|
||||
|
||||
if (is_object($post->quoted_status) && !$noquote) {
|
||||
if (!empty($post->quoted_status) && !$noquote) {
|
||||
$quoted = twitter_createpost($a, $uid, $post->quoted_status, $self, false, false, true);
|
||||
|
||||
$postarray['body'] = $statustext;
|
||||
|
@ -1508,10 +1514,6 @@ function twitter_fetchparentposts(App $a, $uid, $post, TwitterOAuth $connection,
|
|||
|
||||
$item = Item::insert($postarray);
|
||||
|
||||
if ($notify) {
|
||||
$item = $notify;
|
||||
}
|
||||
|
||||
$postarray["id"] = $item;
|
||||
|
||||
logger('twitter_fetchparentpost: User ' . $self["nick"] . ' posted parent timeline item ' . $item);
|
||||
|
@ -1629,7 +1631,7 @@ function twitter_fetchhometimeline(App $a, $uid)
|
|||
|
||||
$postarray = twitter_createpost($a, $uid, $post, $self, $create_user, true, false);
|
||||
|
||||
if (trim($postarray['body']) == "") {
|
||||
if (empty($postarray['body']) || trim($postarray['body']) == "") {
|
||||
continue;
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in a new issue