]> git.mxchange.org Git - friendica.git/blobdiff - include/api.php
Ensure APContact::getByUrl always returns an array
[friendica.git] / include / api.php
index 2c9ae9b387db54b3429d0f0ffe3c2f7b550f4bef..681515cc376cc0a9bfeb11bad6ad05521b972cb4 100644 (file)
@@ -7,6 +7,7 @@
  */
 
 use Friendica\App;
+use Friendica\BaseObject;
 use Friendica\Content\ContactSelector;
 use Friendica\Content\Feature;
 use Friendica\Content\Text\BBCode;
@@ -15,7 +16,6 @@ use Friendica\Core\Config;
 use Friendica\Core\Hook;
 use Friendica\Core\L10n;
 use Friendica\Core\Logger;
-use Friendica\Core\NotificationsManager;
 use Friendica\Core\PConfig;
 use Friendica\Core\Protocol;
 use Friendica\Core\Session;
@@ -26,6 +26,7 @@ use Friendica\Model\Contact;
 use Friendica\Model\Group;
 use Friendica\Model\Item;
 use Friendica\Model\Mail;
+use Friendica\Model\Notify;
 use Friendica\Model\Photo;
 use Friendica\Model\Profile;
 use Friendica\Model\User;
@@ -41,16 +42,18 @@ use Friendica\Network\HTTPException\NotImplementedException;
 use Friendica\Network\HTTPException\TooManyRequestsException;
 use Friendica\Network\HTTPException\UnauthorizedException;
 use Friendica\Object\Image;
+use Friendica\Protocol\Activity;
 use Friendica\Protocol\Diaspora;
 use Friendica\Util\DateTimeFormat;
+use Friendica\Util\Images;
 use Friendica\Util\Network;
 use Friendica\Util\Proxy as ProxyUtils;
 use Friendica\Util\Strings;
 use Friendica\Util\XML;
 
-require_once 'mod/share.php';
-require_once 'mod/item.php';
-require_once 'mod/wall_upload.php';
+require_once __DIR__ . '/../mod/share.php';
+require_once __DIR__ . '/../mod/item.php';
+require_once __DIR__ . '/../mod/wall_upload.php';
 
 define('API_METHOD_ANY', '*');
 define('API_METHOD_GET', 'GET');
@@ -162,6 +165,7 @@ function api_register_func($path, $func, $auth = false, $method = API_METHOD_ANY
  * @brief Login API user
  *
  * @param App $a App
+ * @throws ForbiddenException
  * @throws InternalServerErrorException
  * @throws UnauthorizedException
  * @hook  'authenticate'
@@ -170,8 +174,6 @@ function api_register_func($path, $func, $auth = false, $method = API_METHOD_ANY
  *               'password' => password from login form
  *               'authenticated' => return status,
  *               'user_record' => return authenticated user record
- * @hook  'logged_in'
- *               array $user    logged user record
  */
 function api_login(App $a)
 {
@@ -182,7 +184,7 @@ function api_login(App $a)
                list($consumer, $token) = $oauth1->verify_request($request);
                if (!is_null($token)) {
                        $oauth1->loginUser($token->uid);
-                       Hook::callAll('logged_in', $a->user);
+                       Session::set('allow_api', true);
                        return;
                }
                echo __FILE__.__LINE__.__FUNCTION__ . "<pre>";
@@ -208,8 +210,8 @@ function api_login(App $a)
                throw new UnauthorizedException("This API requires login");
        }
 
-       $user = defaults($_SERVER, 'PHP_AUTH_USER', '');
-       $password = defaults($_SERVER, 'PHP_AUTH_PW', '');
+       $user = $_SERVER['PHP_AUTH_USER'] ?? '';
+       $password = $_SERVER['PHP_AUTH_PW'] ?? '';
 
        // allow "user@server" login (but ignore 'server' part)
        $at = strstr($user, "@", true);
@@ -273,7 +275,7 @@ function api_check_method($method)
        if ($method == "*") {
                return true;
        }
-       return (stripos($method, defaults($_SERVER, 'REQUEST_METHOD', 'GET')) !== false);
+       return (stripos($method, $_SERVER['REQUEST_METHOD'] ?? 'GET') !== false);
 }
 
 /**
@@ -323,7 +325,7 @@ function api_call(App $a)
 
                                $stamp =  microtime(true);
                                $return = call_user_func($info['func'], $type);
-                               $duration = (float) (microtime(true) - $stamp);
+                               $duration = floatval(microtime(true) - $stamp);
 
                                Logger::info(API_LOG_PREFIX . 'username {username}', ['module' => 'api', 'action' => 'call', 'username' => $a->user['username'], 'duration' => round($duration, 2)]);
 
@@ -614,7 +616,7 @@ function api_get_user(App $a, $contact_id = null)
                                'name' => $contact["name"],
                                'screen_name' => (($contact['nick']) ? $contact['nick'] : $contact['name']),
                                'location' => ($contact["location"] != "") ? $contact["location"] : ContactSelector::networkToName($contact['network'], $contact['url']),
-                               'description' => HTML::toPlaintext(BBCode::toPlaintext($contact["about"])),
+                               'description' => BBCode::toPlaintext($contact["about"]),
                                'profile_image_url' => $contact["micro"],
                                'profile_image_url_https' => $contact["micro"],
                                'profile_image_url_profile_size' => $contact["thumb"],
@@ -693,7 +695,7 @@ function api_get_user(App $a, $contact_id = null)
                'name' => (($uinfo[0]['name']) ? $uinfo[0]['name'] : $uinfo[0]['nick']),
                'screen_name' => (($uinfo[0]['nick']) ? $uinfo[0]['nick'] : $uinfo[0]['name']),
                'location' => $location,
-               'description' => HTML::toPlaintext(BBCode::toPlaintext($description)),
+               'description' => BBCode::toPlaintext($description),
                'profile_image_url' => $uinfo[0]['micro'],
                'profile_image_url_https' => $uinfo[0]['micro'],
                'profile_image_url_profile_size' => $uinfo[0]["thumb"],
@@ -776,14 +778,14 @@ function api_get_user(App $a, $contact_id = null)
  */
 function api_item_get_user(App $a, $item)
 {
-       $status_user = api_get_user($a, defaults($item, 'author-id', null));
+       $status_user = api_get_user($a, $item['author-id'] ?? null);
 
        $author_user = $status_user;
 
-       $status_user["protected"] = defaults($item, 'private', 0);
+       $status_user["protected"] = $item['private'] ?? 0;
 
-       if (defaults($item, 'thr-parent', '') == defaults($item, 'uri', '')) {
-               $owner_user = api_get_user($a, defaults($item, 'owner-id', null));
+       if (($item['thr-parent'] ?? '') == ($item['uri'] ?? '')) {
+               $owner_user = api_get_user($a, $item['owner-id'] ?? null);
        } else {
                $owner_user = $author_user;
        }
@@ -947,7 +949,7 @@ function api_account_verify_credentials($type)
        unset($_REQUEST["screen_name"]);
        unset($_GET["screen_name"]);
 
-       $skip_status = defaults($_REQUEST, 'skip_status', false);
+       $skip_status = $_REQUEST['skip_status'] ?? false;
 
        $user_info = api_get_user($a);
 
@@ -957,7 +959,7 @@ function api_account_verify_credentials($type)
        // - Adding last status
        if (!$skip_status) {
                $item = api_get_last_status($user_info['pid'], $user_info['uid']);
-               if ($item) {
+               if (!empty($item)) {
                        $user_info['status'] = api_format_item($item, $type);
                }
        }
@@ -1157,19 +1159,21 @@ function api_statuses_update($type)
                }
        }
 
-       // To-Do: Multiple IDs
        if (requestdata('media_ids')) {
-               $r = q(
-                       "SELECT `resource-id`, `scale`, `nickname`, `type`, `desc` FROM `photo` INNER JOIN `user` ON `user`.`uid` = `photo`.`uid` WHERE `resource-id` IN (SELECT `resource-id` FROM `photo` WHERE `id` = %d) AND `scale` > 0 AND `photo`.`uid` = %d ORDER BY `photo`.`width` DESC LIMIT 1",
-                       intval(requestdata('media_ids')),
-                       api_user()
-               );
-               if (DBA::isResult($r)) {
-                       $phototypes = Image::supportedTypes();
-                       $ext = $phototypes[$r[0]['type']];
-                       $description = $r[0]['desc'] ?? '';
-                       $_REQUEST['body'] .= "\n\n" . '[url=' . System::baseUrl() . '/photos/' . $r[0]['nickname'] . '/image/' . $r[0]['resource-id'] . ']';
-                       $_REQUEST['body'] .= '[img=' . System::baseUrl() . '/photo/' . $r[0]['resource-id'] . '-' . $r[0]['scale'] . '.' . $ext . ']' . $description . '[/img][/url]';
+               $ids = explode(',', requestdata('media_ids'));
+               foreach ($ids as $id) {
+                       $r = q(
+                               "SELECT `resource-id`, `scale`, `nickname`, `type`, `desc` FROM `photo` INNER JOIN `user` ON `user`.`uid` = `photo`.`uid` WHERE `resource-id` IN (SELECT `resource-id` FROM `photo` WHERE `id` = %d) AND `scale` > 0 AND `photo`.`uid` = %d ORDER BY `photo`.`width` DESC LIMIT 1",
+                               intval($id),
+                               api_user()
+                       );
+                       if (DBA::isResult($r)) {
+                               $phototypes = Images::supportedTypes();
+                               $ext = $phototypes[$r[0]['type']];
+                               $description = $r[0]['desc'] ?? '';
+                               $_REQUEST['body'] .= "\n\n" . '[url=' . System::baseUrl() . '/photos/' . $r[0]['nickname'] . '/image/' . $r[0]['resource-id'] . ']';
+                               $_REQUEST['body'] .= '[img=' . System::baseUrl() . '/photo/' . $r[0]['resource-id'] . '-' . $r[0]['scale'] . '.' . $ext . ']' . $description . '[/img][/url]';
+                       }
                }
        }
 
@@ -1314,7 +1318,7 @@ function api_status_show($type, $item_id)
        $status_info = [];
 
        $item = api_get_item(['id' => $item_id]);
-       if ($item) {
+       if (!empty($item)) {
                $status_info = api_format_item($item, $type);
        }
 
@@ -1373,12 +1377,12 @@ function api_get_item(array $condition)
  */
 function api_users_show($type)
 {
-       $a = \Friendica\BaseObject::getApp();
+       $a = BaseObject::getApp();
 
        $user_info = api_get_user($a);
 
        $item = api_get_last_status($user_info['pid'], $user_info['uid']);
-       if ($item) {
+       if (!empty($item)) {
                $user_info['status'] = api_format_item($item, $type);
        }
 
@@ -1516,10 +1520,12 @@ function api_search($type)
                $count = $_REQUEST['count'];
        }
        
-       $since_id = defaults($_REQUEST, 'since_id', 0);
-       $max_id = defaults($_REQUEST, 'max_id', 0);
-       $page = (!empty($_REQUEST['page']) ? $_REQUEST['page'] - 1 : 0);
-       $start = $page * $count;
+       $since_id = $_REQUEST['since_id'] ?? 0;
+       $max_id = $_REQUEST['max_id'] ?? 0;
+       $page = $_REQUEST['page'] ?? 1;
+
+       $start = max(0, ($page - 1) * $count);
+
        $params = ['order' => ['id' => true], 'limit' => [$start, $count]];
        if (preg_match('/^#(\w+)$/', $searchTerm, $matches) === 1 && isset($matches[1])) {
                $searchTerm = $matches[1];
@@ -1607,17 +1613,14 @@ function api_statuses_home_timeline($type)
        // get last network messages
 
        // params
-       $count = defaults($_REQUEST, 'count', 20);
-       $page = (!empty($_REQUEST['page']) ? $_REQUEST['page'] - 1 : 0);
-       if ($page < 0) {
-               $page = 0;
-       }
-       $since_id = defaults($_REQUEST, 'since_id', 0);
-       $max_id = defaults($_REQUEST, 'max_id', 0);
+       $count = $_REQUEST['count'] ?? 20;
+       $page = $_REQUEST['page']?? 0;
+       $since_id = $_REQUEST['since_id'] ?? 0;
+       $max_id = $_REQUEST['max_id'] ?? 0;
        $exclude_replies = !empty($_REQUEST['exclude_replies']);
-       $conversation_id = defaults($_REQUEST, 'conversation_id', 0);
+       $conversation_id = $_REQUEST['conversation_id'] ?? 0;
 
-       $start = $page * $count;
+       $start = max(0, ($page - 1) * $count);
 
        $condition = ["`uid` = ? AND `gravity` IN (?, ?) AND `item`.`id` > ?",
                api_user(), GRAVITY_PARENT, GRAVITY_COMMENT, $since_id];
@@ -1697,17 +1700,14 @@ function api_statuses_public_timeline($type)
        // get last network messages
 
        // params
-       $count = defaults($_REQUEST, 'count', 20);
-       $page = (!empty($_REQUEST['page']) ? $_REQUEST['page'] -1 : 0);
-       if ($page < 0) {
-               $page = 0;
-       }
-       $since_id = defaults($_REQUEST, 'since_id', 0);
-       $max_id = defaults($_REQUEST, 'max_id', 0);
+       $count = $_REQUEST['count'] ?? 20;
+       $page = $_REQUEST['page'] ?? 1;
+       $since_id = $_REQUEST['since_id'] ?? 0;
+       $max_id = $_REQUEST['max_id'] ?? 0;
        $exclude_replies = (!empty($_REQUEST['exclude_replies']) ? 1 : 0);
-       $conversation_id = defaults($_REQUEST, 'conversation_id', 0);
+       $conversation_id = $_REQUEST['conversation_id'] ?? 0;
 
-       $start = $page * $count;
+       $start = max(0, ($page - 1) * $count);
 
        if ($exclude_replies && !$conversation_id) {
                $condition = ["`gravity` IN (?, ?) AND `iid` > ? AND NOT `private` AND `wall` AND NOT `user`.`hidewall` AND NOT `author`.`hidden`",
@@ -1782,16 +1782,14 @@ function api_statuses_networkpublic_timeline($type)
                throw new ForbiddenException();
        }
 
-       $since_id        = defaults($_REQUEST, 'since_id', 0);
-       $max_id          = defaults($_REQUEST, 'max_id', 0);
+       $since_id        = $_REQUEST['since_id'] ?? 0;
+       $max_id          = $_REQUEST['max_id'] ?? 0;
 
        // pagination
-       $count = defaults($_REQUEST, 'count', 20);
-       $page  = defaults($_REQUEST, 'page', 1);
-       if ($page < 1) {
-               $page = 1;
-       }
-       $start = ($page - 1) * $count;
+       $count = $_REQUEST['count'] ?? 20;
+       $page  = $_REQUEST['page'] ?? 1;
+
+       $start = max(0, ($page - 1) * $count);
 
        $condition = ["`uid` = 0 AND `gravity` IN (?, ?) AND `thread`.`iid` > ? AND NOT `private`",
                GRAVITY_PARENT, GRAVITY_COMMENT, $since_id];
@@ -1846,15 +1844,15 @@ function api_statuses_show($type)
        }
 
        // params
-       $id = intval(defaults($a->argv, 3, 0));
+       $id = intval($a->argv[3] ?? 0);
 
        if ($id == 0) {
-               $id = intval(defaults($_REQUEST, 'id', 0));
+               $id = intval($_REQUEST['id'] ?? 0);
        }
 
        // Hotot workaround
        if ($id == 0) {
-               $id = intval(defaults($a->argv, 4, 0));
+               $id = intval($a->argv[4] ?? 0);
        }
 
        Logger::log('API: api_statuses_show: ' . $id);
@@ -1925,24 +1923,21 @@ function api_conversation_show($type)
        }
 
        // params
-       $id       = intval(defaults($a->argv , 3         , 0));
-       $since_id = intval(defaults($_REQUEST, 'since_id', 0));
-       $max_id   = intval(defaults($_REQUEST, 'max_id'  , 0));
-       $count    = intval(defaults($_REQUEST, 'count'   , 20));
-       $page     = intval(defaults($_REQUEST, 'page'    , 1)) - 1;
-       if ($page < 0) {
-               $page = 0;
-       }
+       $id       = intval($a->argv[3]           ?? 0);
+       $since_id = intval($_REQUEST['since_id'] ?? 0);
+       $max_id   = intval($_REQUEST['max_id']   ?? 0);
+       $count    = intval($_REQUEST['count']    ?? 20);
+       $page     = intval($_REQUEST['page']     ?? 1);
 
-       $start = $page * $count;
+       $start = max(0, ($page - 1) * $count);
 
        if ($id == 0) {
-               $id = intval(defaults($_REQUEST, 'id', 0));
+               $id = intval($_REQUEST['id'] ?? 0);
        }
 
        // Hotot workaround
        if ($id == 0) {
-               $id = intval(defaults($a->argv, 4, 0));
+               $id = intval($a->argv[4] ?? 0);
        }
 
        Logger::info(API_LOG_PREFIX . '{subaction}', ['module' => 'api', 'action' => 'conversation', 'subaction' => 'show', 'id' => $id]);
@@ -2011,20 +2006,20 @@ function api_statuses_repeat($type)
        api_get_user($a);
 
        // params
-       $id = intval(defaults($a->argv, 3, 0));
+       $id = intval($a->argv[3] ?? 0);
 
        if ($id == 0) {
-               $id = intval(defaults($_REQUEST, 'id', 0));
+               $id = intval($_REQUEST['id'] ?? 0);
        }
 
        // Hotot workaround
        if ($id == 0) {
-               $id = intval(defaults($a->argv, 4, 0));
+               $id = intval($a->argv[4] ?? 0);
        }
 
        Logger::log('API: api_statuses_repeat: '.$id);
 
-       $fields = ['body', 'author-name', 'author-link', 'author-avatar', 'guid', 'created', 'plink'];
+       $fields = ['body', 'title', 'attach', 'tag', 'author-name', 'author-link', 'author-avatar', 'guid', 'created', 'plink'];
        $item = Item::selectFirst($fields, ['id' => $id, 'private' => false]);
 
        if (DBA::isResult($item) && $item['body'] != "") {
@@ -2034,10 +2029,16 @@ function api_statuses_repeat($type)
                } else {
                        $post = share_header($item['author-name'], $item['author-link'], $item['author-avatar'], $item['guid'], $item['created'], $item['plink']);
 
+                       if (!empty($item['title'])) {
+                               $post .= '[h3]' . $item['title'] . "[/h3]\n";
+                       }
+
                        $post .= $item['body'];
                        $post .= "[/share]";
                }
                $_REQUEST['body'] = $post;
+               $_REQUEST['tag'] = $item['tag'];
+               $_REQUEST['attach'] = $item['attach'];
                $_REQUEST['profile_uid'] = api_user();
                $_REQUEST['api_source'] = true;
 
@@ -2082,15 +2083,15 @@ function api_statuses_destroy($type)
        api_get_user($a);
 
        // params
-       $id = intval(defaults($a->argv, 3, 0));
+       $id = intval($a->argv[3] ?? 0);
 
        if ($id == 0) {
-               $id = intval(defaults($_REQUEST, 'id', 0));
+               $id = intval($_REQUEST['id'] ?? 0);
        }
 
        // Hotot workaround
        if ($id == 0) {
-               $id = intval(defaults($a->argv, 4, 0));
+               $id = intval($a->argv[4] ?? 0);
        }
 
        Logger::log('API: api_statuses_destroy: '.$id);
@@ -2136,15 +2137,12 @@ function api_statuses_mentions($type)
        // get last network messages
 
        // params
-       $since_id = defaults($_REQUEST, 'since_id', 0);
-       $max_id   = defaults($_REQUEST, 'max_id'  , 0);
-       $count    = defaults($_REQUEST, 'count'   , 20);
-       $page     = defaults($_REQUEST, 'page'    , 1);
-       if ($page < 1) {
-               $page = 1;
-       }
+       $since_id = $_REQUEST['since_id'] ?? 0;
+       $max_id   = $_REQUEST['max_id']   ?? 0;
+       $count    = $_REQUEST['count']    ?? 20;
+       $page     = $_REQUEST['page']     ?? 1;
 
-       $start = ($page - 1) * $count;
+       $start = max(0, ($page - 1) * $count);
 
        $condition = ["`uid` = ? AND `gravity` IN (?, ?) AND `item`.`id` > ? AND `author-id` != ?
                AND `item`.`parent` IN (SELECT `iid` FROM `thread` WHERE `thread`.`uid` = ? AND `thread`.`mention` AND NOT `thread`.`ignored`)",
@@ -2206,18 +2204,16 @@ function api_statuses_user_timeline($type)
                Logger::DEBUG
        );
 
-       $since_id        = defaults($_REQUEST, 'since_id', 0);
-       $max_id          = defaults($_REQUEST, 'max_id', 0);
+       $since_id        = $_REQUEST['since_id'] ?? 0;
+       $max_id          = $_REQUEST['max_id'] ?? 0;
        $exclude_replies = !empty($_REQUEST['exclude_replies']);
-       $conversation_id = defaults($_REQUEST, 'conversation_id', 0);
+       $conversation_id = $_REQUEST['conversation_id'] ?? 0;
 
        // pagination
-       $count = defaults($_REQUEST, 'count', 20);
-       $page  = defaults($_REQUEST, 'page', 1);
-       if ($page < 1) {
-               $page = 1;
-       }
-       $start = ($page - 1) * $count;
+       $count = $_REQUEST['count'] ?? 20;
+       $page  = $_REQUEST['page'] ?? 1;
+
+       $start = max(0, ($page - 1) * $count);
 
        $condition = ["`uid` = ? AND `gravity` IN (?, ?) AND `item`.`id` > ? AND `item`.`contact-id` = ?",
                api_user(), GRAVITY_PARENT, GRAVITY_COMMENT, $since_id, $user_info['cid']];
@@ -2296,9 +2292,9 @@ function api_favorites_create_destroy($type)
        }
        $action = str_replace("." . $type, "", $a->argv[$action_argv_id]);
        if ($a->argc == $action_argv_id + 2) {
-               $itemid = intval(defaults($a->argv, $action_argv_id + 1, 0));
+               $itemid = intval($a->argv[$action_argv_id + 1] ?? 0);
        } else {
-               $itemid = intval(defaults($_REQUEST, 'id', 0));
+               $itemid = intval($_REQUEST['id'] ?? 0);
        }
 
        $item = Item::selectFirstForUser(api_user(), [], ['id' => $itemid, 'uid' => api_user()]);
@@ -2378,15 +2374,12 @@ function api_favorites($type)
                $ret = [];
        } else {
                // params
-               $since_id = defaults($_REQUEST, 'since_id', 0);
-               $max_id = defaults($_REQUEST, 'max_id', 0);
-               $count = defaults($_GET, 'count', 20);
-               $page = (!empty($_REQUEST['page']) ? $_REQUEST['page'] -1 : 0);
-               if ($page < 0) {
-                       $page = 0;
-               }
+               $since_id = $_REQUEST['since_id'] ?? 0;
+               $max_id = $_REQUEST['max_id'] ?? 0;
+               $count = $_GET['count'] ?? 20;
+               $page = $_REQUEST['page'] ?? 1;
 
-               $start = $page*$count;
+               $start = max(0, ($page - 1) * $count);
 
                $condition = ["`uid` = ? AND `gravity` IN (?, ?) AND `id` > ? AND `starred`",
                        api_user(), GRAVITY_PARENT, GRAVITY_COMMENT, $since_id];
@@ -2437,14 +2430,14 @@ function api_format_messages($item, $recipient, $sender)
                'sender_id'             => $sender['id'],
                'text'                  => "",
                'recipient_id'          => $recipient['id'],
-               'created_at'            => api_date(defaults($item, 'created', DateTimeFormat::utcNow())),
+               'created_at'            => api_date($item['created'] ?? DateTimeFormat::utcNow()),
                'sender_screen_name'    => $sender['screen_name'],
                'recipient_screen_name' => $recipient['screen_name'],
                'sender'                => $sender,
                'recipient'             => $recipient,
                'title'                 => "",
-               'friendica_seen'        => defaults($item, 'seen', 0),
-               'friendica_parent_uri'  => defaults($item, 'parent-uri', ''),
+               'friendica_seen'        => $item['seen'] ?? 0,
+               'friendica_parent_uri'  => $item['parent-uri'] ?? '',
        ];
 
        // "uid" and "self" are only needed for some internal stuff, so remove it from here
@@ -2507,11 +2500,11 @@ function api_convert_item($item)
                $statustext = trim($statustitle."\n\n".$statusbody);
        }
 
-       if ((defaults($item, 'network', Protocol::PHANTOM) == Protocol::FEED) && (mb_strlen($statustext)> 1000)) {
-               $statustext = mb_substr($statustext, 0, 1000) . "... \n" . defaults($item, 'plink', '');
+       if ((($item['network'] ?? Protocol::PHANTOM) == Protocol::FEED) && (mb_strlen($statustext)> 1000)) {
+               $statustext = mb_substr($statustext, 0, 1000) . "... \n" . ($item['plink'] ?? '');
        }
 
-       $statushtml = BBCode::convert(api_clean_attachments($body), false);
+       $statushtml = BBCode::convert(BBCode::removeAttachment($body), false);
 
        // Workaround for clients with limited HTML parser functionality
        $search = ["<br>", "<blockquote>", "</blockquote>",
@@ -2542,7 +2535,7 @@ function api_convert_item($item)
        }
 
        // feeds without body should contain the link
-       if ((defaults($item, 'network', Protocol::PHANTOM) == Protocol::FEED) && (strlen($item['body']) == 0)) {
+       if ((($item['network'] ?? Protocol::PHANTOM) == Protocol::FEED) && (strlen($item['body']) == 0)) {
                $statushtml .= BBCode::convert($item['plink']);
        }
 
@@ -2567,6 +2560,7 @@ function api_get_attachments(&$body)
 {
        $text = $body;
        $text = preg_replace("/\[img\=([0-9]*)x([0-9]*)\](.*?)\[\/img\]/ism", '[img]$3[/img]', $text);
+       $text = preg_replace("/\[img\=(.*?)\](.*?)\[\/img\]/ism", '[img]$1[/img]', $text);
 
        $URLSearchString = "^\[\]";
        $ret = preg_match_all("/\[img\]([$URLSearchString]*)\[\/img\]/ism", $text, $images);
@@ -2578,14 +2572,14 @@ function api_get_attachments(&$body)
        $attachments = [];
 
        foreach ($images[1] as $image) {
-               $imagedata = Image::getInfoFromURL($image);
+               $imagedata = Images::getInfoFromURLCached($image);
 
                if ($imagedata) {
                        $attachments[] = ["url" => $image, "mimetype" => $imagedata["mime"], "size" => $imagedata["size"]];
                }
        }
 
-       if (strstr(defaults($_SERVER, 'HTTP_USER_AGENT', ''), "AndStatus")) {
+       if (strstr($_SERVER['HTTP_USER_AGENT'] ?? '', 'AndStatus')) {
                foreach ($images[0] as $orig) {
                        $body = str_replace($orig, "", $body);
                }
@@ -2605,7 +2599,7 @@ function api_get_attachments(&$body)
  */
 function api_get_entitities(&$text, $bbcode)
 {
-       $include_entities = strtolower(defaults($_REQUEST, 'include_entities', "false"));
+       $include_entities = strtolower($_REQUEST['include_entities'] ?? 'false');
 
        if ($include_entities != "true") {
                preg_match_all("/\[img](.*?)\[\/img\]/ism", $bbcode, $images);
@@ -2693,19 +2687,29 @@ function api_get_entitities(&$text, $bbcode)
                }
        }
 
-       preg_match_all("/\[img](.*?)\[\/img\]/ism", $bbcode, $images);
+       preg_match_all("/\[img\=(.*?)\](.*?)\[\/img\]/ism", $bbcode, $images, PREG_SET_ORDER);
        $ordered_images = [];
+       foreach ($images as $image) {
+               $start = iconv_strpos($text, $image[1], 0, "UTF-8");
+               if (!($start === false)) {
+                       $ordered_images[$start] = ['url' => $image[1], 'alt' => $image[2]];
+               }
+       }
+
+       preg_match_all("/\[img](.*?)\[\/img\]/ism", $bbcode, $images);
        foreach ($images[1] as $image) {
-               //$start = strpos($text, $url, $offset);
                $start = iconv_strpos($text, $image, 0, "UTF-8");
                if (!($start === false)) {
-                       $ordered_images[$start] = $image;
+                       $ordered_images[$start] = ['url' => $image, 'alt' => ''];
                }
        }
        //$entities["media"] = array();
        $offset = 0;
 
-       foreach ($ordered_images as $url) {
+       foreach ($ordered_images as $image) {
+               $url = $image['url'];
+               $ext_alt_text = $image['alt'];
+
                $display_url = str_replace(["http://www.", "https://www."], ["", ""], $url);
                $display_url = str_replace(["http://", "https://"], ["", ""], $display_url);
 
@@ -2715,7 +2719,7 @@ function api_get_entitities(&$text, $bbcode)
 
                $start = iconv_strpos($text, $url, $offset, "UTF-8");
                if (!($start === false)) {
-                       $image = Image::getInfoFromURL($url);
+                       $image = Images::getInfoFromURLCached($url);
                        if ($image) {
                                // If image cache is activated, then use the following sizes:
                                // thumb  (150), small (340), medium (600) and large (1024)
@@ -2723,19 +2727,19 @@ function api_get_entitities(&$text, $bbcode)
                                        $media_url = ProxyUtils::proxifyUrl($url);
 
                                        $sizes = [];
-                                       $scale = Image::getScalingDimensions($image[0], $image[1], 150);
+                                       $scale = Images::getScalingDimensions($image[0], $image[1], 150);
                                        $sizes["thumb"] = ["w" => $scale["width"], "h" => $scale["height"], "resize" => "fit"];
 
                                        if (($image[0] > 150) || ($image[1] > 150)) {
-                                               $scale = Image::getScalingDimensions($image[0], $image[1], 340);
+                                               $scale = Images::getScalingDimensions($image[0], $image[1], 340);
                                                $sizes["small"] = ["w" => $scale["width"], "h" => $scale["height"], "resize" => "fit"];
                                        }
 
-                                       $scale = Image::getScalingDimensions($image[0], $image[1], 600);
+                                       $scale = Images::getScalingDimensions($image[0], $image[1], 600);
                                        $sizes["medium"] = ["w" => $scale["width"], "h" => $scale["height"], "resize" => "fit"];
 
                                        if (($image[0] > 600) || ($image[1] > 600)) {
-                                               $scale = Image::getScalingDimensions($image[0], $image[1], 1024);
+                                               $scale = Images::getScalingDimensions($image[0], $image[1], 1024);
                                                $sizes["large"] = ["w" => $scale["width"], "h" => $scale["height"], "resize" => "fit"];
                                        }
                                } else {
@@ -2752,6 +2756,7 @@ function api_get_entitities(&$text, $bbcode)
                                                        "url" => $url,
                                                        "display_url" => $display_url,
                                                        "expanded_url" => $url,
+                                                       "ext_alt_text" => $ext_alt_text,
                                                        "type" => "photo",
                                                        "sizes" => $sizes];
                        }
@@ -2844,19 +2849,19 @@ function api_format_items_activities($item, $type = "json")
                // get user data and add it to the array of the activity
                $user = api_get_user($a, $parent_item['author-id']);
                switch ($parent_item['verb']) {
-                       case ACTIVITY_LIKE:
+                       case Activity::LIKE:
                                $activities['like'][] = $user;
                                break;
-                       case ACTIVITY_DISLIKE:
+                       case Activity::DISLIKE:
                                $activities['dislike'][] = $user;
                                break;
-                       case ACTIVITY_ATTEND:
+                       case Activity::ATTEND:
                                $activities['attendyes'][] = $user;
                                break;
-                       case ACTIVITY_ATTENDNO:
+                       case Activity::ATTENDNO:
                                $activities['attendno'][] = $user;
                                break;
-                       case ACTIVITY_ATTENDMAYBE:
+                       case Activity::ATTENDMAYBE:
                                $activities['attendmaybe'][] = $user;
                                break;
                        default:
@@ -2952,7 +2957,7 @@ function api_format_items_profiles($profile_row)
  */
 function api_format_items($items, $user_info, $filter_user = false, $type = "json")
 {
-       $a = \Friendica\BaseObject::getApp();
+       $a = BaseObject::getApp();
 
        $ret = [];
 
@@ -2986,7 +2991,7 @@ function api_format_items($items, $user_info, $filter_user = false, $type = "jso
  */
 function api_format_item($item, $type = "json", $status_user = null, $author_user = null, $owner_user = null)
 {
-       $a = \Friendica\BaseObject::getApp();
+       $a = BaseObject::getApp();
 
        if (empty($status_user) || empty($author_user) || empty($owner_user)) {
                list($status_user, $author_user, $owner_user) = api_item_get_user($a, $item);
@@ -3027,6 +3032,8 @@ function api_format_item($item, $type = "json", $status_user = null, $author_use
                'statusnet_conversation_id' => $item['parent'],
                'external_url' => System::baseUrl() . "/display/" . $item['guid'],
                'friendica_activities' => api_format_items_activities($item, $type),
+               'friendica_title' => $item['title'],
+               'friendica_html' => BBCode::convert($item['body'], false)
        ];
 
        if (count($converted["attachments"]) > 0) {
@@ -3297,17 +3304,14 @@ function api_lists_statuses($type)
        }
 
        // params
-       $count = defaults($_REQUEST, 'count', 20);
-       $page = (!empty($_REQUEST['page']) ? $_REQUEST['page'] - 1 : 0);
-       if ($page < 0) {
-               $page = 0;
-       }
-       $since_id = defaults($_REQUEST, 'since_id', 0);
-       $max_id = defaults($_REQUEST, 'max_id', 0);
+       $count = $_REQUEST['count'] ?? 20;
+       $page = $_REQUEST['page'] ?? 1;
+       $since_id = $_REQUEST['since_id'] ?? 0;
+       $max_id = $_REQUEST['max_id'] ?? 0;
        $exclude_replies = (!empty($_REQUEST['exclude_replies']) ? 1 : 0);
-       $conversation_id = defaults($_REQUEST, 'conversation_id', 0);
+       $conversation_id = $_REQUEST['conversation_id'] ?? 0;
 
-       $start = $page * $count;
+       $start = max(0, ($page - 1) * $count);
 
        $condition = ["`uid` = ? AND `gravity` IN (?, ?) AND `id` > ? AND `group_member`.`gid` = ?",
                api_user(), GRAVITY_PARENT, GRAVITY_COMMENT, $since_id, $_REQUEST['list_id']];
@@ -3367,12 +3371,10 @@ function api_statuses_f($qtype)
        }
 
        // pagination
-       $count = defaults($_GET, 'count', 20);
-       $page = defaults($_GET, 'page', 1);
-       if ($page < 1) {
-               $page = 1;
-       }
-       $start = ($page - 1) * $count;
+       $count = $_GET['count'] ?? 20;
+       $page = $_GET['page'] ?? 1;
+
+       $start = max(0, ($page - 1) * $count);
 
        $user_info = api_get_user($a);
 
@@ -3619,7 +3621,7 @@ function api_ff_ids($type)
 
        api_get_user($a);
 
-       $stringify_ids = defaults($_REQUEST, 'stringify_ids', false);
+       $stringify_ids = $_REQUEST['stringify_ids'] ?? false;
 
        $r = q(
                "SELECT `pcontact`.`id` FROM `contact`
@@ -3794,9 +3796,9 @@ function api_direct_messages_destroy($type)
        // params
        $user_info = api_get_user($a);
        //required
-       $id = defaults($_REQUEST, 'id', 0);
+       $id = $_REQUEST['id'] ?? 0;
        // optional
-       $parenturi = defaults($_REQUEST, 'friendica_parenturi', "");
+       $parenturi = $_REQUEST['friendica_parenturi'] ?? '';
        $verbose = (!empty($_GET['friendica_verbose']) ? strtolower($_GET['friendica_verbose']) : "false");
        /// @todo optional parameter 'include_entities' from Twitter API not yet implemented
 
@@ -3877,7 +3879,7 @@ function api_friendships_destroy($type)
                throw new ForbiddenException();
        }
 
-       $contact_id = defaults($_REQUEST, 'user_id');
+       $contact_id = $_REQUEST['user_id'] ?? 0;
 
        if (empty($contact_id)) {
                Logger::notice(API_LOG_PREFIX . 'No user_id specified', ['module' => 'api', 'action' => 'friendships_destroy']);
@@ -3958,17 +3960,14 @@ function api_direct_messages_box($type, $box, $verbose)
                throw new ForbiddenException();
        }
        // params
-       $count = defaults($_GET, 'count', 20);
-       $page = defaults($_REQUEST, 'page', 1) - 1;
-       if ($page < 0) {
-               $page = 0;
-       }
+       $count = $_GET['count'] ?? 20;
+       $page = $_REQUEST['page'] ?? 1;
 
-       $since_id = defaults($_REQUEST, 'since_id', 0);
-       $max_id = defaults($_REQUEST, 'max_id', 0);
+       $since_id = $_REQUEST['since_id'] ?? 0;
+       $max_id = $_REQUEST['max_id'] ?? 0;
 
-       $user_id = defaults($_REQUEST, 'user_id', '');
-       $screen_name = defaults($_REQUEST, 'screen_name', '');
+       $user_id = $_REQUEST['user_id'] ?? '';
+       $screen_name = $_REQUEST['screen_name'] ?? '';
 
        //  caller user info
        unset($_REQUEST["user_id"]);
@@ -3984,7 +3983,7 @@ function api_direct_messages_box($type, $box, $verbose)
        $profile_url = $user_info["url"];
 
        // pagination
-       $start = $page * $count;
+       $start = max(0, ($page - 1) * $count);
 
        $sql_extra = "";
 
@@ -3992,7 +3991,7 @@ function api_direct_messages_box($type, $box, $verbose)
        if ($box=="sentbox") {
                $sql_extra = "`mail`.`from-url`='" . DBA::escape($profile_url) . "'";
        } elseif ($box == "conversation") {
-               $sql_extra = "`mail`.`parent-uri`='" . DBA::escape(defaults($_GET, 'uri', ''))  . "'";
+               $sql_extra = "`mail`.`parent-uri`='" . DBA::escape($_GET['uri'] ?? '')  . "'";
        } elseif ($box == "all") {
                $sql_extra = "true";
        } elseif ($box == "inbox") {
@@ -4172,7 +4171,7 @@ function api_fr_photoalbum_delete($type)
                throw new ForbiddenException();
        }
        // input params
-       $album = defaults($_REQUEST, 'album', "");
+       $album = $_REQUEST['album'] ?? '';
 
        // we do not allow calls without album string
        if ($album == "") {
@@ -4227,8 +4226,8 @@ function api_fr_photoalbum_update($type)
                throw new ForbiddenException();
        }
        // input params
-       $album = defaults($_REQUEST, 'album', "");
-       $album_new = defaults($_REQUEST, 'album_new', "");
+       $album = $_REQUEST['album'] ?? '';
+       $album_new = $_REQUEST['album_new'] ?? '';
 
        // we do not allow calls without album string
        if ($album == "") {
@@ -4319,14 +4318,14 @@ function api_fr_photo_create_update($type)
                throw new ForbiddenException();
        }
        // input params
-       $photo_id = defaults($_REQUEST, 'photo_id', null);
-       $desc = defaults($_REQUEST, 'desc', (array_key_exists('desc', $_REQUEST) ? "" : null)) ; // extra check necessary to distinguish between 'not provided' and 'empty string'
-       $album = defaults($_REQUEST, 'album', null);
-       $album_new = defaults($_REQUEST, 'album_new', null);
-       $allow_cid = defaults($_REQUEST, 'allow_cid', (array_key_exists('allow_cid', $_REQUEST) ? " " : null));
-       $deny_cid  = defaults($_REQUEST, 'deny_cid' , (array_key_exists('deny_cid' , $_REQUEST) ? " " : null));
-       $allow_gid = defaults($_REQUEST, 'allow_gid', (array_key_exists('allow_gid', $_REQUEST) ? " " : null));
-       $deny_gid  = defaults($_REQUEST, 'deny_gid' , (array_key_exists('deny_gid' , $_REQUEST) ? " " : null));
+       $photo_id  = $_REQUEST['photo_id']  ?? null;
+       $desc      = $_REQUEST['desc']      ?? null;
+       $album     = $_REQUEST['album']     ?? null;
+       $album_new = $_REQUEST['album_new'] ?? null;
+       $allow_cid = $_REQUEST['allow_cid'] ?? null;
+       $deny_cid  = $_REQUEST['deny_cid' ] ?? null;
+       $allow_gid = $_REQUEST['allow_gid'] ?? null;
+       $deny_gid  = $_REQUEST['deny_gid' ] ?? null;
        $visibility = !empty($_REQUEST['visibility']) && $_REQUEST['visibility'] !== "false";
 
        // do several checks on input parameters
@@ -4457,7 +4456,7 @@ function api_fr_photo_delete($type)
        }
 
        // input params
-       $photo_id = defaults($_REQUEST, 'photo_id', null);
+       $photo_id = $_REQUEST['photo_id'] ?? null;
 
        // do several checks on input parameters
        // we do not allow calls without photo id
@@ -4544,7 +4543,7 @@ function api_account_update_profile_image($type)
                throw new ForbiddenException();
        }
        // input params
-       $profile_id = defaults($_REQUEST, 'profile_id', 0);
+       $profile_id = $_REQUEST['profile_id'] ?? 0;
 
        // error if image data is missing
        if (empty($_FILES['image'])) {
@@ -4676,9 +4675,10 @@ api_register_func('api/account/update_profile', 'api_account_update_profile', tr
  */
 function check_acl_input($acl_string)
 {
-       if ($acl_string == null || $acl_string == " ") {
+       if (empty($acl_string)) {
                return false;
        }
+
        $contact_not_found = false;
 
        // split <x><y><z> into array of cid's
@@ -4696,7 +4696,6 @@ function check_acl_input($acl_string)
 }
 
 /**
- *
  * @param string  $mediatype
  * @param array   $media
  * @param string  $type
@@ -4715,6 +4714,7 @@ function check_acl_input($acl_string)
  * @throws ImagickException
  * @throws InternalServerErrorException
  * @throws NotFoundException
+ * @throws UnauthorizedException
  */
 function save_media_to_database($mediatype, $media, $type, $album, $allow_cid, $deny_cid, $allow_gid, $deny_gid, $desc, $profile = 0, $visibility = false, $photo_id = null)
 {
@@ -4748,7 +4748,7 @@ function save_media_to_database($mediatype, $media, $type, $album, $allow_cid, $
        }
 
        if ($filetype == "") {
-               $filetype=Image::guessType($filename);
+               $filetype = Images::guessType($filename);
        }
        $imagedata = @getimagesize($src);
        if ($imagedata) {
@@ -4795,19 +4795,19 @@ function save_media_to_database($mediatype, $media, $type, $album, $allow_cid, $
        $height = $Image->getHeight();
 
        // create a new resource-id if not already provided
-       $hash = ($photo_id == null) ? Photo::newResource() : $photo_id;
+       $resource_id = ($photo_id == null) ? Photo::newResource() : $photo_id;
 
        if ($mediatype == "photo") {
                // upload normal image (scales 0, 1, 2)
                Logger::log("photo upload: starting new photo upload", Logger::DEBUG);
 
-               $r = Photo::store($Image, local_user(), $visitor, $hash, $filename, $album, 0, 0, $allow_cid, $allow_gid, $deny_cid, $deny_gid, $desc);
+               $r = Photo::store($Image, local_user(), $visitor, $resource_id, $filename, $album, 0, 0, $allow_cid, $allow_gid, $deny_cid, $deny_gid, $desc);
                if (!$r) {
                        Logger::log("photo upload: image upload with scale 0 (original size) failed");
                }
                if ($width > 640 || $height > 640) {
                        $Image->scaleDown(640);
-                       $r = Photo::store($Image, local_user(), $visitor, $hash, $filename, $album, 1, 0, $allow_cid, $allow_gid, $deny_cid, $deny_gid, $desc);
+                       $r = Photo::store($Image, local_user(), $visitor, $resource_id, $filename, $album, 1, 0, $allow_cid, $allow_gid, $deny_cid, $deny_gid, $desc);
                        if (!$r) {
                                Logger::log("photo upload: image upload with scale 1 (640x640) failed");
                        }
@@ -4815,7 +4815,7 @@ function save_media_to_database($mediatype, $media, $type, $album, $allow_cid, $
 
                if ($width > 320 || $height > 320) {
                        $Image->scaleDown(320);
-                       $r = Photo::store($Image, local_user(), $visitor, $hash, $filename, $album, 2, 0, $allow_cid, $allow_gid, $deny_cid, $deny_gid, $desc);
+                       $r = Photo::store($Image, local_user(), $visitor, $resource_id, $filename, $album, 2, 0, $allow_cid, $allow_gid, $deny_cid, $deny_gid, $desc);
                        if (!$r) {
                                Logger::log("photo upload: image upload with scale 2 (320x320) failed");
                        }
@@ -4827,7 +4827,7 @@ function save_media_to_database($mediatype, $media, $type, $album, $allow_cid, $
 
                if ($width > 300 || $height > 300) {
                        $Image->scaleDown(300);
-                       $r = Photo::store($Image, local_user(), $visitor, $hash, $filename, $album, 4, $profile, $allow_cid, $allow_gid, $deny_cid, $deny_gid, $desc);
+                       $r = Photo::store($Image, local_user(), $visitor, $resource_id, $filename, $album, 4, $profile, $allow_cid, $allow_gid, $deny_cid, $deny_gid, $desc);
                        if (!$r) {
                                Logger::log("photo upload: profile image upload with scale 4 (300x300) failed");
                        }
@@ -4835,7 +4835,7 @@ function save_media_to_database($mediatype, $media, $type, $album, $allow_cid, $
 
                if ($width > 80 || $height > 80) {
                        $Image->scaleDown(80);
-                       $r = Photo::store($Image, local_user(), $visitor, $hash, $filename, $album, 5, $profile, $allow_cid, $allow_gid, $deny_cid, $deny_gid, $desc);
+                       $r = Photo::store($Image, local_user(), $visitor, $resource_id, $filename, $album, 5, $profile, $allow_cid, $allow_gid, $deny_cid, $deny_gid, $desc);
                        if (!$r) {
                                Logger::log("photo upload: profile image upload with scale 5 (80x80) failed");
                        }
@@ -4843,7 +4843,7 @@ function save_media_to_database($mediatype, $media, $type, $album, $allow_cid, $
 
                if ($width > 48 || $height > 48) {
                        $Image->scaleDown(48);
-                       $r = Photo::store($Image, local_user(), $visitor, $hash, $filename, $album, 6, $profile, $allow_cid, $allow_gid, $deny_cid, $deny_gid, $desc);
+                       $r = Photo::store($Image, local_user(), $visitor, $resource_id, $filename, $album, 6, $profile, $allow_cid, $allow_gid, $deny_cid, $deny_gid, $desc);
                        if (!$r) {
                                Logger::log("photo upload: profile image upload with scale 6 (48x48) failed");
                        }
@@ -4855,10 +4855,10 @@ function save_media_to_database($mediatype, $media, $type, $album, $allow_cid, $
        if (isset($r) && $r) {
                // create entry in 'item'-table on new uploads to enable users to comment/like/dislike the photo
                if ($photo_id == null && $mediatype == "photo") {
-                       post_photo_item($hash, $allow_cid, $deny_cid, $allow_gid, $deny_gid, $filetype, $visibility);
+                       post_photo_item($resource_id, $allow_cid, $deny_cid, $allow_gid, $deny_gid, $filetype, $visibility);
                }
                // on success return image data in json/xml format (like /api/friendica/photo does when no scale is given)
-               return prepare_photo_data($type, false, $hash);
+               return prepare_photo_data($type, false, $resource_id);
        } else {
                throw new InternalServerErrorException("image upload failed");
        }
@@ -5049,8 +5049,8 @@ function prepare_photo_data($type, $scale, $photo_id)
  */
 function api_friendica_remoteauth()
 {
-       $url = defaults($_GET, 'url', '');
-       $c_url = defaults($_GET, 'c_url', '');
+       $url = $_GET['url'] ?? '';
+       $c_url = $_GET['c_url'] ?? '';
 
        if ($url === '' || $c_url === '') {
                throw new BadRequestException("Wrong parameters.");
@@ -5068,7 +5068,7 @@ function api_friendica_remoteauth()
 
        $cid = $contact['id'];
 
-       $dfrn_id = defaults($contact, 'issued-id', $contact['dfrn-id']);
+       $dfrn_id = $contact['issued-id'] ?? $contact['dfrn-id'];
 
        if ($contact['duplex'] && $contact['issued-id']) {
                $orig_id = $contact['issued-id'];
@@ -5120,7 +5120,7 @@ function api_get_announce($item)
        }
 
        $fields = ['author-id', 'author-name', 'author-link', 'author-avatar'];
-       $activity = Item::activityToIndex(ACTIVITY2_ANNOUNCE);
+       $activity = Item::activityToIndex(Activity::ANNOUNCE);
        $condition = ['parent-uri' => $item['uri'], 'gravity' => GRAVITY_ACTIVITY, 'uid' => [0, $item['uid']], 'activity' => $activity];
        $announce = Item::selectFirstForUser($item['uid'], $fields, $condition, ['order' => ['received' => true]]);
        if (!DBA::isResult($announce)) {
@@ -5156,99 +5156,30 @@ function api_share_as_retweet(&$item)
                }
        }
 
-       /// @TODO "$1" should maybe mean '$1' ?
-       $attributes = preg_replace("/\[share(.*?)\]\s?(.*?)\s?\[\/share\]\s?/ism", "$1", $body);
-       /*
-        * Skip if there is no shared message in there
-        * we already checked this in diaspora::isReshare()
-        * but better one more than one less...
-        */
-       if (($body == $attributes) || empty($attributes)) {
+       $reshared = Item::getShareArray($item);
+       if (empty($reshared)) {
                return false;
        }
 
-       // build the fake reshared item
        $reshared_item = $item;
 
-       $author = "";
-       preg_match("/author='(.*?)'/ism", $attributes, $matches);
-       if (!empty($matches[1])) {
-               $author = html_entity_decode($matches[1], ENT_QUOTES, 'UTF-8');
-       }
-
-       preg_match('/author="(.*?)"/ism', $attributes, $matches);
-       if (!empty($matches[1])) {
-               $author = $matches[1];
-       }
-
-       $profile = "";
-       preg_match("/profile='(.*?)'/ism", $attributes, $matches);
-       if (!empty($matches[1])) {
-               $profile = $matches[1];
-       }
-
-       preg_match('/profile="(.*?)"/ism', $attributes, $matches);
-       if (!empty($matches[1])) {
-               $profile = $matches[1];
-       }
-
-       $avatar = "";
-       preg_match("/avatar='(.*?)'/ism", $attributes, $matches);
-       if (!empty($matches[1])) {
-               $avatar = $matches[1];
-       }
-
-       preg_match('/avatar="(.*?)"/ism', $attributes, $matches);
-       if (!empty($matches[1])) {
-               $avatar = $matches[1];
-       }
-
-       $link = "";
-       preg_match("/link='(.*?)'/ism", $attributes, $matches);
-       if (!empty($matches[1])) {
-               $link = $matches[1];
-       }
-
-       preg_match('/link="(.*?)"/ism', $attributes, $matches);
-       if (!empty($matches[1])) {
-               $link = $matches[1];
-       }
-
-       $posted = "";
-       preg_match("/posted='(.*?)'/ism", $attributes, $matches);
-       if (!empty($matches[1])) {
-               $posted = $matches[1];
-       }
-
-       preg_match('/posted="(.*?)"/ism', $attributes, $matches);
-       if (!empty($matches[1])) {
-               $posted = $matches[1];
-       }
-
-       if (!preg_match("/(.*?)\[share.*?\]\s?(.*?)\s?\[\/share\]\s?(.*?)/ism", $body, $matches)) {
+       if (empty($reshared['shared']) || empty($reshared['profile']) || empty($reshared['author']) || empty($reshared['avatar']) || empty($reshared['posted'])) {
                return false;
        }
 
-       $pre_body = trim($matches[1]);
-       if ($pre_body != '') {
-               $item['body'] = $pre_body;
+       if (!empty($reshared['comment'])) {
+               $item['body'] = $reshared['comment'];
        }
 
-       $shared_body = trim($matches[2]);
-
-       if (($shared_body == "") || ($profile == "") || ($author == "") || ($avatar == "") || ($posted == "")) {
-               return false;
-       }
-
-       $reshared_item["share-pre-body"] = $pre_body;
-       $reshared_item["body"] = $shared_body;
-       $reshared_item["author-id"] = Contact::getIdForURL($profile, 0, true);
-       $reshared_item["author-name"] = $author;
-       $reshared_item["author-link"] = $profile;
-       $reshared_item["author-avatar"] = $avatar;
-       $reshared_item["plink"] = $link;
-       $reshared_item["created"] = $posted;
-       $reshared_item["edited"] = $posted;
+       $reshared_item["share-pre-body"] = $reshared['comment'];
+       $reshared_item["body"] = $reshared['shared'];
+       $reshared_item["author-id"] = Contact::getIdForURL($reshared['profile'], 0, true);
+       $reshared_item["author-name"] = $reshared['author'];
+       $reshared_item["author-link"] = $reshared['profile'];
+       $reshared_item["author-avatar"] = $reshared['avatar'];
+       $reshared_item["plink"] = $reshared['link'] ?? '';
+       $reshared_item["created"] = $reshared['posted'];
+       $reshared_item["edited"] = $reshared['posted'];
 
        return $reshared_item;
 }
@@ -5403,7 +5334,7 @@ function api_in_reply_to($item)
  */
 function api_clean_plain_items($text)
 {
-       $include_entities = strtolower(defaults($_REQUEST, 'include_entities', "false"));
+       $include_entities = strtolower($_REQUEST['include_entities'] ?? 'false');
 
        $text = BBCode::cleanPictureLinks($text);
        $URLSearchString = "^\[\]";
@@ -5415,42 +5346,11 @@ function api_clean_plain_items($text)
        }
 
        // Simplify "attachment" element
-       $text = api_clean_attachments($text);
+       $text = BBCode::removeAttachment($text);
 
        return $text;
 }
 
-/**
- * @brief Removes most sharing information for API text export
- *
- * @param string $body The original body
- *
- * @return string Cleaned body
- * @throws InternalServerErrorException
- */
-function api_clean_attachments($body)
-{
-       $data = BBCode::getAttachmentData($body);
-
-       if (empty($data)) {
-               return $body;
-       }
-       $body = "";
-
-       if (isset($data["text"])) {
-               $body = $data["text"];
-       }
-       if (($body == "") && isset($data["title"])) {
-               $body = $data["title"];
-       }
-       if (isset($data["url"])) {
-               $body .= "\n".$data["url"];
-       }
-       $body .= $data["after"];
-
-       return $body;
-}
-
 /**
  *
  * @param array $contacts
@@ -5541,7 +5441,7 @@ function api_friendica_group_show($type)
 
        // params
        $user_info = api_get_user($a);
-       $gid = defaults($_REQUEST, 'gid', 0);
+       $gid = $_REQUEST['gid'] ?? 0;
        $uid = $user_info['uid'];
 
        // get data of the specified group id or all groups if not specified
@@ -5611,8 +5511,8 @@ function api_friendica_group_delete($type)
 
        // params
        $user_info = api_get_user($a);
-       $gid = defaults($_REQUEST, 'gid', 0);
-       $name = defaults($_REQUEST, 'name', "");
+       $gid = $_REQUEST['gid'] ?? 0;
+       $name = $_REQUEST['name'] ?? '';
        $uid = $user_info['uid'];
 
        // error if no gid specified
@@ -5678,7 +5578,7 @@ function api_lists_destroy($type)
 
        // params
        $user_info = api_get_user($a);
-       $gid = defaults($_REQUEST, 'list_id', 0);
+       $gid = $_REQUEST['list_id'] ?? 0;
        $uid = $user_info['uid'];
 
        // error if no gid specified
@@ -5800,7 +5700,7 @@ function api_friendica_group_create($type)
 
        // params
        $user_info = api_get_user($a);
-       $name = defaults($_REQUEST, 'name', "");
+       $name = $_REQUEST['name'] ?? '';
        $uid = $user_info['uid'];
        $json = json_decode($_POST['json'], true);
        $users = $json['user'];
@@ -5834,7 +5734,7 @@ function api_lists_create($type)
 
        // params
        $user_info = api_get_user($a);
-       $name = defaults($_REQUEST, 'name', "");
+       $name = $_REQUEST['name'] ?? '';
        $uid = $user_info['uid'];
 
        $success = group_create($name, $uid);
@@ -5874,8 +5774,8 @@ function api_friendica_group_update($type)
        // params
        $user_info = api_get_user($a);
        $uid = $user_info['uid'];
-       $gid = defaults($_REQUEST, 'gid', 0);
-       $name = defaults($_REQUEST, 'name', "");
+       $gid = $_REQUEST['gid'] ?? 0;
+       $name = $_REQUEST['name'] ?? '';
        $json = json_decode($_POST['json'], true);
        $users = $json['user'];
 
@@ -5952,8 +5852,8 @@ function api_lists_update($type)
 
        // params
        $user_info = api_get_user($a);
-       $gid = defaults($_REQUEST, 'list_id', 0);
-       $name = defaults($_REQUEST, 'name', "");
+       $gid = $_REQUEST['list_id'] ?? 0;
+       $name = $_REQUEST['name'] ?? '';
        $uid = $user_info['uid'];
 
        // error if no gid specified
@@ -6002,7 +5902,7 @@ function api_friendica_activity($type)
        $verb = strtolower($a->argv[3]);
        $verb = preg_replace("|\..*$|", "", $verb);
 
-       $id = defaults($_REQUEST, 'id', 0);
+       $id = $_REQUEST['id'] ?? 0;
 
        $res = Item::performLike($id, $verb);
 
@@ -6049,7 +5949,8 @@ function api_friendica_notification($type)
        if ($a->argc!==3) {
                throw new BadRequestException("Invalid argument count");
        }
-       $nm = new NotificationsManager();
+       /** @var Notify $nm */
+       $nm = BaseObject::getClass(Notify::class);
 
        $notes = $nm->getAll([], ['seen' => 'ASC', 'date' => 'DESC'], 50);
 
@@ -6093,7 +5994,8 @@ function api_friendica_notification_seen($type)
 
        $id = (!empty($_REQUEST['id']) ? intval($_REQUEST['id']) : 0);
 
-       $nm = new NotificationsManager();
+       /** @var Notify $nm */
+       $nm = BaseObject::getClass(Notify::class);
        $note = $nm->getByID($id);
        if (is_null($note)) {
                throw new BadRequestException("Invalid argument");
@@ -6139,7 +6041,7 @@ function api_friendica_direct_messages_setseen($type)
        // params
        $user_info = api_get_user($a);
        $uid = $user_info['uid'];
-       $id = defaults($_REQUEST, 'id', 0);
+       $id = $_REQUEST['id'] ?? 0;
 
        // return error if id is zero
        if ($id == "") {
@@ -6193,7 +6095,7 @@ function api_friendica_direct_messages_search($type, $box = "")
 
        // params
        $user_info = api_get_user($a);
-       $searchstring = defaults($_REQUEST, 'searchstring', "");
+       $searchstring = $_REQUEST['searchstring'] ?? '';
        $uid = $user_info['uid'];
 
        // error if no searchstring specified
@@ -6260,7 +6162,7 @@ function api_friendica_profile_show($type)
        }
 
        // input params
-       $profile_id = defaults($_REQUEST, 'profile_id', 0);
+       $profile_id = $_REQUEST['profile_id'] ?? 0;
 
        // retrieve general information about profiles for user
        $multi_profiles = Feature::isEnabled(api_user(), 'multi_profiles');