Don't store non federated contacts in gcontact
[friendica-addons.git/.git] / twitter / twitter.php
index 9f07fa5..dd2241a 100644 (file)
  *     we do not need "Twitter as login". When you've registered the app you get the
  *     OAuth Consumer key and secret pair for your application/site.
  *
- *     Add this key pair to your global config/addon.ini.php or use the admin panel.
+ *     Add this key pair to your global config/addon.config.php or use the admin panel.
  *
- *     [twitter]
- *     consumerkey = your consumer_key here
- *     consumersecret = your consumer_secret here
+ *             'twitter' => [
+ *                 'consumerkey' => '',
+ *             'consumersecret' => '',
+ *      ],
  *
- *     To activate the addon itself add it to the [system] addon
+ *     To activate the addon itself add it to the system.addon
  *     setting. After this, your user can configure their Twitter account settings
  *     from "Settings -> Addon Settings".
  *
@@ -65,30 +66,28 @@ use Abraham\TwitterOAuth\TwitterOAuth;
 use Abraham\TwitterOAuth\TwitterOAuthException;
 use Friendica\App;
 use Friendica\Content\OEmbed;
+use Friendica\Content\Text\BBCode;
 use Friendica\Content\Text\Plaintext;
-use Friendica\Core\Addon;
 use Friendica\Core\Config;
+use Friendica\Core\Hook;
 use Friendica\Core\L10n;
+use Friendica\Core\Logger;
 use Friendica\Core\PConfig;
 use Friendica\Core\Protocol;
+use Friendica\Core\Renderer;
 use Friendica\Core\Worker;
 use Friendica\Database\DBA;
 use Friendica\Model\Contact;
 use Friendica\Model\Conversation;
-use Friendica\Model\GContact;
 use Friendica\Model\Group;
 use Friendica\Model\Item;
 use Friendica\Model\ItemContent;
-use Friendica\Model\Queue;
 use Friendica\Model\User;
 use Friendica\Object\Image;
+use Friendica\Util\Config\ConfigFileLoader;
 use Friendica\Util\DateTimeFormat;
 use Friendica\Util\Network;
-
-require_once 'boot.php';
-require_once 'include/dba.php';
-require_once 'include/enotify.php';
-require_once 'include/text.php';
+use Friendica\Util\Strings;
 
 require_once __DIR__ . DIRECTORY_SEPARATOR . 'vendor' . DIRECTORY_SEPARATOR . 'autoload.php';
 
@@ -97,45 +96,45 @@ define('TWITTER_DEFAULT_POLL_INTERVAL', 5); // given in minutes
 function twitter_install()
 {
        //  we need some hooks, for the configuration and for sending tweets
-       Addon::registerHook('load_config'            , __FILE__, 'twitter_load_config');
-       Addon::registerHook('connector_settings'     , __FILE__, 'twitter_settings');
-       Addon::registerHook('connector_settings_post', __FILE__, 'twitter_settings_post');
-       Addon::registerHook('post_local'             , __FILE__, 'twitter_post_local');
-       Addon::registerHook('notifier_normal'        , __FILE__, 'twitter_post_hook');
-       Addon::registerHook('jot_networks'           , __FILE__, 'twitter_jot_nets');
-       Addon::registerHook('cron'                   , __FILE__, 'twitter_cron');
-       Addon::registerHook('queue_predeliver'       , __FILE__, 'twitter_queue_hook');
-       Addon::registerHook('follow'                 , __FILE__, 'twitter_follow');
-       Addon::registerHook('expire'                 , __FILE__, 'twitter_expire');
-       Addon::registerHook('prepare_body'           , __FILE__, 'twitter_prepare_body');
-       Addon::registerHook('check_item_notification', __FILE__, 'twitter_check_item_notification');
-       logger("installed twitter");
+       Hook::register('load_config'            , __FILE__, 'twitter_load_config');
+       Hook::register('connector_settings'     , __FILE__, 'twitter_settings');
+       Hook::register('connector_settings_post', __FILE__, 'twitter_settings_post');
+       Hook::register('hook_fork'              , __FILE__, 'twitter_hook_fork');
+       Hook::register('post_local'             , __FILE__, 'twitter_post_local');
+       Hook::register('notifier_normal'        , __FILE__, 'twitter_post_hook');
+       Hook::register('jot_networks'           , __FILE__, 'twitter_jot_nets');
+       Hook::register('cron'                   , __FILE__, 'twitter_cron');
+       Hook::register('follow'                 , __FILE__, 'twitter_follow');
+       Hook::register('expire'                 , __FILE__, 'twitter_expire');
+       Hook::register('prepare_body'           , __FILE__, 'twitter_prepare_body');
+       Hook::register('check_item_notification', __FILE__, 'twitter_check_item_notification');
+       Logger::log("installed twitter");
 }
 
 function twitter_uninstall()
 {
-       Addon::unregisterHook('load_config'            , __FILE__, 'twitter_load_config');
-       Addon::unregisterHook('connector_settings'     , __FILE__, 'twitter_settings');
-       Addon::unregisterHook('connector_settings_post', __FILE__, 'twitter_settings_post');
-       Addon::unregisterHook('post_local'             , __FILE__, 'twitter_post_local');
-       Addon::unregisterHook('notifier_normal'        , __FILE__, 'twitter_post_hook');
-       Addon::unregisterHook('jot_networks'           , __FILE__, 'twitter_jot_nets');
-       Addon::unregisterHook('cron'                   , __FILE__, 'twitter_cron');
-       Addon::unregisterHook('queue_predeliver'       , __FILE__, 'twitter_queue_hook');
-       Addon::unregisterHook('follow'                 , __FILE__, 'twitter_follow');
-       Addon::unregisterHook('expire'                 , __FILE__, 'twitter_expire');
-       Addon::unregisterHook('prepare_body'           , __FILE__, 'twitter_prepare_body');
-       Addon::unregisterHook('check_item_notification', __FILE__, 'twitter_check_item_notification');
+       Hook::unregister('load_config'            , __FILE__, 'twitter_load_config');
+       Hook::unregister('connector_settings'     , __FILE__, 'twitter_settings');
+       Hook::unregister('connector_settings_post', __FILE__, 'twitter_settings_post');
+       Hook::unregister('hook_fork'              , __FILE__, 'twitter_hook_fork');
+       Hook::unregister('post_local'             , __FILE__, 'twitter_post_local');
+       Hook::unregister('notifier_normal'        , __FILE__, 'twitter_post_hook');
+       Hook::unregister('jot_networks'           , __FILE__, 'twitter_jot_nets');
+       Hook::unregister('cron'                   , __FILE__, 'twitter_cron');
+       Hook::unregister('follow'                 , __FILE__, 'twitter_follow');
+       Hook::unregister('expire'                 , __FILE__, 'twitter_expire');
+       Hook::unregister('prepare_body'           , __FILE__, 'twitter_prepare_body');
+       Hook::unregister('check_item_notification', __FILE__, 'twitter_check_item_notification');
 
        // old setting - remove only
-       Addon::unregisterHook('post_local_end'     , __FILE__, 'twitter_post_hook');
-       Addon::unregisterHook('addon_settings'     , __FILE__, 'twitter_settings');
-       Addon::unregisterHook('addon_settings_post', __FILE__, 'twitter_settings_post');
+       Hook::unregister('post_local_end'     , __FILE__, 'twitter_post_hook');
+       Hook::unregister('addon_settings'     , __FILE__, 'twitter_settings');
+       Hook::unregister('addon_settings_post', __FILE__, 'twitter_settings_post');
 }
 
-function twitter_load_config(App $a)
+function twitter_load_config(App $a, ConfigFileLoader $loader)
 {
-       $a->loadConfigFile(__DIR__ . '/config/twitter.ini.php');
+       $a->getConfigCache()->load($loader->loadAddonConfig('twitter'));
 }
 
 function twitter_check_item_notification(App $a, array &$notification_data)
@@ -154,7 +153,7 @@ function twitter_check_item_notification(App $a, array &$notification_data)
 
 function twitter_follow(App $a, array &$contact)
 {
-       logger("twitter_follow: Check if contact is twitter contact. " . $contact["url"], LOGGER_DEBUG);
+       Logger::log("twitter_follow: Check if contact is twitter contact. " . $contact["url"], Logger::DEBUG);
 
        if (!strstr($contact["url"], "://twitter.com") && !strstr($contact["url"], "@twitter.com")) {
                return;
@@ -191,21 +190,25 @@ function twitter_follow(App $a, array &$contact)
        }
 }
 
-function twitter_jot_nets(App $a, &$b)
+function twitter_jot_nets(App $a, array &$jotnets_fields)
 {
        if (!local_user()) {
                return;
        }
 
-       $tw_post = PConfig::get(local_user(), 'twitter', 'post');
-       if (intval($tw_post) == 1) {
-               $tw_defpost = PConfig::get(local_user(), 'twitter', 'post_by_default');
-               $selected = ((intval($tw_defpost) == 1) ? ' checked="checked" ' : '');
-               $b .= '<div class="profile-jot-net"><input type="checkbox" name="twitter_enable"' . $selected . ' value="1" /> '
-                       . L10n::t('Post to Twitter') . '</div>';
+       if (PConfig::get(local_user(), 'twitter', 'post')) {
+               $jotnets_fields[] = [
+                       'type' => 'checkbox',
+                       'field' => [
+                               'twitter_enable',
+                               L10n::t('Post to Twitter'),
+                               PConfig::get(local_user(), 'twitter', 'post_by_default')
+                       ]
+               ];
        }
 }
 
+
 function twitter_settings_post(App $a)
 {
        if (!local_user()) {
@@ -235,7 +238,7 @@ function twitter_settings_post(App $a)
        } else {
                if (isset($_POST['twitter-pin'])) {
                        //  if the user supplied us with a PIN from Twitter, let the magic of OAuth happen
-                       logger('got a Twitter PIN');
+                       Logger::log('got a Twitter PIN');
                        $ckey    = Config::get('twitter', 'consumerkey');
                        $csecret = Config::get('twitter', 'consumersecret');
                        //  the token and secret for which the PIN was generated were hidden in the settings
@@ -258,7 +261,7 @@ function twitter_settings_post(App $a)
                                info($e->getMessage());
                        }
                        //  reload the Addon Settings page, if we don't do it see Bug #42
-                       goaway('settings/connectors');
+                       $a->internalRedirect('settings/connectors');
                } else {
                        //  if no PIN is supplied in the POST variables, the user has changed the setting
                        //  to post a tweet for every new __public__ posting to the wall
@@ -346,7 +349,7 @@ function twitter_settings(App $a, &$s)
                        try {
                                $details = $connection->get('account/verify_credentials');
 
-                               $field_checkbox = get_markup_template('field_checkbox.tpl');
+                               $field_checkbox = Renderer::getMarkupTemplate('field_checkbox.tpl');
 
                                $s .= '<div id="twitter-info" >
                                        <p>' . L10n::t('Currently connected to: ') . '<a href="https://twitter.com/' . $details->screen_name . '" target="_twitter">' . $details->screen_name . '</a>
@@ -359,22 +362,22 @@ function twitter_settings(App $a, &$s)
                                </div>';
                                $s .= '<div class="clear"></div>';
 
-                               $s .= replace_macros($field_checkbox, [
+                               $s .= Renderer::replaceMacros($field_checkbox, [
                                        '$field' => ['twitter-enable', L10n::t('Allow posting to Twitter'), $enabled, L10n::t('If enabled all your <strong>public</strong> postings can be posted to the associated Twitter account. You can choose to do so by default (here) or for every posting separately in the posting options when writing the entry.')]
                                ]);
                                if ($a->user['hidewall']) {
                                        $s .= '<p>' . L10n::t('<strong>Note</strong>: Due to your privacy settings (<em>Hide your profile details from unknown viewers?</em>) the link potentially included in public postings relayed to Twitter will lead the visitor to a blank page informing the visitor that the access to your profile has been restricted.') . '</p>';
                                }
-                               $s .= replace_macros($field_checkbox, [
+                               $s .= Renderer::replaceMacros($field_checkbox, [
                                        '$field' => ['twitter-default', L10n::t('Send public postings to Twitter by default'), $defenabled, '']
                                ]);
-                               $s .= replace_macros($field_checkbox, [
+                               $s .= Renderer::replaceMacros($field_checkbox, [
                                        '$field' => ['twitter-mirror', L10n::t('Mirror all posts from twitter that are no replies'), $mirrorenabled, '']
                                ]);
-                               $s .= replace_macros($field_checkbox, [
+                               $s .= Renderer::replaceMacros($field_checkbox, [
                                        '$field' => ['twitter-import', L10n::t('Import the remote timeline'), $importenabled, '']
                                ]);
-                               $s .= replace_macros($field_checkbox, [
+                               $s .= Renderer::replaceMacros($field_checkbox, [
                                        '$field' => ['twitter-create_user', L10n::t('Automatically create contacts'), $create_userenabled, L10n::t('This will automatically create a contact in Friendica as soon as you receive a message from an existing contact via the Twitter network. If you do not enable this, you need to manually add those Twitter contacts in Friendica from whom you would like to see posts here. However if enabled, you cannot merely remove a twitter contact from the Friendica contact list, as it will recreate this contact when they post again.')]
                                ]);
                                $s .= '<div class="clear"></div>';
@@ -387,6 +390,47 @@ function twitter_settings(App $a, &$s)
        $s .= '</div><div class="clear"></div>';
 }
 
+function twitter_hook_fork(App $a, array &$b)
+{
+       if ($b['name'] != 'notifier_normal') {
+               return;
+       }
+
+       $post = $b['data'];
+
+       // Deleting and editing is not supported by the addon (deleting could, but isn't by now)
+       if ($post['deleted'] || ($post['created'] !== $post['edited'])) {
+               $b['execute'] = false;
+               return;
+       }
+
+       // if post comes from twitter don't send it back
+       if ($post['extid'] == Protocol::TWITTER) {
+               $b['execute'] = false;
+               return;
+       }
+
+       if ($post['app'] == 'Twitter') {
+               $b['execute'] = false;
+               return;
+       }
+
+       if (PConfig::get($post['uid'], 'twitter', 'import')) {
+               // Don't fork if it isn't a reply to a twitter post
+               if (($post['parent'] != $post['id']) && !Item::exists(['id' => $post['parent'], 'network' => Protocol::TWITTER])) {
+                       Logger::log('No twitter parent found for item ' . $post['id']);
+                       $b['execute'] = false;
+                       return;
+               }
+       } else {
+               // Comments are never exported when we don't import the twitter timeline
+               if (!strstr($post['postopts'], 'twitter') || ($post['parent'] != $post['id']) || $post['private']) {
+                       $b['execute'] = false;
+                       return;
+               }
+        }
+}
+
 function twitter_post_local(App $a, array &$b)
 {
        if ($b['edit']) {
@@ -398,7 +442,7 @@ function twitter_post_local(App $a, array &$b)
        }
 
        $twitter_post = intval(PConfig::get(local_user(), 'twitter', 'post'));
-       $twitter_enable = (($twitter_post && x($_REQUEST, 'twitter_enable')) ? intval($_REQUEST['twitter_enable']) : 0);
+       $twitter_enable = (($twitter_post && !empty($_REQUEST['twitter_enable'])) ? intval($_REQUEST['twitter_enable']) : 0);
 
        // if API is used, default to the chosen settings
        if ($b['api_source'] && intval(PConfig::get(local_user(), 'twitter', 'post_by_default'))) {
@@ -427,7 +471,7 @@ function twitter_action(App $a, $uid, $pid, $action)
 
        $post = ['id' => $pid];
 
-       logger("twitter_action '" . $action . "' ID: " . $pid . " data: " . print_r($post, true), LOGGER_DATA);
+       Logger::log("twitter_action '" . $action . "' ID: " . $pid . " data: " . print_r($post, true), Logger::DATA);
 
        switch ($action) {
                case "delete":
@@ -441,10 +485,10 @@ function twitter_action(App $a, $uid, $pid, $action)
                        $result = $connection->post('favorites/destroy', $post);
                        break;
                default:
-                       logger('Unhandled action ' . $action, LOGGER_DEBUG);
+                       Logger::log('Unhandled action ' . $action, Logger::DEBUG);
                        $result = [];
        }
-       logger("twitter_action '" . $action . "' send, result: " . print_r($result, true), LOGGER_DEBUG);
+       Logger::log("twitter_action '" . $action . "' send, result: " . print_r($result, true), Logger::DEBUG);
 }
 
 function twitter_post_hook(App $a, array &$b)
@@ -456,21 +500,21 @@ function twitter_post_hook(App $a, array &$b)
        }
 
        if ($b['parent'] != $b['id']) {
-               logger("twitter_post_hook: parameter " . print_r($b, true), LOGGER_DATA);
+               Logger::log("twitter_post_hook: parameter " . print_r($b, true), Logger::DATA);
 
                // Looking if its a reply to a twitter post
                if ((substr($b["parent-uri"], 0, 9) != "twitter::")
                        && (substr($b["extid"], 0, 9) != "twitter::")
                        && (substr($b["thr-parent"], 0, 9) != "twitter::"))
                {
-                       logger("twitter_post_hook: no twitter post " . $b["parent"]);
+                       Logger::log("twitter_post_hook: no twitter post " . $b["parent"]);
                        return;
                }
 
                $condition = ['uri' => $b["thr-parent"], 'uid' => $b["uid"]];
                $orig_post = Item::selectFirst([], $condition);
                if (!DBA::isResult($orig_post)) {
-                       logger("twitter_post_hook: no parent found " . $b["thr-parent"]);
+                       Logger::log("twitter_post_hook: no parent found " . $b["thr-parent"]);
                        return;
                } else {
                        $iscomment = true;
@@ -481,12 +525,12 @@ function twitter_post_hook(App $a, array &$b)
                $nickname = "@[url=" . $orig_post["author-link"] . "]" . $nicknameplain . "[/url]";
                $nicknameplain = "@" . $nicknameplain;
 
-               logger("twitter_post_hook: comparing " . $nickname . " and " . $nicknameplain . " with " . $b["body"], LOGGER_DEBUG);
+               Logger::log("twitter_post_hook: comparing " . $nickname . " and " . $nicknameplain . " with " . $b["body"], Logger::DEBUG);
                if ((strpos($b["body"], $nickname) === false) && (strpos($b["body"], $nicknameplain) === false)) {
                        $b["body"] = $nickname . " " . $b["body"];
                }
 
-               logger("twitter_post_hook: parent found " . print_r($orig_post, true), LOGGER_DATA);
+               Logger::log("twitter_post_hook: parent found " . print_r($orig_post, true), Logger::DATA);
        } else {
                $iscomment = false;
 
@@ -507,7 +551,7 @@ function twitter_post_hook(App $a, array &$b)
        }
 
        if ($b['verb'] == ACTIVITY_LIKE) {
-               logger("twitter_post_hook: parameter 2 " . substr($b["thr-parent"], 9), LOGGER_DEBUG);
+               Logger::log("twitter_post_hook: parameter 2 " . substr($b["thr-parent"], 9), Logger::DEBUG);
                if ($b['deleted']) {
                        twitter_action($a, $b["uid"], substr($b["thr-parent"], 9), "unlike");
                } else {
@@ -530,7 +574,7 @@ function twitter_post_hook(App $a, array &$b)
                return;
        }
 
-       logger('twitter post invoked');
+       Logger::notice('twitter post invoked', ['id' => $b['id'], 'guid' => $b['guid']]);
 
        PConfig::load($b['uid'], 'twitter');
 
@@ -540,7 +584,7 @@ function twitter_post_hook(App $a, array &$b)
        $osecret = PConfig::get($b['uid'], 'twitter', 'oauthsecret');
 
        if ($ckey && $csecret && $otoken && $osecret) {
-               logger('twitter: we have customer key and oauth stuff, going to send.', LOGGER_DEBUG);
+               Logger::log('twitter: we have customer key and oauth stuff, going to send.', Logger::DEBUG);
 
                // If it's a repeated message from twitter then do a native retweet and exit
                if (twitter_is_retweet($a, $b['uid'], $b['body'])) {
@@ -553,7 +597,19 @@ function twitter_post_hook(App $a, array &$b)
                $connection->setTimeouts(10, 30);
 
                $max_char = 280;
+
+               // Handling non-native reshares
+               $b['body'] = Friendica\Content\Text\BBCode::convertShare(
+                       $b['body'],
+                       function (array $attributes, array $author_contact, $content, $is_quote_share) {
+                               return twitter_convert_share($attributes, $author_contact, $content, $is_quote_share);
+                       }
+               );
+
+               $b['body'] = twitter_update_mentions($b['body']);
+
                $msgarr = ItemContent::getPlaintextPost($b, $max_char, true, 8);
+               Logger::info('Got plaintext', $msgarr);
                $msg = $msgarr["text"];
 
                if (($msg == "") && isset($msgarr["title"])) {
@@ -597,7 +653,7 @@ function twitter_post_hook(App $a, array &$b)
                                        throw new Exception('Failed upload of ' . $image);
                                }
                        } catch (Exception $e) {
-                               logger('Exception when trying to send to Twitter: ' . $e->getMessage());
+                               Logger::log('Exception when trying to send to Twitter: ' . $e->getMessage());
 
                                // Workaround: Remove the picture link so that the post can be reposted without it
                                // When there is another url already added, a second url would be superfluous.
@@ -617,26 +673,17 @@ function twitter_post_hook(App $a, array &$b)
 
                $url = 'statuses/update';
                $result = $connection->post($url, $post);
-               logger('twitter_post send, result: ' . print_r($result, true), LOGGER_DEBUG);
+               Logger::log('twitter_post send, result: ' . print_r($result, true), Logger::DEBUG);
 
                if (!empty($result->source)) {
                        Config::set("twitter", "application_name", strip_tags($result->source));
                }
 
                if (!empty($result->errors)) {
-                       logger('Send to Twitter failed: "' . print_r($result->errors, true) . '"');
-
-                       $r = q("SELECT `id` FROM `contact` WHERE `uid` = %d AND `self`", intval($b['uid']));
-                       if (DBA::isResult($r)) {
-                               $a->contact = $r[0]["id"];
-                       }
-
-                       $s = serialize(['url' => $url, 'item' => $b['id'], 'post' => $post]);
-
-                       Queue::add($a->contact, Protocol::TWITTER, $s);
-                       notice(L10n::t('Twitter post failed. Queued for retry.') . EOL);
+                       Logger::log('Send to Twitter failed: "' . print_r($result->errors, true) . '"');
+                       Worker::defer();
                } elseif ($iscomment) {
-                       logger('twitter_post: Update extid ' . $result->id_str . " for post id " . $b['id']);
+                       Logger::log('twitter_post: Update extid ' . $result->id_str . " for post id " . $b['id']);
                        Item::update(['extid' => "twitter::" . $result->id_str], ['id' => $b['id']]);
                }
        }
@@ -644,8 +691,8 @@ function twitter_post_hook(App $a, array &$b)
 
 function twitter_addon_admin_post(App $a)
 {
-       $consumerkey    = x($_POST, 'consumerkey')    ? notags(trim($_POST['consumerkey']))    : '';
-       $consumersecret = x($_POST, 'consumersecret') ? notags(trim($_POST['consumersecret'])) : '';
+       $consumerkey    = !empty($_POST['consumerkey'])    ? Strings::escapeTags(trim($_POST['consumerkey']))    : '';
+       $consumersecret = !empty($_POST['consumersecret']) ? Strings::escapeTags(trim($_POST['consumersecret'])) : '';
        Config::set('twitter', 'consumerkey', $consumerkey);
        Config::set('twitter', 'consumersecret', $consumersecret);
        info(L10n::t('Settings updated.') . EOL);
@@ -653,9 +700,9 @@ function twitter_addon_admin_post(App $a)
 
 function twitter_addon_admin(App $a, &$o)
 {
-       $t = get_markup_template("admin.tpl", "addon/twitter/");
+       $t = Renderer::getMarkupTemplate("admin.tpl", "addon/twitter/");
 
-       $o = replace_macros($t, [
+       $o = Renderer::replaceMacros($t, [
                '$submit' => L10n::t('Save Settings'),
                // name, label, value, help, [extra values]
                '$consumerkey' => ['consumerkey', L10n::t('Consumer key'), Config::get('twitter', 'consumerkey'), ''],
@@ -675,17 +722,17 @@ function twitter_cron(App $a)
        if ($last) {
                $next = $last + ($poll_interval * 60);
                if ($next > time()) {
-                       logger('twitter: poll intervall not reached');
+                       Logger::log('twitter: poll intervall not reached');
                        return;
                }
        }
-       logger('twitter: cron_start');
+       Logger::log('twitter: cron_start');
 
        $r = q("SELECT * FROM `pconfig` WHERE `cat` = 'twitter' AND `k` = 'mirror_posts' AND `v` = '1'");
        if (DBA::isResult($r)) {
                foreach ($r as $rr) {
-                       logger('twitter: fetching for user ' . $rr['uid']);
-                       Worker::add(PRIORITY_MEDIUM, "addon/twitter/twitter_sync.php", 1, (int) $rr['uid']);
+                       Logger::log('twitter: fetching for user ' . $rr['uid']);
+                       Worker::add(['priority' => PRIORITY_MEDIUM, 'force_priority' => true], "addon/twitter/twitter_sync.php", 1, (int) $rr['uid']);
                }
        }
 
@@ -702,13 +749,13 @@ function twitter_cron(App $a)
                        if ($abandon_days != 0) {
                                $user = q("SELECT `login_date` FROM `user` WHERE uid=%d AND `login_date` >= '%s'", $rr['uid'], $abandon_limit);
                                if (!DBA::isResult($user)) {
-                                       logger('abandoned account: timeline from user ' . $rr['uid'] . ' will not be imported');
+                                       Logger::log('abandoned account: timeline from user ' . $rr['uid'] . ' will not be imported');
                                        continue;
                                }
                        }
 
-                       logger('twitter: importing timeline from user ' . $rr['uid']);
-                       Worker::add(PRIORITY_MEDIUM, "addon/twitter/twitter_sync.php", 2, (int) $rr['uid']);
+                       Logger::log('twitter: importing timeline from user ' . $rr['uid']);
+                       Worker::add(['priority' => PRIORITY_MEDIUM, 'force_priority' => true], "addon/twitter/twitter_sync.php", 2, (int) $rr['uid']);
                        /*
                          // To-Do
                          // check for new contacts once a day
@@ -726,7 +773,7 @@ function twitter_cron(App $a)
                }
        }
 
-       logger('twitter: cron_end');
+       Logger::log('twitter: cron_end');
 
        Config::set('twitter', 'last_poll', time());
 }
@@ -745,19 +792,17 @@ function twitter_expire(App $a)
        }
        DBA::close($r);
 
-       require_once "include/items.php";
-
-       logger('twitter_expire: expire_start');
+       Logger::log('twitter_expire: expire_start');
 
        $r = q("SELECT * FROM `pconfig` WHERE `cat` = 'twitter' AND `k` = 'import' AND `v` = '1' ORDER BY RAND()");
        if (DBA::isResult($r)) {
                foreach ($r as $rr) {
-                       logger('twitter_expire: user ' . $rr['uid']);
+                       Logger::log('twitter_expire: user ' . $rr['uid']);
                        Item::expire($rr['uid'], $days, Protocol::TWITTER, true);
                }
        }
 
-       logger('twitter_expire: expire_end');
+       Logger::log('twitter_expire: expire_end');
 }
 
 function twitter_prepare_body(App $a, array &$b)
@@ -769,7 +814,7 @@ function twitter_prepare_body(App $a, array &$b)
        if ($b["preview"]) {
                $max_char = 280;
                $item = $b["item"];
-               $item["plink"] = $a->getBaseURL() . "/display/" . $a->user["nickname"] . "/" . $item["parent"];
+               $item["plink"] = $a->getBaseURL() . "/display/" . $item["guid"];
 
                $condition = ['uri' => $item["thr-parent"], 'uid' => local_user()];
                $orig_post = Item::selectFirst(['author-link'], $condition);
@@ -876,7 +921,6 @@ function twitter_fetchtimeline(App $a, $uid)
        $has_picture = false;
 
        require_once 'mod/item.php';
-       require_once 'include/items.php';
        require_once 'mod/share.php';
 
        $connection = new TwitterOAuth($ckey, $csecret, $otoken, $osecret);
@@ -892,18 +936,18 @@ function twitter_fetchtimeline(App $a, $uid)
        try {
                $items = $connection->get('statuses/user_timeline', $parameters);
        } catch (TwitterOAuthException $e) {
-               logger('Error fetching timeline for user ' . $uid . ': ' . $e->getMessage());
+               Logger::log('Error fetching timeline for user ' . $uid . ': ' . $e->getMessage());
                return;
        }
 
        if (!is_array($items)) {
-               logger('No items for user ' . $uid, LOGGER_INFO);
+               Logger::log('No items for user ' . $uid, Logger::INFO);
                return;
        }
 
        $posts = array_reverse($items);
 
-       logger('Starting from ID ' . $lastid . ' for user ' . $uid, LOGGER_DEBUG);
+       Logger::log('Starting from ID ' . $lastid . ' for user ' . $uid, Logger::DEBUG);
 
        if (count($posts)) {
                foreach ($posts as $post) {
@@ -920,7 +964,7 @@ function twitter_fetchtimeline(App $a, $uid)
                                $_SESSION["authenticated"] = true;
                                $_SESSION["uid"] = $uid;
 
-                               logger('Preparing Twitter ID ' . $post->id_str . ' for user ' . $uid, LOGGER_DEBUG);
+                               Logger::log('Preparing Twitter ID ' . $post->id_str . ' for user ' . $uid, Logger::DEBUG);
 
                                $_REQUEST = twitter_do_mirrorpost($a, $uid, $post);
 
@@ -928,74 +972,14 @@ function twitter_fetchtimeline(App $a, $uid)
                                        continue;
                                }
 
-                               logger('Posting Twitter ID ' . $post->id_str . ' for user ' . $uid);
+                               Logger::log('Posting Twitter ID ' . $post->id_str . ' for user ' . $uid);
 
                                item_post($a);
                        }
                }
        }
        PConfig::set($uid, 'twitter', 'lastid', $lastid);
-       logger('Last ID for user ' . $uid . ' is now ' . $lastid, LOGGER_DEBUG);
-}
-
-function twitter_queue_hook(App $a)
-{
-       $qi = q("SELECT * FROM `queue` WHERE `network` = '%s'",
-               DBA::escape(Protocol::TWITTER)
-       );
-       if (!DBA::isResult($qi)) {
-               return;
-       }
-
-       foreach ($qi as $x) {
-               if ($x['network'] !== Protocol::TWITTER) {
-                       continue;
-               }
-
-               logger('twitter_queue: run');
-
-               $r = q("SELECT `user`.* FROM `user` LEFT JOIN `contact` on `contact`.`uid` = `user`.`uid`
-                       WHERE `contact`.`self` = 1 AND `contact`.`id` = %d LIMIT 1",
-                       intval($x['cid'])
-               );
-               if (!DBA::isResult($r)) {
-                       continue;
-               }
-
-               $user = $r[0];
-
-               $ckey    = Config::get('twitter', 'consumerkey');
-               $csecret = Config::get('twitter', 'consumersecret');
-               $otoken  = PConfig::get($user['uid'], 'twitter', 'oauthtoken');
-               $osecret = PConfig::get($user['uid'], 'twitter', 'oauthsecret');
-
-               $success = false;
-
-               if ($ckey && $csecret && $otoken && $osecret) {
-                       logger('twitter_queue: able to post');
-
-                       $z = unserialize($x['content']);
-
-                       $connection = new TwitterOAuth($ckey, $csecret, $otoken, $osecret);
-                       $result = $connection->post($z['url'], $z['post']);
-
-                       logger('twitter_queue: post result: ' . print_r($result, true), LOGGER_DEBUG);
-
-                       if ($result->errors) {
-                               logger('twitter_queue: Send to Twitter failed: "' . print_r($result->errors, true) . '"');
-                       } else {
-                               $success = true;
-                               Queue::removeItem($x['id']);
-                       }
-               } else {
-                       logger("twitter_queue: Error getting tokens for user " . $user['uid']);
-               }
-
-               if (!$success) {
-                       logger('twitter_queue: delayed');
-                       Queue::updateTime($x['id']);
-               }
-       }
+       Logger::log('Last ID for user ' . $uid . ' is now ' . $lastid, Logger::DEBUG);
 }
 
 function twitter_fix_avatar($avatar)
@@ -1020,12 +1004,6 @@ function twitter_fetch_contact($uid, $data, $create_user)
        $url = "https://twitter.com/" . $data->screen_name;
        $addr = $data->screen_name . "@twitter.com";
 
-       GContact::update(["url" => $url, "network" => Protocol::TWITTER,
-               "photo" => $avatar, "hide" => true,
-               "name" => $data->name, "nick" => $data->screen_name,
-               "location" => $data->location, "about" => $data->description,
-               "addr" => $addr, "generation" => 2]);
-
        $fields = ['url' => $url, 'network' => Protocol::TWITTER,
                'name' => $data->name, 'nick' => $data->screen_name, 'addr' => $addr,
                 'location' => $data->location, 'about' => $data->description];
@@ -1045,7 +1023,7 @@ function twitter_fetch_contact($uid, $data, $create_user)
                // create contact record
                $fields['uid'] = $uid;
                $fields['created'] = DateTimeFormat::utcNow();
-               $fields['nurl'] = normalise_link($url);
+               $fields['nurl'] = Strings::normaliseLink($url);
                $fields['alias'] = 'twitter::' . $data->id_str;
                $fields['poll'] = 'twitter::' . $data->id_str;
                $fields['rel'] = Contact::FRIEND;
@@ -1066,7 +1044,7 @@ function twitter_fetch_contact($uid, $data, $create_user)
                Contact::updateAvatar($avatar, $uid, $contact_id);
        } else {
                if ($contact["readonly"] || $contact["blocked"]) {
-                       logger("twitter_fetch_contact: Contact '" . $contact["nick"] . "' is blocked or readonly.", LOGGER_DEBUG);
+                       Logger::log("twitter_fetch_contact: Contact '" . $contact["nick"] . "' is blocked or readonly.", Logger::DEBUG);
                        return -1;
                }
 
@@ -1077,7 +1055,7 @@ function twitter_fetch_contact($uid, $data, $create_user)
 
                // check that we have all the photos, this has been known to fail on occasion
                if (empty($contact['photo']) || empty($contact['thumb']) || empty($contact['micro']) || $update_photo) {
-                       logger("twitter_fetch_contact: Updating contact " . $data->screen_name, LOGGER_DEBUG);
+                       Logger::log("twitter_fetch_contact: Updating contact " . $data->screen_name, Logger::DEBUG);
 
                        Contact::updateAvatar($avatar, $uid, $contact['id']);
 
@@ -1122,7 +1100,7 @@ function twitter_fetchuser(App $a, $uid, $screen_name = "", $user_id = "")
        try {
                $user = $connection->get('users/show', $parameters);
        } catch (TwitterOAuthException $e) {
-               logger('twitter_fetchuser: Error fetching user ' . $uid . ': ' . $e->getMessage());
+               Logger::log('twitter_fetchuser: Error fetching user ' . $uid . ': ' . $e->getMessage());
                return;
        }
 
@@ -1142,7 +1120,7 @@ function twitter_expand_entities(App $a, $body, $item, $picture)
        $tags_arr = [];
 
        foreach ($item->entities->hashtags AS $hashtag) {
-               $url = '#[url=' . $a->getBaseURL() . '/search?tag=' . rawurlencode($hashtag->text) . ']' . $hashtag->text . '[/url]';
+               $url = '#[url=' . $a->getBaseURL() . '/search?tag=' . $hashtag->text . ']' . $hashtag->text . '[/url]';
                $tags_arr['#' . $hashtag->text] = $url;
                $body = str_replace('#' . $hashtag->text, $url, $body);
        }
@@ -1170,9 +1148,11 @@ function twitter_expand_entities(App $a, $body, $item, $picture)
                                        continue;
                                }
 
-                               $expanded_url = Network::finalUrl($url->expanded_url);
+                               $expanded_url = $url->expanded_url;
+
+                               $final_url = Network::finalUrl($url->expanded_url);
 
-                               $oembed_data = OEmbed::fetchURL($expanded_url);
+                               $oembed_data = OEmbed::fetchURL($final_url);
 
                                if (empty($oembed_data) || empty($oembed_data->type)) {
                                        continue;
@@ -1198,7 +1178,7 @@ function twitter_expand_entities(App $a, $body, $item, $picture)
                                } elseif ($oembed_data->type != 'link') {
                                        $body = str_replace($url->url, '[url=' . $expanded_url . ']' . $url->display_url . '[/url]', $body);
                                } else {
-                                       $img_str = Network::fetchUrl($expanded_url, true, $redirects, 4);
+                                       $img_str = Network::fetchUrl($final_url, true, 4);
 
                                        $tempfile = tempnam(get_temppath(), 'cache');
                                        file_put_contents($tempfile, $img_str);
@@ -1214,7 +1194,7 @@ function twitter_expand_entities(App $a, $body, $item, $picture)
 
                                        if (substr($mime, 0, 6) == 'image/') {
                                                $type = 'photo';
-                                               $body = str_replace($url->url, '[img]' . $expanded_url . '[/img]', $body);
+                                               $body = str_replace($url->url, '[img]' . $final_url . '[/img]', $body);
                                        } else {
                                                $type = $oembed_data->type;
                                                $footerurl = $expanded_url;
@@ -1251,7 +1231,7 @@ function twitter_expand_entities(App $a, $body, $item, $picture)
        }
 
        // it seems as if the entities aren't always covering all mentions. So the rest will be checked here
-       $tags = get_tags($body);
+       $tags = BBCode::getTags($body);
 
        if (count($tags)) {
                foreach ($tags as $tag) {
@@ -1273,7 +1253,7 @@ function twitter_expand_entities(App $a, $body, $item, $picture)
                                }
 
                                $basetag = str_replace('_', ' ', substr($tag, 1));
-                               $url = '#[url=' . $a->getBaseURL() . '/search?tag=' . rawurlencode($basetag) . ']' . $basetag . '[/url]';
+                               $url = '#[url=' . $a->getBaseURL() . '/search?tag=' . $basetag . ']' . $basetag . '[/url]';
                                $body = str_replace($tag, $url, $body);
                                $tags_arr['#' . $basetag] = $url;
                        } elseif (strpos($tag, '@') === 0) {
@@ -1376,7 +1356,7 @@ function twitter_createpost(App $a, $uid, $post, array $self, $create_user, $onl
 
        // Don't import our own comments
        if (Item::exists(['extid' => $postarray['uri'], 'uid' => $uid])) {
-               logger("Item with extid " . $postarray['uri'] . " found.", LOGGER_DEBUG);
+               Logger::log("Item with extid " . $postarray['uri'] . " found.", Logger::DEBUG);
                return [];
        }
 
@@ -1416,7 +1396,7 @@ function twitter_createpost(App $a, $uid, $post, array $self, $create_user, $onl
                                $postarray['owner-link']   = $r[0]["url"];
                                $postarray['owner-avatar'] = $r[0]["photo"];
                        } else {
-                               logger("No self contact for user " . $uid, LOGGER_DEBUG);
+                               Logger::log("No self contact for user " . $uid, Logger::DEBUG);
                                return [];
                        }
                }
@@ -1438,7 +1418,7 @@ function twitter_createpost(App $a, $uid, $post, array $self, $create_user, $onl
        if (($contactid == 0) && !$only_existing_contact) {
                $contactid = $self['id'];
        } elseif ($contactid <= 0) {
-               logger("Contact ID is zero or less than zero.", LOGGER_DEBUG);
+               Logger::log("Contact ID is zero or less than zero.", Logger::DEBUG);
                return [];
        }
 
@@ -1500,15 +1480,28 @@ function twitter_createpost(App $a, $uid, $post, array $self, $create_user, $onl
                        return [];
                }
 
-               $retweet['source'] = $postarray['source'];
-               $retweet['private'] = $postarray['private'];
-               $retweet['allow_cid'] = $postarray['allow_cid'];
-               $retweet['contact-id'] = $postarray['contact-id'];
-               $retweet['owner-name'] = $postarray['owner-name'];
-               $retweet['owner-link'] = $postarray['owner-link'];
-               $retweet['owner-avatar'] = $postarray['owner-avatar'];
+               if (!$noquote) {
+                       // Store the original tweet
+                       Item::insert($retweet);
 
-               $postarray = $retweet;
+                       // CHange the other post into a reshare activity
+                       $postarray['verb'] = ACTIVITY2_ANNOUNCE;
+                       $postarray['gravity'] = GRAVITY_ACTIVITY;
+                       $postarray['object-type'] = ACTIVITY_OBJ_NOTE;
+
+                       $postarray['thr-parent'] = $retweet['uri'];
+                       $postarray['parent-uri'] = $retweet['uri'];
+               } else {
+                       $retweet['source'] = $postarray['source'];
+                       $retweet['private'] = $postarray['private'];
+                       $retweet['allow_cid'] = $postarray['allow_cid'];
+                       $retweet['contact-id'] = $postarray['contact-id'];
+                       $retweet['owner-name'] = $postarray['owner-name'];
+                       $retweet['owner-link'] = $postarray['owner-link'];
+                       $retweet['owner-avatar'] = $postarray['owner-avatar'];
+
+                       $postarray = $retweet;
+               }
        }
 
        if (!empty($post->quoted_status) && !$noquote) {
@@ -1535,7 +1528,7 @@ function twitter_createpost(App $a, $uid, $post, array $self, $create_user, $onl
 
 function twitter_fetchparentposts(App $a, $uid, $post, TwitterOAuth $connection, array $self)
 {
-       logger("twitter_fetchparentposts: Fetching for user " . $uid . " and post " . $post->id_str, LOGGER_DEBUG);
+       Logger::log("twitter_fetchparentposts: Fetching for user " . $uid . " and post " . $post->id_str, Logger::DEBUG);
 
        $posts = [];
 
@@ -1545,17 +1538,17 @@ function twitter_fetchparentposts(App $a, $uid, $post, TwitterOAuth $connection,
                try {
                        $post = $connection->get('statuses/show', $parameters);
                } catch (TwitterOAuthException $e) {
-                       logger('twitter_fetchparentposts: Error fetching for user ' . $uid . ' and post ' . $post->id_str . ': ' . $e->getMessage());
+                       Logger::log('twitter_fetchparentposts: Error fetching for user ' . $uid . ' and post ' . $post->id_str . ': ' . $e->getMessage());
                        break;
                }
 
                if (empty($post)) {
-                       logger("twitter_fetchparentposts: Can't fetch post " . $parameters->id, LOGGER_DEBUG);
+                       Logger::log("twitter_fetchparentposts: Can't fetch post " . $parameters['id'], Logger::DEBUG);
                        break;
                }
 
                if (empty($post->id_str)) {
-                       logger("twitter_fetchparentposts: This is not a post " . json_encode($post), LOGGER_DEBUG);
+                       Logger::log("twitter_fetchparentposts: This is not a post " . json_encode($post), Logger::DEBUG);
                        break;
                }
 
@@ -1566,7 +1559,7 @@ function twitter_fetchparentposts(App $a, $uid, $post, TwitterOAuth $connection,
                $posts[] = $post;
        }
 
-       logger("twitter_fetchparentposts: Fetching " . count($posts) . " parents", LOGGER_DEBUG);
+       Logger::log("twitter_fetchparentposts: Fetching " . count($posts) . " parents", Logger::DEBUG);
 
        $posts = array_reverse($posts);
 
@@ -1582,7 +1575,7 @@ function twitter_fetchparentposts(App $a, $uid, $post, TwitterOAuth $connection,
 
                        $postarray["id"] = $item;
 
-                       logger('twitter_fetchparentpost: User ' . $self["nick"] . ' posted parent timeline item ' . $item);
+                       Logger::log('twitter_fetchparentpost: User ' . $self["nick"] . ' posted parent timeline item ' . $item);
                }
        }
 }
@@ -1596,7 +1589,7 @@ function twitter_fetchhometimeline(App $a, $uid)
        $create_user = PConfig::get($uid, 'twitter', 'create_user');
        $mirror_posts = PConfig::get($uid, 'twitter', 'mirror_posts');
 
-       logger("Fetching timeline for user " . $uid, LOGGER_DEBUG);
+       Logger::log("Fetching timeline for user " . $uid, Logger::DEBUG);
 
        $application_name = Config::get('twitter', 'application_name');
 
@@ -1604,14 +1597,12 @@ function twitter_fetchhometimeline(App $a, $uid)
                $application_name = $a->getHostName();
        }
 
-       require_once 'include/items.php';
-
        $connection = new TwitterOAuth($ckey, $csecret, $otoken, $osecret);
 
        try {
                $own_contact = twitter_fetch_own_contact($a, $uid);
        } catch (TwitterOAuthException $e) {
-               logger('Error fetching own contact for user ' . $uid . ': ' . $e->getMessage());
+               Logger::log('Error fetching own contact for user ' . $uid . ': ' . $e->getMessage());
                return;
        }
 
@@ -1622,13 +1613,13 @@ function twitter_fetchhometimeline(App $a, $uid)
        if (DBA::isResult($r)) {
                $own_id = $r[0]["nick"];
        } else {
-               logger("Own twitter contact not found for user " . $uid);
+               Logger::log("Own twitter contact not found for user " . $uid);
                return;
        }
 
        $self = User::getOwnerDataById($uid);
        if ($self === false) {
-               logger("Own contact not found for user " . $uid);
+               Logger::log("Own contact not found for user " . $uid);
                return;
        }
 
@@ -1646,23 +1637,23 @@ function twitter_fetchhometimeline(App $a, $uid)
        try {
                $items = $connection->get('statuses/home_timeline', $parameters);
        } catch (TwitterOAuthException $e) {
-               logger('Error fetching home timeline for user ' . $uid . ': ' . $e->getMessage());
+               Logger::log('Error fetching home timeline for user ' . $uid . ': ' . $e->getMessage());
                return;
        }
 
        if (!is_array($items)) {
-               logger('No array while fetching home timeline for user ' . $uid . ': ' . print_r($items, true));
+               Logger::log('No array while fetching home timeline for user ' . $uid . ': ' . print_r($items, true));
                return;
        }
 
        if (empty($items)) {
-               logger('No new timeline content for user ' . $uid, LOGGER_INFO);
+               Logger::log('No new timeline content for user ' . $uid, Logger::INFO);
                return;
        }
 
        $posts = array_reverse($items);
 
-       logger('Fetching timeline from ID ' . $lastid . ' for user ' . $uid . ' ' . sizeof($posts) . ' items', LOGGER_DEBUG);
+       Logger::log('Fetching timeline from ID ' . $lastid . ' for user ' . $uid . ' ' . sizeof($posts) . ' items', Logger::DEBUG);
 
        if (count($posts)) {
                foreach ($posts as $post) {
@@ -1676,12 +1667,12 @@ function twitter_fetchhometimeline(App $a, $uid)
                        }
 
                        if (stristr($post->source, $application_name) && $post->user->screen_name == $own_id) {
-                               logger("Skip previously sent post", LOGGER_DEBUG);
+                               Logger::log("Skip previously sent post", Logger::DEBUG);
                                continue;
                        }
 
                        if ($mirror_posts && $post->user->screen_name == $own_id && $post->in_reply_to_status_id_str == "") {
-                               logger("Skip post that will be mirrored", LOGGER_DEBUG);
+                               Logger::log("Skip post that will be mirrored", Logger::DEBUG);
                                continue;
                        }
 
@@ -1689,12 +1680,12 @@ function twitter_fetchhometimeline(App $a, $uid)
                                twitter_fetchparentposts($a, $uid, $post, $connection, $self);
                        }
 
-                       logger('Preparing post ' . $post->id_str . ' for user ' . $uid, LOGGER_DEBUG);
+                       Logger::log('Preparing post ' . $post->id_str . ' for user ' . $uid, Logger::DEBUG);
 
                        $postarray = twitter_createpost($a, $uid, $post, $self, $create_user, true, false);
 
                        if (empty($postarray['body']) || trim($postarray['body']) == "") {
-                               logger('Empty body for post ' . $post->id_str . ' and user ' . $uid, LOGGER_DEBUG);
+                               Logger::log('Empty body for post ' . $post->id_str . ' and user ' . $uid, Logger::DEBUG);
                                continue;
                        }
 
@@ -1710,12 +1701,12 @@ function twitter_fetchhometimeline(App $a, $uid)
                        $item = Item::insert($postarray, false, $notify);
                        $postarray["id"] = $item;
 
-                       logger('User ' . $uid . ' posted home timeline item ' . $item);
+                       Logger::log('User ' . $uid . ' posted home timeline item ' . $item);
                }
        }
        PConfig::set($uid, 'twitter', 'lasthometimelineid', $lastid);
 
-       logger('Last timeline ID for user ' . $uid . ' is now ' . $lastid, LOGGER_DEBUG);
+       Logger::log('Last timeline ID for user ' . $uid . ' is now ' . $lastid, Logger::DEBUG);
 
        // Fetching mentions
        $lastid = PConfig::get($uid, 'twitter', 'lastmentionid');
@@ -1729,18 +1720,18 @@ function twitter_fetchhometimeline(App $a, $uid)
        try {
                $items = $connection->get('statuses/mentions_timeline', $parameters);
        } catch (TwitterOAuthException $e) {
-               logger('Error fetching mentions: ' . $e->getMessage());
+               Logger::log('Error fetching mentions: ' . $e->getMessage());
                return;
        }
 
        if (!is_array($items)) {
-               logger("Error fetching mentions: " . print_r($items, true), LOGGER_DEBUG);
+               Logger::log("Error fetching mentions: " . print_r($items, true), Logger::DEBUG);
                return;
        }
 
        $posts = array_reverse($items);
 
-       logger("Fetching mentions for user " . $uid . " " . sizeof($posts) . " items", LOGGER_DEBUG);
+       Logger::log("Fetching mentions for user " . $uid . " " . sizeof($posts) . " items", Logger::DEBUG);
 
        if (count($posts)) {
                foreach ($posts as $post) {
@@ -1764,13 +1755,13 @@ function twitter_fetchhometimeline(App $a, $uid)
 
                        $item = Item::insert($postarray);
 
-                       logger('User ' . $uid . ' posted mention timeline item ' . $item);
+                       Logger::log('User ' . $uid . ' posted mention timeline item ' . $item);
                }
        }
 
        PConfig::set($uid, 'twitter', 'lastmentionid', $lastid);
 
-       logger('Last mentions ID for user ' . $uid . ' is now ' . $lastid, LOGGER_DEBUG);
+       Logger::log('Last mentions ID for user ' . $uid . ' is now ' . $lastid, Logger::DEBUG);
 }
 
 function twitter_fetch_own_contact(App $a, $uid)
@@ -1790,6 +1781,9 @@ function twitter_fetch_own_contact(App $a, $uid)
                // Fetching user data
                // get() may throw TwitterOAuthException, but we will catch it later
                $user = $connection->get('account/verify_credentials');
+               if (empty($user) || empty($user->id_str)) {
+                       return false;
+               }
 
                PConfig::set($uid, 'twitter', 'own_id', $user->id_str);
 
@@ -1845,7 +1839,7 @@ function twitter_is_retweet(App $a, $uid, $body)
                return false;
        }
 
-       logger('twitter_is_retweet: Retweeting id ' . $id . ' for user ' . $uid, LOGGER_DEBUG);
+       Logger::log('twitter_is_retweet: Retweeting id ' . $id . ' for user ' . $uid, Logger::DEBUG);
 
        $ckey    = Config::get('twitter', 'consumerkey');
        $csecret = Config::get('twitter', 'consumersecret');
@@ -1855,7 +1849,38 @@ function twitter_is_retweet(App $a, $uid, $body)
        $connection = new TwitterOAuth($ckey, $csecret, $otoken, $osecret);
        $result = $connection->post('statuses/retweet/' . $id);
 
-       logger('twitter_is_retweet: result ' . print_r($result, true), LOGGER_DEBUG);
+       Logger::log('twitter_is_retweet: result ' . print_r($result, true), Logger::DEBUG);
 
        return !isset($result->errors);
 }
+
+function twitter_update_mentions($body)
+{
+       $URLSearchString = "^\[\]";
+       $return = preg_replace_callback(
+               "/@\[url\=([$URLSearchString]*)\](.*?)\[\/url\]/ism",
+               function ($matches) {
+                       if (strpos($matches[1], 'twitter.com')) {
+                               $return = '@' . substr($matches[1], strrpos($matches[1], '/') + 1);
+                       } else {
+                               $return = $matches[2] . ' (' . $matches[1] . ')';
+                       }
+
+                       return $return;
+               },
+               $body
+       );
+
+       return $return;
+}
+
+function twitter_convert_share(array $attributes, array $author_contact, $content, $is_quote_share)
+{
+       if ($author_contact['network'] == Protocol::TWITTER) {
+               $mention = '@' . $author_contact['nickname'];
+       } else {
+               $mention = $author_contact['addr'];
+       }
+
+       return ($is_quote_share ? "\n\n" : '' ) . 'RT ' . $mention . ': ' . $content . "\n\n" . $attributes['link'];
+}