Merge pull request #564 from annando/twitter-exception

Handle exception when posting pictures to twitter
This commit is contained in:
Tobias Diekershoff 2018-03-31 14:37:19 +02:00 committed by GitHub
commit 0dfe67ca3e
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -548,32 +548,39 @@ function twitter_post_hook(App $a, &$b)
// and now tweet it :-) // and now tweet it :-)
if (strlen($msg) && ($image != "")) { if (strlen($msg) && ($image != "")) {
$connection = new TwitterOAuth($ckey, $csecret, $otoken, $osecret); try {
$media = $connection->upload('media/upload', ['media' => $image]); $media = $connection->upload('media/upload', ['media' => $image]);
$post = ['status' => $msg, 'media_ids' => $media->media_id_string]; $post = ['status' => $msg, 'media_ids' => $media->media_id_string];
if ($iscomment) { if ($iscomment) {
$post["in_reply_to_status_id"] = substr($orig_post["uri"], 9); $post["in_reply_to_status_id"] = substr($orig_post["uri"], 9);
} }
$result = $connection->post('statuses/update', $post); $result = $connection->post('statuses/update', $post);
logger('twitter_post_with_media send, result: ' . print_r($result, true), LOGGER_DEBUG); logger('twitter_post_with_media send, result: ' . print_r($result, true), LOGGER_DEBUG);
if ($result->source) { if ($result->source) {
Config::set("twitter", "application_name", strip_tags($result->source)); Config::set("twitter", "application_name", strip_tags($result->source));
} }
if ($result->errors || $result->error) { if ($result->errors || $result->error) {
logger('Send to Twitter failed: "' . print_r($result->errors, true) . '"'); logger('Send to Twitter failed: "' . print_r($result->errors, true) . '"');
// Workaround: Remove the picture link so that the post can be reposted without it
$msg .= " " . $image;
$image = "";
} elseif ($iscomment) {
logger('twitter_post: Update extid ' . $result->id_str . " for post id " . $b['id']);
Item::update(['extid' => "twitter::" . $result->id_str, 'body' => $result->text], ['id' => $b['id']]);
}
} catch (Exception $e) {
logger('Exception when trying to send to Twitter: ' . $e->getMessage());
// Workaround: Remove the picture link so that the post can be reposted without it // Workaround: Remove the picture link so that the post can be reposted without it
$msg .= " " . $image; $msg .= " " . $image;
$image = ""; $image = "";
} elseif ($iscomment) {
logger('twitter_post: Update extid ' . $result->id_str . " for post id " . $b['id']);
Item::update(['extid' => "twitter::" . $result->id_str, 'body' => $result->text], ['id' => $b['id']]);
} }
} }