]> git.mxchange.org Git - friendica.git/blobdiff - include/api.php
linking new Admin FAQ in home
[friendica.git] / include / api.php
index af9fe7736b180471e7e04244f2d343774506a655..164a9de5320f59f988c0d622034f7a714c4d7192 100644 (file)
@@ -27,12 +27,9 @@ use Friendica\App;
 use Friendica\Content\ContactSelector;
 use Friendica\Content\Text\BBCode;
 use Friendica\Content\Text\HTML;
-use Friendica\Core\Hook;
 use Friendica\Core\Logger;
 use Friendica\Core\Protocol;
-use Friendica\Core\Session;
 use Friendica\Core\System;
-use Friendica\Core\Worker;
 use Friendica\Database\DBA;
 use Friendica\DI;
 use Friendica\Model\Contact;
@@ -45,9 +42,10 @@ use Friendica\Model\Post;
 use Friendica\Model\Profile;
 use Friendica\Model\User;
 use Friendica\Model\Verb;
+use Friendica\Module\Api\ApiResponse;
+use Friendica\Module\BaseApi;
 use Friendica\Network\HTTPException;
 use Friendica\Network\HTTPException\BadRequestException;
-use Friendica\Network\HTTPException\ExpectationFailedException;
 use Friendica\Network\HTTPException\ForbiddenException;
 use Friendica\Network\HTTPException\InternalServerErrorException;
 use Friendica\Network\HTTPException\MethodNotAllowedException;
@@ -56,13 +54,13 @@ use Friendica\Network\HTTPException\TooManyRequestsException;
 use Friendica\Network\HTTPException\UnauthorizedException;
 use Friendica\Object\Image;
 use Friendica\Protocol\Activity;
-use Friendica\Protocol\Diaspora;
+use Friendica\Security\BasicAuth;
 use Friendica\Security\OAuth;
 use Friendica\Util\DateTimeFormat;
 use Friendica\Util\Images;
 use Friendica\Util\Network;
+use Friendica\Util\Proxy;
 use Friendica\Util\Strings;
-use Friendica\Util\XML;
 
 require_once __DIR__ . '/../mod/item.php';
 require_once __DIR__ . '/../mod/wall_upload.php';
@@ -173,94 +171,6 @@ function api_register_func($path, $func, $auth = false, $method = API_METHOD_ANY
        ];
 }
 
-/**
- * Log in user via Simple HTTP Auth.
- * Simple Auth allow username in form of <pre>user@server</pre>, ignoring server part
- *
- * @param App $a App
- * @throws ForbiddenException
- * @throws InternalServerErrorException
- * @throws UnauthorizedException
- * @hook  'authenticate'
- *               array $addon_auth
- *               'username' => username from login form
- *               'password' => password from login form
- *               'authenticated' => return status,
- *               'user_record' => return authenticated user record
- */
-function api_login(App $a)
-{
-       $_SESSION["allow_api"] = false;
-
-       // workaround for HTTP-auth in CGI mode
-       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;
-                       $_SERVER['PHP_AUTH_PW'] = $password;
-               }
-       }
-
-       if (empty($_SERVER['PHP_AUTH_USER'])) {
-               Logger::debug(API_LOG_PREFIX . 'failed', ['module' => 'api', 'action' => 'login', 'parameters' => $_SERVER]);
-               header('WWW-Authenticate: Basic realm="Friendica"');
-               throw new UnauthorizedException("This API requires login");
-       }
-
-       $user = $_SERVER['PHP_AUTH_USER'] ?? '';
-       $password = $_SERVER['PHP_AUTH_PW'] ?? '';
-
-       // allow "user@server" login (but ignore 'server' part)
-       $at = strstr($user, "@", true);
-       if ($at) {
-               $user = $at;
-       }
-
-       // next code from mod/auth.php. needs better solution
-       $record = null;
-
-       $addon_auth = [
-               'username' => trim($user),
-               'password' => trim($password),
-               'authenticated' => 0,
-               'user_record' => null,
-       ];
-
-       /*
-       * An addon indicates successful login by setting 'authenticated' to non-zero value and returning a user record
-       * Addons should never set 'authenticated' except to indicate success - as hooks may be chained
-       * and later addons should not interfere with an earlier one that succeeded.
-       */
-       Hook::callAll('authenticate', $addon_auth);
-
-       if ($addon_auth['authenticated'] && !empty($addon_auth['user_record'])) {
-               $record = $addon_auth['user_record'];
-       } else {
-               $user_id = User::authenticate(trim($user), trim($password), true);
-               if ($user_id !== false) {
-                       $record = DBA::selectFirst('user', [], ['uid' => $user_id]);
-               }
-       }
-
-       if (!DBA::isResult($record)) {
-               Logger::debug(API_LOG_PREFIX . 'failed', ['module' => 'api', 'action' => 'login', 'parameters' => $_SERVER]);
-               header('WWW-Authenticate: Basic realm="Friendica"');
-               //header('HTTP/1.0 401 Unauthorized');
-               //die('This api requires login');
-               throw new UnauthorizedException("This API requires login");
-       }
-
-       // Don't refresh the login date more often than twice a day to spare database writes
-       $login_refresh = strcmp(DateTimeFormat::utc('now - 12 hours'), $record['login_date']) > 0;
-
-       DI::auth()->setForUser($a, $record, false, false, $login_refresh);
-
-       $_SESSION["allow_api"] = true;
-
-       Hook::callAll('logged_in', $record);
-}
-
 /**
  * Check HTTP method of called API
  *
@@ -321,7 +231,7 @@ function api_call(App $a, App\Arguments $args = null)
                                $called_api = explode("/", $p);
 
                                if (!empty($info['auth']) && api_user() === false) {
-                                       api_login($a);
+                                       BasicAuth::getCurrentUserID(true);
                                        Logger::info(API_LOG_PREFIX . 'nickname {nickname}', ['module' => 'api', 'action' => 'call', 'nickname' => $a->getLoggedInUserNickname()]);
                                }
 
@@ -373,49 +283,10 @@ function api_call(App $a, App\Arguments $args = null)
                Logger::warning(API_LOG_PREFIX . 'not implemented', ['module' => 'api', 'action' => 'call', 'query' => DI::args()->getQueryString()]);
                throw new NotFoundException();
        } catch (HTTPException $e) {
-               header("HTTP/1.1 {$e->getCode()} {$e->httpdesc}");
-               return api_error($type, $e, $args);
+               DI::apiResponse()->error($e->getCode(), $e->getDescription(), $e->getMessage(), $type);
        }
 }
 
-/**
- * Format API error string
- *
- * @param string $type Return type (xml, json, rss, as)
- * @param object $e    HTTPException Error object
- * @param App\Arguments $args The App arguments
- * @return string|array error message formatted as $type
- */
-function api_error($type, $e, App\Arguments $args)
-{
-       $error = ($e->getMessage() !== "" ? $e->getMessage() : $e->httpdesc);
-       /// @TODO:  https://dev.twitter.com/overview/api/response-codes
-
-       $error = ["error" => $error,
-                       "code" => $e->getCode() . " " . $e->httpdesc,
-                       "request" => $args->getQueryString()];
-
-       $return = api_format_data('status', $type, ['status' => $error]);
-
-       switch ($type) {
-               case "xml":
-                       header("Content-Type: text/xml");
-                       break;
-               case "json":
-                       header("Content-Type: application/json");
-                       $return = json_encode($return);
-                       break;
-               case "rss":
-                       header("Content-Type: application/rss+xml");
-                       break;
-               case "atom":
-                       header("Content-Type: application/atom+xml");
-                       break;
-       }
-
-       return $return;
-}
-
 /**
  * Set values for RSS template
  *
@@ -432,7 +303,7 @@ function api_error($type, $e, App\Arguments $args)
 function api_rss_extra(App $a, $arr, $user_info)
 {
        if (is_null($user_info)) {
-               $user_info = api_get_user($a);
+               $user_info = api_get_user();
        }
 
        $arr['$user'] = $user_info;
@@ -480,7 +351,7 @@ function api_unique_id_to_nurl($id)
  * @throws InternalServerErrorException
  * @throws UnauthorizedException
  */
-function api_get_user(App $a, $contact_id = null)
+function api_get_user($contact_id = null)
 {
        global $called_api;
 
@@ -492,9 +363,9 @@ function api_get_user(App $a, $contact_id = null)
 
        // Searching for contact URL
        if (!is_null($contact_id) && (intval($contact_id) == 0)) {
-               $user = DBA::escape(Strings::normaliseLink($contact_id));
+               $user = Strings::normaliseLink($contact_id);
                $url = $user;
-               $extra_query = "AND `contact`.`nurl` = '%s' ";
+               $extra_query = "AND `contact`.`nurl` = ? ";
                if (api_user() !== false) {
                        $extra_query .= "AND `contact`.`uid`=" . intval(api_user());
                }
@@ -502,43 +373,43 @@ function api_get_user(App $a, $contact_id = null)
 
        // Searching for contact id with uid = 0
        if (!is_null($contact_id) && (intval($contact_id) != 0)) {
-               $user = DBA::escape(api_unique_id_to_nurl(intval($contact_id)));
+               $user = api_unique_id_to_nurl(intval($contact_id));
 
                if ($user == "") {
                        throw new BadRequestException("User ID ".$contact_id." not found.");
                }
 
                $url = $user;
-               $extra_query = "AND `contact`.`nurl` = '%s' ";
+               $extra_query = "AND `contact`.`nurl` = ? ";
                if (api_user() !== false) {
                        $extra_query .= "AND `contact`.`uid`=" . intval(api_user());
                }
        }
 
        if (is_null($user) && !empty($_GET['user_id'])) {
-               $user = DBA::escape(api_unique_id_to_nurl($_GET['user_id']));
+               $user = api_unique_id_to_nurl($_GET['user_id']);
 
                if ($user == "") {
                        throw new BadRequestException("User ID ".$_GET['user_id']." not found.");
                }
 
                $url = $user;
-               $extra_query = "AND `contact`.`nurl` = '%s' ";
+               $extra_query = "AND `contact`.`nurl` = ? ";
                if (api_user() !== false) {
                        $extra_query .= "AND `contact`.`uid`=" . intval(api_user());
                }
        }
        if (is_null($user) && !empty($_GET['screen_name'])) {
-               $user = DBA::escape($_GET['screen_name']);
-               $extra_query = "AND `contact`.`nick` = '%s' ";
+               $user = $_GET['screen_name'];
+               $extra_query = "AND `contact`.`nick` = ? ";
                if (api_user() !== false) {
                        $extra_query .= "AND `contact`.`uid`=".intval(api_user());
                }
        }
 
        if (is_null($user) && !empty($_GET['profileurl'])) {
-               $user = DBA::escape(Strings::normaliseLink($_GET['profileurl']));
-               $extra_query = "AND `contact`.`nurl` = '%s' ";
+               $user = Strings::normaliseLink($_GET['profileurl']);
+               $extra_query = "AND `contact`.`nurl` = ? ";
                if (api_user() !== false) {
                        $extra_query .= "AND `contact`.`uid`=".intval(api_user());
                }
@@ -550,22 +421,21 @@ function api_get_user(App $a, $contact_id = null)
                if (!empty(DI::args()->getArgv()[$argid])) {
                        $data = explode(".", DI::args()->getArgv()[$argid]);
                        if (count($data) > 1) {
-                               list($user, $null) = $data;
+                               [$user, $null] = $data;
                        }
                }
                if (is_numeric($user)) {
-                       $user = DBA::escape(api_unique_id_to_nurl(intval($user)));
+                       $user = api_unique_id_to_nurl(intval($user));
 
                        if ($user != "") {
                                $url = $user;
-                               $extra_query = "AND `contact`.`nurl` = '%s' ";
+                               $extra_query = "AND `contact`.`nurl` = ? ";
                                if (api_user() !== false) {
                                        $extra_query .= "AND `contact`.`uid`=" . intval(api_user());
                                }
                        }
                } else {
-                       $user = DBA::escape($user);
-                       $extra_query = "AND `contact`.`nick` = '%s' ";
+                       $extra_query = "AND `contact`.`nick` = ? ";
                        if (api_user() !== false) {
                                $extra_query .= "AND `contact`.`uid`=" . intval(api_user());
                        }
@@ -576,23 +446,23 @@ function api_get_user(App $a, $contact_id = null)
 
        if (!$user) {
                if (api_user() === false) {
-                       api_login($a);
+                       BasicAuth::getCurrentUserID(true);
                        return false;
                } else {
                        $user = api_user();
-                       $extra_query = "AND `contact`.`uid` = %d AND `contact`.`self` ";
+                       $extra_query = "AND `contact`.`uid` = ? AND `contact`.`self` ";
                }
        }
 
        Logger::info(API_LOG_PREFIX . 'found user {user}', ['module' => 'api', 'action' => 'get_user', 'user' => $user, 'extra_query' => $extra_query]);
 
        // user info
-       $uinfo = q(
+       $uinfo = DBA::toArray(DBA::p(
                "SELECT *, `contact`.`id` AS `cid` FROM `contact`
                        WHERE 1
                $extra_query",
                $user
-       );
+       ));
 
        // Selecting the id by priority, friendica first
        if (is_array($uinfo)) {
@@ -615,10 +485,10 @@ function api_get_user(App $a, $contact_id = null)
                                'screen_name' => (($contact['nick']) ? $contact['nick'] : $contact['name']),
                                'location' => ($contact["location"] != "") ? $contact["location"] : ContactSelector::networkToName($contact['network'], $contact['url'], $contact['protocol']),
                                'description' => BBCode::toPlaintext($contact["about"] ?? ''),
-                               'profile_image_url' => $contact["micro"],
-                               'profile_image_url_https' => $contact["micro"],
-                               'profile_image_url_profile_size' => $contact["thumb"],
-                               'profile_image_url_large' => $contact["photo"],
+                               'profile_image_url' => Contact::getAvatarUrlForUrl($contact['url'], api_user(), Proxy::SIZE_MICRO),
+                               'profile_image_url_https' => Contact::getAvatarUrlForUrl($contact['url'], api_user(), Proxy::SIZE_MICRO),
+                               'profile_image_url_profile_size' => Contact::getAvatarUrlForUrl($contact['url'], api_user(), Proxy::SIZE_THUMB),
+                               'profile_image_url_large' => Contact::getAvatarUrlForUrl($contact['url'], api_user(), Proxy::SIZE_SMALL),
                                'url' => $contact["url"],
                                'protected' => false,
                                'followers_count' => 0,
@@ -689,10 +559,10 @@ function api_get_user(App $a, $contact_id = null)
                'screen_name' => (($uinfo[0]['nick']) ? $uinfo[0]['nick'] : $uinfo[0]['name']),
                'location' => $location,
                '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"],
-               'profile_image_url_large' => $uinfo[0]["photo"],
+               'profile_image_url' => Contact::getAvatarUrlForUrl($uinfo[0]['url'], api_user(), Proxy::SIZE_MICRO),
+               'profile_image_url_https' => Contact::getAvatarUrlForUrl($uinfo[0]['url'], api_user(), Proxy::SIZE_MICRO),
+               'profile_image_url_profile_size' => Contact::getAvatarUrlForUrl($uinfo[0]['url'], api_user(), Proxy::SIZE_THUMB),
+               'profile_image_url_large' => Contact::getAvatarUrlForUrl($uinfo[0]['url'], api_user(), Proxy::SIZE_SMALL),
                'url' => $uinfo[0]['url'],
                'protected' => false,
                'followers_count' => intval($countfollowers),
@@ -771,14 +641,14 @@ function api_get_user(App $a, $contact_id = null)
  */
 function api_item_get_user(App $a, $item)
 {
-       $status_user = api_get_user($a, $item['author-id'] ?? null);
+       $status_user = api_get_user($item['author-id'] ?? null);
 
        $author_user = $status_user;
 
        $status_user["protected"] = isset($item['private']) && ($item['private'] == Item::PRIVATE);
 
        if (($item['thr-parent'] ?? '') == ($item['uri'] ?? '')) {
-               $owner_user = api_get_user($a, $item['owner-id'] ?? null);
+               $owner_user = api_get_user($item['owner-id'] ?? null);
        } else {
                $owner_user = $author_user;
        }
@@ -786,130 +656,6 @@ function api_item_get_user(App $a, $item)
        return ([$status_user, $author_user, $owner_user]);
 }
 
-/**
- * walks recursively through an array with the possibility to change value and key
- *
- * @param array    $array    The array to walk through
- * @param callable $callback The callback function
- *
- * @return array the transformed array
- */
-function api_walk_recursive(array &$array, callable $callback)
-{
-       $new_array = [];
-
-       foreach ($array as $k => $v) {
-               if (is_array($v)) {
-                       if ($callback($v, $k)) {
-                               $new_array[$k] = api_walk_recursive($v, $callback);
-                       }
-               } else {
-                       if ($callback($v, $k)) {
-                               $new_array[$k] = $v;
-                       }
-               }
-       }
-       $array = $new_array;
-
-       return $array;
-}
-
-/**
- * Callback function to transform the array in an array that can be transformed in a XML file
- *
- * @param mixed  $item Array item value
- * @param string $key  Array key
- *
- * @return boolean Should the array item be deleted?
- */
-function api_reformat_xml(&$item, &$key)
-{
-       if (is_bool($item)) {
-               $item = ($item ? "true" : "false");
-       }
-
-       if (substr($key, 0, 10) == "statusnet_") {
-               $key = "statusnet:".substr($key, 10);
-       } elseif (substr($key, 0, 10) == "friendica_") {
-               $key = "friendica:".substr($key, 10);
-       }
-       /// @TODO old-lost code?
-       //else
-       //      $key = "default:".$key;
-
-       return true;
-}
-
-/**
- * Creates the XML from a JSON style array
- *
- * @param array  $data         JSON style array
- * @param string $root_element Name of the root element
- *
- * @return string The XML data
- */
-function api_create_xml(array $data, $root_element)
-{
-       $childname = key($data);
-       $data2 = array_pop($data);
-
-       $namespaces = ["" => "http://api.twitter.com",
-                               "statusnet" => "http://status.net/schema/api/1/",
-                               "friendica" => "http://friendi.ca/schema/api/1/",
-                               "georss" => "http://www.georss.org/georss"];
-
-       /// @todo Auto detection of needed namespaces
-       if (in_array($root_element, ["ok", "hash", "config", "version", "ids", "notes", "photos"])) {
-               $namespaces = [];
-       }
-
-       if (is_array($data2)) {
-               $key = key($data2);
-               api_walk_recursive($data2, "api_reformat_xml");
-
-               if ($key == "0") {
-                       $data4 = [];
-                       $i = 1;
-
-                       foreach ($data2 as $item) {
-                               $data4[$i++ . ":" . $childname] = $item;
-                       }
-
-                       $data2 = $data4;
-               }
-       }
-
-       $data3 = [$root_element => $data2];
-
-       $ret = XML::fromArray($data3, $xml, false, $namespaces);
-       return $ret;
-}
-
-/**
- * Formats the data according to the data type
- *
- * @param string $root_element Name of the root element
- * @param string $type         Return type (atom, rss, xml, json)
- * @param array  $data         JSON style array
- *
- * @return array|string (string|array) XML data or JSON data
- */
-function api_format_data($root_element, $type, $data)
-{
-       switch ($type) {
-               case "atom":
-               case "rss":
-               case "xml":
-                       $ret = api_create_xml($data, $root_element);
-                       break;
-               case "json":
-               default:
-                       $ret = $data;
-                       break;
-       }
-       return $ret;
-}
-
 /**
  * TWITTER API
  */
@@ -944,7 +690,7 @@ function api_account_verify_credentials($type)
 
        $skip_status = $_REQUEST['skip_status'] ?? false;
 
-       $user_info = api_get_user($a);
+       $user_info = api_get_user();
 
        // "verified" isn't used here in the standard
        unset($user_info["verified"]);
@@ -961,7 +707,7 @@ function api_account_verify_credentials($type)
        unset($user_info["uid"]);
        unset($user_info["self"]);
 
-       return api_format_data("user", $type, ['user' => $user_info]);
+       return DI::apiResponse()->formatData("user", $type, ['user' => $user_info]);
 }
 
 /// @TODO move to top of file or somewhere better
@@ -1001,10 +747,10 @@ function api_statuses_mediap($type)
        $a = DI::app();
 
        if (api_user() === false) {
-               Logger::log('api_statuses_update: no user');
+               logger::notice('api_statuses_update: no user');
                throw new ForbiddenException();
        }
-       $user_info = api_get_user($a);
+       $user_info = api_get_user();
 
        $_REQUEST['profile_uid'] = api_user();
        $_REQUEST['api_source'] = true;
@@ -1055,11 +801,11 @@ function api_statuses_update($type)
        $a = DI::app();
 
        if (api_user() === false) {
-               Logger::log('api_statuses_update: no user');
+               logger::notice('api_statuses_update: no user');
                throw new ForbiddenException();
        }
 
-       api_get_user($a);
+       api_get_user();
 
        // convert $_POST array items to the form we use for web posts.
        if (requestdata('htmlstatus')) {
@@ -1109,7 +855,7 @@ function api_statuses_update($type)
                        $posts_day = Post::count($condition);
 
                        if ($posts_day > $throttle_day) {
-                               Logger::log('Daily posting limit reached for user '.api_user(), Logger::DEBUG);
+                               logger::info('Daily posting limit reached for user '.api_user());
                                // die(api_error($type, DI::l10n()->t("Daily posting limit of %d posts reached. The post was rejected.", $throttle_day));
                                throw new TooManyRequestsException(DI::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));
                        }
@@ -1123,7 +869,7 @@ function api_statuses_update($type)
                        $posts_week = Post::count($condition);
 
                        if ($posts_week > $throttle_week) {
-                               Logger::log('Weekly posting limit reached for user '.api_user(), Logger::DEBUG);
+                               logger::info('Weekly posting limit reached for user '.api_user());
                                // die(api_error($type, DI::l10n()->t("Weekly posting limit of %d posts reached. The post was rejected.", $throttle_week)));
                                throw new TooManyRequestsException(DI::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));
                        }
@@ -1137,7 +883,7 @@ function api_statuses_update($type)
                        $posts_month = Post::count($condition);
 
                        if ($posts_month > $throttle_month) {
-                               Logger::log('Monthly posting limit reached for user '.api_user(), Logger::DEBUG);
+                               logger::info('Monthly posting limit reached for user '.api_user());
                                // die(api_error($type, DI::l10n()->t("Monthly posting limit of %d posts reached. The post was rejected.", $throttle_month));
                                throw new TooManyRequestsException(DI::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));
                        }
@@ -1238,11 +984,11 @@ function api_media_upload()
        $a = DI::app();
 
        if (api_user() === false) {
-               Logger::log('no user');
+               logger::notice('no user');
                throw new ForbiddenException();
        }
 
-       api_get_user($a);
+       api_get_user();
 
        if (empty($_FILES['media'])) {
                // Output error
@@ -1297,7 +1043,7 @@ function api_media_metadata_create($type)
                throw new ForbiddenException();
        }
 
-       api_get_user($a);
+       api_get_user();
 
        $postdata = Network::postdata();
 
@@ -1350,7 +1096,7 @@ function api_status_show($type, $item_id)
 
        Logger::info(API_LOG_PREFIX . 'End', ['action' => 'get_status', 'status_info' => $status_info]);
 
-       return api_format_data('statuses', $type, ['status' => $status_info]);
+       return DI::apiResponse()->formatData('statuses', $type, ['status' => $status_info]);
 }
 
 /**
@@ -1405,7 +1151,7 @@ function api_users_show($type)
 {
        $a = Friendica\DI::app();
 
-       $user_info = api_get_user($a);
+       $user_info = api_get_user();
 
        $item = api_get_last_status($user_info['pid'], $user_info['uid']);
        if (!empty($item)) {
@@ -1416,7 +1162,7 @@ function api_users_show($type)
        unset($user_info['uid']);
        unset($user_info['self']);
 
-       return api_format_data('user', $type, ['user' => $user_info]);
+       return DI::apiResponse()->formatData('user', $type, ['user' => $user_info]);
 }
 
 /// @TODO move to top of file or somewhere better
@@ -1456,7 +1202,7 @@ function api_users_search($type)
                if (DBA::isResult($contacts)) {
                        $k = 0;
                        foreach ($contacts as $contact) {
-                               $user_info = api_get_user($a, $contact['id']);
+                               $user_info = api_get_user($contact['id']);
 
                                if ($type == 'xml') {
                                        $userlist[$k++ . ':user'] = $user_info;
@@ -1472,7 +1218,7 @@ function api_users_search($type)
                throw new BadRequestException('No search term specified.');
        }
 
-       return api_format_data('users', $type, $userlist);
+       return DI::apiResponse()->formatData('users', $type, $userlist);
 }
 
 /// @TODO move to top of file or somewhere better
@@ -1499,7 +1245,7 @@ function api_users_lookup($type)
        if (!empty($_REQUEST['user_id'])) {
                foreach (explode(',', $_REQUEST['user_id']) as $id) {
                        if (!empty($id)) {
-                               $users[] = api_get_user(DI::app(), $id);
+                               $users[] = api_get_user($id);
                        }
                }
        }
@@ -1508,7 +1254,7 @@ function api_users_lookup($type)
                throw new NotFoundException;
        }
 
-       return api_format_data("users", $type, ['users' => $users]);
+       return DI::apiResponse()->formatData("users", $type, ['users' => $users]);
 }
 
 /// @TODO move to top of file or somewhere better
@@ -1531,7 +1277,7 @@ api_register_func('api/users/lookup', 'api_users_lookup', true);
 function api_search($type)
 {
        $a = DI::app();
-       $user_info = api_get_user($a);
+       $user_info = api_get_user();
 
        if (api_user() === false || $user_info === false) {
                throw new ForbiddenException();
@@ -1571,7 +1317,7 @@ function api_search($type)
                DBA::close($tags);
 
                if (empty($uriids)) {
-                       return api_format_data('statuses', $type, $data);
+                       return DI::apiResponse()->formatData('statuses', $type, $data);
                }
 
                $condition = ['uri-id' => $uriids];
@@ -1612,7 +1358,7 @@ function api_search($type)
 
        bindComments($data['status']);
 
-       return api_format_data('statuses', $type, $data);
+       return DI::apiResponse()->formatData('statuses', $type, $data);
 }
 
 /// @TODO move to top of file or somewhere better
@@ -1638,7 +1384,7 @@ api_register_func('api/search', 'api_search', true);
 function api_statuses_home_timeline($type)
 {
        $a = DI::app();
-       $user_info = api_get_user($a);
+       $user_info = api_get_user();
 
        if (api_user() === false || $user_info === false) {
                throw new ForbiddenException();
@@ -1709,7 +1455,7 @@ function api_statuses_home_timeline($type)
                        break;
        }
 
-       return api_format_data("statuses", $type, $data);
+       return DI::apiResponse()->formatData("statuses", $type, $data);
 }
 
 
@@ -1732,7 +1478,7 @@ api_register_func('api/statuses/friends_timeline', 'api_statuses_home_timeline',
 function api_statuses_public_timeline($type)
 {
        $a = DI::app();
-       $user_info = api_get_user($a);
+       $user_info = api_get_user();
 
        if (api_user() === false || $user_info === false) {
                throw new ForbiddenException();
@@ -1795,7 +1541,7 @@ function api_statuses_public_timeline($type)
                        break;
        }
 
-       return api_format_data("statuses", $type, $data);
+       return DI::apiResponse()->formatData("statuses", $type, $data);
 }
 
 /// @TODO move to top of file or somewhere better
@@ -1815,7 +1561,7 @@ api_register_func('api/statuses/public_timeline', 'api_statuses_public_timeline'
 function api_statuses_networkpublic_timeline($type)
 {
        $a = DI::app();
-       $user_info = api_get_user($a);
+       $user_info = api_get_user();
 
        if (api_user() === false || $user_info === false) {
                throw new ForbiddenException();
@@ -1854,7 +1600,7 @@ function api_statuses_networkpublic_timeline($type)
                        break;
        }
 
-       return api_format_data("statuses", $type, $data);
+       return DI::apiResponse()->formatData("statuses", $type, $data);
 }
 
 /// @TODO move to top of file or somewhere better
@@ -1876,7 +1622,7 @@ api_register_func('api/statuses/networkpublic_timeline', 'api_statuses_networkpu
 function api_statuses_show($type)
 {
        $a = DI::app();
-       $user_info = api_get_user($a);
+       $user_info = api_get_user();
 
        if (api_user() === false || $user_info === false) {
                throw new ForbiddenException();
@@ -1894,7 +1640,7 @@ function api_statuses_show($type)
                $id = intval(DI::args()->getArgv()[4] ?? 0);
        }
 
-       Logger::log('API: api_statuses_show: ' . $id);
+       logger::notice('API: api_statuses_show: ' . $id);
 
        $conversation = !empty($_REQUEST['conversation']);
 
@@ -1930,10 +1676,10 @@ function api_statuses_show($type)
 
        if ($conversation) {
                $data = ['status' => $ret];
-               return api_format_data("statuses", $type, $data);
+               return DI::apiResponse()->formatData("statuses", $type, $data);
        } else {
                $data = ['status' => $ret[0]];
-               return api_format_data("status", $type, $data);
+               return DI::apiResponse()->formatData("status", $type, $data);
        }
 }
 
@@ -1955,7 +1701,7 @@ api_register_func('api/statuses/show', 'api_statuses_show', true);
 function api_conversation_show($type)
 {
        $a = DI::app();
-       $user_info = api_get_user($a);
+       $user_info = api_get_user();
 
        if (api_user() === false || $user_info === false) {
                throw new ForbiddenException();
@@ -1984,7 +1730,7 @@ function api_conversation_show($type)
        // try to fetch the item for the local user - or the public item, if there is no local one
        $item = Post::selectFirst(['parent-uri-id'], ['id' => $id]);
        if (!DBA::isResult($item)) {
-               throw new BadRequestException("There is no status with this id.");
+               throw new BadRequestException("There is no status with the id $id.");
        }
 
        $parent = Post::selectFirst(['id'], ['uri-id' => $item['parent-uri-id'], 'uid' => [0, api_user()]], ['order' => ['uid' => true]]);
@@ -2012,7 +1758,7 @@ function api_conversation_show($type)
        $ret = api_format_items(Post::toArray($statuses), $user_info, false, $type);
 
        $data = ['status' => $ret];
-       return api_format_data("statuses", $type, $data);
+       return DI::apiResponse()->formatData("statuses", $type, $data);
 }
 
 /// @TODO move to top of file or somewhere better
@@ -2042,7 +1788,7 @@ function api_statuses_repeat($type)
                throw new ForbiddenException();
        }
 
-       api_get_user($a);
+       api_get_user();
 
        // params
        $id = intval(DI::args()->getArgv()[3] ?? 0);
@@ -2056,7 +1802,7 @@ function api_statuses_repeat($type)
                $id = intval(DI::args()->getArgv()[4] ?? 0);
        }
 
-       Logger::log('API: api_statuses_repeat: '.$id);
+       logger::notice('API: api_statuses_repeat: ' . $id);
 
        $fields = ['uri-id', 'network', 'body', 'title', 'author-name', 'author-link', 'author-avatar', 'guid', 'created', 'plink'];
        $item = Post::selectFirst($fields, ['id' => $id, 'private' => [Item::PUBLIC, Item::UNLISTED]]);
@@ -2125,7 +1871,7 @@ function api_statuses_destroy($type)
                throw new ForbiddenException();
        }
 
-       api_get_user($a);
+       api_get_user();
 
        // params
        $id = intval(DI::args()->getArgv()[3] ?? 0);
@@ -2139,7 +1885,7 @@ function api_statuses_destroy($type)
                $id = intval(DI::args()->getArgv()[4] ?? 0);
        }
 
-       Logger::log('API: api_statuses_destroy: '.$id);
+       logger::notice('API: api_statuses_destroy: ' . $id);
 
        $ret = api_statuses_show($type);
 
@@ -2167,7 +1913,7 @@ api_register_func('api/statuses/destroy', 'api_statuses_destroy', true, API_METH
 function api_statuses_mentions($type)
 {
        $a = DI::app();
-       $user_info = api_get_user($a);
+       $user_info = api_get_user();
 
        if (api_user() === false || $user_info === false) {
                throw new ForbiddenException();
@@ -2193,11 +1939,14 @@ function api_statuses_mentions($type)
                (SELECT `uri-id` FROM `post-user-notification` WHERE `uid` = ? AND `notification-type` & ? != 0 ORDER BY `uri-id`)
                AND (`uid` = 0 OR (`uid` = ? AND NOT `global`)) AND `id` > ?";
 
-       $condition = [GRAVITY_PARENT, GRAVITY_COMMENT, api_user(),
-               Post\UserNotification::NOTIF_EXPLICIT_TAGGED | Post\UserNotification::NOTIF_IMPLICIT_TAGGED |
-               Post\UserNotification::NOTIF_THREAD_COMMENT | Post\UserNotification::NOTIF_DIRECT_COMMENT |
-               Post\UserNotification::NOTIF_DIRECT_THREAD_COMMENT,
-               api_user(), $since_id];
+       $condition = [
+               GRAVITY_PARENT, GRAVITY_COMMENT,
+               api_user(),
+               Post\UserNotification::TYPE_EXPLICIT_TAGGED | Post\UserNotification::TYPE_IMPLICIT_TAGGED |
+               Post\UserNotification::TYPE_THREAD_COMMENT | Post\UserNotification::TYPE_DIRECT_COMMENT |
+               Post\UserNotification::TYPE_DIRECT_THREAD_COMMENT,
+               api_user(), $since_id,
+       ];
 
        if ($max_id > 0) {
                $query .= " AND `id` <= ?";
@@ -2220,7 +1969,7 @@ function api_statuses_mentions($type)
                        break;
        }
 
-       return api_format_data("statuses", $type, $data);
+       return DI::apiResponse()->formatData("statuses", $type, $data);
 }
 
 /// @TODO move to top of file or somewhere better
@@ -2242,7 +1991,7 @@ api_register_func('api/statuses/replies', 'api_statuses_mentions', true);
 function api_statuses_user_timeline($type)
 {
        $a = DI::app();
-       $user_info = api_get_user($a);
+       $user_info = api_get_user();
 
        if (api_user() === false || $user_info === false) {
                throw new ForbiddenException();
@@ -2298,7 +2047,7 @@ function api_statuses_user_timeline($type)
                        break;
        }
 
-       return api_format_data("statuses", $type, $data);
+       return DI::apiResponse()->formatData("statuses", $type, $data);
 }
 
 /// @TODO move to top of file or somewhere better
@@ -2367,7 +2116,7 @@ function api_favorites_create_destroy($type)
        }
 
 
-       $user_info = api_get_user($a);
+       $user_info = api_get_user();
        $rets = api_format_items([$item], $user_info, false, $type);
        $ret = $rets[0];
 
@@ -2380,7 +2129,7 @@ function api_favorites_create_destroy($type)
                        break;
        }
 
-       return api_format_data("status", $type, $data);
+       return DI::apiResponse()->formatData("status", $type, $data);
 }
 
 /// @TODO move to top of file or somewhere better
@@ -2404,7 +2153,7 @@ function api_favorites($type)
        global $called_api;
 
        $a = DI::app();
-       $user_info = api_get_user($a);
+       $user_info = api_get_user();
 
        if (api_user() === false || $user_info === false) {
                throw new ForbiddenException();
@@ -2453,7 +2202,7 @@ function api_favorites($type)
                        break;
        }
 
-       return api_format_data("statuses", $type, $data);
+       return DI::apiResponse()->formatData("statuses", $type, $data);
 }
 
 /// @TODO move to top of file or somewhere better
@@ -2894,7 +2643,7 @@ function api_format_items_activities($item, $type = "json")
                //builtin_activity_puller($i, $activities);
 
                // get user data and add it to the array of the activity
-               $user = api_get_user($a, $parent_item['author-id']);
+               $user = api_get_user($parent_item['author-id']);
                switch ($parent_item['verb']) {
                        case Activity::LIKE:
                                $activities['like'][] = $user;
@@ -2962,7 +2711,7 @@ function api_format_items($items, $user_info, $filter_user = false, $type = "jso
        }
 
        foreach ((array)$items as $item) {
-               list($status_user, $author_user, $owner_user) = api_item_get_user($a, $item);
+               [$status_user, $author_user, $owner_user] = api_item_get_user($a, $item);
 
                // Look if the posts are matching if they should be filtered by user id
                if ($filter_user && ($status_user["id"] != $user_info["id"])) {
@@ -2994,7 +2743,7 @@ function api_format_item($item, $type = "json", $status_user = null, $author_use
        $a = Friendica\DI::app();
 
        if (empty($status_user) || empty($author_user) || empty($owner_user)) {
-               list($status_user, $author_user, $owner_user) = api_item_get_user($a, $item);
+               [$status_user, $author_user, $owner_user] = api_item_get_user($a, $item);
        }
 
        DI::contentItem()->localize($item);
@@ -3058,7 +2807,7 @@ function api_format_item($item, $type = "json", $status_user = null, $author_use
                if (!empty($announce)) {
                        $retweeted_item = $item;
                        $item = $announce;
-                       $status['friendica_owner'] = api_get_user($a, $announce['author-id']);
+                       $status['friendica_owner'] = api_get_user($announce['author-id']);
                }
        }
 
@@ -3077,7 +2826,7 @@ function api_format_item($item, $type = "json", $status_user = null, $author_use
                        $quoted_status['text'] = $conv_quoted['text'];
                        $quoted_status['statusnet_html'] = $conv_quoted['html'];
                        try {
-                               $quoted_status["user"] = api_get_user($a, $quoted_item["author-id"]);
+                               $quoted_status["user"] = api_get_user($quoted_item["author-id"]);
                        } catch (BadRequestException $e) {
                                // user not found. should be found?
                                /// @todo check if the user should be always found
@@ -3099,7 +2848,7 @@ function api_format_item($item, $type = "json", $status_user = null, $author_use
                unset($retweeted_status['statusnet_conversation_id']);
                $status['user'] = $status['friendica_owner'];
                try {
-                       $retweeted_status["user"] = api_get_user($a, $retweeted_item["author-id"]);
+                       $retweeted_status["user"] = api_get_user($retweeted_item["author-id"]);
                } catch (BadRequestException $e) {
                        // user not found. should be found?
                        /// @todo check if the user should be always found
@@ -3147,63 +2896,6 @@ function api_format_item($item, $type = "json", $status_user = null, $author_use
        return $status;
 }
 
-/**
- * Returns the remaining number of API requests available to the user before the API limit is reached.
- *
- * @param string $type Return type (atom, rss, xml, json)
- *
- * @return array|string
- * @throws Exception
- */
-function api_account_rate_limit_status($type)
-{
-       if ($type == "xml") {
-               $hash = [
-                               'remaining-hits' => '150',
-                               '@attributes' => ["type" => "integer"],
-                               'hourly-limit' => '150',
-                               '@attributes2' => ["type" => "integer"],
-                               'reset-time' => DateTimeFormat::utc('now + 1 hour', DateTimeFormat::ATOM),
-                               '@attributes3' => ["type" => "datetime"],
-                               'reset_time_in_seconds' => strtotime('now + 1 hour'),
-                               '@attributes4' => ["type" => "integer"],
-                       ];
-       } else {
-               $hash = [
-                               'reset_time_in_seconds' => strtotime('now + 1 hour'),
-                               'remaining_hits' => '150',
-                               'hourly_limit' => '150',
-                               'reset_time' => api_date(DateTimeFormat::utc('now + 1 hour', DateTimeFormat::ATOM)),
-                       ];
-       }
-
-       return api_format_data('hash', $type, ['hash' => $hash]);
-}
-
-/// @TODO move to top of file or somewhere better
-api_register_func('api/account/rate_limit_status', 'api_account_rate_limit_status', true);
-
-/**
- * Returns the string "ok" in the requested format with a 200 OK HTTP status code.
- *
- * @param string $type Return type (atom, rss, xml, json)
- *
- * @return array|string
- */
-function api_help_test($type)
-{
-       if ($type == 'xml') {
-               $ok = "true";
-       } else {
-               $ok = "ok";
-       }
-
-       return api_format_data('ok', $type, ["ok" => $ok]);
-}
-
-/// @TODO move to top of file or somewhere better
-api_register_func('api/help/test', 'api_help_test', false);
-
 /**
  * Returns all lists the user subscribes to.
  *
@@ -3216,7 +2908,7 @@ function api_lists_list($type)
 {
        $ret = [];
        /// @TODO $ret is not filled here?
-       return api_format_data('lists', $type, ["lists_list" => $ret]);
+       return DI::apiResponse()->formatData('lists', $type, ["lists_list" => $ret]);
 }
 
 /// @TODO move to top of file or somewhere better
@@ -3245,7 +2937,7 @@ function api_lists_ownerships($type)
        }
 
        // params
-       $user_info = api_get_user($a);
+       $user_info = api_get_user();
        $uid = $user_info['uid'];
 
        $groups = DBA::select('group', [], ['deleted' => 0, 'uid' => $uid]);
@@ -3266,7 +2958,7 @@ function api_lists_ownerships($type)
                        'mode' => $mode
                ];
        }
-       return api_format_data("lists", $type, ['lists' => ['lists' => $lists]]);
+       return DI::apiResponse()->formatData("lists", $type, ['lists' => ['lists' => $lists]]);
 }
 
 /// @TODO move to top of file or somewhere better
@@ -3289,7 +2981,7 @@ function api_lists_statuses($type)
 {
        $a = DI::app();
 
-       $user_info = api_get_user($a);
+       $user_info = api_get_user();
        if (api_user() === false || $user_info === false) {
                throw new ForbiddenException();
        }
@@ -3346,7 +3038,7 @@ function api_lists_statuses($type)
                        break;
        }
 
-       return api_format_data("statuses", $type, $data);
+       return DI::apiResponse()->formatData("statuses", $type, $data);
 }
 
 /// @TODO move to top of file or somewhere better
@@ -3380,7 +3072,7 @@ function api_statuses_f($qtype)
 
        $start = max(0, ($page - 1) * $count);
 
-       $user_info = api_get_user($a);
+       $user_info = api_get_user();
 
        if (!empty($_GET['cursor']) && $_GET['cursor'] == 'undefined') {
                /* this is to stop Hotot to load friends multiple times
@@ -3413,23 +3105,24 @@ function api_statuses_f($qtype)
                $sql_filter = 'AND (NOT `blocked` OR `pending`)';
        }
 
-       $r = q(
+       // @todo This query most likely can be replaced with a Contact::select...
+       $r = DBA::toArray(DBA::p(
                "SELECT `nurl`
                FROM `contact`
-               WHERE `uid` = %d
+               WHERE `uid` = ?
                AND NOT `self`
                $sql_filter
                $sql_extra
                ORDER BY `nick`
-               LIMIT %d, %d",
-               intval(api_user()),
-               intval($start),
-               intval($count)
-       );
+               LIMIT ?, ?",
+               api_user(),
+               $start,
+               $count
+       ));
 
        $ret = [];
        foreach ($r as $cid) {
-               $user = api_get_user($a, $cid['nurl']);
+               $user = api_get_user($cid['nurl']);
                // "uid" and "self" are only needed for some internal stuff, so remove it from here
                unset($user["uid"]);
                unset($user["self"]);
@@ -3459,7 +3152,7 @@ function api_statuses_friends($type)
        if ($data === false) {
                return false;
        }
-       return api_format_data("users", $type, $data);
+       return DI::apiResponse()->formatData("users", $type, $data);
 }
 
 /**
@@ -3478,7 +3171,7 @@ function api_statuses_followers($type)
        if ($data === false) {
                return false;
        }
-       return api_format_data("users", $type, $data);
+       return DI::apiResponse()->formatData("users", $type, $data);
 }
 
 /// @TODO move to top of file or somewhere better
@@ -3502,7 +3195,7 @@ function api_blocks_list($type)
        if ($data === false) {
                return false;
        }
-       return api_format_data("users", $type, $data);
+       return DI::apiResponse()->formatData("users", $type, $data);
 }
 
 /// @TODO move to top of file or somewhere better
@@ -3531,72 +3224,12 @@ function api_friendships_incoming($type)
                $ids[] = $user['id'];
        }
 
-       return api_format_data("ids", $type, ['id' => $ids]);
+       return DI::apiResponse()->formatData("ids", $type, ['id' => $ids]);
 }
 
 /// @TODO move to top of file or somewhere better
 api_register_func('api/friendships/incoming', 'api_friendships_incoming', true);
 
-/**
- * Returns the instance's configuration information.
- *
- * @param string $type Return type (atom, rss, xml, json)
- *
- * @return array|string
- * @throws InternalServerErrorException
- */
-function api_statusnet_config($type)
-{
-       $name      = DI::config()->get('config', 'sitename');
-       $server    = DI::baseUrl()->getHostname();
-       $logo      = DI::baseUrl() . '/images/friendica-64.png';
-       $email     = DI::config()->get('config', 'admin_email');
-       $closed    = intval(DI::config()->get('config', 'register_policy')) === \Friendica\Module\Register::CLOSED ? 'true' : 'false';
-       $private   = DI::config()->get('system', 'block_public') ? 'true' : 'false';
-       $textlimit = (string) DI::config()->get('config', 'api_import_size', DI::config()->get('config', 'max_import_size', 200000));
-       $ssl       = DI::config()->get('system', 'have_ssl') ? 'true' : 'false';
-       $sslserver = DI::config()->get('system', 'have_ssl') ? str_replace('http:', 'https:', DI::baseUrl()) : '';
-
-       $config = [
-               'site' => ['name' => $name,'server' => $server, 'theme' => 'default', 'path' => '',
-                       'logo' => $logo, 'fancy' => true, 'language' => 'en', 'email' => $email, 'broughtby' => '',
-                       'broughtbyurl' => '', 'timezone' => 'UTC', 'closed' => $closed, 'inviteonly' => false,
-                       'private' => $private, 'textlimit' => $textlimit, 'sslserver' => $sslserver, 'ssl' => $ssl,
-                       'shorturllength' => '30',
-                       'friendica' => [
-                                       'FRIENDICA_PLATFORM' => FRIENDICA_PLATFORM,
-                                       'FRIENDICA_VERSION' => FRIENDICA_VERSION,
-                                       'DFRN_PROTOCOL_VERSION' => DFRN_PROTOCOL_VERSION,
-                                       'DB_UPDATE_VERSION' => DB_UPDATE_VERSION
-                                       ]
-               ],
-       ];
-
-       return api_format_data('config', $type, ['config' => $config]);
-}
-
-/// @TODO move to top of file or somewhere better
-api_register_func('api/gnusocial/config', 'api_statusnet_config', false);
-api_register_func('api/statusnet/config', 'api_statusnet_config', false);
-
-/**
- *
- * @param string $type Return type (atom, rss, xml, json)
- *
- * @return array|string
- */
-function api_statusnet_version($type)
-{
-       // liar
-       $fake_statusnet_version = "0.9.7";
-
-       return api_format_data('version', $type, ['version' => $fake_statusnet_version]);
-}
-
-/// @TODO move to top of file or somewhere better
-api_register_func('api/gnusocial/version', 'api_statusnet_version', false);
-api_register_func('api/statusnet/version', 'api_statusnet_version', false);
-
 /**
  * Sends a new direct message.
  *
@@ -3623,24 +3256,19 @@ function api_direct_messages_new($type)
                return;
        }
 
-       $sender = api_get_user($a);
+       $sender = api_get_user();
 
        $recipient = null;
        if (!empty($_POST['screen_name'])) {
-               $r = q(
-                       "SELECT `id`, `nurl`, `network` FROM `contact` WHERE `uid`=%d AND `nick`='%s'",
-                       intval(api_user()),
-                       DBA::escape($_POST['screen_name'])
-               );
-
-               if (DBA::isResult($r)) {
+               $contacts = Contact::selectToArray(['id', 'nurl', 'network'], ['uid' => api_user(), 'nick' => $_POST['screen_name']]);
+               if (DBA::isResult($contacts)) {
                        // Selecting the id by priority, friendica first
-                       api_best_nickname($r);
+                       api_best_nickname($contacts);
 
-                       $recipient = api_get_user($a, $r[0]['nurl']);
+                       $recipient = api_get_user($contacts[0]['nurl']);
                }
        } else {
-               $recipient = api_get_user($a, $_POST['user_id']);
+               $recipient = api_get_user($_POST['user_id']);
        }
 
        if (empty($recipient)) {
@@ -3649,13 +3277,9 @@ function api_direct_messages_new($type)
 
        $replyto = '';
        if (!empty($_REQUEST['replyto'])) {
-               $r = q(
-                       'SELECT `parent-uri`, `title` FROM `mail` WHERE `uid`=%d AND `id`=%d',
-                       intval(api_user()),
-                       intval($_REQUEST['replyto'])
-               );
-               $replyto = $r[0]['parent-uri'];
-               $sub     = $r[0]['title'];
+               $mail = DBA::selectFirst('mail', ['parent-uri', 'title'], ['uid' => api_user(), 'id' => $_REQUEST['replyto']]);
+               $replyto = $mail['parent-uri'];
+               $sub     = $mail['title'];
        } else {
                if (!empty($_REQUEST['title'])) {
                        $sub = $_REQUEST['title'];
@@ -3667,10 +3291,10 @@ function api_direct_messages_new($type)
        $id = Mail::send($recipient['cid'], $_POST['text'], $sub, $replyto);
 
        if ($id > -1) {
-               $r = q("SELECT * FROM `mail` WHERE id=%d", intval($id));
-               $ret = api_format_messages($r[0], $recipient, $sender);
+               $mail = DBA::selectFirst('mail', [], ['id' => $id]);
+               $ret = api_format_messages($mail, $recipient, $sender);
        } else {
-               $ret = ["error"=>$id];
+               $ret = ["error" => $id];
        }
 
        $data = ['direct_message'=>$ret];
@@ -3683,7 +3307,7 @@ function api_direct_messages_new($type)
                        break;
        }
 
-       return api_format_data("direct-messages", $type, $data);
+       return DI::apiResponse()->formatData("direct-messages", $type, $data);
 }
 
 /// @TODO move to top of file or somewhere better
@@ -3710,7 +3334,7 @@ function api_direct_messages_destroy($type)
        }
 
        // params
-       $user_info = api_get_user($a);
+       $user_info = api_get_user();
        //required
        $id = $_REQUEST['id'] ?? 0;
        // optional
@@ -3722,7 +3346,7 @@ function api_direct_messages_destroy($type)
        // error if no id or parenturi specified (for clients posting parent-uri as well)
        if ($verbose == "true" && ($id == 0 || $parenturi == "")) {
                $answer = ['result' => 'error', 'message' => 'message id or parenturi not specified'];
-               return api_format_data("direct_messages_delete", $type, ['$result' => $answer]);
+               return DI::apiResponse()->formatData("direct_messages_delete", $type, ['$result' => $answer]);
        }
 
        // BadRequestException if no id specified (for clients using Twitter API)
@@ -3733,38 +3357,27 @@ function api_direct_messages_destroy($type)
        // add parent-uri to sql command if specified by calling app
        $sql_extra = ($parenturi != "" ? " AND `parent-uri` = '" . DBA::escape($parenturi) . "'" : "");
 
-       // get data of the specified message id
-       $r = q(
-               "SELECT `id` FROM `mail` WHERE `uid` = %d AND `id` = %d" . $sql_extra,
-               intval($uid),
-               intval($id)
-       );
-
        // error message if specified id is not in database
-       if (!DBA::isResult($r)) {
+       if (!DBA::exists('mail', ["`uid` = ? AND `id` = ? " . $sql_extra, $uid, $id])) {
                if ($verbose == "true") {
                        $answer = ['result' => 'error', 'message' => 'message id not in database'];
-                       return api_format_data("direct_messages_delete", $type, ['$result' => $answer]);
+                       return DI::apiResponse()->formatData("direct_messages_delete", $type, ['$result' => $answer]);
                }
                /// @todo BadRequestException ok for Twitter API clients?
                throw new BadRequestException('message id not in database');
        }
 
        // delete message
-       $result = q(
-               "DELETE FROM `mail` WHERE `uid` = %d AND `id` = %d" . $sql_extra,
-               intval($uid),
-               intval($id)
-       );
+       $result = DBA::delete('mail', ["`uid` = ? AND `id` = ? " . $sql_extra, $uid, $id]);
 
        if ($verbose == "true") {
                if ($result) {
                        // return success
                        $answer = ['result' => 'ok', 'message' => 'message deleted'];
-                       return api_format_data("direct_message_delete", $type, ['$result' => $answer]);
+                       return DI::apiResponse()->formatData("direct_message_delete", $type, ['$result' => $answer]);
                } else {
                        $answer = ['result' => 'error', 'message' => 'unknown error'];
-                       return api_format_data("direct_messages_delete", $type, ['$result' => $answer]);
+                       return DI::apiResponse()->formatData("direct_messages_delete", $type, ['$result' => $answer]);
                }
        }
        /// @todo return JSON data like Twitter API not yet implemented
@@ -3849,8 +3462,9 @@ function api_friendships_destroy($type)
        // Set screen_name since Twidere requests it
        $contact['screen_name'] = $contact['nick'];
 
-       return api_format_data('friendships-destroy', $type, ['user' => $contact]);
+       return DI::apiResponse()->formatData('friendships-destroy', $type, ['user' => $contact]);
 }
+
 api_register_func('api/friendships/destroy', 'api_friendships_destroy', true, API_METHOD_POST);
 
 /**
@@ -3889,7 +3503,7 @@ function api_direct_messages_box($type, $box, $verbose)
        unset($_REQUEST["screen_name"]);
        unset($_GET["screen_name"]);
 
-       $user_info = api_get_user($a);
+       $user_info = api_get_user();
        if ($user_info === false) {
                throw new ForbiddenException();
        }
@@ -3921,25 +3535,25 @@ function api_direct_messages_box($type, $box, $verbose)
                $sql_extra .= " AND `contact`.`nick` = '" . DBA::escape($screen_name). "'";
        }
 
-       $r = q(
-               "SELECT `mail`.*, `contact`.`nurl` AS `contact-url` FROM `mail`,`contact` WHERE `mail`.`contact-id` = `contact`.`id` AND `mail`.`uid`=%d AND $sql_extra AND `mail`.`id` > %d ORDER BY `mail`.`id` DESC LIMIT %d,%d",
-               intval(api_user()),
-               intval($since_id),
-               intval($start),
-               intval($count)
-       );
+       $r = DBA::toArray(DBA::p(
+               "SELECT `mail`.*, `contact`.`nurl` AS `contact-url` FROM `mail`,`contact` WHERE `mail`.`contact-id` = `contact`.`id` AND `mail`.`uid` = ? AND $sql_extra AND `mail`.`id` > ? ORDER BY `mail`.`id` DESC LIMIT ?,?",
+               api_user(),
+               $since_id,
+               $start,
+               $count
+       ));
        if ($verbose == "true" && !DBA::isResult($r)) {
                $answer = ['result' => 'error', 'message' => 'no mails available'];
-               return api_format_data("direct_messages_all", $type, ['$result' => $answer]);
+               return DI::apiResponse()->formatData("direct_messages_all", $type, ['$result' => $answer]);
        }
 
        $ret = [];
        foreach ($r as $item) {
                if ($box == "inbox" || $item['from-url'] != $profile_url) {
                        $recipient = $user_info;
-                       $sender = api_get_user($a, Strings::normaliseLink($item['contact-url']));
+                       $sender = api_get_user(Strings::normaliseLink($item['contact-url']));
                } elseif ($box == "sentbox" || $item['from-url'] == $profile_url) {
-                       $recipient = api_get_user($a, Strings::normaliseLink($item['contact-url']));
+                       $recipient = api_get_user(Strings::normaliseLink($item['contact-url']));
                        $sender = $user_info;
                }
 
@@ -3958,7 +3572,7 @@ function api_direct_messages_box($type, $box, $verbose)
                        break;
        }
 
-       return api_format_data("direct-messages", $type, $data);
+       return DI::apiResponse()->formatData("direct-messages", $type, $data);
 }
 
 /**
@@ -4027,95 +3641,6 @@ api_register_func('api/direct_messages/all', 'api_direct_messages_all', true);
 api_register_func('api/direct_messages/sent', 'api_direct_messages_sentbox', true);
 api_register_func('api/direct_messages', 'api_direct_messages_inbox', true);
 
-/**
- * delete a complete photoalbum with all containing photos from database through api
- *
- * @param string $type Known types are 'atom', 'rss', 'xml' and 'json'
- * @return string|array
- * @throws BadRequestException
- * @throws ForbiddenException
- * @throws InternalServerErrorException
- */
-function api_fr_photoalbum_delete($type)
-{
-       if (api_user() === false) {
-               throw new ForbiddenException();
-       }
-       // input params
-       $album = $_REQUEST['album'] ?? '';
-
-       // we do not allow calls without album string
-       if ($album == "") {
-               throw new BadRequestException("no albumname specified");
-       }
-       // check if album is existing
-
-       $photos = DBA::selectToArray('photo', ['resource-id'], ['uid' => api_user(), 'album' => $album], ['group_by' => ['resource-id']]);
-       if (!DBA::isResult($photos)) {
-               throw new BadRequestException("album not available");
-       }
-
-       $resourceIds = array_column($photos, 'resource-id');
-
-       // function for setting the items to "deleted = 1" which ensures that comments, likes etc. are not shown anymore
-       // to the user and the contacts of the users (drop_items() performs the federation of the deletion to other networks
-       $condition = ['uid' => api_user(), 'resource-id' => $resourceIds, 'type' => 'photo'];
-       Item::deleteForUser($condition, api_user());
-
-       // now let's delete all photos from the album
-       $result = Photo::delete(['uid' => api_user(), 'album' => $album]);
-
-       // return success of deletion or error message
-       if ($result) {
-               $answer = ['result' => 'deleted', 'message' => 'album `' . $album . '` with all containing photos has been deleted.'];
-               return api_format_data("photoalbum_delete", $type, ['$result' => $answer]);
-       } else {
-               throw new InternalServerErrorException("unknown error - deleting from database failed");
-       }
-}
-
-/**
- * update the name of the album for all photos of an album
- *
- * @param string $type Known types are 'atom', 'rss', 'xml' and 'json'
- * @return string|array
- * @throws BadRequestException
- * @throws ForbiddenException
- * @throws InternalServerErrorException
- */
-function api_fr_photoalbum_update($type)
-{
-       if (api_user() === false) {
-               throw new ForbiddenException();
-       }
-       // input params
-       $album = $_REQUEST['album'] ?? '';
-       $album_new = $_REQUEST['album_new'] ?? '';
-
-       // we do not allow calls without album string
-       if ($album == "") {
-               throw new BadRequestException("no albumname specified");
-       }
-       if ($album_new == "") {
-               throw new BadRequestException("no new albumname specified");
-       }
-       // check if album is existing
-       if (!Photo::exists(['uid' => api_user(), 'album' => $album])) {
-               throw new BadRequestException("album not available");
-       }
-       // now let's update all photos to the albumname
-       $result = Photo::update(['album' => $album_new], ['uid' => api_user(), 'album' => $album]);
-
-       // return success of updating or error message
-       if ($result) {
-               $answer = ['result' => 'updated', 'message' => 'album `' . $album . '` with all containing photos has been renamed to `' . $album_new . '`.'];
-               return api_format_data("photoalbum_update", $type, ['$result' => $answer]);
-       } else {
-               throw new InternalServerErrorException("unknown error - updating in database failed");
-       }
-}
-
-
 /**
  * list all photos of the authenticated user
  *
@@ -4129,12 +3654,12 @@ function api_fr_photos_list($type)
        if (api_user() === false) {
                throw new ForbiddenException();
        }
-       $r = q(
+       $r = DBA::toArray(DBA::p(
                "SELECT `resource-id`, MAX(scale) AS `scale`, `album`, `filename`, `type`, MAX(`created`) AS `created`,
                MAX(`edited`) AS `edited`, MAX(`desc`) AS `desc` FROM `photo`
-               WHERE `uid` = %d AND `album` != 'Contact Photos' GROUP BY `resource-id`, `album`, `filename`, `type`",
-               intval(local_user())
-       );
+               WHERE `uid` = ? AND NOT `photo-type` IN (?, ?) GROUP BY `resource-id`, `album`, `filename`, `type`",
+               local_user(), Photo::CONTACT_AVATAR, Photo::CONTACT_BANNER
+       ));
        $typetoext = [
                'image/jpeg' => 'jpg',
                'image/png' => 'png',
@@ -4161,7 +3686,7 @@ function api_fr_photos_list($type)
                        }
                }
        }
-       return api_format_data("photos", $type, $data);
+       return DI::apiResponse()->formatData("photos", $type, $data);
 }
 
 /**
@@ -4229,11 +3754,11 @@ function api_fr_photo_create_update($type)
        // now let's upload the new media in create-mode
        if ($mode == "create") {
                $media = $_FILES['media'];
-               $data = save_media_to_database("photo", $media, $type, $album, trim($allow_cid), trim($deny_cid), trim($allow_gid), trim($deny_gid), $desc, $visibility);
+               $data = save_media_to_database("photo", $media, $type, $album, trim($allow_cid), trim($deny_cid), trim($allow_gid), trim($deny_gid), $desc, Photo::DEFAULT, $visibility);
 
                // return success of updating or error message
                if (!is_null($data)) {
-                       return api_format_data("photo_create", $type, $data);
+                       return DI::apiResponse()->formatData("photo_create", $type, $data);
                } else {
                        throw new InternalServerErrorException("unknown error - uploading photo failed, see Friendica log for more information");
                }
@@ -4282,20 +3807,20 @@ function api_fr_photo_create_update($type)
                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);
+                       $data = save_media_to_database("photo", $media, $type, $album, $allow_cid, $deny_cid, $allow_gid, $deny_gid, $desc, Photo::DEFAULT, $visibility, $photo_id);
                        if (!is_null($data)) {
-                               return api_format_data("photo_update", $type, $data);
+                               return DI::apiResponse()->formatData("photo_update", $type, $data);
                        }
                }
 
                // return success of updating or error message
                if ($result) {
                        $answer = ['result' => 'updated', 'message' => 'Image id `' . $photo_id . '` has been updated.'];
-                       return api_format_data("photo_update", $type, ['$result' => $answer]);
+                       return DI::apiResponse()->formatData("photo_update", $type, ['$result' => $answer]);
                } else {
                        if ($nothingtodo) {
                                $answer = ['result' => 'cancelled', 'message' => 'Nothing to update for image id `' . $photo_id . '`.'];
-                               return api_format_data("photo_update", $type, ['$result' => $answer]);
+                               return DI::apiResponse()->formatData("photo_update", $type, ['$result' => $answer]);
                        }
                        throw new InternalServerErrorException("unknown error - update photo entry in database failed");
                }
@@ -4303,53 +3828,6 @@ function api_fr_photo_create_update($type)
        throw new InternalServerErrorException("unknown error - this error on uploading or updating a photo should never happen");
 }
 
-/**
- * delete a single photo from the database through api
- *
- * @param string $type Known types are 'atom', 'rss', 'xml' and 'json'
- * @return string|array
- * @throws BadRequestException
- * @throws ForbiddenException
- * @throws InternalServerErrorException
- */
-function api_fr_photo_delete($type)
-{
-       if (api_user() === false) {
-               throw new ForbiddenException();
-       }
-
-       // input params
-       $photo_id = $_REQUEST['photo_id'] ?? null;
-
-       // do several checks on input parameters
-       // we do not allow calls without photo id
-       if ($photo_id == null) {
-               throw new BadRequestException("no photo_id specified");
-       }
-
-       // check if photo is existing in database
-       if (!Photo::exists(['resource-id' => $photo_id, 'uid' => api_user()])) {
-               throw new BadRequestException("photo not available");
-       }
-
-       // now we can perform on the deletion of the photo
-       $result = Photo::delete(['uid' => api_user(), 'resource-id' => $photo_id]);
-
-       // return success of deletion or error message
-       if ($result) {
-               // function for setting the items to "deleted = 1" which ensures that comments, likes etc. are not shown anymore
-               // to the user and the contacts of the users (drop_items() do all the necessary magic to avoid orphans in database and federate deletion)
-               $condition = ['uid' => api_user(), 'resource-id' => $photo_id, 'type' => 'photo'];
-               Item::deleteForUser($condition, api_user());
-
-               $result = ['result' => 'deleted', 'message' => 'photo with id `' . $photo_id . '` has been deleted from server.'];
-               return api_format_data("photo_delete", $type, ['$result' => $result]);
-       } else {
-               throw new InternalServerErrorException("unknown error on deleting photo from database table");
-       }
-}
-
-
 /**
  * returns the details of a specified photo id, if scale is given, returns the photo data in base 64
  *
@@ -4375,7 +3853,7 @@ function api_fr_photo_detail($type)
        // prepare json/xml output with data from database for the requested photo
        $data = prepare_photo_data($type, $scale, $photo_id);
 
-       return api_format_data("photo_detail", $type, $data);
+       return DI::apiResponse()->formatData("photo_detail", $type, $data);
 }
 
 
@@ -4425,7 +3903,7 @@ function api_account_update_profile_image($type)
                $media = $_FILES['media'];
        }
        // save new profile image
-       $data = save_media_to_database("profileimage", $media, $type, DI::l10n()->t('Profile Photos'), "", "", "", "", "", $is_default_profile);
+       $data = save_media_to_database("profileimage", $media, $type, DI::l10n()->t(Photo::PROFILE_PHOTOS), "", "", "", "", "", Photo::USER_AVATAR);
 
        // get filetype
        if (is_array($media['type'])) {
@@ -4444,7 +3922,7 @@ function api_account_update_profile_image($type)
        // change specified profile or all profiles to the new resource-id
        if ($is_default_profile) {
                $condition = ["`profile` AND `resource-id` != ? AND `uid` = ?", $data['photo']['id'], api_user()];
-               Photo::update(['profile' => false], $condition);
+               Photo::update(['profile' => false, 'photo-type' => Photo::DEFAULT], $condition);
        } else {
                $fields = ['photo' => DI::baseUrl() . '/photo/' . $data['photo']['id'] . '-4.' . $fileext,
                        'thumb' => DI::baseUrl() . '/photo/' . $data['photo']['id'] . '-5.' . $fileext];
@@ -4466,12 +3944,9 @@ function api_account_update_profile_image($type)
 }
 
 // place api-register for photoalbum calls before 'api/friendica/photo', otherwise this function is never reached
-api_register_func('api/friendica/photoalbum/delete', 'api_fr_photoalbum_delete', true, API_METHOD_DELETE);
-api_register_func('api/friendica/photoalbum/update', 'api_fr_photoalbum_update', true, API_METHOD_POST);
 api_register_func('api/friendica/photos/list', 'api_fr_photos_list', true);
 api_register_func('api/friendica/photo/create', 'api_fr_photo_create_update', true, API_METHOD_POST);
 api_register_func('api/friendica/photo/update', 'api_fr_photo_create_update', true, API_METHOD_POST);
-api_register_func('api/friendica/photo/delete', 'api_fr_photo_delete', true, API_METHOD_DELETE);
 api_register_func('api/friendica/photo', 'api_fr_photo_detail', true);
 api_register_func('api/account/update_profile_image', 'api_account_update_profile_image', true, API_METHOD_POST);
 
@@ -4490,7 +3965,7 @@ api_register_func('api/account/update_profile_image', 'api_account_update_profil
 function api_account_update_profile($type)
 {
        $local_user = api_user();
-       $api_user = api_get_user(DI::app());
+       $api_user = api_get_user();
 
        if (!empty($_POST['name'])) {
                DBA::update('profile', ['name' => $_POST['name']], ['uid' => $local_user]);
@@ -4551,7 +4026,7 @@ function check_acl_input($acl_string)
  * @param string  $allow_gid
  * @param string  $deny_gid
  * @param string  $desc
- * @param integer $profile
+ * @param integer $phototype
  * @param boolean $visibility
  * @param string  $photo_id
  * @return array
@@ -4562,7 +4037,7 @@ function check_acl_input($acl_string)
  * @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)
+function save_media_to_database($mediatype, $media, $type, $album, $allow_cid, $deny_cid, $allow_gid, $deny_gid, $desc, $phototype = 0, $visibility = false, $photo_id = null)
 {
        $visitor   = 0;
        $src = "";
@@ -4595,11 +4070,9 @@ function save_media_to_database($mediatype, $media, $type, $album, $allow_cid, $
 
        $filetype = Images::getMimeTypeBySource($src, $filename, $filetype);
 
-       Logger::log(
+       logger::info(
                "File upload src: " . $src . " - filename: " . $filename .
-               " - size: " . $filesize . " - type: " . $filetype,
-               Logger::DEBUG
-       );
+               " - size: " . $filesize . " - type: " . $filetype);
 
        // check if there was a php upload error
        if ($filesize == 0 && $media['error'] == 1) {
@@ -4625,12 +4098,9 @@ function save_media_to_database($mediatype, $media, $type, $album, $allow_cid, $
 
        // check max length of images on server
        $max_length = DI::config()->get('system', 'max_image_length');
-       if (!$max_length) {
-               $max_length = MAX_IMAGE_LENGTH;
-       }
        if ($max_length > 0) {
                $Image->scaleDown($max_length);
-               Logger::log("File upload: Scaling picture to new size " . $max_length, Logger::DEBUG);
+               logger::info("File upload: Scaling picture to new size " . $max_length);
        }
        $width = $Image->getWidth();
        $height = $Image->getHeight();
@@ -4640,57 +4110,57 @@ function save_media_to_database($mediatype, $media, $type, $album, $allow_cid, $
 
        if ($mediatype == "photo") {
                // upload normal image (scales 0, 1, 2)
-               Logger::log("photo upload: starting new photo upload", Logger::DEBUG);
+               logger::info("photo upload: starting new photo upload");
 
-               $r = Photo::store($Image, local_user(), $visitor, $resource_id, $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, Photo::DEFAULT, $allow_cid, $allow_gid, $deny_cid, $deny_gid, $desc);
                if (!$r) {
-                       Logger::log("photo upload: image upload with scale 0 (original size) failed");
+                       logger::notice("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, $resource_id, $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, Photo::DEFAULT, $allow_cid, $allow_gid, $deny_cid, $deny_gid, $desc);
                        if (!$r) {
-                               Logger::log("photo upload: image upload with scale 1 (640x640) failed");
+                               logger::notice("photo upload: image upload with scale 1 (640x640) failed");
                        }
                }
 
                if ($width > 320 || $height > 320) {
                        $Image->scaleDown(320);
-                       $r = Photo::store($Image, local_user(), $visitor, $resource_id, $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, Photo::DEFAULT, $allow_cid, $allow_gid, $deny_cid, $deny_gid, $desc);
                        if (!$r) {
-                               Logger::log("photo upload: image upload with scale 2 (320x320) failed");
+                               logger::notice("photo upload: image upload with scale 2 (320x320) failed");
                        }
                }
-               Logger::log("photo upload: new photo upload ended", Logger::DEBUG);
+               logger::info("photo upload: new photo upload ended");
        } elseif ($mediatype == "profileimage") {
                // upload profile image (scales 4, 5, 6)
-               Logger::log("photo upload: starting new profile image upload", Logger::DEBUG);
+               logger::info("photo upload: starting new profile image upload");
 
                if ($width > 300 || $height > 300) {
                        $Image->scaleDown(300);
-                       $r = Photo::store($Image, local_user(), $visitor, $resource_id, $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, $phototype, $allow_cid, $allow_gid, $deny_cid, $deny_gid, $desc);
                        if (!$r) {
-                               Logger::log("photo upload: profile image upload with scale 4 (300x300) failed");
+                               logger::notice("photo upload: profile image upload with scale 4 (300x300) failed");
                        }
                }
 
                if ($width > 80 || $height > 80) {
                        $Image->scaleDown(80);
-                       $r = Photo::store($Image, local_user(), $visitor, $resource_id, $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, $phototype, $allow_cid, $allow_gid, $deny_cid, $deny_gid, $desc);
                        if (!$r) {
-                               Logger::log("photo upload: profile image upload with scale 5 (80x80) failed");
+                               logger::notice("photo upload: profile image upload with scale 5 (80x80) failed");
                        }
                }
 
                if ($width > 48 || $height > 48) {
                        $Image->scaleDown(48);
-                       $r = Photo::store($Image, local_user(), $visitor, $resource_id, $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, $phototype, $allow_cid, $allow_gid, $deny_cid, $deny_gid, $desc);
                        if (!$r) {
-                               Logger::log("photo upload: profile image upload with scale 6 (48x48) failed");
+                               logger::notice("photo upload: profile image upload with scale 6 (48x48) failed");
                        }
                }
                $Image->__destruct();
-               Logger::log("photo upload: new profile image upload ended", Logger::DEBUG);
+               logger::info("photo upload: new profile image upload ended");
        }
 
        if (!empty($r)) {
@@ -4776,7 +4246,7 @@ function post_photo_item($hash, $allow_cid, $deny_cid, $allow_gid, $deny_gid, $f
 function prepare_photo_data($type, $scale, $photo_id)
 {
        $a = DI::app();
-       $user_info = api_get_user($a);
+       $user_info = api_get_user();
 
        if ($user_info === false) {
                throw new ForbiddenException();
@@ -4787,18 +4257,16 @@ function prepare_photo_data($type, $scale, $photo_id)
 
        // added allow_cid, allow_gid, deny_cid, deny_gid to output as string like stored in database
        // clients needs to convert this in their way for further processing
-       $r = q(
-               "SELECT %s `resource-id`, `created`, `edited`, `title`, `desc`, `album`, `filename`,
+       $r = DBA::toArray(DBA::p(
+               "SELECT $data_sql `resource-id`, `created`, `edited`, `title`, `desc`, `album`, `filename`,
                                        `type`, `height`, `width`, `datasize`, `profile`, `allow_cid`, `deny_cid`, `allow_gid`, `deny_gid`,
                                        MIN(`scale`) AS `minscale`, MAX(`scale`) AS `maxscale`
-                       FROM `photo` WHERE `uid` = %d AND `resource-id` = '%s' %s GROUP BY
-                              `resource-id`, `created`, `edited`, `title`, `desc`, `album`, `filename`,
-                              `type`, `height`, `width`, `datasize`, `profile`, `allow_cid`, `deny_cid`, `allow_gid`, `deny_gid`",
-               $data_sql,
-               intval(local_user()),
-               DBA::escape($photo_id),
-               $scale_sql
-       );
+                       FROM `photo` WHERE `uid` = ? AND `resource-id` = ? $scale_sql GROUP BY
+                                  `resource-id`, `created`, `edited`, `title`, `desc`, `album`, `filename`,
+                                  `type`, `height`, `width`, `datasize`, `profile`, `allow_cid`, `deny_cid`, `allow_gid`, `deny_gid`",
+               local_user(),
+               $photo_id
+       ));
 
        $typetoext = [
                'image/jpeg' => 'jpg',
@@ -5078,31 +4546,25 @@ function api_friendica_group_show($type)
        }
 
        // params
-       $user_info = api_get_user($a);
+       $user_info = api_get_user();
        $gid = $_REQUEST['gid'] ?? 0;
        $uid = $user_info['uid'];
 
        // get data of the specified group id or all groups if not specified
        if ($gid != 0) {
-               $r = q(
-                       "SELECT * FROM `group` WHERE `deleted` = 0 AND `uid` = %d AND `id` = %d",
-                       intval($uid),
-                       intval($gid)
-               );
+               $groups = DBA::selectToArray('group', [], ['deleted' => false, 'uid' => $uid, 'id' => $gid]);
+
                // error message if specified gid is not in database
-               if (!DBA::isResult($r)) {
+               if (!DBA::isResult($groups)) {
                        throw new BadRequestException("gid not available");
                }
        } else {
-               $r = q(
-                       "SELECT * FROM `group` WHERE `deleted` = 0 AND `uid` = %d",
-                       intval($uid)
-               );
+               $groups = DBA::selectToArray('group', [], ['deleted' => false, 'uid' => $uid]);
        }
 
        // loop through all groups and retrieve all members for adding data in the user array
        $grps = [];
-       foreach ($r as $rr) {
+       foreach ($groups as $rr) {
                $members = Contact\Group::getById($rr['id']);
                $users = [];
 
@@ -5110,88 +4572,22 @@ function api_friendica_group_show($type)
                        $user_element = "users";
                        $k = 0;
                        foreach ($members as $member) {
-                               $user = api_get_user($a, $member['nurl']);
+                               $user = api_get_user($member['nurl']);
                                $users[$k++.":user"] = $user;
                        }
                } else {
                        $user_element = "user";
                        foreach ($members as $member) {
-                               $user = api_get_user($a, $member['nurl']);
+                               $user = api_get_user($member['nurl']);
                                $users[] = $user;
                        }
                }
                $grps[] = ['name' => $rr['name'], 'gid' => $rr['id'], $user_element => $users];
        }
-       return api_format_data("groups", $type, ['group' => $grps]);
+       return DI::apiResponse()->formatData("groups", $type, ['group' => $grps]);
 }
-api_register_func('api/friendica/group_show', 'api_friendica_group_show', true);
 
-
-/**
- * Delete the specified group of the user.
- *
- * @param string $type Return type (atom, rss, xml, json)
- *
- * @return array|string
- * @throws BadRequestException
- * @throws ForbiddenException
- * @throws ImagickException
- * @throws InternalServerErrorException
- * @throws UnauthorizedException
- */
-function api_friendica_group_delete($type)
-{
-       $a = DI::app();
-
-       if (api_user() === false) {
-               throw new ForbiddenException();
-       }
-
-       // params
-       $user_info = api_get_user($a);
-       $gid = $_REQUEST['gid'] ?? 0;
-       $name = $_REQUEST['name'] ?? '';
-       $uid = $user_info['uid'];
-
-       // error if no gid specified
-       if ($gid == 0 || $name == "") {
-               throw new BadRequestException('gid or name not specified');
-       }
-
-       // get data of the specified group id
-       $r = q(
-               "SELECT * FROM `group` WHERE `uid` = %d AND `id` = %d",
-               intval($uid),
-               intval($gid)
-       );
-       // error message if specified gid is not in database
-       if (!DBA::isResult($r)) {
-               throw new BadRequestException('gid not available');
-       }
-
-       // get data of the specified group id and group name
-       $rname = q(
-               "SELECT * FROM `group` WHERE `uid` = %d AND `id` = %d AND `name` = '%s'",
-               intval($uid),
-               intval($gid),
-               DBA::escape($name)
-       );
-       // error message if specified gid is not in database
-       if (!DBA::isResult($rname)) {
-               throw new BadRequestException('wrong group name');
-       }
-
-       // delete group
-       $ret = Group::removeByName($uid, $name);
-       if ($ret) {
-               // return success
-               $success = ['success' => $ret, 'gid' => $gid, 'name' => $name, 'status' => 'deleted', 'wrong users' => []];
-               return api_format_data("group_delete", $type, ['result' => $success]);
-       } else {
-               throw new BadRequestException('other API error');
-       }
-}
-api_register_func('api/friendica/group_delete', 'api_friendica_group_delete', true, API_METHOD_DELETE);
+api_register_func('api/friendica/group_show', 'api_friendica_group_show', true);
 
 /**
  * Delete a group.
@@ -5215,7 +4611,7 @@ function api_lists_destroy($type)
        }
 
        // params
-       $user_info = api_get_user($a);
+       $user_info = api_get_user();
        $gid = $_REQUEST['list_id'] ?? 0;
        $uid = $user_info['uid'];
 
@@ -5239,9 +4635,10 @@ function api_lists_destroy($type)
                        'user' => $user_info
                ];
 
-               return api_format_data("lists", $type, ['lists' => $list]);
+               return DI::apiResponse()->formatData("lists", $type, ['lists' => $list]);
        }
 }
+
 api_register_func('api/lists/destroy', 'api_lists_destroy', true, API_METHOD_DELETE);
 
 /**
@@ -5261,25 +4658,13 @@ function group_create($name, $uid, $users = [])
                throw new BadRequestException('group name not specified');
        }
 
-       // get data of the specified group name
-       $rname = q(
-               "SELECT * FROM `group` WHERE `uid` = %d AND `name` = '%s' AND `deleted` = 0",
-               intval($uid),
-               DBA::escape($name)
-       );
        // error message if specified group name already exists
-       if (DBA::isResult($rname)) {
+       if (DBA::exists('group', ['uid' => $uid, 'name' => $name, 'deleted' => false])) {
                throw new BadRequestException('group name already exists');
        }
 
-       // check if specified group name is a deleted group
-       $rname = q(
-               "SELECT * FROM `group` WHERE `uid` = %d AND `name` = '%s' AND `deleted` = 1",
-               intval($uid),
-               DBA::escape($name)
-       );
-       // error message if specified group name already exists
-       if (DBA::isResult($rname)) {
+       // Check if the group needs to be reactivated
+       if (DBA::exists('group', ['uid' => $uid, 'name' => $name, 'deleted' => true])) {
                $reactivate_group = true;
        }
 
@@ -5296,13 +4681,7 @@ function group_create($name, $uid, $users = [])
        $errorusers = [];
        foreach ($users as $user) {
                $cid = $user['cid'];
-               // check if user really exists as contact
-               $contact = q(
-                       "SELECT * FROM `contact` WHERE `id` = %d AND `uid` = %d",
-                       intval($cid),
-                       intval($uid)
-               );
-               if (count($contact)) {
+               if (DBA::exists('contact', ['id' => $cid, 'uid' => $uid])) {
                        Group::addMember($gid, $cid);
                } else {
                        $erroraddinguser = true;
@@ -5337,7 +4716,7 @@ function api_friendica_group_create($type)
        }
 
        // params
-       $user_info = api_get_user($a);
+       $user_info = api_get_user();
        $name = $_REQUEST['name'] ?? '';
        $uid = $user_info['uid'];
        $json = json_decode($_POST['json'], true);
@@ -5345,8 +4724,9 @@ function api_friendica_group_create($type)
 
        $success = group_create($name, $uid, $users);
 
-       return api_format_data("group_create", $type, ['result' => $success]);
+       return DI::apiResponse()->formatData("group_create", $type, ['result' => $success]);
 }
+
 api_register_func('api/friendica/group_create', 'api_friendica_group_create', true, API_METHOD_POST);
 
 /**
@@ -5371,7 +4751,7 @@ function api_lists_create($type)
        }
 
        // params
-       $user_info = api_get_user($a);
+       $user_info = api_get_user();
        $name = $_REQUEST['name'] ?? '';
        $uid = $user_info['uid'];
 
@@ -5384,9 +4764,10 @@ function api_lists_create($type)
                        'user' => $user_info
                ];
 
-               return api_format_data("lists", $type, ['lists'=>$grp]);
+               return DI::apiResponse()->formatData("lists", $type, ['lists' => $grp]);
        }
 }
+
 api_register_func('api/lists/create', 'api_lists_create', true, API_METHOD_POST);
 
 /**
@@ -5410,7 +4791,7 @@ function api_friendica_group_update($type)
        }
 
        // params
-       $user_info = api_get_user($a);
+       $user_info = api_get_user();
        $uid = $user_info['uid'];
        $gid = $_REQUEST['gid'] ?? 0;
        $name = $_REQUEST['name'] ?? '';
@@ -5435,7 +4816,8 @@ function api_friendica_group_update($type)
                        $found = ($user['cid'] == $cid ? true : false);
                }
                if (!isset($found) || !$found) {
-                       Group::removeMemberByName($uid, $name, $cid);
+                       $gid = Group::getIdByName($uid, $name);
+                       Group::removeMember($gid, $cid);
                }
        }
 
@@ -5444,14 +4826,8 @@ function api_friendica_group_update($type)
        $errorusers = [];
        foreach ($users as $user) {
                $cid = $user['cid'];
-               // check if user really exists as contact
-               $contact = q(
-                       "SELECT * FROM `contact` WHERE `id` = %d AND `uid` = %d",
-                       intval($cid),
-                       intval($uid)
-               );
 
-               if (count($contact)) {
+               if (DBA::exists('contact', ['id' => $cid, 'uid' => $uid])) {
                        Group::addMember($gid, $cid);
                } else {
                        $erroraddinguser = true;
@@ -5462,7 +4838,7 @@ function api_friendica_group_update($type)
        // return success message incl. missing users in array
        $status = ($erroraddinguser ? "missing user" : "ok");
        $success = ['success' => true, 'gid' => $gid, 'name' => $name, 'status' => $status, 'wrong users' => $errorusers];
-       return api_format_data("group_update", $type, ['result' => $success]);
+       return DI::apiResponse()->formatData("group_update", $type, ['result' => $success]);
 }
 
 api_register_func('api/friendica/group_update', 'api_friendica_group_update', true, API_METHOD_POST);
@@ -5489,7 +4865,7 @@ function api_lists_update($type)
        }
 
        // params
-       $user_info = api_get_user($a);
+       $user_info = api_get_user();
        $gid = $_REQUEST['list_id'] ?? 0;
        $name = $_REQUEST['name'] ?? '';
        $uid = $user_info['uid'];
@@ -5514,101 +4890,12 @@ function api_lists_update($type)
                        'user' => $user_info
                ];
 
-               return api_format_data("lists", $type, ['lists' => $list]);
+               return DI::apiResponse()->formatData("lists", $type, ['lists' => $list]);
        }
 }
 
 api_register_func('api/lists/update', 'api_lists_update', true, API_METHOD_POST);
 
-/**
- *
- * @param string $type Return type (atom, rss, xml, json)
- *
- * @return array|string
- * @throws BadRequestException
- * @throws ForbiddenException
- * @throws ImagickException
- * @throws InternalServerErrorException
- */
-function api_friendica_activity($type)
-{
-       $a = DI::app();
-
-       if (api_user() === false) {
-               throw new ForbiddenException();
-       }
-       $verb = strtolower(DI::args()->getArgv()[3]);
-       $verb = preg_replace("|\..*$|", "", $verb);
-
-       $id = $_REQUEST['id'] ?? 0;
-
-       $res = Item::performActivity($id, $verb, api_user());
-
-       if ($res) {
-               if ($type == "xml") {
-                       $ok = "true";
-               } else {
-                       $ok = "ok";
-               }
-               return api_format_data('ok', $type, ['ok' => $ok]);
-       } else {
-               throw new BadRequestException('Error adding activity');
-       }
-}
-
-/// @TODO move to top of file or somewhere better
-api_register_func('api/friendica/activity/like', 'api_friendica_activity', true, API_METHOD_POST);
-api_register_func('api/friendica/activity/dislike', 'api_friendica_activity', true, API_METHOD_POST);
-api_register_func('api/friendica/activity/attendyes', 'api_friendica_activity', true, API_METHOD_POST);
-api_register_func('api/friendica/activity/attendno', 'api_friendica_activity', true, API_METHOD_POST);
-api_register_func('api/friendica/activity/attendmaybe', 'api_friendica_activity', true, API_METHOD_POST);
-api_register_func('api/friendica/activity/unlike', 'api_friendica_activity', true, API_METHOD_POST);
-api_register_func('api/friendica/activity/undislike', 'api_friendica_activity', true, API_METHOD_POST);
-api_register_func('api/friendica/activity/unattendyes', 'api_friendica_activity', true, API_METHOD_POST);
-api_register_func('api/friendica/activity/unattendno', 'api_friendica_activity', true, API_METHOD_POST);
-api_register_func('api/friendica/activity/unattendmaybe', 'api_friendica_activity', true, API_METHOD_POST);
-
-/**
- * Returns notifications
- *
- * @param string $type Known types are 'atom', 'rss', 'xml' and 'json'
- *
- * @return string|array
- * @throws ForbiddenException
- * @throws BadRequestException
- * @throws Exception
- */
-function api_friendica_notification($type)
-{
-       $a = DI::app();
-
-       if (api_user() === false) {
-               throw new ForbiddenException();
-       }
-       if (DI::args()->getArgc()!==3) {
-               throw new BadRequestException("Invalid argument count");
-       }
-
-       $notifications = DI::notification()->getApiList(local_user());
-
-       if ($type == "xml") {
-               $xmlnotes = false;
-               if (!empty($notifications)) {
-                       foreach ($notifications as $notification) {
-                               $xmlnotes[] = ["@attributes" => $notification->toArray()];
-                       }
-               }
-
-               $result = $xmlnotes;
-       } elseif (count($notifications) > 0) {
-               $result = $notifications->getArrayCopy();
-       } else {
-               $result = false;
-       }
-
-       return api_format_data("notes", $type, ['note' => $result]);
-}
-
 /**
  * Set notification as seen and returns associated item (if possible)
  *
@@ -5625,32 +4912,42 @@ function api_friendica_notification($type)
 function api_friendica_notification_seen($type)
 {
        $a         = DI::app();
-       $user_info = api_get_user($a);
+       $user_info = api_get_user();
 
        if (api_user() === false || $user_info === false) {
                throw new ForbiddenException();
        }
        if (DI::args()->getArgc() !== 4) {
-               throw new BadRequestException("Invalid argument count");
+               throw new BadRequestException('Invalid argument count');
        }
 
-       $id = (!empty($_REQUEST['id']) ? intval($_REQUEST['id']) : 0);
+       $id = intval($_REQUEST['id'] ?? 0);
 
        try {
-               $notify = DI::notify()->getByID($id, api_user());
-               DI::notify()->setSeen(true, $notify);
+               $Notify = DI::notify()->selectOneById($id);
+               if ($Notify->uid !== api_user()) {
+                       throw new NotFoundException();
+               }
+
+               if ($Notify->uriId) {
+                       DI::notification()->setAllSeenForUser($Notify->uid, ['target-uri-id' => $Notify->uriId]);
+               }
 
-               if ($notify->otype === Notification\ObjectType::ITEM) {
-                       $item = Post::selectFirstForUser(api_user(), [], ['id' => $notify->iid, 'uid' => api_user()]);
+               $Notify->setSeen();
+               DI::notify()->save($Notify);
+
+               if ($Notify->otype === Notification\ObjectType::ITEM) {
+                       $item = Post::selectFirstForUser(api_user(), [], ['id' => $Notify->iid, 'uid' => api_user()]);
                        if (DBA::isResult($item)) {
                                // we found the item, return it to the user
                                $ret  = api_format_items([$item], $user_info, false, $type);
                                $data = ['status' => $ret];
-                               return api_format_data("status", $type, $data);
+                               return DI::apiResponse()->formatData('status', $type, $data);
                        }
                        // the item can't be found, but we set the notification as seen, so we count this as a success
                }
-               return api_format_data('result', $type, ['result' => "success"]);
+
+               return DI::apiResponse()->formatData('result', $type, ['result' => 'success']);
        } catch (NotFoundException $e) {
                throw new BadRequestException('Invalid argument', $e);
        } catch (Exception $e) {
@@ -5660,58 +4957,6 @@ function api_friendica_notification_seen($type)
 
 /// @TODO move to top of file or somewhere better
 api_register_func('api/friendica/notification/seen', 'api_friendica_notification_seen', true, API_METHOD_POST);
-api_register_func('api/friendica/notification', 'api_friendica_notification', true, API_METHOD_GET);
-
-/**
- * update a direct_message to seen state
- *
- * @param string $type Known types are 'atom', 'rss', 'xml' and 'json'
- * @return string|array (success result=ok, error result=error with error message)
- * @throws BadRequestException
- * @throws ForbiddenException
- * @throws ImagickException
- * @throws InternalServerErrorException
- * @throws UnauthorizedException
- */
-function api_friendica_direct_messages_setseen($type)
-{
-       $a = DI::app();
-       if (api_user() === false) {
-               throw new ForbiddenException();
-       }
-
-       // params
-       $user_info = api_get_user($a);
-       $uid = $user_info['uid'];
-       $id = $_REQUEST['id'] ?? 0;
-
-       // return error if id is zero
-       if ($id == "") {
-               $answer = ['result' => 'error', 'message' => 'message id not specified'];
-               return api_format_data("direct_messages_setseen", $type, ['$result' => $answer]);
-       }
-
-       // error message if specified id is not in database
-       if (!DBA::exists('mail', ['id' => $id, 'uid' => $uid])) {
-               $answer = ['result' => 'error', 'message' => 'message id not in database'];
-               return api_format_data("direct_messages_setseen", $type, ['$result' => $answer]);
-       }
-
-       // update seen indicator
-       $result = DBA::update('mail', ['seen' => true], ['id' => $id]);
-
-       if ($result) {
-               // return success
-               $answer = ['result' => 'ok', 'message' => 'message set to seen'];
-               return api_format_data("direct_message_setseen", $type, ['$result' => $answer]);
-       } else {
-               $answer = ['result' => 'error', 'message' => 'unknown error'];
-               return api_format_data("direct_messages_setseen", $type, ['$result' => $answer]);
-       }
-}
-
-/// @TODO move to top of file or somewhere better
-api_register_func('api/friendica/direct_messages_setseen', 'api_friendica_direct_messages_setseen', true);
 
 /**
  * search for direct_messages containing a searchstring through api
@@ -5736,22 +4981,22 @@ function api_friendica_direct_messages_search($type, $box = "")
        }
 
        // params
-       $user_info = api_get_user($a);
+       $user_info = api_get_user();
        $searchstring = $_REQUEST['searchstring'] ?? '';
        $uid = $user_info['uid'];
 
        // error if no searchstring specified
        if ($searchstring == "") {
                $answer = ['result' => 'error', 'message' => 'searchstring not specified'];
-               return api_format_data("direct_messages_search", $type, ['$result' => $answer]);
+               return DI::apiResponse()->formatData("direct_messages_search", $type, ['$result' => $answer]);
        }
 
        // get data for the specified searchstring
-       $r = q(
-               "SELECT `mail`.*, `contact`.`nurl` AS `contact-url` FROM `mail`,`contact` WHERE `mail`.`contact-id` = `contact`.`id` AND `mail`.`uid`=%d AND `body` LIKE '%s' ORDER BY `mail`.`id` DESC",
-               intval($uid),
-               DBA::escape('%'.$searchstring.'%')
-       );
+       $r = DBA::toArray(DBA::p(
+               "SELECT `mail`.*, `contact`.`nurl` AS `contact-url` FROM `mail`,`contact` WHERE `mail`.`contact-id` = `contact`.`id` AND `mail`.`uid` = ? AND `body` LIKE ? ORDER BY `mail`.`id` DESC",
+               $uid,
+               '%'.$searchstring.'%'
+       ));
 
        $profile_url = $user_info["url"];
 
@@ -5765,9 +5010,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, Strings::normaliseLink($item['contact-url']));
+                               $sender = api_get_user(Strings::normaliseLink($item['contact-url']));
                        } elseif ($box == "sentbox" || $item['from-url'] == $profile_url) {
-                               $recipient = api_get_user($a, Strings::normaliseLink($item['contact-url']));
+                               $recipient = api_get_user(Strings::normaliseLink($item['contact-url']));
                                $sender = $user_info;
                        }
 
@@ -5778,46 +5023,12 @@ function api_friendica_direct_messages_search($type, $box = "")
                $success = ['success' => true, 'search_results' => $ret];
        }
 
-       return api_format_data("direct_message_search", $type, ['$result' => $success]);
+       return DI::apiResponse()->formatData("direct_message_search", $type, ['$result' => $success]);
 }
 
 /// @TODO move to top of file or somewhere better
 api_register_func('api/friendica/direct_messages_search', 'api_friendica_direct_messages_search', true);
 
-/**
- * Returns a list of saved searches.
- *
- * @see https://developer.twitter.com/en/docs/accounts-and-users/manage-account-settings/api-reference/get-saved_searches-list
- *
- * @param  string $type Return format: json or xml
- *
- * @return string|array
- * @throws Exception
- */
-function api_saved_searches_list($type)
-{
-       $terms = DBA::select('search', ['id', 'term'], ['uid' => local_user()]);
-
-       $result = [];
-       while ($term = DBA::fetch($terms)) {
-               $result[] = [
-                       'created_at' => api_date(time()),
-                       'id' => intval($term['id']),
-                       'id_str' => $term['id'],
-                       'name' => $term['term'],
-                       'position' => null,
-                       'query' => $term['term']
-               ];
-       }
-
-       DBA::close($terms);
-
-       return api_format_data("terms", $type, ['terms' => $result]);
-}
-
-/// @TODO move to top of file or somewhere better
-api_register_func('api/saved_searches/list', 'api_saved_searches_list', true);
-
 /*
  * Number of comments
  *