]> git.mxchange.org Git - friendica.git/blobdiff - include/api.php
Code standards
[friendica.git] / include / api.php
index 7e54fa382a643ea38c85fff7f7d3a5050a893eba..22ed066bafb7a968aac62d897bbeff832c6d3d55 100644 (file)
@@ -15,6 +15,7 @@ use Friendica\Core\Addon;
 use Friendica\Core\Authentication;
 use Friendica\Core\Config;
 use Friendica\Core\L10n;
+use Friendica\Core\Logger;
 use Friendica\Core\NotificationsManager;
 use Friendica\Core\PConfig;
 use Friendica\Core\Protocol;
@@ -42,9 +43,9 @@ use Friendica\Protocol\Diaspora;
 use Friendica\Util\DateTimeFormat;
 use Friendica\Util\Network;
 use Friendica\Util\Proxy as ProxyUtils;
+use Friendica\Util\Strings;
 use Friendica\Util\XML;
 
-require_once 'include/conversation.php';
 require_once 'mod/share.php';
 require_once 'mod/item.php';
 require_once 'mod/wall_upload.php';
@@ -66,7 +67,7 @@ $called_api = [];
  */
 function api_user()
 {
-       if (x($_SESSION, 'allow_api')) {
+       if (!empty($_SESSION['allow_api'])) {
                return local_user();
        }
 
@@ -96,9 +97,9 @@ function api_source()
                        return "Twidere";
                }
 
-               logger("Unrecognized user-agent ".$_SERVER['HTTP_USER_AGENT'], LOGGER_DEBUG);
+               Logger::log("Unrecognized user-agent ".$_SERVER['HTTP_USER_AGENT'], Logger::DEBUG);
        } else {
-               logger("Empty user-agent", LOGGER_DEBUG);
+               Logger::log("Empty user-agent", Logger::DEBUG);
        }
 
        return "api";
@@ -180,12 +181,12 @@ function api_login(App $a)
                var_dump($consumer, $token);
                die();
        } catch (Exception $e) {
-               logger($e);
+               Logger::log($e);
        }
 
        // workaround for HTTP-auth in CGI mode
-       if (x($_SERVER, 'REDIRECT_REMOTE_USER')) {
-               $userpass = base64_decode(substr($_SERVER["REDIRECT_REMOTE_USER"], 6)) ;
+       if (!empty($_SERVER['REDIRECT_REMOTE_USER'])) {
+               $userpass = base64_decode(substr($_SERVER["REDIRECT_REMOTE_USER"], 6));
                if (strlen($userpass)) {
                        list($name, $password) = explode(':', $userpass);
                        $_SERVER['PHP_AUTH_USER'] = $name;
@@ -193,8 +194,8 @@ function api_login(App $a)
                }
        }
 
-       if (!x($_SERVER, 'PHP_AUTH_USER')) {
-               logger('API_login: ' . print_r($_SERVER, true), LOGGER_DEBUG);
+       if (empty($_SERVER['PHP_AUTH_USER'])) {
+               Logger::log('API_login: ' . print_r($_SERVER, true), Logger::DEBUG);
                header('WWW-Authenticate: Basic realm="Friendica"');
                throw new UnauthorizedException("This API requires login");
        }
@@ -235,7 +236,7 @@ function api_login(App $a)
        }
 
        if (!DBA::isResult($record)) {
-               logger('API_login failure: ' . print_r($_SERVER, true), LOGGER_DEBUG);
+               Logger::log('API_login failure: ' . print_r($_SERVER, true), Logger::DEBUG);
                header('WWW-Authenticate: Basic realm="Friendica"');
                //header('HTTP/1.0 401 Unauthorized');
                //die('This api requires login');
@@ -308,19 +309,19 @@ function api_call(App $a)
                                        api_login($a);
                                }
 
-                               logger('API call for ' . $a->user['username'] . ': ' . $a->query_string);
-                               logger('API parameters: ' . print_r($_REQUEST, true));
+                               Logger::log('API call for ' . $a->user['username'] . ': ' . $a->query_string);
+                               Logger::log('API parameters: ' . print_r($_REQUEST, true));
 
                                $stamp =  microtime(true);
                                $return = call_user_func($info['func'], $type);
                                $duration = (float) (microtime(true) - $stamp);
-                               logger("API call duration: " . round($duration, 2) . "\t" . $a->query_string, LOGGER_DEBUG);
+                               Logger::log("API call duration: " . round($duration, 2) . "\t" . $a->query_string, Logger::DEBUG);
 
                                if (Config::get("system", "profiler")) {
                                        $duration = microtime(true)-$a->performance["start"];
 
                                        /// @TODO round() really everywhere?
-                                       logger(
+                                       Logger::log(
                                                parse_url($a->query_string, PHP_URL_PATH) . ": " . sprintf(
                                                        "Database: %s/%s, Cache %s/%s, Network: %s, I/O: %s, Other: %s, Total: %s",
                                                        round($a->performance["database"] - $a->performance["database_write"], 3),
@@ -334,7 +335,7 @@ function api_call(App $a)
                                                                + $a->performance["network"] + $a->performance["file"]), 2),
                                                        round($duration, 2)
                                                ),
-                                               LOGGER_DEBUG
+                                               Logger::DEBUG
                                        );
 
                                        if (Config::get("rendertime", "callstack")) {
@@ -375,7 +376,7 @@ function api_call(App $a)
                                                                $o .= $func . ": " . $time . "\n";
                                                        }
                                                }
-                                               logger($o, LOGGER_DEBUG);
+                                               Logger::log($o, Logger::DEBUG);
                                        }
                                }
 
@@ -394,7 +395,7 @@ function api_call(App $a)
                                        case "json":
                                                header("Content-Type: application/json");
                                                $json = json_encode(end($return));
-                                               if (x($_GET, 'callback')) {
+                                               if (!empty($_GET['callback'])) {
                                                        $json = $_GET['callback'] . "(" . $json . ")";
                                                }
                                                $return = $json;
@@ -412,7 +413,7 @@ function api_call(App $a)
                        }
                }
 
-               logger('API call not implemented: ' . $a->query_string);
+               Logger::log('API call not implemented: ' . $a->query_string);
                throw new NotImplementedException();
        } catch (HTTPException $e) {
                header("HTTP/1.1 {$e->httpcode} {$e->httpdesc}");
@@ -429,7 +430,7 @@ function api_call(App $a)
  */
 function api_error($type, $e)
 {
-       $a = get_app();
+       $a = \get_app();
 
        $error = ($e->getMessage() !== "" ? $e->getMessage() : $e->httpdesc);
        /// @TODO:  https://dev.twitter.com/overview/api/response-codes
@@ -521,11 +522,11 @@ function api_get_user(App $a, $contact_id = null)
        $extra_query = "";
        $url = "";
 
-       logger("api_get_user: Fetching user data for user ".$contact_id, LOGGER_DEBUG);
+       Logger::log("api_get_user: Fetching user data for user ".$contact_id, Logger::DEBUG);
 
        // Searching for contact URL
        if (!is_null($contact_id) && (intval($contact_id) == 0)) {
-               $user = DBA::escape(normalise_link($contact_id));
+               $user = DBA::escape(Strings::normaliseLink($contact_id));
                $url = $user;
                $extra_query = "AND `contact`.`nurl` = '%s' ";
                if (api_user() !== false) {
@@ -548,7 +549,7 @@ function api_get_user(App $a, $contact_id = null)
                }
        }
 
-       if (is_null($user) && x($_GET, 'user_id')) {
+       if (is_null($user) && !empty($_GET['user_id'])) {
                $user = DBA::escape(api_unique_id_to_nurl($_GET['user_id']));
 
                if ($user == "") {
@@ -561,7 +562,7 @@ function api_get_user(App $a, $contact_id = null)
                        $extra_query .= "AND `contact`.`uid`=" . intval(api_user());
                }
        }
-       if (is_null($user) && x($_GET, 'screen_name')) {
+       if (is_null($user) && !empty($_GET['screen_name'])) {
                $user = DBA::escape($_GET['screen_name']);
                $extra_query = "AND `contact`.`nick` = '%s' ";
                if (api_user() !== false) {
@@ -569,8 +570,8 @@ function api_get_user(App $a, $contact_id = null)
                }
        }
 
-       if (is_null($user) && x($_GET, 'profileurl')) {
-               $user = DBA::escape(normalise_link($_GET['profileurl']));
+       if (is_null($user) && !empty($_GET['profileurl'])) {
+               $user = DBA::escape(Strings::normaliseLink($_GET['profileurl']));
                $extra_query = "AND `contact`.`nurl` = '%s' ";
                if (api_user() !== false) {
                        $extra_query .= "AND `contact`.`uid`=".intval(api_user());
@@ -605,7 +606,7 @@ function api_get_user(App $a, $contact_id = null)
                }
        }
 
-       logger("api_get_user: user ".$user, LOGGER_DEBUG);
+       Logger::log("api_get_user: user ".$user, Logger::DEBUG);
 
        if (!$user) {
                if (api_user() === false) {
@@ -617,7 +618,7 @@ function api_get_user(App $a, $contact_id = null)
                }
        }
 
-       logger('api_user: ' . $extra_query . ', user: ' . $user);
+       Logger::log('api_user: ' . $extra_query . ', user: ' . $user);
 
        // user info
        $uinfo = q(
@@ -638,11 +639,9 @@ function api_get_user(App $a, $contact_id = null)
                        throw new BadRequestException("User not found.");
                }
 
-               $contact = DBA::selectFirst('contact', [], ['uid' => 0, 'nurl' => normalise_link($url)]);
+               $contact = DBA::selectFirst('contact', [], ['uid' => 0, 'nurl' => Strings::normaliseLink($url)]);
 
                if (DBA::isResult($contact)) {
-                       $network_name = ContactSelector::networkToName($contact['network'], $contact['url']);
-
                        // If no nick where given, extract it from the address
                        if (($contact['nick'] == "") || ($contact['name'] == $contact['nick'])) {
                                $contact['nick'] = api_get_nick($contact["url"]);
@@ -653,7 +652,7 @@ function api_get_user(App $a, $contact_id = null)
                                'id_str' => (string) $contact["id"],
                                'name' => $contact["name"],
                                'screen_name' => (($contact['nick']) ? $contact['nick'] : $contact['name']),
-                               'location' => ($contact["location"] != "") ? $contact["location"] : $network_name,
+                               'location' => ($contact["location"] != "") ? $contact["location"] : ContactSelector::networkToName($contact['network'], $contact['url']),
                                'description' => $contact["about"],
                                'profile_image_url' => $contact["micro"],
                                'profile_image_url_https' => $contact["micro"],
@@ -711,8 +710,6 @@ function api_get_user(App $a, $contact_id = null)
                $uinfo[0]['nick'] = api_get_nick($uinfo[0]["url"]);
        }
 
-       $network_name = ContactSelector::networkToName($uinfo[0]['network'], $uinfo[0]['url']);
-
        $pcontact_id  = Contact::getIdForURL($uinfo[0]['url'], 0, true);
 
        if (!empty($profile['about'])) {
@@ -726,7 +723,7 @@ function api_get_user(App $a, $contact_id = null)
        } elseif (!empty($uinfo[0]["location"])) {
                $location = $uinfo[0]["location"];
        } else {
-               $location = $network_name;
+               $location = ContactSelector::networkToName($uinfo[0]['network'], $uinfo[0]['url']);
        }
 
        $ret = [
@@ -966,7 +963,7 @@ function api_format_data($root_element, $type, $data)
 function api_account_verify_credentials($type)
 {
 
-       $a = get_app();
+       $a = \get_app();
 
        if (api_user() === false) {
                throw new ForbiddenException();
@@ -978,7 +975,7 @@ function api_account_verify_credentials($type)
        unset($_REQUEST["screen_name"]);
        unset($_GET["screen_name"]);
 
-       $skip_status = (x($_REQUEST, 'skip_status')?$_REQUEST['skip_status'] : false);
+       $skip_status = defaults($_REQUEST, 'skip_status', false);
 
        $user_info = api_get_user($a);
 
@@ -1012,10 +1009,10 @@ api_register_func('api/account/verify_credentials', 'api_account_verify_credenti
  */
 function requestdata($k)
 {
-       if (x($_POST, $k)) {
+       if (!empty($_POST[$k])) {
                return $_POST[$k];
        }
-       if (x($_GET, $k)) {
+       if (!empty($_GET[$k])) {
                return $_GET[$k];
        }
        return null;
@@ -1030,10 +1027,10 @@ function requestdata($k)
  */
 function api_statuses_mediap($type)
 {
-       $a = get_app();
+       $a = \get_app();
 
        if (api_user() === false) {
-               logger('api_statuses_update: no user');
+               Logger::log('api_statuses_update: no user');
                throw new ForbiddenException();
        }
        $user_info = api_get_user($a);
@@ -1045,7 +1042,7 @@ function api_statuses_mediap($type)
        //$txt = urldecode(requestdata('status'));
 
        if ((strpos($txt, '<') !== false) || (strpos($txt, '>') !== false)) {
-               $txt = html2bb_video($txt);
+               $txt = HTML::toBBCodeVideo($txt);
                $config = HTMLPurifier_Config::createDefault();
                $config->set('Cache.DefinitionImpl', null);
                $purifier = new HTMLPurifier($config);
@@ -1078,10 +1075,10 @@ api_register_func('api/statuses/mediap', 'api_statuses_mediap', true, API_METHOD
  */
 function api_statuses_update($type)
 {
-       $a = get_app();
+       $a = \get_app();
 
        if (api_user() === false) {
-               logger('api_statuses_update: no user');
+               Logger::log('api_statuses_update: no user');
                throw new ForbiddenException();
        }
 
@@ -1091,7 +1088,7 @@ function api_statuses_update($type)
        if (requestdata('htmlstatus')) {
                $txt = requestdata('htmlstatus');
                if ((strpos($txt, '<') !== false) || (strpos($txt, '>') !== false)) {
-                       $txt = html2bb_video($txt);
+                       $txt = HTML::toBBCodeVideo($txt);
 
                        $config = HTMLPurifier_Config::createDefault();
                        $config->set('Cache.DefinitionImpl', null);
@@ -1135,7 +1132,7 @@ function api_statuses_update($type)
                        $posts_day = DBA::count('thread', $condition);
 
                        if ($posts_day > $throttle_day) {
-                               logger('Daily posting limit reached for user '.api_user(), LOGGER_DEBUG);
+                               Logger::log('Daily posting limit reached for user '.api_user(), Logger::DEBUG);
                                // die(api_error($type, L10n::t("Daily posting limit of %d posts reached. The post was rejected.", $throttle_day));
                                throw new TooManyRequestsException(L10n::tt("Daily posting limit of %d post reached. The post was rejected.", "Daily posting limit of %d posts reached. The post was rejected.", $throttle_day));
                        }
@@ -1149,7 +1146,7 @@ function api_statuses_update($type)
                        $posts_week = DBA::count('thread', $condition);
 
                        if ($posts_week > $throttle_week) {
-                               logger('Weekly posting limit reached for user '.api_user(), LOGGER_DEBUG);
+                               Logger::log('Weekly posting limit reached for user '.api_user(), Logger::DEBUG);
                                // die(api_error($type, L10n::t("Weekly posting limit of %d posts reached. The post was rejected.", $throttle_week)));
                                throw new TooManyRequestsException(L10n::tt("Weekly posting limit of %d post reached. The post was rejected.", "Weekly posting limit of %d posts reached. The post was rejected.", $throttle_week));
                        }
@@ -1163,14 +1160,14 @@ function api_statuses_update($type)
                        $posts_month = DBA::count('thread', $condition);
 
                        if ($posts_month > $throttle_month) {
-                               logger('Monthly posting limit reached for user '.api_user(), LOGGER_DEBUG);
+                               Logger::log('Monthly posting limit reached for user '.api_user(), Logger::DEBUG);
                                // die(api_error($type, L10n::t("Monthly posting limit of %d posts reached. The post was rejected.", $throttle_month));
                                throw new TooManyRequestsException(L10n::t("Monthly posting limit of %d post reached. The post was rejected.", "Monthly posting limit of %d posts reached. The post was rejected.", $throttle_month));
                        }
                }
        }
 
-       if (x($_FILES, 'media')) {
+       if (!empty($_FILES['media'])) {
                // upload the image if we have one
                $picture = wall_upload_post($a, false);
                if (is_array($picture)) {
@@ -1197,7 +1194,7 @@ function api_statuses_update($type)
 
        $_REQUEST['api_source'] = true;
 
-       if (!x($_REQUEST, "source")) {
+       if (empty($_REQUEST['source'])) {
                $_REQUEST["source"] = api_source();
        }
 
@@ -1220,16 +1217,16 @@ api_register_func('api/statuses/update_with_media', 'api_statuses_update', true,
  */
 function api_media_upload()
 {
-       $a = get_app();
+       $a = \get_app();
 
        if (api_user() === false) {
-               logger('no user');
+               Logger::log('no user');
                throw new ForbiddenException();
        }
 
        api_get_user($a);
 
-       if (!x($_FILES, 'media')) {
+       if (empty($_FILES['media'])) {
                // Output error
                throw new BadRequestException("No media.");
        }
@@ -1248,7 +1245,7 @@ function api_media_upload()
                                        "h" => $media["height"],
                                        "image_type" => $media["type"]];
 
-       logger("Media uploaded: " . print_r($returndata, true), LOGGER_DEBUG);
+       Logger::log("Media uploaded: " . print_r($returndata, true), Logger::DEBUG);
 
        return ["media" => $returndata];
 }
@@ -1264,11 +1261,11 @@ api_register_func('api/media/upload', 'api_media_upload', true, API_METHOD_POST)
  */
 function api_status_show($type, $item_id = 0)
 {
-       $a = get_app();
+       $a = \get_app();
 
        $user_info = api_get_user($a);
 
-       logger('api_status_show: user_info: '.print_r($user_info, true), LOGGER_DEBUG);
+       Logger::log('api_status_show: user_info: '.print_r($user_info, true), Logger::DEBUG);
 
        if ($type == "raw") {
                $privacy_sql = "AND NOT `private`";
@@ -1344,7 +1341,7 @@ function api_status_show($type, $item_id = 0)
                unset($status_info["user"]["uid"]);
                unset($status_info["user"]["self"]);
 
-               logger('status_info: '.print_r($status_info, true), LOGGER_DEBUG);
+               Logger::log('status_info: '.print_r($status_info, true), Logger::DEBUG);
 
                if ($type == "raw") {
                        return $status_info;
@@ -1363,7 +1360,7 @@ function api_status_show($type, $item_id = 0)
  */
 function api_users_show($type)
 {
-       $a = get_app();
+       $a = \get_app();
 
        $user_info = api_get_user($a);
 
@@ -1439,11 +1436,11 @@ api_register_func('api/externalprofile/show', 'api_users_show');
  */
 function api_users_search($type)
 {
-       $a = get_app();
+       $a = \get_app();
 
        $userlist = [];
 
-       if (x($_GET, 'q')) {
+       if (!empty($_GET['q'])) {
                $r = q("SELECT id FROM `contact` WHERE `uid` = 0 AND `name` = '%s'", DBA::escape($_GET["q"]));
 
                if (!DBA::isResult($r)) {
@@ -1519,7 +1516,7 @@ api_register_func('api/users/lookup', 'api_users_lookup', true);
  */
 function api_search($type)
 {
-       $a = get_app();
+       $a = \get_app();
        $user_info = api_get_user($a);
 
        if (api_user() === false || $user_info === false) {
@@ -1528,21 +1525,21 @@ function api_search($type)
 
        $data = [];
 
-       if (!x($_REQUEST, 'q')) {
+       if (empty($_REQUEST['q'])) {
                throw new BadRequestException("q parameter is required.");
        }
 
-       if (x($_REQUEST, 'rpp')) {
+       if (!empty($_REQUEST['rpp'])) {
                $count = $_REQUEST['rpp'];
-       } elseif (x($_REQUEST, 'count')) {
+       } elseif (!empty($_REQUEST['count'])) {
                $count = $_REQUEST['count'];
        } else {
                $count = 15;
        }
 
-       $since_id = (x($_REQUEST, 'since_id') ? $_REQUEST['since_id'] : 0);
-       $max_id = (x($_REQUEST, 'max_id') ? $_REQUEST['max_id'] : 0);
-       $page = (x($_REQUEST, 'page') ? $_REQUEST['page'] - 1 : 0);
+       $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;
 
@@ -1580,7 +1577,7 @@ api_register_func('api/search', 'api_search', true);
  */
 function api_statuses_home_timeline($type)
 {
-       $a = get_app();
+       $a = \get_app();
        $user_info = api_get_user($a);
 
        if (api_user() === false || $user_info === false) {
@@ -1596,16 +1593,15 @@ function api_statuses_home_timeline($type)
        // get last network messages
 
        // params
-       $count = (x($_REQUEST, 'count') ? $_REQUEST['count'] : 20);
-       $page = (x($_REQUEST, 'page') ? $_REQUEST['page'] - 1 : 0);
+       $count = defaults($_REQUEST, 'count', 20);
+       $page = (!empty($_REQUEST['page']) ? $_REQUEST['page'] - 1 : 0);
        if ($page < 0) {
                $page = 0;
        }
-       $since_id = (x($_REQUEST, 'since_id') ? $_REQUEST['since_id'] : 0);
-       $max_id = (x($_REQUEST, 'max_id') ? $_REQUEST['max_id'] : 0);
-       //$since_id = 0;//$since_id = (x($_REQUEST, 'since_id')?$_REQUEST['since_id'] : 0);
-       $exclude_replies = (x($_REQUEST, 'exclude_replies') ? 1 : 0);
-       $conversation_id = (x($_REQUEST, 'conversation_id') ? $_REQUEST['conversation_id'] : 0);
+       $since_id = defaults($_REQUEST, 'since_id', 0);
+       $max_id = defaults($_REQUEST, 'max_id', 0);
+       $exclude_replies = !empty($_REQUEST['exclude_replies']);
+       $conversation_id = defaults($_REQUEST, 'conversation_id', 0);
 
        $start = $page * $count;
 
@@ -1616,7 +1612,7 @@ function api_statuses_home_timeline($type)
                $condition[0] .= " AND `item`.`id` <= ?";
                $condition[] = $max_id;
        }
-       if ($exclude_replies > 0) {
+       if ($exclude_replies) {
                $condition[0] .= ' AND `item`.`parent` = `item`.`id`';
        }
        if ($conversation_id > 0) {
@@ -1647,6 +1643,7 @@ function api_statuses_home_timeline($type)
        $data = ['status' => $ret];
        switch ($type) {
                case "atom":
+                       break;
                case "rss":
                        $data = api_rss_extra($a, $data, $user_info);
                        break;
@@ -1668,7 +1665,7 @@ api_register_func('api/statuses/friends_timeline', 'api_statuses_home_timeline',
  */
 function api_statuses_public_timeline($type)
 {
-       $a = get_app();
+       $a = \get_app();
        $user_info = api_get_user($a);
 
        if (api_user() === false || $user_info === false) {
@@ -1678,19 +1675,17 @@ function api_statuses_public_timeline($type)
        // get last network messages
 
        // params
-       $count = (x($_REQUEST, 'count') ? $_REQUEST['count'] : 20);
-       $page = (x($_REQUEST, 'page') ? $_REQUEST['page'] -1 : 0);
+       $count = defaults($_REQUEST, 'count', 20);
+       $page = (!empty($_REQUEST['page']) ? $_REQUEST['page'] -1 : 0);
        if ($page < 0) {
                $page = 0;
        }
-       $since_id = (x($_REQUEST, 'since_id') ? $_REQUEST['since_id'] : 0);
-       $max_id = (x($_REQUEST, 'max_id') ? $_REQUEST['max_id'] : 0);
-       //$since_id = 0;//$since_id = (x($_REQUEST, 'since_id')?$_REQUEST['since_id'] : 0);
-       $exclude_replies = (x($_REQUEST, 'exclude_replies') ? 1 : 0);
-       $conversation_id = (x($_REQUEST, 'conversation_id') ? $_REQUEST['conversation_id'] : 0);
+       $since_id = defaults($_REQUEST, 'since_id', 0);
+       $max_id = defaults($_REQUEST, 'max_id', 0);
+       $exclude_replies = (!empty($_REQUEST['exclude_replies']) ? 1 : 0);
+       $conversation_id = defaults($_REQUEST, 'conversation_id', 0);
 
        $start = $page * $count;
-       $sql_extra = '';
 
        if ($exclude_replies && !$conversation_id) {
                $condition = ["`gravity` IN (?, ?) AND `iid` > ? AND NOT `private` AND `wall` AND NOT `user`.`hidewall`",
@@ -1729,6 +1724,7 @@ function api_statuses_public_timeline($type)
        $data = ['status' => $ret];
        switch ($type) {
                case "atom":
+                       break;
                case "rss":
                        $data = api_rss_extra($a, $data, $user_info);
                        break;
@@ -1751,19 +1747,19 @@ api_register_func('api/statuses/public_timeline', 'api_statuses_public_timeline'
  */
 function api_statuses_networkpublic_timeline($type)
 {
-       $a = get_app();
+       $a = \get_app();
        $user_info = api_get_user($a);
 
        if (api_user() === false || $user_info === false) {
                throw new ForbiddenException();
        }
 
-       $since_id        = x($_REQUEST, 'since_id')        ? $_REQUEST['since_id']        : 0;
-       $max_id          = x($_REQUEST, 'max_id')          ? $_REQUEST['max_id']          : 0;
+       $since_id        = defaults($_REQUEST, 'since_id', 0);
+       $max_id          = defaults($_REQUEST, 'max_id', 0);
 
        // pagination
-       $count = x($_REQUEST, 'count') ? $_REQUEST['count']   : 20;
-       $page  = x($_REQUEST, 'page')  ? $_REQUEST['page']    : 1;
+       $count = defaults($_REQUEST, 'count', 20);
+       $page  = defaults($_REQUEST, 'page', 1);
        if ($page < 1) {
                $page = 1;
        }
@@ -1785,6 +1781,7 @@ function api_statuses_networkpublic_timeline($type)
        $data = ['status' => $ret];
        switch ($type) {
                case "atom":
+                       break;
                case "rss":
                        $data = api_rss_extra($a, $data, $user_info);
                        break;
@@ -1805,7 +1802,7 @@ api_register_func('api/statuses/networkpublic_timeline', 'api_statuses_networkpu
  */
 function api_statuses_show($type)
 {
-       $a = get_app();
+       $a = \get_app();
        $user_info = api_get_user($a);
 
        if (api_user() === false || $user_info === false) {
@@ -1824,7 +1821,7 @@ function api_statuses_show($type)
                $id = intval(defaults($a->argv, 4, 0));
        }
 
-       logger('API: api_statuses_show: ' . $id);
+       Logger::log('API: api_statuses_show: ' . $id);
 
        $conversation = !empty($_REQUEST['conversation']);
 
@@ -1878,7 +1875,7 @@ api_register_func('api/statuses/show', 'api_statuses_show', true);
  */
 function api_conversation_show($type)
 {
-       $a = get_app();
+       $a = \get_app();
        $user_info = api_get_user($a);
 
        if (api_user() === false || $user_info === false) {
@@ -1906,7 +1903,7 @@ function api_conversation_show($type)
                $id = intval(defaults($a->argv, 4, 0));
        }
 
-       logger('API: api_conversation_show: '.$id);
+       Logger::log('API: api_conversation_show: '.$id);
 
        // try to fetch the item for the local user - or the public item, if there is no local one
        $item = Item::selectFirst(['parent-uri'], ['id' => $id]);
@@ -1957,7 +1954,7 @@ function api_statuses_repeat($type)
 {
        global $called_api;
 
-       $a = get_app();
+       $a = \get_app();
 
        if (api_user() === false) {
                throw new ForbiddenException();
@@ -1977,7 +1974,7 @@ function api_statuses_repeat($type)
                $id = intval(defaults($a->argv, 4, 0));
        }
 
-       logger('API: api_statuses_repeat: '.$id);
+       Logger::log('API: api_statuses_repeat: '.$id);
 
        $fields = ['body', 'author-name', 'author-link', 'author-avatar', 'guid', 'created', 'plink'];
        $item = Item::selectFirst($fields, ['id' => $id, 'private' => false]);
@@ -1996,7 +1993,7 @@ function api_statuses_repeat($type)
                $_REQUEST['profile_uid'] = api_user();
                $_REQUEST['api_source'] = true;
 
-               if (!x($_REQUEST, "source")) {
+               if (empty($_REQUEST['source'])) {
                        $_REQUEST["source"] = api_source();
                }
 
@@ -2022,7 +2019,7 @@ api_register_func('api/statuses/retweet', 'api_statuses_repeat', true, API_METHO
  */
 function api_statuses_destroy($type)
 {
-       $a = get_app();
+       $a = \get_app();
 
        if (api_user() === false) {
                throw new ForbiddenException();
@@ -2042,7 +2039,7 @@ function api_statuses_destroy($type)
                $id = intval(defaults($a->argv, 4, 0));
        }
 
-       logger('API: api_statuses_destroy: '.$id);
+       Logger::log('API: api_statuses_destroy: '.$id);
 
        $ret = api_statuses_show($type);
 
@@ -2063,7 +2060,7 @@ api_register_func('api/statuses/destroy', 'api_statuses_destroy', true, API_METH
  */
 function api_statuses_mentions($type)
 {
-       $a = get_app();
+       $a = \get_app();
        $user_info = api_get_user($a);
 
        if (api_user() === false || $user_info === false) {
@@ -2106,6 +2103,7 @@ function api_statuses_mentions($type)
        $data = ['status' => $ret];
        switch ($type) {
                case "atom":
+                       break;
                case "rss":
                        $data = api_rss_extra($a, $data, $user_info);
                        break;
@@ -2130,28 +2128,28 @@ api_register_func('api/statuses/replies', 'api_statuses_mentions', true);
  */
 function api_statuses_user_timeline($type)
 {
-       $a = get_app();
+       $a = \get_app();
        $user_info = api_get_user($a);
 
        if (api_user() === false || $user_info === false) {
                throw new ForbiddenException();
        }
 
-       logger(
+       Logger::log(
                "api_statuses_user_timeline: api_user: ". api_user() .
                        "\nuser_info: ".print_r($user_info, true) .
                        "\n_REQUEST:  ".print_r($_REQUEST, true),
-               LOGGER_DEBUG
+               Logger::DEBUG
        );
 
-       $since_id        = x($_REQUEST, 'since_id')        ? $_REQUEST['since_id']        : 0;
-       $max_id          = x($_REQUEST, 'max_id')          ? $_REQUEST['max_id']          : 0;
-       $exclude_replies = x($_REQUEST, 'exclude_replies') ? 1                            : 0;
-       $conversation_id = x($_REQUEST, 'conversation_id') ? $_REQUEST['conversation_id'] : 0;
+       $since_id        = defaults($_REQUEST, 'since_id', 0);
+       $max_id          = defaults($_REQUEST, 'max_id', 0);
+       $exclude_replies = !empty($_REQUEST['exclude_replies']);
+       $conversation_id = defaults($_REQUEST, 'conversation_id', 0);
 
        // pagination
-       $count = x($_REQUEST, 'count') ? $_REQUEST['count'] : 20;
-       $page  = x($_REQUEST, 'page')  ? $_REQUEST['page']  : 1;
+       $count = defaults($_REQUEST, 'count', 20);
+       $page  = defaults($_REQUEST, 'page', 1);
        if ($page < 1) {
                $page = 1;
        }
@@ -2164,7 +2162,7 @@ function api_statuses_user_timeline($type)
                $condition[0] .= ' AND `item`.`wall` ';
        }
 
-       if ($exclude_replies > 0) {
+       if ($exclude_replies) {
                $condition[0] .= ' AND `item`.`parent` = `item`.`id`';
        }
 
@@ -2186,6 +2184,7 @@ function api_statuses_user_timeline($type)
        $data = ['status' => $ret];
        switch ($type) {
                case "atom":
+                       break;
                case "rss":
                        $data = api_rss_extra($a, $data, $user_info);
                        break;
@@ -2207,7 +2206,7 @@ api_register_func('api/statuses/user_timeline', 'api_statuses_user_timeline', tr
  */
 function api_favorites_create_destroy($type)
 {
-       $a = get_app();
+       $a = \get_app();
 
        if (api_user() === false) {
                throw new ForbiddenException();
@@ -2261,8 +2260,10 @@ function api_favorites_create_destroy($type)
        $data = ['status' => $ret];
        switch ($type) {
                case "atom":
+                       break;
                case "rss":
                        $data = api_rss_extra($a, $data, $user_info);
+                       break;
        }
 
        return api_format_data("status", $type, $data);
@@ -2283,7 +2284,7 @@ function api_favorites($type)
 {
        global $called_api;
 
-       $a = get_app();
+       $a = \get_app();
        $user_info = api_get_user($a);
 
        if (api_user() === false || $user_info === false) {
@@ -2294,16 +2295,16 @@ function api_favorites($type)
 
        // in friendica starred item are private
        // return favorites only for self
-       logger('api_favorites: self:' . $user_info['self']);
+       Logger::log('api_favorites: self:' . $user_info['self']);
 
        if ($user_info['self'] == 0) {
                $ret = [];
        } else {
                // params
-               $since_id = (x($_REQUEST, 'since_id') ? $_REQUEST['since_id'] : 0);
-               $max_id = (x($_REQUEST, 'max_id') ? $_REQUEST['max_id'] : 0);
-               $count = (x($_GET, 'count') ? $_GET['count'] : 20);
-               $page = (x($_REQUEST, 'page') ? $_REQUEST['page'] -1 : 0);
+               $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;
                }
@@ -2328,8 +2329,10 @@ function api_favorites($type)
        $data = ['status' => $ret];
        switch ($type) {
                case "atom":
+                       break;
                case "rss":
                        $data = api_rss_extra($a, $data, $user_info);
+                       break;
        }
 
        return api_format_data("statuses", $type, $data);
@@ -2379,7 +2382,7 @@ function api_format_messages($item, $recipient, $sender)
        }
 
        //don't send title to regular StatusNET requests to avoid confusing these apps
-       if (x($_GET, 'getText')) {
+       if (!empty($_GET['getText'])) {
                $ret['title'] = $item['title'];
                if ($_GET['getText'] == 'html') {
                        $ret['text'] = BBCode::convert($item['body'], false);
@@ -2389,7 +2392,7 @@ function api_format_messages($item, $recipient, $sender)
        } else {
                $ret['text'] = $item['title'] . "\n" . HTML::toPlaintext(BBCode::convert(api_clean_plain_items($item['body']), false, 2, true), 0);
        }
-       if (x($_GET, 'getUserObjects') && $_GET['getUserObjects'] == 'false') {
+       if (!empty($_GET['getUserObjects']) && $_GET['getUserObjects'] == 'false') {
                unset($ret['sender']);
                unset($ret['recipient']);
        }
@@ -2519,7 +2522,7 @@ function api_get_attachments(&$body)
  */
 function api_get_entitities(&$text, $bbcode)
 {
-       $include_entities = strtolower(x($_REQUEST, 'include_entities') ? $_REQUEST['include_entities'] : "false");
+       $include_entities = strtolower(defaults($_REQUEST, 'include_entities', "false"));
 
        if ($include_entities != "true") {
                preg_match_all("/\[img](.*?)\[\/img\]/ism", $bbcode, $images);
@@ -2661,7 +2664,7 @@ function api_get_entitities(&$text, $bbcode)
                                                        "id" => $start+1,
                                                        "id_str" => (string)$start+1,
                                                        "indices" => [$start, $start+strlen($url)],
-                                                       "media_url" => normalise_link($media_url),
+                                                       "media_url" => Strings::normaliseLink($media_url),
                                                        "media_url_https" => $media_url,
                                                        "url" => $url,
                                                        "display_url" => $display_url,
@@ -2734,7 +2737,7 @@ function api_contactlink_to_array($txt)
  */
 function api_format_items_activities($item, $type = "json")
 {
-       $a = get_app();
+       $a = \get_app();
 
        $activities = [
                'like' => [],
@@ -2856,7 +2859,7 @@ function api_format_items_profiles($profile_row)
  */
 function api_format_items($r, $user_info, $filter_user = false, $type = "json")
 {
-       $a = get_app();
+       $a = \get_app();
 
        $ret = [];
 
@@ -2921,6 +2924,7 @@ function api_format_items($r, $user_info, $filter_user = false, $type = "json")
                        $retweeted_item = api_share_as_retweet($item);
                        if ($retweeted_item !== false) {
                                $retweeted_status = $status;
+                               $status['user'] = $status['friendica_owner'];
                                try {
                                        $retweeted_status["user"] = api_get_user($a, $retweeted_item["author-id"]);
                                } catch (BadRequestException $e) {
@@ -3045,7 +3049,7 @@ api_register_func('api/lists/subscriptions', 'api_lists_list', true);
  */
 function api_lists_ownerships($type)
 {
-       $a = get_app();
+       $a = \get_app();
 
        if (api_user() === false) {
                throw new ForbiddenException();
@@ -3089,7 +3093,7 @@ api_register_func('api/lists/ownerships', 'api_lists_ownerships', true);
  */
 function api_lists_statuses($type)
 {
-       $a = get_app();
+       $a = \get_app();
 
        $user_info = api_get_user($a);
        if (api_user() === false || $user_info === false) {
@@ -3107,15 +3111,15 @@ function api_lists_statuses($type)
        }
 
        // params
-       $count = (x($_REQUEST, 'count') ? $_REQUEST['count'] : 20);
-       $page = (x($_REQUEST, 'page') ? $_REQUEST['page'] - 1 : 0);
+       $count = defaults($_REQUEST, 'count', 20);
+       $page = (!empty($_REQUEST['page']) ? $_REQUEST['page'] - 1 : 0);
        if ($page < 0) {
                $page = 0;
        }
-       $since_id = (x($_REQUEST, 'since_id') ? $_REQUEST['since_id'] : 0);
-       $max_id = (x($_REQUEST, 'max_id') ? $_REQUEST['max_id'] : 0);
-       $exclude_replies = (x($_REQUEST, 'exclude_replies') ? 1 : 0);
-       $conversation_id = (x($_REQUEST, 'conversation_id') ? $_REQUEST['conversation_id'] : 0);
+       $since_id = defaults($_REQUEST, 'since_id', 0);
+       $max_id = defaults($_REQUEST, 'max_id', 0);
+       $exclude_replies = (!empty($_REQUEST['exclude_replies']) ? 1 : 0);
+       $conversation_id = defaults($_REQUEST, 'conversation_id', 0);
 
        $start = $page * $count;
 
@@ -3142,6 +3146,7 @@ function api_lists_statuses($type)
        $data = ['status' => $items];
        switch ($type) {
                case "atom":
+                       break;
                case "rss":
                        $data = api_rss_extra($a, $data, $user_info);
                        break;
@@ -3165,15 +3170,15 @@ api_register_func('api/lists/statuses', 'api_lists_statuses', true);
  */
 function api_statuses_f($qtype)
 {
-       $a = get_app();
+       $a = \get_app();
 
        if (api_user() === false) {
                throw new ForbiddenException();
        }
 
        // pagination
-       $count = x($_GET, 'count') ? $_GET['count'] : 20;
-       $page = x($_GET, 'page') ? $_GET['page'] : 1;
+       $count = defaults($_GET, 'count', 20);
+       $page = defaults($_GET, 'page', 1);
        if ($page < 1) {
                $page = 1;
        }
@@ -3181,7 +3186,7 @@ function api_statuses_f($qtype)
 
        $user_info = api_get_user($a);
 
-       if (x($_GET, 'cursor') && $_GET['cursor'] == 'undefined') {
+       if (!empty($_GET['cursor']) && $_GET['cursor'] == 'undefined') {
                /* this is to stop Hotot to load friends multiple times
                *  I'm not sure if I'm missing return something or
                *  is a bug in hotot. Workaround, meantime
@@ -3341,7 +3346,7 @@ api_register_func('api/friendships/incoming', 'api_friendships_incoming', true);
  */
 function api_statusnet_config($type)
 {
-       $a = get_app();
+       $a = \get_app();
 
        $name      = Config::get('config', 'sitename');
        $server    = $a->getHostName();
@@ -3405,7 +3410,7 @@ function api_ff_ids($type)
                throw new ForbiddenException();
        }
 
-       $a = get_app();
+       $a = \get_app();
 
        api_get_user($a);
 
@@ -3473,7 +3478,7 @@ api_register_func('api/followers/ids', 'api_followers_ids', true);
  */
 function api_direct_messages_new($type)
 {
-       $a = get_app();
+       $a = \get_app();
 
        if (api_user() === false) {
                throw new ForbiddenException();
@@ -3509,7 +3514,7 @@ function api_direct_messages_new($type)
 
        $replyto = '';
        $sub     = '';
-       if (x($_REQUEST, 'replyto')) {
+       if (!empty($_REQUEST['replyto'])) {
                $r = q(
                        'SELECT `parent-uri`, `title` FROM `mail` WHERE `uid`=%d AND `id`=%d',
                        intval(api_user()),
@@ -3518,7 +3523,7 @@ function api_direct_messages_new($type)
                $replyto = $r[0]['parent-uri'];
                $sub     = $r[0]['title'];
        } else {
-               if (x($_REQUEST, 'title')) {
+               if (!empty($_REQUEST['title'])) {
                        $sub = $_REQUEST['title'];
                } else {
                        $sub = ((strlen($_POST['text'])>10) ? substr($_POST['text'], 0, 10)."...":$_POST['text']);
@@ -3538,8 +3543,10 @@ function api_direct_messages_new($type)
 
        switch ($type) {
                case "atom":
+                       break;
                case "rss":
                        $data = api_rss_extra($a, $data, $sender);
+                       break;
        }
 
        return api_format_data("direct-messages", $type, $data);
@@ -3559,7 +3566,7 @@ api_register_func('api/direct_messages/new', 'api_direct_messages_new', true, AP
  */
 function api_direct_messages_destroy($type)
 {
-       $a = get_app();
+       $a = \get_app();
 
        if (api_user() === false) {
                throw new ForbiddenException();
@@ -3568,10 +3575,10 @@ function api_direct_messages_destroy($type)
        // params
        $user_info = api_get_user($a);
        //required
-       $id = (x($_REQUEST, 'id') ? $_REQUEST['id'] : 0);
+       $id = defaults($_REQUEST, 'id', 0);
        // optional
-       $parenturi = (x($_REQUEST, 'friendica_parenturi') ? $_REQUEST['friendica_parenturi'] : "");
-       $verbose = (x($_GET, 'friendica_verbose') ? strtolower($_GET['friendica_verbose']) : "false");
+       $parenturi = defaults($_REQUEST, 'friendica_parenturi', "");
+       $verbose = (!empty($_GET['friendica_verbose']) ? strtolower($_GET['friendica_verbose']) : "false");
        /// @todo optional parameter 'include_entities' from Twitter API not yet implemented
 
        $uid = $user_info['uid'];
@@ -3632,7 +3639,7 @@ api_register_func('api/direct_messages/destroy', 'api_direct_messages_destroy',
 /**
  * Unfollow Contact
  *
- * @brief unfollow contact 
+ * @brief unfollow contact
  *
  * @param string $type Known types are 'atom', 'rss', 'xml' and 'json'
  * @return string|array
@@ -3649,7 +3656,7 @@ function api_friendships_destroy($type)
        $contact_id = defaults($_REQUEST, 'user_id');
 
        if (empty($contact_id)) {
-               logger("No user_id specified", LOGGER_DEBUG);
+               Logger::log("No user_id specified", Logger::DEBUG);
                throw new BadRequestException("no user_id specified");
        }
 
@@ -3657,24 +3664,24 @@ function api_friendships_destroy($type)
        $contact = DBA::selectFirst('contact', ['url'], ['id' => $contact_id, 'uid' => 0, 'self' => false]);
 
        if(!DBA::isResult($contact)) {
-               logger("No contact found for ID" . $contact_id, LOGGER_DEBUG);
+               Logger::log("No contact found for ID" . $contact_id, Logger::DEBUG);
                throw new NotFoundException("no contact found to given ID");
        }
 
        $url = $contact["url"];
 
        $condition = ["`uid` = ? AND (`rel` = ? OR `rel` = ?) AND (`nurl` = ? OR `alias` = ? OR `alias` = ?)",
-                       $uid, Contact::SHARING, Contact::FRIEND, normalise_link($url),
-                       normalise_link($url), $url];
+                       $uid, Contact::SHARING, Contact::FRIEND, Strings::normaliseLink($url),
+                       Strings::normaliseLink($url), $url];
        $contact = DBA::selectFirst('contact', [], $condition);
 
        if (!DBA::isResult($contact)) {
-               logger("Not following Contact", LOGGER_DEBUG);
+               Logger::log("Not following Contact", Logger::DEBUG);
                throw new NotFoundException("Not following Contact");
        }
 
        if (!in_array($contact['network'], Protocol::NATIVE_SUPPORT)) {
-               logger("Not supported", LOGGER_DEBUG);
+               Logger::log("Not supported", Logger::DEBUG);
                throw new ExpectationFailedException("Not supported");
        }
 
@@ -3685,7 +3692,7 @@ function api_friendships_destroy($type)
                Contact::terminateFriendship($owner, $contact, $dissolve);
        }
        else {
-               logger("No owner found", LOGGER_DEBUG);
+               Logger::log("No owner found", Logger::DEBUG);
                throw new NotFoundException("Error Processing Request");
        }
 
@@ -3717,7 +3724,7 @@ api_register_func('api/friendships/destroy', 'api_friendships_destroy', true, AP
  */
 function api_direct_messages_box($type, $box, $verbose)
 {
-       $a = get_app();
+       $a = \get_app();
        if (api_user() === false) {
                throw new ForbiddenException();
        }
@@ -3789,9 +3796,9 @@ function api_direct_messages_box($type, $box, $verbose)
        foreach ($r as $item) {
                if ($box == "inbox" || $item['from-url'] != $profile_url) {
                        $recipient = $user_info;
-                       $sender = api_get_user($a, normalise_link($item['contact-url']));
+                       $sender = api_get_user($a, Strings::normaliseLink($item['contact-url']));
                } elseif ($box == "sentbox" || $item['from-url'] == $profile_url) {
-                       $recipient = api_get_user($a, normalise_link($item['contact-url']));
+                       $recipient = api_get_user($a, Strings::normaliseLink($item['contact-url']));
                        $sender = $user_info;
                }
 
@@ -3804,8 +3811,10 @@ function api_direct_messages_box($type, $box, $verbose)
        $data = ['direct_message' => $ret];
        switch ($type) {
                case "atom":
+                       break;
                case "rss":
                        $data = api_rss_extra($a, $data, $user_info);
+                       break;
        }
 
        return api_format_data("direct-messages", $type, $data);
@@ -3821,7 +3830,7 @@ function api_direct_messages_box($type, $box, $verbose)
  */
 function api_direct_messages_sentbox($type)
 {
-       $verbose = (x($_GET, 'friendica_verbose') ? strtolower($_GET['friendica_verbose']) : "false");
+       $verbose = !empty($_GET['friendica_verbose']) ? strtolower($_GET['friendica_verbose']) : "false";
        return api_direct_messages_box($type, "sentbox", $verbose);
 }
 
@@ -3835,7 +3844,7 @@ function api_direct_messages_sentbox($type)
  */
 function api_direct_messages_inbox($type)
 {
-       $verbose = (x($_GET, 'friendica_verbose') ? strtolower($_GET['friendica_verbose']) : "false");
+       $verbose = !empty($_GET['friendica_verbose']) ? strtolower($_GET['friendica_verbose']) : "false";
        return api_direct_messages_box($type, "inbox", $verbose);
 }
 
@@ -3847,7 +3856,7 @@ function api_direct_messages_inbox($type)
  */
 function api_direct_messages_all($type)
 {
-       $verbose = (x($_GET, 'friendica_verbose') ? strtolower($_GET['friendica_verbose']) : "false");
+       $verbose = !empty($_GET['friendica_verbose']) ? strtolower($_GET['friendica_verbose']) : "false";
        return api_direct_messages_box($type, "all", $verbose);
 }
 
@@ -3859,7 +3868,7 @@ function api_direct_messages_all($type)
  */
 function api_direct_messages_conversation($type)
 {
-       $verbose = (x($_GET, 'friendica_verbose') ? strtolower($_GET['friendica_verbose']) : "false");
+       $verbose = !empty($_GET['friendica_verbose']) ? strtolower($_GET['friendica_verbose']) : "false";
        return api_direct_messages_box($type, "conversation", $verbose);
 }
 
@@ -3923,7 +3932,7 @@ function api_fr_photoalbum_delete($type)
                throw new ForbiddenException();
        }
        // input params
-       $album = (x($_REQUEST, 'album') ? $_REQUEST['album'] : "");
+       $album = defaults($_REQUEST, 'album', "");
 
        // we do not allow calls without album string
        if ($album == "") {
@@ -3975,8 +3984,8 @@ function api_fr_photoalbum_update($type)
                throw new ForbiddenException();
        }
        // input params
-       $album = (x($_REQUEST, 'album') ? $_REQUEST['album'] : "");
-       $album_new = (x($_REQUEST, 'album_new') ? $_REQUEST['album_new'] : "");
+       $album = defaults($_REQUEST, 'album', "");
+       $album_new = defaults($_REQUEST, 'album_new', "");
 
        // we do not allow calls without album string
        if ($album == "") {
@@ -4060,15 +4069,15 @@ function api_fr_photo_create_update($type)
                throw new ForbiddenException();
        }
        // input params
-       $photo_id = (x($_REQUEST, 'photo_id') ? $_REQUEST['photo_id'] : null);
-       $desc = (x($_REQUEST, 'desc') ? $_REQUEST['desc'] : (array_key_exists('desc', $_REQUEST) ? "" : null)); // extra check necessary to distinguish between 'not provided' and 'empty string'
-       $album = (x($_REQUEST, 'album') ? $_REQUEST['album'] : null);
-       $album_new = (x($_REQUEST, 'album_new') ? $_REQUEST['album_new'] : null);
-       $allow_cid = (x($_REQUEST, 'allow_cid') ? $_REQUEST['allow_cid'] : (array_key_exists('allow_cid', $_REQUEST) ? " " : null));
-       $deny_cid = (x($_REQUEST, 'deny_cid') ? $_REQUEST['deny_cid'] : (array_key_exists('deny_cid', $_REQUEST) ? " " : null));
-       $allow_gid = (x($_REQUEST, 'allow_gid') ? $_REQUEST['allow_gid'] : (array_key_exists('allow_gid', $_REQUEST) ? " " : null));
-       $deny_gid = (x($_REQUEST, 'deny_gid') ? $_REQUEST['deny_gid'] : (array_key_exists('deny_gid', $_REQUEST) ? " " : null));
-       $visibility = (x($_REQUEST, 'visibility') ? (($_REQUEST['visibility'] == "true" || $_REQUEST['visibility'] == 1) ? true : false) : false);
+       $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));
+       $visibility = !empty($_REQUEST['visibility']) && $_REQUEST['visibility'] !== "false";
 
        // do several checks on input parameters
        // we do not allow calls without album string
@@ -4080,7 +4089,7 @@ function api_fr_photo_create_update($type)
                $mode = "create";
 
                // error if no media posted in create-mode
-               if (!x($_FILES, 'media')) {
+               if (empty($_FILES['media'])) {
                        // Output error
                        throw new BadRequestException("no media data submitted");
                }
@@ -4171,7 +4180,7 @@ function api_fr_photo_create_update($type)
                        $nothingtodo = true;
                }
 
-               if (x($_FILES, 'media')) {
+               if (!empty($_FILES['media'])) {
                        $nothingtodo = false;
                        $media = $_FILES['media'];
                        $data = save_media_to_database("photo", $media, $type, $album, $allow_cid, $deny_cid, $allow_gid, $deny_gid, $desc, 0, $visibility, $photo_id);
@@ -4207,7 +4216,7 @@ function api_fr_photo_delete($type)
                throw new ForbiddenException();
        }
        // input params
-       $photo_id = (x($_REQUEST, 'photo_id') ? $_REQUEST['photo_id'] : null);
+       $photo_id = defaults($_REQUEST, 'photo_id', null);
 
        // do several checks on input parameters
        // we do not allow calls without photo id
@@ -4258,11 +4267,11 @@ function api_fr_photo_detail($type)
        if (api_user() === false) {
                throw new ForbiddenException();
        }
-       if (!x($_REQUEST, 'photo_id')) {
+       if (empty($_REQUEST['photo_id'])) {
                throw new BadRequestException("No photo id.");
        }
 
-       $scale = (x($_REQUEST, 'scale') ? intval($_REQUEST['scale']) : false);
+       $scale = (!empty($_REQUEST['scale']) ? intval($_REQUEST['scale']) : false);
        $photo_id = $_REQUEST['photo_id'];
 
        // prepare json/xml output with data from database for the requested photo
@@ -4291,7 +4300,7 @@ function api_account_update_profile_image($type)
        $profile_id = defaults($_REQUEST, 'profile_id', 0);
 
        // error if image data is missing
-       if (!x($_FILES, 'image')) {
+       if (empty($_FILES['image'])) {
                throw new BadRequestException("no media data submitted");
        }
 
@@ -4309,9 +4318,9 @@ function api_account_update_profile_image($type)
 
        // get mediadata from image or media (Twitter call api/account/update_profile_image provides image)
        $media = null;
-       if (x($_FILES, 'image')) {
+       if (!empty($_FILES['image'])) {
                $media = $_FILES['image'];
-       } elseif (x($_FILES, 'media')) {
+       } elseif (!empty($_FILES['media'])) {
                $media = $_FILES['media'];
        }
        // save new profile image
@@ -4344,7 +4353,7 @@ function api_account_update_profile_image($type)
        Contact::updateSelfFromUserID(api_user(), true);
 
        // Update global directory in background
-       $url = System::baseUrl() . '/profile/' . get_app()->user['nickname'];
+       $url = System::baseUrl() . '/profile/' . \get_app()->user['nickname'];
        if ($url && strlen(Config::get('system', 'directory'))) {
                Worker::add(PRIORITY_LOW, "Directory", $url);
        }
@@ -4485,10 +4494,10 @@ function save_media_to_database($mediatype, $media, $type, $album, $allow_cid, $
        if ($imagedata) {
                $filetype = $imagedata['mime'];
        }
-       logger(
+       Logger::log(
                "File upload src: " . $src . " - filename: " . $filename .
                " - size: " . $filesize . " - type: " . $filetype,
-               LOGGER_DEBUG
+               Logger::DEBUG
        );
 
        // check if there was a php upload error
@@ -4498,7 +4507,7 @@ function save_media_to_database($mediatype, $media, $type, $album, $allow_cid, $
        // check against max upload size within Friendica instance
        $maximagesize = Config::get('system', 'maximagesize');
        if ($maximagesize && ($filesize > $maximagesize)) {
-               $formattedBytes = formatBytes($maximagesize);
+               $formattedBytes = Strings::formatBytes($maximagesize);
                throw new InternalServerErrorException("image size exceeds Friendica config setting (uploaded size: $formattedBytes)");
        }
 
@@ -4520,7 +4529,7 @@ function save_media_to_database($mediatype, $media, $type, $album, $allow_cid, $
        }
        if ($max_length > 0) {
                $Image->scaleDown($max_length);
-               logger("File upload: Scaling picture to new size " . $max_length, LOGGER_DEBUG);
+               Logger::log("File upload: Scaling picture to new size " . $max_length, Logger::DEBUG);
        }
        $width = $Image->getWidth();
        $height = $Image->getHeight();
@@ -4530,17 +4539,17 @@ function save_media_to_database($mediatype, $media, $type, $album, $allow_cid, $
 
        if ($mediatype == "photo") {
                // upload normal image (scales 0, 1, 2)
-               logger("photo upload: starting new photo upload", LOGGER_DEBUG);
+               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);
                if (!$r) {
-                       logger("photo upload: image upload with scale 0 (original size) failed");
+                       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);
                        if (!$r) {
-                               logger("photo upload: image upload with scale 1 (640x640) failed");
+                               Logger::log("photo upload: image upload with scale 1 (640x640) failed");
                        }
                }
 
@@ -4548,19 +4557,19 @@ function save_media_to_database($mediatype, $media, $type, $album, $allow_cid, $
                        $Image->scaleDown(320);
                        $r = Photo::store($Image, local_user(), $visitor, $hash, $filename, $album, 2, 0, $allow_cid, $allow_gid, $deny_cid, $deny_gid, $desc);
                        if (!$r) {
-                               logger("photo upload: image upload with scale 2 (320x320) failed");
+                               Logger::log("photo upload: image upload with scale 2 (320x320) failed");
                        }
                }
-               logger("photo upload: new photo upload ended", LOGGER_DEBUG);
+               Logger::log("photo upload: new photo upload ended", Logger::DEBUG);
        } elseif ($mediatype == "profileimage") {
                // upload profile image (scales 4, 5, 6)
-               logger("photo upload: starting new profile image upload", LOGGER_DEBUG);
+               Logger::log("photo upload: starting new profile image upload", Logger::DEBUG);
 
-               if ($width > 175 || $height > 175) {
-                       $Image->scaleDown(175);
+               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);
                        if (!$r) {
-                               logger("photo upload: profile image upload with scale 4 (175x175) failed");
+                               Logger::log("photo upload: profile image upload with scale 4 (300x300) failed");
                        }
                }
 
@@ -4568,7 +4577,7 @@ function save_media_to_database($mediatype, $media, $type, $album, $allow_cid, $
                        $Image->scaleDown(80);
                        $r = Photo::store($Image, local_user(), $visitor, $hash, $filename, $album, 5, $profile, $allow_cid, $allow_gid, $deny_cid, $deny_gid, $desc);
                        if (!$r) {
-                               logger("photo upload: profile image upload with scale 5 (80x80) failed");
+                               Logger::log("photo upload: profile image upload with scale 5 (80x80) failed");
                        }
                }
 
@@ -4576,11 +4585,11 @@ function save_media_to_database($mediatype, $media, $type, $album, $allow_cid, $
                        $Image->scaleDown(48);
                        $r = Photo::store($Image, local_user(), $visitor, $hash, $filename, $album, 6, $profile, $allow_cid, $allow_gid, $deny_cid, $deny_gid, $desc);
                        if (!$r) {
-                               logger("photo upload: profile image upload with scale 6 (48x48) failed");
+                               Logger::log("photo upload: profile image upload with scale 6 (48x48) failed");
                        }
                }
                $Image->__destruct();
-               logger("photo upload: new profile image upload ended", LOGGER_DEBUG);
+               Logger::log("photo upload: new profile image upload ended", Logger::DEBUG);
        }
 
        if (isset($r) && $r) {
@@ -4659,7 +4668,7 @@ function post_photo_item($hash, $allow_cid, $deny_cid, $allow_gid, $deny_gid, $f
  */
 function prepare_photo_data($type, $scale, $photo_id)
 {
-       $a = get_app();
+       $a = \get_app();
        $user_info = api_get_user($a);
 
        if ($user_info === false) {
@@ -4771,14 +4780,14 @@ function prepare_photo_data($type, $scale, $photo_id)
  */
 function api_friendica_remoteauth()
 {
-       $url = (x($_GET, 'url') ? $_GET['url'] : '');
-       $c_url = (x($_GET, 'c_url') ? $_GET['c_url'] : '');
+       $url = defaults($_GET, 'url', '');
+       $c_url = defaults($_GET, 'c_url', '');
 
        if ($url === '' || $c_url === '') {
                throw new BadRequestException("Wrong parameters.");
        }
 
-       $c_url = normalise_link($c_url);
+       $c_url = Strings::normaliseLink($c_url);
 
        // traditional DFRN
 
@@ -4801,13 +4810,13 @@ function api_friendica_remoteauth()
                $dfrn_id = '0:' . $orig_id;
        }
 
-       $sec = random_string();
+       $sec = Strings::getRandomHex();
 
        $fields = ['uid' => api_user(), 'cid' => $cid, 'dfrn_id' => $dfrn_id,
                'sec' => $sec, 'expire' => time() + 45];
        DBA::insert('profile_check', $fields);
 
-       logger($contact['name'] . ' ' . $sec, LOGGER_DEBUG);
+       Logger::log($contact['name'] . ' ' . $sec, Logger::DEBUG);
        $dest = ($url ? '&destination_url=' . $url : '');
 
        System::externalRedirect(
@@ -4918,6 +4927,7 @@ function api_share_as_retweet(&$item)
        }
 
        $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;
@@ -4942,7 +4952,7 @@ function api_get_nick($profile)
 
        $r = q(
                "SELECT `nick` FROM `contact` WHERE `uid` = 0 AND `nurl` = '%s'",
-               DBA::escape(normalise_link($profile))
+               DBA::escape(Strings::normaliseLink($profile))
        );
 
        if (DBA::isResult($r)) {
@@ -4952,7 +4962,7 @@ function api_get_nick($profile)
        if (!$nick == "") {
                $r = q(
                        "SELECT `nick` FROM `contact` WHERE `uid` = 0 AND `nurl` = '%s'",
-                       DBA::escape(normalise_link($profile))
+                       DBA::escape(Strings::normaliseLink($profile))
                );
 
                if (DBA::isResult($r)) {
@@ -5055,7 +5065,7 @@ function api_in_reply_to($item)
                // https://github.com/friendica/friendica/issues/1010
                // This is a bugfix for that.
                if (intval($in_reply_to['status_id']) == intval($item['id'])) {
-                       logger('this message should never appear: id: '.$item['id'].' similar to reply-to: '.$in_reply_to['status_id'], LOGGER_DEBUG);
+                       Logger::log('this message should never appear: id: '.$item['id'].' similar to reply-to: '.$in_reply_to['status_id'], Logger::DEBUG);
                        $in_reply_to['status_id'] = null;
                        $in_reply_to['user_id'] = null;
                        $in_reply_to['status_id_str'] = null;
@@ -5075,7 +5085,7 @@ function api_in_reply_to($item)
  */
 function api_clean_plain_items($text)
 {
-       $include_entities = strtolower(x($_REQUEST, 'include_entities') ? $_REQUEST['include_entities'] : "false");
+       $include_entities = strtolower(defaults($_REQUEST, 'include_entities', "false"));
 
        $text = BBCode::cleanPictureLinks($text);
        $URLSearchString = "^\[\]";
@@ -5199,7 +5209,7 @@ function api_best_nickname(&$contacts)
  */
 function api_friendica_group_show($type)
 {
-       $a = get_app();
+       $a = \get_app();
 
        if (api_user() === false) {
                throw new ForbiddenException();
@@ -5207,7 +5217,7 @@ function api_friendica_group_show($type)
 
        // params
        $user_info = api_get_user($a);
-       $gid = (x($_REQUEST, 'gid') ? $_REQUEST['gid'] : 0);
+       $gid = defaults($_REQUEST, 'gid', 0);
        $uid = $user_info['uid'];
 
        // get data of the specified group id or all groups if not specified
@@ -5264,7 +5274,7 @@ api_register_func('api/friendica/group_show', 'api_friendica_group_show', true);
  */
 function api_friendica_group_delete($type)
 {
-       $a = get_app();
+       $a = \get_app();
 
        if (api_user() === false) {
                throw new ForbiddenException();
@@ -5272,8 +5282,8 @@ function api_friendica_group_delete($type)
 
        // params
        $user_info = api_get_user($a);
-       $gid = (x($_REQUEST, 'gid') ? $_REQUEST['gid'] : 0);
-       $name = (x($_REQUEST, 'name') ? $_REQUEST['name'] : "");
+       $gid = defaults($_REQUEST, 'gid', 0);
+       $name = defaults($_REQUEST, 'name', "");
        $uid = $user_info['uid'];
 
        // error if no gid specified
@@ -5326,7 +5336,7 @@ api_register_func('api/friendica/group_delete', 'api_friendica_group_delete', tr
  */
 function api_lists_destroy($type)
 {
-       $a = get_app();
+       $a = \get_app();
 
        if (api_user() === false) {
                throw new ForbiddenException();
@@ -5334,7 +5344,7 @@ function api_lists_destroy($type)
 
        // params
        $user_info = api_get_user($a);
-       $gid = (x($_REQUEST, 'list_id') ? $_REQUEST['list_id'] : 0);
+       $gid = defaults($_REQUEST, 'list_id', 0);
        $uid = $user_info['uid'];
 
        // error if no gid specified
@@ -5442,7 +5452,7 @@ function group_create($name, $uid, $users = [])
  */
 function api_friendica_group_create($type)
 {
-       $a = get_app();
+       $a = \get_app();
 
        if (api_user() === false) {
                throw new ForbiddenException();
@@ -5450,7 +5460,7 @@ function api_friendica_group_create($type)
 
        // params
        $user_info = api_get_user($a);
-       $name = (x($_REQUEST, 'name') ? $_REQUEST['name'] : "");
+       $name = defaults($_REQUEST, 'name', "");
        $uid = $user_info['uid'];
        $json = json_decode($_POST['json'], true);
        $users = $json['user'];
@@ -5471,7 +5481,7 @@ api_register_func('api/friendica/group_create', 'api_friendica_group_create', tr
  */
 function api_lists_create($type)
 {
-       $a = get_app();
+       $a = \get_app();
 
        if (api_user() === false) {
                throw new ForbiddenException();
@@ -5479,7 +5489,7 @@ function api_lists_create($type)
 
        // params
        $user_info = api_get_user($a);
-       $name = (x($_REQUEST, 'name') ? $_REQUEST['name'] : "");
+       $name = defaults($_REQUEST, 'name', "");
        $uid = $user_info['uid'];
 
        $success = group_create($name, $uid);
@@ -5505,7 +5515,7 @@ api_register_func('api/lists/create', 'api_lists_create', true, API_METHOD_POST)
  */
 function api_friendica_group_update($type)
 {
-       $a = get_app();
+       $a = \get_app();
 
        if (api_user() === false) {
                throw new ForbiddenException();
@@ -5514,8 +5524,8 @@ function api_friendica_group_update($type)
        // params
        $user_info = api_get_user($a);
        $uid = $user_info['uid'];
-       $gid = (x($_REQUEST, 'gid') ? $_REQUEST['gid'] : 0);
-       $name = (x($_REQUEST, 'name') ? $_REQUEST['name'] : "");
+       $gid = defaults($_REQUEST, 'gid', 0);
+       $name = defaults($_REQUEST, 'name', "");
        $json = json_decode($_POST['json'], true);
        $users = $json['user'];
 
@@ -5579,7 +5589,7 @@ api_register_func('api/friendica/group_update', 'api_friendica_group_update', tr
  */
 function api_lists_update($type)
 {
-       $a = get_app();
+       $a = \get_app();
 
        if (api_user() === false) {
                throw new ForbiddenException();
@@ -5587,8 +5597,8 @@ function api_lists_update($type)
 
        // params
        $user_info = api_get_user($a);
-       $gid = (x($_REQUEST, 'list_id') ? $_REQUEST['list_id'] : 0);
-       $name = (x($_REQUEST, 'name') ? $_REQUEST['name'] : "");
+       $gid = defaults($_REQUEST, 'list_id', 0);
+       $name = defaults($_REQUEST, 'name', "");
        $uid = $user_info['uid'];
 
        // error if no gid specified
@@ -5625,7 +5635,7 @@ api_register_func('api/lists/update', 'api_lists_update', true, API_METHOD_POST)
  */
 function api_friendica_activity($type)
 {
-       $a = get_app();
+       $a = \get_app();
 
        if (api_user() === false) {
                throw new ForbiddenException();
@@ -5633,7 +5643,7 @@ function api_friendica_activity($type)
        $verb = strtolower($a->argv[3]);
        $verb = preg_replace("|\..*$|", "", $verb);
 
-       $id = (x($_REQUEST, 'id') ? $_REQUEST['id'] : 0);
+       $id = defaults($_REQUEST, 'id', 0);
 
        $res = Item::performLike($id, $verb);
 
@@ -5669,7 +5679,7 @@ api_register_func('api/friendica/activity/unattendmaybe', 'api_friendica_activit
 */
 function api_friendica_notification($type)
 {
-       $a = get_app();
+       $a = \get_app();
 
        if (api_user() === false) {
                throw new ForbiddenException();
@@ -5705,7 +5715,7 @@ function api_friendica_notification($type)
  */
 function api_friendica_notification_seen($type)
 {
-       $a = get_app();
+       $a = \get_app();
        $user_info = api_get_user($a);
 
        if (api_user() === false || $user_info === false) {
@@ -5715,7 +5725,7 @@ function api_friendica_notification_seen($type)
                throw new BadRequestException("Invalid argument count");
        }
 
-       $id = (x($_REQUEST, 'id') ? intval($_REQUEST['id']) : 0);
+       $id = (!empty($_REQUEST['id']) ? intval($_REQUEST['id']) : 0);
 
        $nm = new NotificationsManager();
        $note = $nm->getByID($id);
@@ -5750,7 +5760,7 @@ api_register_func('api/friendica/notification', 'api_friendica_notification', tr
  */
 function api_friendica_direct_messages_setseen($type)
 {
-       $a = get_app();
+       $a = \get_app();
        if (api_user() === false) {
                throw new ForbiddenException();
        }
@@ -5758,7 +5768,7 @@ function api_friendica_direct_messages_setseen($type)
        // params
        $user_info = api_get_user($a);
        $uid = $user_info['uid'];
-       $id = (x($_REQUEST, 'id') ? $_REQUEST['id'] : 0);
+       $id = defaults($_REQUEST, 'id', 0);
 
        // return error if id is zero
        if ($id == "") {
@@ -5799,7 +5809,7 @@ api_register_func('api/friendica/direct_messages_setseen', 'api_friendica_direct
  */
 function api_friendica_direct_messages_search($type, $box = "")
 {
-       $a = get_app();
+       $a = \get_app();
 
        if (api_user() === false) {
                throw new ForbiddenException();
@@ -5807,7 +5817,7 @@ function api_friendica_direct_messages_search($type, $box = "")
 
        // params
        $user_info = api_get_user($a);
-       $searchstring = (x($_REQUEST, 'searchstring') ? $_REQUEST['searchstring'] : "");
+       $searchstring = defaults($_REQUEST, 'searchstring', "");
        $uid = $user_info['uid'];
 
        // error if no searchstring specified
@@ -5835,9 +5845,9 @@ function api_friendica_direct_messages_search($type, $box = "")
                foreach ($r as $item) {
                        if ($box == "inbox" || $item['from-url'] != $profile_url) {
                                $recipient = $user_info;
-                               $sender = api_get_user($a, normalise_link($item['contact-url']));
+                               $sender = api_get_user($a, Strings::normaliseLink($item['contact-url']));
                        } elseif ($box == "sentbox" || $item['from-url'] == $profile_url) {
-                               $recipient = api_get_user($a, normalise_link($item['contact-url']));
+                               $recipient = api_get_user($a, Strings::normaliseLink($item['contact-url']));
                                $sender = $user_info;
                        }
 
@@ -5862,14 +5872,14 @@ api_register_func('api/friendica/direct_messages_search', 'api_friendica_direct_
  */
 function api_friendica_profile_show($type)
 {
-       $a = get_app();
+       $a = \get_app();
 
        if (api_user() === false) {
                throw new ForbiddenException();
        }
 
        // input params
-       $profile_id = (x($_REQUEST, 'profile_id') ? $_REQUEST['profile_id'] : 0);
+       $profile_id = defaults($_REQUEST, 'profile_id', 0);
 
        // retrieve general information about profiles for user
        $multi_profiles = Feature::isEnabled(api_user(), 'multi_profiles');