]> git.mxchange.org Git - friendica.git/blobdiff - include/api.php
Issue 3857: There is the possibility of a bad handling of dislikes
[friendica.git] / include / api.php
index 75f8ab069a83b1d4f8286d0183f1017d533c3ee2..d1ec25eabe279764762f1b51a24b9d3160e9dd93 100644 (file)
@@ -6,41 +6,40 @@
  * @todo Automatically detect if incoming data is HTML or BBCode
  */
 
-use \Friendica\Core\Config;
-
-       require_once('include/HTTPExceptions.php');
-
-       require_once('include/bbcode.php');
-       require_once('include/datetime.php');
-       require_once('include/conversation.php');
-       require_once('include/oauth.php');
-       require_once('include/html2plain.php');
-       require_once('mod/share.php');
-       require_once('include/Photo.php');
-       require_once('mod/item.php');
-       require_once('include/security.php');
-       require_once('include/contact_selectors.php');
-       require_once('include/html2bbcode.php');
-       require_once('mod/wall_upload.php');
-       require_once('mod/proxy.php');
-       require_once('include/message.php');
-       require_once('include/group.php');
-       require_once('include/like.php');
-       require_once('include/NotificationsManager.php');
-       require_once('include/plaintext.php');
-       require_once('include/xml.php');
-
-
-       define('API_METHOD_ANY','*');
-       define('API_METHOD_GET','GET');
-       define('API_METHOD_POST','POST,PUT');
-       define('API_METHOD_DELETE','POST,DELETE');
-
-
-
-       $API = Array();
-       $called_api = Null;
-
+use Friendica\App;
+use Friendica\Core\System;
+use Friendica\Core\Config;
+
+require_once 'include/HTTPExceptions.php';
+require_once 'include/bbcode.php';
+require_once 'include/datetime.php';
+require_once 'include/conversation.php';
+require_once 'include/oauth.php';
+require_once 'include/html2plain.php';
+require_once 'mod/share.php';
+require_once 'include/Photo.php';
+require_once 'mod/item.php';
+require_once 'include/security.php';
+require_once 'include/contact_selectors.php';
+require_once 'include/html2bbcode.php';
+require_once 'mod/wall_upload.php';
+require_once 'mod/proxy.php';
+require_once 'include/message.php';
+require_once 'include/group.php';
+require_once 'include/like.php';
+require_once 'include/NotificationsManager.php';
+require_once 'include/plaintext.php';
+require_once 'include/xml.php';
+
+define('API_METHOD_ANY', '*');
+define('API_METHOD_GET', 'GET');
+define('API_METHOD_POST', 'POST,PUT');
+define('API_METHOD_DELETE', 'POST,DELETE');
+
+$API = array();
+$called_api = null;
+
+/// @TODO Fix intending
        /**
         * @brief Auth API user
         *
@@ -49,8 +48,9 @@ use \Friendica\Core\Config;
         * into a page, and visitors will post something without noticing it).
         */
        function api_user() {
-               if ($_SESSION['allow_api'])
+               if (x($_SESSION, 'allow_api')) {
                        return local_user();
+               }
 
                return false;
        }
@@ -67,16 +67,18 @@ use \Friendica\Core\Config;
         *              Client source name, default to "api" if unset/unknown
         */
        function api_source() {
-               if (requestdata('source'))
-                       return (requestdata('source'));
+               if (requestdata('source')) {
+                       return requestdata('source');
+               }
 
                // Support for known clients that doesn't send a source name
-               if (strstr($_SERVER['HTTP_USER_AGENT'], "Twidere"))
-                       return ("Twidere");
+               if (strpos($_SERVER['HTTP_USER_AGENT'], "Twidere") !== false) {
+                       return "Twidere";
+               }
 
                logger("Unrecognized user-agent ".$_SERVER['HTTP_USER_AGENT'], LOGGER_DEBUG);
 
-               return ("api");
+               return "api";
        }
 
        /**
@@ -85,8 +87,8 @@ use \Friendica\Core\Config;
         * @param string $str Source date, as UTC
         * @return string Date in UTC formatted as "D M d H:i:s +0000 Y"
         */
-       function api_date($str){
-               //Wed May 23 06:01:13 +0000 2007
+       function api_date($str) {
+               // Wed May 23 06:01:13 +0000 2007
                return datetime_convert('UTC', 'UTC', $str, "D M d H:i:s +0000 Y" );
        }
 
@@ -95,28 +97,30 @@ use \Friendica\Core\Config;
         *
         * Register a function to be the endpont for defined API path.
         *
-        * @param string $path API URL path, relative to App::get_baseurl()
+        * @param string $path API URL path, relative to System::baseUrl()
         * @param string $func Function name to call on path request
         * @param bool $auth API need logged user
-        * @param string $method
-        *      HTTP method reqiured to call this endpoint.
-        *      One of API_METHOD_ANY, API_METHOD_GET, API_METHOD_POST.
-        *  Default to API_METHOD_ANY
+        * @param string $method HTTP method reqiured to call this endpoint.
+        *
+        * One of API_METHOD_ANY, API_METHOD_GET, API_METHOD_POST.
+        * Default to API_METHOD_ANY
         */
-       function api_register_func($path, $func, $auth=false, $method=API_METHOD_ANY){
+       function api_register_func($path, $func, $auth = false, $method = API_METHOD_ANY) {
                global $API;
+
                $API[$path] = array(
-                       'func'=>$func,
-                       'auth'=>$auth,
-                       'method'=> $method
+                       'func'   => $func,
+                       'auth'   => $auth,
+                       'method' => $method,
                );
 
                // Workaround for hotot
                $path = str_replace("api/", "api/1.1/", $path);
+
                $API[$path] = array(
-                       'func'=>$func,
-                       'auth'=>$auth,
-                       'method'=> $method
+                       'func'   => $func,
+                       'auth'   => $auth,
+                       'method' => $method,
                );
        }
 
@@ -136,34 +140,34 @@ use \Friendica\Core\Config;
         * @hook 'logged_in'
         *              array $user     logged user record
         */
-       function api_login(App $a){
+       function api_login(App $a) {
                // login with oauth
-               try{
+               try {
                        $oauth = new FKOAuth1();
                        list($consumer,$token) = $oauth->verify_request(OAuthRequest::from_request());
-                       if (!is_null($token)){
+                       if (!is_null($token)) {
                                $oauth->loginUser($token->uid);
                                call_hooks('logged_in', $a->user);
                                return;
                        }
-                       echo __file__.__line__.__function__."<pre>"; var_dump($consumer, $token); die();
-               }catch(Exception $e){
+                       echo __FILE__.__LINE__.__FUNCTION__ . "<pre>";
+                       var_dump($consumer, $token);
+                       die();
+               } catch (Exception $e) {
                        logger($e);
                }
 
-
-
                // workaround for HTTP-auth in CGI mode
-               if(x($_SERVER,'REDIRECT_REMOTE_USER')) {
-                       $userpass = base64_decode(substr($_SERVER["REDIRECT_REMOTE_USER"],6)) ;
-                       if(strlen($userpass)) {
+               if (x($_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 (!isset($_SERVER['PHP_AUTH_USER'])) {
+               if (!x($_SERVER, 'PHP_AUTH_USER')) {
                        logger('API_login: ' . print_r($_SERVER,true), LOGGER_DEBUG);
                        header('WWW-Authenticate: Basic realm="Friendica"');
                        throw new UnauthorizedException("This API requires login");
@@ -171,55 +175,51 @@ use \Friendica\Core\Config;
 
                $user = $_SERVER['PHP_AUTH_USER'];
                $password = $_SERVER['PHP_AUTH_PW'];
-               $encrypted = hash('whirlpool',trim($password));
+               $encrypted = hash('whirlpool', trim($password));
 
                // allow "user@server" login (but ignore 'server' part)
-               $at=strstr($user, "@", true);
-               if ( $at ) $user=$at;
+               $at = strstr($user, "@", true);
+               if ($at) {
+                       $user = $at;
+               }
 
-               /**
-                *  next code from mod/auth.php. needs better solution
-                */
+               // next code from mod/auth.php. needs better solution
                $record = null;
 
                $addon_auth = array(
                        'username' => trim($user),
                        'password' => trim($password),
                        'authenticated' => 0,
-                       'user_record' => null
+                       'user_record' => null,
                );
 
-               /**
-                *
+               /*
                 * A plugin indicates successful login by setting 'authenticated' to non-zero value and returning a user record
                 * Plugins should never set 'authenticated' except to indicate success - as hooks may be chained
                 * and later plugins should not interfere with an earlier one that succeeded.
-                *
                 */
-
                call_hooks('authenticate', $addon_auth);
 
-               if(($addon_auth['authenticated']) && (count($addon_auth['user_record']))) {
+               if (($addon_auth['authenticated']) && (count($addon_auth['user_record']))) {
                        $record = $addon_auth['user_record'];
-               }
-               else {
+               } else {
                        // process normal login request
-
                        $r = q("SELECT * FROM `user` WHERE (`email` = '%s' OR `nickname` = '%s')
                                AND `password` = '%s' AND NOT `blocked` AND NOT `account_expired` AND NOT `account_removed` AND `verified` LIMIT 1",
                                dbesc(trim($user)),
                                dbesc(trim($user)),
                                dbesc($encrypted)
                        );
-                       if (dbm::is_result($r))
+                       if (dbm::is_result($r)) {
                                $record = $r[0];
+                       }
                }
 
-               if((! $record) || (! count($record))) {
-                       logger('API_login failure: ' . print_r($_SERVER,true), LOGGER_DEBUG);
+               if ((! $record) || (! count($record))) {
+                       logger('API_login failure: ' . print_r($_SERVER, true), LOGGER_DEBUG);
                        header('WWW-Authenticate: Basic realm="Friendica"');
-                       #header('HTTP/1.0 401 Unauthorized');
-                       #die('This api requires login');
+                       //header('HTTP/1.0 401 Unauthorized');
+                       //die('This api requires login');
                        throw new UnauthorizedException("This API requires login");
                }
 
@@ -242,8 +242,10 @@ use \Friendica\Core\Config;
         * @return bool
         */
         function api_check_method($method) {
-               if ($method=="*") return True;
-               return strpos($method, $_SERVER['REQUEST_METHOD']) !== false;
+               if ($method == "*") {
+                       return true;
+               }
+               return (strpos($method, $_SERVER['REQUEST_METHOD']) !== false);
         }
 
        /**
@@ -254,39 +256,51 @@ use \Friendica\Core\Config;
         * @param App $a
         * @return string API call result
         */
-       function api_call(App $a){
+       function api_call(App $a) {
                global $API, $called_api;
 
-               $type="json";
-               if (strpos($a->query_string, ".xml")>0) $type="xml";
-               if (strpos($a->query_string, ".json")>0) $type="json";
-               if (strpos($a->query_string, ".rss")>0) $type="rss";
-               if (strpos($a->query_string, ".atom")>0) $type="atom";
+               $type = "json";
+               if (strpos($a->query_string, ".xml") > 0) {
+                       $type = "xml";
+               }
+               if (strpos($a->query_string, ".json") > 0) {
+                       $type = "json";
+               }
+               if (strpos($a->query_string, ".rss") > 0) {
+                       $type = "rss";
+               }
+               if (strpos($a->query_string, ".atom") > 0) {
+                       $type = "atom";
+               }
+
                try {
-                       foreach ($API as $p=>$info){
-                               if (strpos($a->query_string, $p)===0){
-                                       if (!api_check_method($info['method'])){
+                       foreach ($API as $p => $info) {
+                               if (strpos($a->query_string, $p) === 0) {
+                                       if (!api_check_method($info['method'])) {
                                                throw new MethodNotAllowedException();
                                        }
 
-                                       $called_api= explode("/",$p);
+                                       $called_api = explode("/", $p);
                                        //unset($_SERVER['PHP_AUTH_USER']);
-                                       if ($info['auth']===true && api_user()===false) {
-                                                       api_login($a);
+
+                                       /// @TODO should be "true ==[=] $info['auth']", if you miss only one = character, you assign a variable (only with ==). Let's make all this even.
+                                       if ($info['auth'] === true && api_user() === false) {
+                                               api_login($a);
                                        }
 
                                        logger('API call for ' . $a->user['username'] . ': ' . $a->query_string);
-                                       logger('API parameters: ' . print_r($_REQUEST,true));
+                                       logger('API parameters: ' . print_r($_REQUEST, true));
 
                                        $stamp =  microtime(true);
                                        $r = call_user_func($info['func'], $type);
-                                       $duration = (float)(microtime(true)-$stamp);
-                                       logger("API call duration: ".round($duration, 2)."\t".$a->query_string, LOGGER_DEBUG);
+                                       $duration = (float) (microtime(true) - $stamp);
+                                       logger("API call duration: " . round($duration, 2) . "\t" . $a->query_string, LOGGER_DEBUG);
 
                                        if (get_config("system", "profiler")) {
                                                $duration = microtime(true)-$a->performance["start"];
 
-                                               logger(parse_url($a->query_string, PHP_URL_PATH).": ".sprintf("Database: %s/%s, Network: %s, I/O: %s, Other: %s, Total: %s",
+                                               /// @TODO round() really everywhere?
+                                               logger(parse_url($a->query_string, PHP_URL_PATH) . ": " . sprintf("Database: %s/%s, Network: %s, I/O: %s, Other: %s, Total: %s",
                                                        round($a->performance["database"] - $a->performance["database_write"], 3),
                                                        round($a->performance["database_write"], 3),
                                                        round($a->performance["network"], 2),
@@ -294,65 +308,71 @@ use \Friendica\Core\Config;
                                                        round($duration - ($a->performance["database"] + $a->performance["network"]
                                                                + $a->performance["file"]), 2),
                                                        round($duration, 2)),
-                                                       LOGGER_DEBUG);
+                                                       LOGGER_DEBUG
+                                               );
 
                                                if (get_config("rendertime", "callstack")) {
                                                        $o = "Database Read:\n";
                                                        foreach ($a->callstack["database"] AS $func => $time) {
                                                                $time = round($time, 3);
-                                                               if ($time > 0)
-                                                                       $o .= $func.": ".$time."\n";
+                                                               if ($time > 0) {
+                                                                       $o .= $func . ": " . $time . "\n";
+                                                               }
                                                        }
                                                        $o .= "\nDatabase Write:\n";
                                                        foreach ($a->callstack["database_write"] AS $func => $time) {
                                                                $time = round($time, 3);
-                                                               if ($time > 0)
-                                                                       $o .= $func.": ".$time."\n";
+                                                               if ($time > 0) {
+                                                                       $o .= $func . ": " . $time . "\n";
+                                                               }
                                                        }
 
                                                        $o .= "\nNetwork:\n";
                                                        foreach ($a->callstack["network"] AS $func => $time) {
                                                                $time = round($time, 3);
-                                                               if ($time > 0)
-                                                                       $o .= $func.": ".$time."\n";
+                                                               if ($time > 0) {
+                                                                       $o .= $func . ": " . $time . "\n";
+                                                               }
                                                        }
                                                        logger($o, LOGGER_DEBUG);
                                                }
                                        }
 
-
-                                       if ($r===false) {
-                                               // api function returned false withour throw an
-                                               // exception. This should not happend, throw a 500
+                                       if (false === $r) {
+                                               /*
+                                                * api function returned false withour throw an
+                                                * exception. This should not happend, throw a 500
+                                                */
                                                throw new InternalServerErrorException();
                                        }
 
-                                       switch($type){
+                                       switch ($type) {
                                                case "xml":
                                                        header ("Content-Type: text/xml");
                                                        return $r;
                                                        break;
                                                case "json":
                                                        header ("Content-Type: application/json");
-                                                       foreach($r as $rr)
+                                                       foreach ($r as $rr)
                                                                $json = json_encode($rr);
-                                                               if ($_GET['callback'])
-                                                                       $json = $_GET['callback']."(".$json.")";
+                                                               if (x($_GET, 'callback')) {
+                                                                       $json = $_GET['callback'] . "(" . $json . ")";
+                                                               }
                                                                return $json;
                                                        break;
                                                case "rss":
                                                        header ("Content-Type: application/rss+xml");
-                                                       return '<?xml version="1.0" encoding="UTF-8"?>'."\n".$r;
+                                                       return '<?xml version="1.0" encoding="UTF-8"?>' . "\n" . $r;
                                                        break;
                                                case "atom":
                                                        header ("Content-Type: application/atom+xml");
-                                                       return '<?xml version="1.0" encoding="UTF-8"?>'."\n".$r;
+                                                       return '<?xml version="1.0" encoding="UTF-8"?>' . "\n" . $r;
                                                        break;
-
                                        }
                                }
                        }
-                       logger('API call not implemented: '.$a->query_string);
+
+                       logger('API call not implemented: ' . $a->query_string);
                        throw new NotImplementedException();
                } catch (HTTPException $e) {
                        header("HTTP/1.1 {$e->httpcode} {$e->httpdesc}");
@@ -371,16 +391,16 @@ use \Friendica\Core\Config;
 
                $a = get_app();
 
-               $error = ($e->getMessage()!==""?$e->getMessage():$e->httpdesc);
-               TODO:  https://dev.twitter.com/overview/api/response-codes
+               $error = ($e->getMessage() !== "" ? $e->getMessage() : $e->httpdesc);
+               /// @TODO:  https://dev.twitter.com/overview/api/response-codes
 
                $error = array("error" => $error,
-                               "code" => $e->httpcode." ".$e->httpdesc,
+                               "code" => $e->httpcode . " " . $e->httpdesc,
                                "request" => $a->query_string);
 
                $ret = api_format_data('status', $type, array('status' => $error));
 
-               switch($type){
+               switch ($type) {
                        case "xml":
                                header ("Content-Type: text/xml");
                                return $ret;
@@ -407,18 +427,22 @@ use \Friendica\Core\Config;
         * @param array $arr Array to be passed to template
         * @param array $user_info
         * @return array
+        * @todo find proper type-hints
         */
-       function api_rss_extra(App $a, $arr, $user_info){
-               if (is_null($user_info)) $user_info = api_get_user($a);
+       function api_rss_extra(App $a, $arr, $user_info) {
+               if (is_null($user_info)) {
+                       $user_info = api_get_user($a);
+               }
+
                $arr['$user'] = $user_info;
                $arr['$rss'] = array(
                        'alternate'    => $user_info['url'],
-                       'self'         => App::get_baseurl(). "/". $a->query_string,
-                       'base'         => App::get_baseurl(),
+                       'self'         => System::baseUrl() . "/" . $a->query_string,
+                       'base'         => System::baseUrl(),
                        'updated'      => api_date(null),
-                       'atom_updated' => datetime_convert('UTC','UTC','now',ATOM_TIME),
+                       'atom_updated' => datetime_convert('UTC', 'UTC', 'now', ATOM_TIME),
                        'language'     => $user_info['language'],
-                       'logo'         => App::get_baseurl()."/images/friendica-32.png",
+                       'logo'         => System::baseUrl() . "/images/friendica-32.png",
                );
 
                return $arr;
@@ -432,13 +456,14 @@ use \Friendica\Core\Config;
         * @return bool|string
         *              Contact url or False if contact id is unknown
         */
-       function api_unique_id_to_url($id){
-               $r = q("SELECT `url` FROM `contact` WHERE `uid` = 0 AND `id` = %d LIMIT 1",
-                       intval($id));
-               if ($r)
-                       return ($r[0]["url"]);
-               else
+       function api_unique_id_to_url($id) {
+               $r = dba::select('contact', array('url'), array('uid' => 0, 'id' => $id), array('limit' => 1));
+
+               if (dbm::is_result($r)) {
+                       return $r["url"];
+               } else {
                        return false;
+               }
        }
 
        /**
@@ -448,8 +473,9 @@ use \Friendica\Core\Config;
         * @param int|string $contact_id Contact ID or URL
         * @param string $type Return type (for errors)
         */
-       function api_get_user(App $a, $contact_id = Null, $type = "json"){
+       function api_get_user(App $a, $contact_id = null, $type = "json") {
                global $called_api;
+
                $user = null;
                $extra_query = "";
                $url = "";
@@ -458,68 +484,92 @@ use \Friendica\Core\Config;
                logger("api_get_user: Fetching user data for user ".$contact_id, LOGGER_DEBUG);
 
                // Searching for contact URL
-               if(!is_null($contact_id) AND (intval($contact_id) == 0)){
+               if (!is_null($contact_id) && (intval($contact_id) == 0)) {
                        $user = dbesc(normalise_link($contact_id));
                        $url = $user;
                        $extra_query = "AND `contact`.`nurl` = '%s' ";
-                       if (api_user()!==false)  $extra_query .= "AND `contact`.`uid`=".intval(api_user());
+                       if (api_user() !== false) {
+                               $extra_query .= "AND `contact`.`uid`=" . intval(api_user());
+                       }
                }
 
                // Searching for contact id with uid = 0
-               if(!is_null($contact_id) AND (intval($contact_id) != 0)){
+               if (!is_null($contact_id) && (intval($contact_id) != 0)) {
                        $user = dbesc(api_unique_id_to_url($contact_id));
 
-                       if ($user == "")
+                       if ($user == "") {
                                throw new BadRequestException("User not found.");
+                       }
 
                        $url = $user;
                        $extra_query = "AND `contact`.`nurl` = '%s' ";
-                       if (api_user()!==false)  $extra_query .= "AND `contact`.`uid`=".intval(api_user());
+                       if (api_user() !== false) {
+                               $extra_query .= "AND `contact`.`uid`=" . intval(api_user());
+                       }
                }
 
-               if(is_null($user) && x($_GET, 'user_id')) {
+               if (is_null($user) && x($_GET, 'user_id')) {
                        $user = dbesc(api_unique_id_to_url($_GET['user_id']));
 
-                       if ($user == "")
+                       if ($user == "") {
                                throw new BadRequestException("User not found.");
+                       }
 
                        $url = $user;
                        $extra_query = "AND `contact`.`nurl` = '%s' ";
-                       if (api_user()!==false)  $extra_query .= "AND `contact`.`uid`=".intval(api_user());
+                       if (api_user() !== false) {
+                               $extra_query .= "AND `contact`.`uid`=" . intval(api_user());
+                       }
                }
-               if(is_null($user) && x($_GET, 'screen_name')) {
+               if (is_null($user) && x($_GET, 'screen_name')) {
                        $user = dbesc($_GET['screen_name']);
                        $nick = $user;
                        $extra_query = "AND `contact`.`nick` = '%s' ";
-                       if (api_user()!==false)  $extra_query .= "AND `contact`.`uid`=".intval(api_user());
+                       if (api_user() !== false) {
+                               $extra_query .= "AND `contact`.`uid`=".intval(api_user());
+                       }
+               }
+
+               if (is_null($user) && x($_GET, 'profileurl')) {
+                       $user = dbesc(normalise_link($_GET['profileurl']));
+                       $nick = $user;
+                       $extra_query = "AND `contact`.`nurl` = '%s' ";
+                       if (api_user() !== false) {
+                               $extra_query .= "AND `contact`.`uid`=".intval(api_user());
+                       }
                }
 
-               if (is_null($user) AND ($a->argc > (count($called_api)-1)) AND (count($called_api) > 0)){
+               if (is_null($user) && ($a->argc > (count($called_api) - 1)) && (count($called_api) > 0)) {
                        $argid = count($called_api);
-                       list($user, $null) = explode(".",$a->argv[$argid]);
-                       if(is_numeric($user)){
+                       list($user, $null) = explode(".", $a->argv[$argid]);
+                       if (is_numeric($user)) {
                                $user = dbesc(api_unique_id_to_url($user));
 
-                               if ($user == "")
+                               if ($user == "") {
                                        return false;
+                               }
 
                                $url = $user;
                                $extra_query = "AND `contact`.`nurl` = '%s' ";
-                               if (api_user()!==false)  $extra_query .= "AND `contact`.`uid`=".intval(api_user());
+                               if (api_user() !== false) {
+                                       $extra_query .= "AND `contact`.`uid`=" . intval(api_user());
+                               }
                        } else {
                                $user = dbesc($user);
                                $nick = $user;
                                $extra_query = "AND `contact`.`nick` = '%s' ";
-                               if (api_user()!==false)  $extra_query .= "AND `contact`.`uid`=".intval(api_user());
+                               if (api_user() !== false) {
+                                       $extra_query .= "AND `contact`.`uid`=" . intval(api_user());
+                               }
                        }
                }
 
                logger("api_get_user: user ".$user, LOGGER_DEBUG);
 
                if (!$user) {
-                       if (api_user()===false) {
+                       if (api_user() === false) {
                                api_login($a);
-                               return False;
+                               return false;
                        } else {
                                $user = $_SESSION['uid'];
                                $extra_query = "AND `contact`.`uid` = %d AND `contact`.`self` ";
@@ -528,8 +578,9 @@ use \Friendica\Core\Config;
                }
 
                logger('api_user: ' . $extra_query . ', user: ' . $user);
+
                // user info
-               $uinfo = q("SELECT *, `contact`.`id` as `cid` FROM `contact`
+               $uinfo = q("SELECT *, `contact`.`id` AS `cid` FROM `contact`
                                WHERE 1
                                $extra_query",
                                $user
@@ -539,18 +590,20 @@ use \Friendica\Core\Config;
                api_best_nickname($uinfo);
 
                // if the contact wasn't found, fetch it from the contacts with uid = 0
-               if (count($uinfo)==0) {
+               if (!dbm::is_result($uinfo)) {
                        $r = array();
 
-                       if ($url != "")
+                       if ($url != "") {
                                $r = q("SELECT * FROM `contact` WHERE `uid` = 0 AND `nurl` = '%s' LIMIT 1", dbesc(normalise_link($url)));
+                       }
 
-                       if ($r) {
+                       if (dbm::is_result($r)) {
                                $network_name = network_to_name($r[0]['network'], $r[0]['url']);
 
                                // If no nick where given, extract it from the address
-                               if (($r[0]['nick'] == "") OR ($r[0]['name'] == $r[0]['nick']))
+                               if (($r[0]['nick'] == "") || ($r[0]['name'] == $r[0]['nick'])) {
                                        $r[0]['nick'] = api_get_nick($r[0]["url"]);
+                               }
 
                                $ret = array(
                                        'id' => $r[0]["id"],
@@ -594,18 +647,20 @@ use \Friendica\Core\Config;
                        }
                }
 
-               if($uinfo[0]['self']) {
+               if ($uinfo[0]['self']) {
 
-                       if ($uinfo[0]['network'] == "")
+                       if ($uinfo[0]['network'] == "") {
                                $uinfo[0]['network'] = NETWORK_DFRN;
+                       }
 
-                       $usr = q("select * from user where uid = %d limit 1",
+                       $usr = q("SELECT * FROM `user` WHERE `uid` = %d LIMIT 1",
                                intval(api_user())
                        );
-                       $profile = q("select * from profile where uid = %d and `is-default` = 1 limit 1",
+                       $profile = q("SELECT * FROM `profile` WHERE `uid` = %d AND `is-default` = 1 LIMIT 1",
                                intval(api_user())
                        );
 
+                       /// @TODO old-lost code? (twice)
                        // Counting is deactivated by now, due to performance issues
                        // count public wall messages
                        //$r = q("SELECT COUNT(*) as `count` FROM `item` WHERE `uid` = %d AND `wall`",
@@ -622,6 +677,8 @@ use \Friendica\Core\Config;
                        //$countitms = $r[0]['count'];
                        $countitms = 0;
                }
+
+               /// @TODO old-lost code? (twice)
 /*
                // Counting is deactivated by now, due to performance issues
                // count friends
@@ -649,7 +706,7 @@ use \Friendica\Core\Config;
                $starred = $r[0]['count'];
 
 
-               if(! $uinfo[0]['self']) {
+               if (! $uinfo[0]['self']) {
                        $countfriends = 0;
                        $countfollowers = 0;
                        $starred = 0;
@@ -660,7 +717,7 @@ use \Friendica\Core\Config;
                $starred = 0;
 
                // Add a nick if it isn't present there
-               if (($uinfo[0]['nick'] == "") OR ($uinfo[0]['name'] == $uinfo[0]['nick'])) {
+               if (($uinfo[0]['nick'] == "") || ($uinfo[0]['name'] == $uinfo[0]['nick'])) {
                        $uinfo[0]['nick'] = api_get_nick($uinfo[0]["url"]);
                }
 
@@ -668,7 +725,7 @@ use \Friendica\Core\Config;
 
                $pcontact_id  = get_contact($uinfo[0]['url'], 0, true);
 
-               $ret = Array(
+               $ret = array(
                        'id' => intval($pcontact_id),
                        'id_str' => (string) intval($pcontact_id),
                        'name' => (($uinfo[0]['name']) ? $uinfo[0]['name'] : $uinfo[0]['nick']),
@@ -693,11 +750,12 @@ use \Friendica\Core\Config;
                        'contributors_enabled' => false,
                        'is_translator' => false,
                        'is_translation_enabled' => false,
-                       'following' => (($uinfo[0]['rel'] == CONTACT_IS_FOLLOWER) OR ($uinfo[0]['rel'] == CONTACT_IS_FRIEND)),
+                       'following' => (($uinfo[0]['rel'] == CONTACT_IS_FOLLOWER) || ($uinfo[0]['rel'] == CONTACT_IS_FRIEND)),
                        'follow_request_sent' => false,
                        'statusnet_blocking' => false,
                        'notifications' => false,
-                       //'statusnet_profile_url' => App::get_baseurl()."/contacts/".$uinfo[0]['cid'],
+                       /// @TODO old way?
+                       //'statusnet_profile_url' => System::baseUrl()."/contacts/".$uinfo[0]['cid'],
                        'statusnet_profile_url' => $uinfo[0]['url'],
                        'uid' => intval($uinfo[0]['uid']),
                        'cid' => intval($uinfo[0]['cid']),
@@ -720,13 +778,17 @@ use \Friendica\Core\Config;
 
                $status_user = api_get_user($a, $item["author-link"]);
 
-               $status_user["protected"] = (($item["allow_cid"] != "") OR
-                                               ($item["allow_gid"] != "") OR
-                                               ($item["deny_cid"] != "") OR
-                                               ($item["deny_gid"] != "") OR
+               $status_user["protected"] = (($item["allow_cid"] != "") ||
+                                               ($item["allow_gid"] != "") ||
+                                               ($item["deny_cid"] != "") ||
+                                               ($item["deny_gid"] != "") ||
                                                $item["private"]);
 
-               $owner_user = api_get_user($a, $item["owner-link"]);
+               if ($item['thr-parent'] == $item['uri']) {
+                       $owner_user = api_get_user($a, $item["owner-link"]);
+               } else {
+                       $owner_user = $status_user;
+               }
 
                return (array($status_user, $owner_user));
        }
@@ -745,11 +807,13 @@ use \Friendica\Core\Config;
 
                foreach ($array as $k => $v) {
                        if (is_array($v)) {
-                               if ($callback($v, $k))
+                               if ($callback($v, $k)) {
                                        $new_array[$k] = api_walk_recursive($v, $callback);
+                               }
                        } else {
-                               if ($callback($v, $k))
+                               if ($callback($v, $k)) {
                                        $new_array[$k] = $v;
+                               }
                        }
                }
                $array = $new_array;
@@ -766,13 +830,16 @@ use \Friendica\Core\Config;
         * @return boolean Should the array item be deleted?
         */
        function api_reformat_xml(&$item, &$key) {
-               if (is_bool($item))
+               if (is_bool($item)) {
                        $item = ($item ? "true" : "false");
+               }
 
-               if (substr($key, 0, 10) == "statusnet_")
+               if (substr($key, 0, 10) == "statusnet_") {
                        $key = "statusnet:".substr($key, 10);
-               elseif (substr($key, 0, 10) == "friendica_")
+               } elseif (substr($key, 0, 10) == "friendica_") {
                        $key = "friendica:".substr($key, 10);
+               }
+               /// @TODO old-lost code?
                //else
                //      $key = "default:".$key;
 
@@ -798,18 +865,21 @@ use \Friendica\Core\Config;
                                        "georss" => "http://www.georss.org/georss");
 
                /// @todo Auto detection of needed namespaces
-               if (in_array($root_element, array("ok", "hash", "config", "version", "ids", "notes", "photos")))
+               if (in_array($root_element, array("ok", "hash", "config", "version", "ids", "notes", "photos"))) {
                        $namespaces = array();
+               }
 
-               if (is_array($data2))
+               if (is_array($data2)) {
                        api_walk_recursive($data2, "api_reformat_xml");
+               }
 
                if ($key == "0") {
                        $data4 = array();
                        $i = 1;
 
-                       foreach ($data2 AS $item)
+                       foreach ($data2 AS $item) {
                                $data4[$i++.":".$childname] = $item;
+                       }
 
                        $data2 = $data4;
                }
@@ -829,11 +899,11 @@ use \Friendica\Core\Config;
         *
         * @return (string|object) XML data or JSON data
         */
-       function api_format_data($root_element, $type, $data){
+       function api_format_data($root_element, $type, $data) {
 
                $a = get_app();
 
-               switch($type){
+               switch ($type) {
                        case "atom":
                        case "rss":
                        case "xml":
@@ -856,11 +926,13 @@ use \Friendica\Core\Config;
         * returns a 401 status code and an error message if not.
         * http://developer.twitter.com/doc/get/account/verify_credentials
         */
-       function api_account_verify_credentials($type){
+       function api_account_verify_credentials($type) {
 
                $a = get_app();
 
-               if (api_user()===false) throw new ForbiddenException();
+               if (api_user() === false) {
+                       throw new ForbiddenException();
+               }
 
                unset($_REQUEST["user_id"]);
                unset($_GET["user_id"]);
@@ -868,7 +940,7 @@ use \Friendica\Core\Config;
                unset($_REQUEST["screen_name"]);
                unset($_GET["screen_name"]);
 
-               $skip_status = (x($_REQUEST,'skip_status')?$_REQUEST['skip_status']:false);
+               $skip_status = (x($_REQUEST, 'skip_status')?$_REQUEST['skip_status'] : false);
 
                $user_info = api_get_user($a);
 
@@ -878,10 +950,11 @@ use \Friendica\Core\Config;
                // - Adding last status
                if (!$skip_status) {
                        $user_info["status"] = api_status_show("raw");
-                       if (!count($user_info["status"]))
+                       if (!count($user_info["status"])) {
                                unset($user_info["status"]);
-                       else
+                       } else {
                                unset($user_info["status"]["user"]);
+                       }
                }
 
                // "uid" and "self" are only needed for some internal stuff, so remove it from here
@@ -891,17 +964,18 @@ use \Friendica\Core\Config;
                return api_format_data("user", $type, array('user' => $user_info));
 
        }
-       api_register_func('api/account/verify_credentials','api_account_verify_credentials', true);
 
+       /// @TODO move to top of file or somwhere better
+       api_register_func('api/account/verify_credentials','api_account_verify_credentials', true);
 
        /**
         * get data from $_POST or $_GET
         */
-       function requestdata($k){
-               if (isset($_POST[$k])){
+       function requestdata($k) {
+               if (x($_POST, $k)) {
                        return $_POST[$k];
                }
-               if (isset($_GET[$k])){
+               if (x($_GET, $k)) {
                        return $_GET[$k];
                }
                return null;
@@ -912,7 +986,7 @@ use \Friendica\Core\Config;
 
                $a = get_app();
 
-               if (api_user()===false) {
+               if (api_user() === false) {
                        logger('api_statuses_update: no user');
                        throw new ForbiddenException();
                }
@@ -922,9 +996,10 @@ use \Friendica\Core\Config;
                $_REQUEST['profile_uid'] = api_user();
                $_REQUEST['api_source'] = true;
                $txt = requestdata('status');
+               /// @TODO old-lost code?
                //$txt = urldecode(requestdata('status'));
 
-               if((strpos($txt,'<') !== false) || (strpos($txt,'>') !== false)) {
+               if ((strpos($txt,'<') !== false) || (strpos($txt,'>') !== false)) {
 
                        $txt = html2bb_video($txt);
                        $config = HTMLPurifier_Config::createDefault();
@@ -936,25 +1011,26 @@ use \Friendica\Core\Config;
 
                $a->argv[1]=$user_info['screen_name']; //should be set to username?
 
-               $_REQUEST['hush']='yeah'; //tell wall_upload function to return img info instead of echo
+               // tell wall_upload function to return img info instead of echo
+               $_REQUEST['hush'] = 'yeah';
                $bebop = wall_upload_post($a);
 
-               //now that we have the img url in bbcode we can add it to the status and insert the wall item.
-               $_REQUEST['body']=$txt."\n\n".$bebop;
+               // now that we have the img url in bbcode we can add it to the status and insert the wall item.
+               $_REQUEST['body'] = $txt . "\n\n" . $bebop;
                item_post($a);
 
                // this should output the last post (the one we just posted).
                return api_status_show($type);
        }
-       api_register_func('api/statuses/mediap','api_statuses_mediap', true, API_METHOD_POST);
-/*Waitman Gobble Mod*/
 
+       /// @TODO move this to top of file or somewhere better!
+       api_register_func('api/statuses/mediap','api_statuses_mediap', true, API_METHOD_POST);
 
        function api_statuses_update($type) {
 
                $a = get_app();
 
-               if (api_user()===false) {
+               if (api_user() === false) {
                        logger('api_statuses_update: no user');
                        throw new ForbiddenException();
                }
@@ -965,9 +1041,9 @@ use \Friendica\Core\Config;
 
                // logger('api_post: ' . print_r($_POST,true));
 
-               if(requestdata('htmlstatus')) {
+               if (requestdata('htmlstatus')) {
                        $txt = requestdata('htmlstatus');
-                       if((strpos($txt,'<') !== false) || (strpos($txt,'>') !== false)) {
+                       if ((strpos($txt, '<') !== false) || (strpos($txt, '>') !== false)) {
                                $txt = html2bb_video($txt);
 
                                $config = HTMLPurifier_Config::createDefault();
@@ -979,29 +1055,33 @@ use \Friendica\Core\Config;
                                $_REQUEST['body'] = html2bbcode($txt);
                        }
 
-               } else
+               } else {
                        $_REQUEST['body'] = requestdata('status');
+               }
 
                $_REQUEST['title'] = requestdata('title');
 
                $parent = requestdata('in_reply_to_status_id');
 
                // Twidere sends "-1" if it is no reply ...
-               if ($parent == -1)
+               if ($parent == -1) {
                        $parent = "";
+               }
 
-               if(ctype_digit($parent))
+               if (ctype_digit($parent)) {
                        $_REQUEST['parent'] = $parent;
-               else
+               } else {
                        $_REQUEST['parent_uri'] = $parent;
+               }
 
-               if(requestdata('lat') && requestdata('long'))
-                       $_REQUEST['coord'] = sprintf("%s %s",requestdata('lat'),requestdata('long'));
+               if (requestdata('lat') && requestdata('long')) {
+                       $_REQUEST['coord'] = sprintf("%s %s", requestdata('lat'), requestdata('long'));
+               }
                $_REQUEST['profile_uid'] = api_user();
 
-               if($parent)
+               if ($parent) {
                        $_REQUEST['type'] = 'net-comment';
-               else {
+               else {
                        // Check for throttling (maximum posts per day, week and month)
                        $throttle_day = get_config('system','throttle_limit_day');
                        if ($throttle_day > 0) {
@@ -1011,14 +1091,15 @@ use \Friendica\Core\Config;
                                        AND `created` > '%s' AND `id` = `parent`",
                                        intval(api_user()), dbesc($datefrom));
 
-                               if ($r)
+                               if (dbm::is_result($r)) {
                                        $posts_day = $r[0]["posts_day"];
-                               else
+                               } else {
                                        $posts_day = 0;
+                               }
 
                                if ($posts_day > $throttle_day) {
                                        logger('Daily posting limit reached for user '.api_user(), LOGGER_DEBUG);
-                                       #die(api_error($type, sprintf(t("Daily posting limit of %d posts reached. The post was rejected."), $throttle_day)));
+                                       // die(api_error($type, sprintf(t("Daily posting limit of %d posts reached. The post was rejected."), $throttle_day)));
                                        throw new TooManyRequestsException(sprintf(t("Daily posting limit of %d posts reached. The post was rejected."), $throttle_day));
                                }
                        }
@@ -1031,16 +1112,16 @@ use \Friendica\Core\Config;
                                        AND `created` > '%s' AND `id` = `parent`",
                                        intval(api_user()), dbesc($datefrom));
 
-                               if ($r)
+                               if (dbm::is_result($r)) {
                                        $posts_week = $r[0]["posts_week"];
-                               else
+                               } else {
                                        $posts_week = 0;
+                               }
 
                                if ($posts_week > $throttle_week) {
                                        logger('Weekly posting limit reached for user '.api_user(), LOGGER_DEBUG);
-                                       #die(api_error($type, sprintf(t("Weekly posting limit of %d posts reached. The post was rejected."), $throttle_week)));
+                                       // die(api_error($type, sprintf(t("Weekly posting limit of %d posts reached. The post was rejected."), $throttle_week)));
                                        throw new TooManyRequestsException(sprintf(t("Weekly posting limit of %d posts reached. The post was rejected."), $throttle_week));
-
                                }
                        }
 
@@ -1052,14 +1133,15 @@ use \Friendica\Core\Config;
                                        AND `created` > '%s' AND `id` = `parent`",
                                        intval(api_user()), dbesc($datefrom));
 
-                               if ($r)
+                               if (dbm::is_result($r)) {
                                        $posts_month = $r[0]["posts_month"];
-                               else
+                               } else {
                                        $posts_month = 0;
+                               }
 
                                if ($posts_month > $throttle_month) {
                                        logger('Monthly posting limit reached for user '.api_user(), LOGGER_DEBUG);
-                                       #die(api_error($type, sprintf(t("Monthly posting limit of %d posts reached. The post was rejected."), $throttle_month)));
+                                       // die(api_error($type, sprintf(t("Monthly posting limit of %d posts reached. The post was rejected."), $throttle_month)));
                                        throw new TooManyRequestsException(sprintf(t("Monthly posting limit of %d posts reached. The post was rejected."), $throttle_month));
                                }
                        }
@@ -1067,23 +1149,24 @@ use \Friendica\Core\Config;
                        $_REQUEST['type'] = 'wall';
                }
 
-               if(x($_FILES,'media')) {
+               if (x($_FILES, 'media')) {
                        // upload the image if we have one
-                       $_REQUEST['hush']='yeah'; //tell wall_upload function to return img info instead of echo
+                       $_REQUEST['hush'] = 'yeah'; //tell wall_upload function to return img info instead of echo
                        $media = wall_upload_post($a);
-                       if(strlen($media)>0)
-                               $_REQUEST['body'] .= "\n\n".$media;
+                       if (strlen($media) > 0) {
+                               $_REQUEST['body'] .= "\n\n" . $media;
+                       }
                }
 
                // To-Do: Multiple IDs
                if (requestdata('media_ids')) {
                        $r = q("SELECT `resource-id`, `scale`, `nickname`, `type` FROM `photo` INNER JOIN `user` ON `user`.`uid` = `photo`.`uid` WHERE `resource-id` IN (SELECT `resource-id` FROM `photo` WHERE `id` = %d) AND `scale` > 0 AND `photo`.`uid` = %d ORDER BY `photo`.`width` DESC LIMIT 1",
                                intval(requestdata('media_ids')), api_user());
-                       if ($r) {
+                       if (dbm::is_result($r)) {
                                $phototypes = Photo::supportedTypes();
                                $ext = $phototypes[$r[0]['type']];
-                               $_REQUEST['body'] .= "\n\n".'[url='.App::get_baseurl().'/photos/'.$r[0]['nickname'].'/image/'.$r[0]['resource-id'].']';
-                               $_REQUEST['body'] .= '[img]'.App::get_baseurl()."/photo/".$r[0]['resource-id']."-".$r[0]['scale'].".".$ext."[/img][/url]";
+                               $_REQUEST['body'] .= "\n\n" . '[url=' . System::baseUrl() . '/photos/' . $r[0]['nickname'] . '/image/' . $r[0]['resource-id'] . ']';
+                               $_REQUEST['body'] .= '[img]' . System::baseUrl() . '/photo/' . $r[0]['resource-id'] . '-' . $r[0]['scale'] . '.' . $ext . '[/img][/url]';
                        }
                }
 
@@ -1091,38 +1174,39 @@ use \Friendica\Core\Config;
 
                $_REQUEST['api_source'] = true;
 
-               if (!x($_REQUEST, "source"))
+               if (!x($_REQUEST, "source")) {
                        $_REQUEST["source"] = api_source();
+               }
 
                // call out normal post function
-
                item_post($a);
 
                // this should output the last post (the one we just posted).
                return api_status_show($type);
        }
+
+       /// @TODO move to top of file or somwhere better
        api_register_func('api/statuses/update','api_statuses_update', true, API_METHOD_POST);
        api_register_func('api/statuses/update_with_media','api_statuses_update', true, API_METHOD_POST);
 
-
        function api_media_upload($type) {
 
                $a = get_app();
 
-               if (api_user()===false) {
+               if (api_user() === false) {
                        logger('no user');
                        throw new ForbiddenException();
                }
 
                $user_info = api_get_user($a);
 
-               if(!x($_FILES,'media')) {
+               if (!x($_FILES, 'media')) {
                        // Output error
                        throw new BadRequestException("No media.");
                }
 
                $media = wall_upload_post($a, false);
-               if(!$media) {
+               if (!$media) {
                        // Output error
                        throw new InternalServerErrorException();
                }
@@ -1135,13 +1219,15 @@ use \Friendica\Core\Config;
                                                "h" => $media["height"],
                                                "image_type" => $media["type"]);
 
-               logger("Media uploaded: ".print_r($returndata, true), LOGGER_DEBUG);
+               logger("Media uploaded: " . print_r($returndata, true), LOGGER_DEBUG);
 
                return array("media" => $returndata);
        }
+
+       /// @TODO move to top of file or somwhere better
        api_register_func('api/media/upload','api_media_upload', true, API_METHOD_POST);
 
-       function api_status_show($type){
+       function api_status_show($type) {
 
                $a = get_app();
 
@@ -1149,10 +1235,11 @@ use \Friendica\Core\Config;
 
                logger('api_status_show: user_info: '.print_r($user_info, true), LOGGER_DEBUG);
 
-               if ($type == "raw")
+               if ($type == "raw") {
                        $privacy_sql = "AND `item`.`allow_cid`='' AND `item`.`allow_gid`='' AND `item`.`deny_cid`='' AND `item`.`deny_gid`=''";
-               else
+               } else {
                        $privacy_sql = "";
+               }
 
                // get last public wall message
                $lastwall = q("SELECT `item`.*
@@ -1170,17 +1257,18 @@ use \Friendica\Core\Config;
                                dbesc(normalise_link($user_info['url']))
                );
 
-               if (count($lastwall)>0){
+               if (dbm::is_result($lastwall)) {
                        $lastwall = $lastwall[0];
 
                        $in_reply_to = api_in_reply_to($lastwall);
 
                        $converted = api_convert_item($lastwall);
 
-                       if ($type == "xml")
+                       if ($type == "xml") {
                                $geo = "georss:point";
-                       else
+                       } else {
                                $geo = "geo";
+                       }
 
                        $status_info = array(
                                'created_at' => api_date($lastwall['created']),
@@ -1210,16 +1298,19 @@ use \Friendica\Core\Config;
                                'statusnet_conversation_id'     => $lastwall['parent'],
                        );
 
-                       if (count($converted["attachments"]) > 0)
+                       if (count($converted["attachments"]) > 0) {
                                $status_info["attachments"] = $converted["attachments"];
+                       }
 
-                       if (count($converted["entities"]) > 0)
+                       if (count($converted["entities"]) > 0) {
                                $status_info["entities"] = $converted["entities"];
+                       }
 
-                       if (($lastwall['item_network'] != "") AND ($status["source"] == 'web'))
+                       if (($lastwall['item_network'] != "") && ($status["source"] == 'web')) {
                                $status_info["source"] = network_to_name($lastwall['item_network'], $user_info['url']);
-                       elseif (($lastwall['item_network'] != "") AND (network_to_name($lastwall['item_network'], $user_info['url']) != $status_info["source"]))
+                       } elseif (($lastwall['item_network'] != "") && (network_to_name($lastwall['item_network'], $user_info['url']) != $status_info["source"])) {
                                $status_info["source"] = trim($status_info["source"].' ('.network_to_name($lastwall['item_network'], $user_info['url']).')');
+                       }
 
                        // "uid" and "self" are only needed for some internal stuff, so remove it from here
                        unset($status_info["user"]["uid"]);
@@ -1228,23 +1319,20 @@ use \Friendica\Core\Config;
 
                logger('status_info: '.print_r($status_info, true), LOGGER_DEBUG);
 
-               if ($type == "raw")
-                       return($status_info);
+               if ($type == "raw") {
+                       return $status_info;
+               }
 
-               return  api_format_data("statuses", $type, array('status' => $status_info));
+               return api_format_data("statuses", $type, array('status' => $status_info));
 
        }
 
-
-
-
-
        /**
         * Returns extended information of a given user, specified by ID or screen name as per the required id parameter.
         * The author's most recent status will be returned inline.
         * http://developer.twitter.com/doc/get/users/show
         */
-       function api_users_show($type){
+       function api_users_show($type) {
 
                $a = get_app();
 
@@ -1267,17 +1355,18 @@ use \Friendica\Core\Config;
                                dbesc(normalise_link($user_info['url']))
                );
 
-               if (count($lastwall)>0){
+               if (dbm::is_result($lastwall)) {
                        $lastwall = $lastwall[0];
 
                        $in_reply_to = api_in_reply_to($lastwall);
 
                        $converted = api_convert_item($lastwall);
 
-                       if ($type == "xml")
+                       if ($type == "xml") {
                                $geo = "georss:point";
-                       else
+                       } else {
                                $geo = "geo";
+                       }
 
                        $user_info['status'] = array(
                                'text' => $converted["text"],
@@ -1293,20 +1382,25 @@ use \Friendica\Core\Config;
                                'in_reply_to_screen_name' => $in_reply_to['screen_name'],
                                $geo => NULL,
                                'favorited' => $lastwall['starred'] ? true : false,
-                               'statusnet_html'                => $converted["html"],
+                               'statusnet_html' => $converted["html"],
                                'statusnet_conversation_id'     => $lastwall['parent'],
                        );
 
-                       if (count($converted["attachments"]) > 0)
+                       if (count($converted["attachments"]) > 0) {
                                $user_info["status"]["attachments"] = $converted["attachments"];
+                       }
 
-                       if (count($converted["entities"]) > 0)
+                       if (count($converted["entities"]) > 0) {
                                $user_info["status"]["entities"] = $converted["entities"];
+                       }
 
-                       if (($lastwall['item_network'] != "") AND ($user_info["status"]["source"] == 'web'))
+                       if (($lastwall['item_network'] != "") && ($user_info["status"]["source"] == 'web')) {
                                $user_info["status"]["source"] = network_to_name($lastwall['item_network'], $user_info['url']);
-                       if (($lastwall['item_network'] != "") AND (network_to_name($lastwall['item_network'], $user_info['url']) != $user_info["status"]["source"]))
-                               $user_info["status"]["source"] = trim($user_info["status"]["source"].' ('.network_to_name($lastwall['item_network'], $user_info['url']).')');
+                       }
+
+                       if (($lastwall['item_network'] != "") && (network_to_name($lastwall['item_network'], $user_info['url']) != $user_info["status"]["source"])) {
+                               $user_info["status"]["source"] = trim($user_info["status"]["source"] . ' (' . network_to_name($lastwall['item_network'], $user_info['url']) . ')');
+                       }
 
                }
 
@@ -1314,34 +1408,39 @@ use \Friendica\Core\Config;
                unset($user_info["uid"]);
                unset($user_info["self"]);
 
-               return  api_format_data("user", $type, array('user' => $user_info));
+               return api_format_data("user", $type, array('user' => $user_info));
 
        }
-       api_register_func('api/users/show','api_users_show');
 
+       /// @TODO move to top of file or somewhere better
+       api_register_func('api/users/show','api_users_show');
+       api_register_func('api/externalprofile/show','api_users_show');
 
        function api_users_search($type) {
 
                $a = get_app();
 
-               $page = (x($_REQUEST,'page')?$_REQUEST['page']-1:0);
+               $page = (x($_REQUEST, 'page') ? $_REQUEST['page'] - 1 : 0);
 
                $userlist = array();
 
-               if (isset($_GET["q"])) {
+               if (x($_GET, 'q')) {
                        $r = q("SELECT id FROM `contact` WHERE `uid` = 0 AND `name` = '%s'", dbesc($_GET["q"]));
-                       if (!dbm::is_result($r))
+
+                       if (!dbm::is_result($r)) {
                                $r = q("SELECT `id` FROM `contact` WHERE `uid` = 0 AND `nick` = '%s'", dbesc($_GET["q"]));
+                       }
 
                        if (dbm::is_result($r)) {
                                $k = 0;
                                foreach ($r AS $user) {
                                        $user_info = api_get_user($a, $user["id"], "json");
 
-                                       if ($type == "xml")
+                                       if ($type == "xml") {
                                                $userlist[$k++.":user"] = $user_info;
-                                       else
+                                       } else {
                                                $userlist[] = $user_info;
+                                       }
                                }
                                $userlist = array("users" => $userlist);
                        } else {
@@ -1353,6 +1452,7 @@ use \Friendica\Core\Config;
                return api_format_data("users", $type, $userlist);
        }
 
+       /// @TODO move to top of file or somewhere better
        api_register_func('api/users/search','api_users_search');
 
        /**
@@ -1362,11 +1462,13 @@ use \Friendica\Core\Config;
         * TODO: Optional parameters
         * TODO: Add reply info
         */
-       function api_statuses_home_timeline($type){
+       function api_statuses_home_timeline($type) {
 
                $a = get_app();
 
-               if (api_user()===false) throw new ForbiddenException();
+               if (api_user() === false) {
+                       throw new ForbiddenException();
+               }
 
                unset($_REQUEST["user_id"]);
                unset($_GET["user_id"]);
@@ -1378,24 +1480,29 @@ use \Friendica\Core\Config;
                // get last newtork messages
 
                // params
-               $count = (x($_REQUEST,'count')?$_REQUEST['count']:20);
-               $page = (x($_REQUEST,'page')?$_REQUEST['page']-1:0);
-               if ($page<0) $page=0;
-               $since_id = (x($_REQUEST,'since_id')?$_REQUEST['since_id']:0);
-               $max_id = (x($_REQUEST,'max_id')?$_REQUEST['max_id']:0);
-               //$since_id = 0;//$since_id = (x($_REQUEST,'since_id')?$_REQUEST['since_id']:0);
-               $exclude_replies = (x($_REQUEST,'exclude_replies')?1:0);
-               $conversation_id = (x($_REQUEST,'conversation_id')?$_REQUEST['conversation_id']:0);
+               $count = (x($_REQUEST, 'count') ? $_REQUEST['count'] : 20);
+               $page = (x($_REQUEST, 'page') ? $_REQUEST['page'] - 1 : 0);
+               if ($page < 0) {
+                       $page = 0;
+               }
+               $since_id = (x($_REQUEST, 'since_id') ? $_REQUEST['since_id'] : 0);
+               $max_id = (x($_REQUEST, 'max_id') ? $_REQUEST['max_id'] : 0);
+               //$since_id = 0;//$since_id = (x($_REQUEST, 'since_id')?$_REQUEST['since_id'] : 0);
+               $exclude_replies = (x($_REQUEST, 'exclude_replies') ? 1 : 0);
+               $conversation_id = (x($_REQUEST, 'conversation_id') ? $_REQUEST['conversation_id'] : 0);
 
-               $start = $page*$count;
+               $start = $page * $count;
 
                $sql_extra = '';
-               if ($max_id > 0)
-                       $sql_extra .= ' AND `item`.`id` <= '.intval($max_id);
-               if ($exclude_replies > 0)
+               if ($max_id > 0) {
+                       $sql_extra .= ' AND `item`.`id` <= ' . intval($max_id);
+               }
+               if ($exclude_replies > 0) {
                        $sql_extra .= ' AND `item`.`parent` = `item`.`id`';
-               if ($conversation_id > 0)
-                       $sql_extra .= ' AND `item`.`parent` = '.intval($conversation_id);
+               }
+               if ($conversation_id > 0) {
+                       $sql_extra .= ' AND `item`.`parent` = ' . intval($conversation_id);
+               }
 
                $r = q("SELECT `item`.*, `item`.`id` AS `item_id`, `item`.`network` AS `item_network`,
                        `contact`.`name`, `contact`.`photo`, `contact`.`url`, `contact`.`rel`,
@@ -1415,63 +1522,73 @@ use \Friendica\Core\Config;
                        intval($start), intval($count)
                );
 
-               $ret = api_format_items($r,$user_info, false, $type);
+               $ret = api_format_items($r, $user_info, false, $type);
 
                // Set all posts from the query above to seen
                $idarray = array();
-               foreach ($r AS $item)
+               foreach ($r AS $item) {
                        $idarray[] = intval($item["id"]);
+               }
 
                $idlist = implode(",", $idarray);
 
                if ($idlist != "") {
                        $unseen = q("SELECT `id` FROM `item` WHERE `unseen` AND `id` IN (%s)", $idlist);
 
-                       if ($unseen)
+                       if ($unseen) {
                                $r = q("UPDATE `item` SET `unseen` = 0 WHERE `unseen` AND `id` IN (%s)", $idlist);
+                       }
                }
 
                $data = array('status' => $ret);
-               switch($type){
+               switch ($type) {
                        case "atom":
                        case "rss":
                                $data = api_rss_extra($a, $data, $user_info);
                                break;
                }
 
-               return  api_format_data("statuses", $type, $data);
+               return api_format_data("statuses", $type, $data);
        }
+
+       /// @TODO move to top of file or somewhere better
        api_register_func('api/statuses/home_timeline','api_statuses_home_timeline', true);
        api_register_func('api/statuses/friends_timeline','api_statuses_home_timeline', true);
 
-       function api_statuses_public_timeline($type){
+       function api_statuses_public_timeline($type) {
 
                $a = get_app();
 
-               if (api_user()===false) throw new ForbiddenException();
+               if (api_user() === false) {
+                       throw new ForbiddenException();
+               }
 
                $user_info = api_get_user($a);
                // get last newtork messages
 
-
                // params
-               $count = (x($_REQUEST,'count')?$_REQUEST['count']:20);
-               $page = (x($_REQUEST,'page')?$_REQUEST['page']-1:0);
-               if ($page<0) $page=0;
-               $since_id = (x($_REQUEST,'since_id')?$_REQUEST['since_id']:0);
-               $max_id = (x($_REQUEST,'max_id')?$_REQUEST['max_id']:0);
-               //$since_id = 0;//$since_id = (x($_REQUEST,'since_id')?$_REQUEST['since_id']:0);
-               $exclude_replies = (x($_REQUEST,'exclude_replies')?1:0);
-               $conversation_id = (x($_REQUEST,'conversation_id')?$_REQUEST['conversation_id']:0);
+               $count = (x($_REQUEST, 'count') ? $_REQUEST['count'] : 20);
+               $page = (x($_REQUEST, 'page') ? $_REQUEST['page'] -1 : 0);
+               if ($page < 0) {
+                       $page = 0;
+               }
+               $since_id = (x($_REQUEST, 'since_id') ? $_REQUEST['since_id'] : 0);
+               $max_id = (x($_REQUEST, 'max_id') ? $_REQUEST['max_id'] : 0);
+               //$since_id = 0;//$since_id = (x($_REQUEST, 'since_id')?$_REQUEST['since_id'] : 0);
+               $exclude_replies = (x($_REQUEST, 'exclude_replies') ? 1 : 0);
+               $conversation_id = (x($_REQUEST, 'conversation_id') ? $_REQUEST['conversation_id'] : 0);
 
-               $start = $page*$count;
+               $start = $page * $count;
 
-               if ($max_id > 0)
-                       $sql_extra = 'AND `item`.`id` <= '.intval($max_id);
-               if ($exclude_replies > 0)
+               if ($max_id > 0) {
+                       $sql_extra = 'AND `item`.`id` <= ' . intval($max_id);
+               }
+               if ($exclude_replies > 0) {
                        $sql_extra .= ' AND `item`.`parent` = `item`.`id`';
-               if ($conversation_id > 0)
-                       $sql_extra .= ' AND `item`.`parent` = '.intval($conversation_id);
+               }
+               if ($conversation_id > 0) {
+                       $sql_extra .= ' AND `item`.`parent` = ' . intval($conversation_id);
+               }
 
                $r = q("SELECT `item`.*, `item`.`id` AS `item_id`, `item`.`network` AS `item_network`,
                        `contact`.`name`, `contact`.`photo`, `contact`.`url`, `contact`.`rel`,
@@ -1495,51 +1612,57 @@ use \Friendica\Core\Config;
                        intval($start),
                        intval($count));
 
-               $ret = api_format_items($r,$user_info, false, $type);
-
+               $ret = api_format_items($r, $user_info, false, $type);
 
                $data = array('status' => $ret);
-               switch($type){
+               switch ($type) {
                        case "atom":
                        case "rss":
                                $data = api_rss_extra($a, $data, $user_info);
                                break;
                }
 
-               return  api_format_data("statuses", $type, $data);
+               return api_format_data("statuses", $type, $data);
        }
+
+       /// @TODO move to top of file or somewhere better
        api_register_func('api/statuses/public_timeline','api_statuses_public_timeline', true);
 
        /**
-        *
+        * @TODO nothing to say?
         */
-       function api_statuses_show($type){
+       function api_statuses_show($type) {
 
                $a = get_app();
 
-               if (api_user()===false) throw new ForbiddenException();
+               if (api_user() === false) {
+                       throw new ForbiddenException();
+               }
 
                $user_info = api_get_user($a);
 
                // params
                $id = intval($a->argv[3]);
 
-               if ($id == 0)
+               if ($id == 0) {
                        $id = intval($_REQUEST["id"]);
+               }
 
                // Hotot workaround
-               if ($id == 0)
+               if ($id == 0) {
                        $id = intval($a->argv[4]);
+               }
 
-               logger('API: api_statuses_show: '.$id);
+               logger('API: api_statuses_show: ' . $id);
 
-               $conversation = (x($_REQUEST,'conversation')?1:0);
+               $conversation = (x($_REQUEST, 'conversation') ? 1 : 0);
 
                $sql_extra = '';
-               if ($conversation)
+               if ($conversation) {
                        $sql_extra .= " AND `item`.`parent` = %d ORDER BY `id` ASC ";
-               else
+               } else {
                        $sql_extra .= " AND `item`.`id` = %d";
+               }
 
                $r = q("SELECT `item`.*, `item`.`id` AS `item_id`, `item`.`network` AS `item_network`,
                        `contact`.`name`, `contact`.`photo`, `contact`.`url`, `contact`.`rel`,
@@ -1556,7 +1679,8 @@ use \Friendica\Core\Config;
                        intval($id)
                );
 
-               if (!$r) {
+               /// @TODO How about copying this to above methods which don't check $r ?
+               if (!dbm::is_result($r)) {
                        throw new BadRequestException("There is no status with this id.");
                }
 
@@ -1567,50 +1691,59 @@ use \Friendica\Core\Config;
                        return api_format_data("statuses", $type, $data);
                } else {
                        $data = array('status' => $ret[0]);
-                       return  api_format_data("status", $type, $data);
+                       return api_format_data("status", $type, $data);
                }
        }
-       api_register_func('api/statuses/show','api_statuses_show', true);
 
+       /// @TODO move to top of file or somewhere better
+       api_register_func('api/statuses/show','api_statuses_show', true);
 
        /**
-        *
+        * @TODO nothing to say?
         */
-       function api_conversation_show($type){
+       function api_conversation_show($type) {
 
                $a = get_app();
 
-               if (api_user()===false) throw new ForbiddenException();
+               if (api_user() === false) {
+                       throw new ForbiddenException();
+               }
 
                $user_info = api_get_user($a);
 
                // params
                $id = intval($a->argv[3]);
-               $count = (x($_REQUEST,'count')?$_REQUEST['count']:20);
-               $page = (x($_REQUEST,'page')?$_REQUEST['page']-1:0);
-               if ($page<0) $page=0;
-               $since_id = (x($_REQUEST,'since_id')?$_REQUEST['since_id']:0);
-               $max_id = (x($_REQUEST,'max_id')?$_REQUEST['max_id']:0);
+               $count = (x($_REQUEST, 'count') ? $_REQUEST['count'] : 20);
+               $page = (x($_REQUEST, 'page') ? $_REQUEST['page'] - 1 : 0);
+               if ($page < 0) {
+                       $page = 0;
+               }
+               $since_id = (x($_REQUEST, 'since_id') ? $_REQUEST['since_id'] : 0);
+               $max_id = (x($_REQUEST, 'max_id') ? $_REQUEST['max_id'] : 0);
 
                $start = $page*$count;
 
-               if ($id == 0)
+               if ($id == 0) {
                        $id = intval($_REQUEST["id"]);
+               }
 
                // Hotot workaround
-               if ($id == 0)
+               if ($id == 0) {
                        $id = intval($a->argv[4]);
+               }
 
                logger('API: api_conversation_show: '.$id);
 
                $r = q("SELECT `parent` FROM `item` WHERE `id` = %d", intval($id));
-               if ($r)
+               if (dbm::is_result($r)) {
                        $id = $r[0]["parent"];
+               }
 
                $sql_extra = '';
 
-               if ($max_id > 0)
-                       $sql_extra = ' AND `item`.`id` <= '.intval($max_id);
+               if ($max_id > 0) {
+                       $sql_extra = ' AND `item`.`id` <= ' . intval($max_id);
+               }
 
                // Not sure why this query was so complicated. We should keep it here for a while,
                // just to make sure that we really don't need it.
@@ -1635,39 +1768,45 @@ use \Friendica\Core\Config;
                        intval($start), intval($count)
                );
 
-               if (!$r)
-                       throw new BadRequestException("There is no conversation with this id.");
+               if (!dbm::is_result($r)) {
+                       throw new BadRequestException("There is no status with this id.");
+               }
 
-               $ret = api_format_items($r,$user_info, false, $type);
+               $ret = api_format_items($r, $user_info, false, $type);
 
                $data = array('status' => $ret);
                return api_format_data("statuses", $type, $data);
        }
+
+       /// @TODO move to top of file or somewhere better
        api_register_func('api/conversation/show','api_conversation_show', true);
        api_register_func('api/statusnet/conversation','api_conversation_show', true);
 
-
        /**
-        *
+        * @TODO nothing to say?
         */
-       function api_statuses_repeat($type){
+       function api_statuses_repeat($type) {
                global $called_api;
 
                $a = get_app();
 
-               if (api_user()===false) throw new ForbiddenException();
+               if (api_user() === false) {
+                       throw new ForbiddenException();
+               }
 
                $user_info = api_get_user($a);
 
                // params
                $id = intval($a->argv[3]);
 
-               if ($id == 0)
+               if ($id == 0) {
                        $id = intval($_REQUEST["id"]);
+               }
 
                // Hotot workaround
-               if ($id == 0)
+               if ($id == 0) {
                        $id = intval($a->argv[4]);
+               }
 
                logger('API: api_statuses_repeat: '.$id);
 
@@ -1679,14 +1818,15 @@ use \Friendica\Core\Config;
                        INNER JOIN `contact` ON `contact`.`id` = `item`.`contact-id` AND `contact`.`uid` = `item`.`uid`
                                AND (NOT `contact`.`blocked` OR `contact`.`pending`)
                        WHERE `item`.`visible` AND NOT `item`.`moderated` AND NOT `item`.`deleted`
-                       AND NOT `item`.`private` AND `item`.`allow_cid` = '' AND `item`.`allow`.`gid` = ''
+                       AND NOT `item`.`private` AND `item`.`allow_cid` = '' AND `item`.`allow_gid` = ''
                        AND `item`.`deny_cid` = '' AND `item`.`deny_gid` = ''
                        $sql_extra
                        AND `item`.`id`=%d",
                        intval($id)
                );
 
-               if ($r[0]['body'] != "") {
+               /// @TODO other style than above functions!
+               if (dbm::is_result($r) && $r[0]['body'] != "") {
                        if (strpos($r[0]['body'], "[/share]") !== false) {
                                $pos = strpos($r[0]['body'], "[share");
                                $post = substr($r[0]['body'], $pos);
@@ -1701,39 +1841,47 @@ use \Friendica\Core\Config;
                        $_REQUEST['type'] = 'wall';
                        $_REQUEST['api_source'] = true;
 
-                       if (!x($_REQUEST, "source"))
+                       if (!x($_REQUEST, "source")) {
                                $_REQUEST["source"] = api_source();
+                       }
 
                        item_post($a);
-               } else
+               } else {
                        throw new ForbiddenException();
+               }
 
                // this should output the last post (the one we just posted).
                $called_api = null;
-               return(api_status_show($type));
+               return api_status_show($type);
        }
+
+       /// @TODO move to top of file or somewhere better
        api_register_func('api/statuses/retweet','api_statuses_repeat', true, API_METHOD_POST);
 
        /**
-        *
+        * @TODO nothing to say?
         */
-       function api_statuses_destroy($type){
+       function api_statuses_destroy($type) {
 
                $a = get_app();
 
-               if (api_user()===false) throw new ForbiddenException();
+               if (api_user() === false) {
+                       throw new ForbiddenException();
+               }
 
                $user_info = api_get_user($a);
 
                // params
                $id = intval($a->argv[3]);
 
-               if ($id == 0)
+               if ($id == 0) {
                        $id = intval($_REQUEST["id"]);
+               }
 
                // Hotot workaround
-               if ($id == 0)
+               if ($id == 0) {
                        $id = intval($a->argv[4]);
+               }
 
                logger('API: api_statuses_destroy: '.$id);
 
@@ -1741,20 +1889,23 @@ use \Friendica\Core\Config;
 
                drop_item($id, false);
 
-               return($ret);
+               return $ret;
        }
+
+       /// @TODO move to top of file or somewhere better
        api_register_func('api/statuses/destroy','api_statuses_destroy', true, API_METHOD_DELETE);
 
        /**
-        *
+        * @TODO Nothing more than an URL to say?
         * http://developer.twitter.com/doc/get/statuses/mentions
-        *
         */
-       function api_statuses_mentions($type){
+       function api_statuses_mentions($type) {
 
                $a = get_app();
 
-               if (api_user()===false) throw new ForbiddenException();
+               if (api_user() === false) {
+                       throw new ForbiddenException();
+               }
 
                unset($_REQUEST["user_id"]);
                unset($_GET["user_id"]);
@@ -1767,24 +1918,27 @@ use \Friendica\Core\Config;
 
 
                // params
-               $count = (x($_REQUEST,'count')?$_REQUEST['count']:20);
-               $page = (x($_REQUEST,'page')?$_REQUEST['page']-1:0);
-               if ($page<0) $page=0;
-               $since_id = (x($_REQUEST,'since_id')?$_REQUEST['since_id']:0);
-               $max_id = (x($_REQUEST,'max_id')?$_REQUEST['max_id']:0);
-               //$since_id = 0;//$since_id = (x($_REQUEST,'since_id')?$_REQUEST['since_id']:0);
+               $count = (x($_REQUEST, 'count') ? $_REQUEST['count'] : 20);
+               $page = (x($_REQUEST, 'page') ? $_REQUEST['page'] -1 : 0);
+               if ($page < 0) {
+                       $page = 0;
+               }
+               $since_id = (x($_REQUEST, 'since_id') ? $_REQUEST['since_id'] : 0);
+               $max_id = (x($_REQUEST, 'max_id') ? $_REQUEST['max_id'] : 0);
+               //$since_id = 0;//$since_id = (x($_REQUEST, 'since_id')?$_REQUEST['since_id'] : 0);
 
-               $start = $page*$count;
+               $start = $page * $count;
 
                // Ugly code - should be changed
-               $myurl = App::get_baseurl() . '/profile/'. $a->user['nickname'];
-               $myurl = substr($myurl,strpos($myurl,'://')+3);
+               $myurl = System::baseUrl() . '/profile/'. $a->user['nickname'];
+               $myurl = substr($myurl,strpos($myurl, '://') + 3);
                //$myurl = str_replace(array('www.','.'),array('','\\.'),$myurl);
-               $myurl = str_replace('www.','',$myurl);
-               $diasp_url = str_replace('/profile/','/u/',$myurl);
+               $myurl = str_replace('www.', '', $myurl);
+               $diasp_url = str_replace('/profile/', '/u/', $myurl);
 
-               if ($max_id > 0)
-                       $sql_extra = ' AND `item`.`id` <= '.intval($max_id);
+               if ($max_id > 0) {
+                       $sql_extra = ' AND `item`.`id` <= ' . intval($max_id);
+               }
 
                $r = q("SELECT `item`.*, `item`.`id` AS `item_id`, `item`.`network` AS `item_network`,
                        `contact`.`name`, `contact`.`photo`, `contact`.`url`, `contact`.`rel`,
@@ -1809,28 +1963,30 @@ use \Friendica\Core\Config;
                        intval($start), intval($count)
                );
 
-               $ret = api_format_items($r,$user_info, false, $type);
-
+               $ret = api_format_items($r, $user_info, false, $type);
 
                $data = array('status' => $ret);
-               switch($type){
+               switch ($type) {
                        case "atom":
                        case "rss":
                                $data = api_rss_extra($a, $data, $user_info);
                                break;
                }
 
-               return  api_format_data("statuses", $type, $data);
+               return api_format_data("statuses", $type, $data);
        }
+
+       /// @TODO move to top of file or somewhere better
        api_register_func('api/statuses/mentions','api_statuses_mentions', true);
        api_register_func('api/statuses/replies','api_statuses_mentions', true);
 
-
-       function api_statuses_user_timeline($type){
+       function api_statuses_user_timeline($type) {
 
                $a = get_app();
 
-               if (api_user()===false) throw new ForbiddenException();
+               if (api_user() === false) {
+                       throw new ForbiddenException();
+               }
 
                $user_info = api_get_user($a);
                // get last network messages
@@ -1841,24 +1997,29 @@ use \Friendica\Core\Config;
                           LOGGER_DEBUG);
 
                // params
-               $count = (x($_REQUEST,'count')?$_REQUEST['count']:20);
-               $page = (x($_REQUEST,'page')?$_REQUEST['page']-1:0);
-               if ($page<0) $page=0;
-               $since_id = (x($_REQUEST,'since_id')?$_REQUEST['since_id']:0);
-               //$since_id = 0;//$since_id = (x($_REQUEST,'since_id')?$_REQUEST['since_id']:0);
-               $exclude_replies = (x($_REQUEST,'exclude_replies')?1:0);
-               $conversation_id = (x($_REQUEST,'conversation_id')?$_REQUEST['conversation_id']:0);
+               $count = (x($_REQUEST, 'count') ? $_REQUEST['count'] : 20);
+               $page = (x($_REQUEST, 'page') ? $_REQUEST['page'] -1 : 0);
+               if ($page < 0) {
+                       $page = 0;
+               }
+               $since_id = (x($_REQUEST, 'since_id') ? $_REQUEST['since_id'] : 0);
+               //$since_id = 0;//$since_id = (x($_REQUEST, 'since_id')?$_REQUEST['since_id'] : 0);
+               $exclude_replies = (x($_REQUEST, 'exclude_replies') ? 1 : 0);
+               $conversation_id = (x($_REQUEST, 'conversation_id') ? $_REQUEST['conversation_id'] : 0);
 
-               $start = $page*$count;
+               $start = $page * $count;
 
                $sql_extra = '';
-               if ($user_info['self']==1)
+               if ($user_info['self'] == 1) {
                        $sql_extra .= " AND `item`.`wall` = 1 ";
+               }
 
-               if ($exclude_replies > 0)
+               if ($exclude_replies > 0) {
                        $sql_extra .= ' AND `item`.`parent` = `item`.`id`';
-               if ($conversation_id > 0)
-                       $sql_extra .= ' AND `item`.`parent` = '.intval($conversation_id);
+               }
+               if ($conversation_id > 0) {
+                       $sql_extra .= ' AND `item`.`parent` = ' . intval($conversation_id);
+               }
 
                $r = q("SELECT `item`.*, `item`.`id` AS `item_id`, `item`.`network` AS `item_network`,
                        `contact`.`name`, `contact`.`photo`, `contact`.`url`, `contact`.`rel`,
@@ -1880,19 +2041,21 @@ use \Friendica\Core\Config;
                        intval($start), intval($count)
                );
 
-               $ret = api_format_items($r,$user_info, true, $type);
+               $ret = api_format_items($r, $user_info, true, $type);
 
                $data = array('status' => $ret);
-               switch($type){
+               switch ($type) {
                        case "atom":
                        case "rss":
                                $data = api_rss_extra($a, $data, $user_info);
+                               break;
                }
 
-               return  api_format_data("statuses", $type, $data);
+               return api_format_data("statuses", $type, $data);
        }
-       api_register_func('api/statuses/user_timeline','api_statuses_user_timeline', true);
 
+       /// @TODO move to top of file or somwhere better
+       api_register_func('api/statuses/user_timeline','api_statuses_user_timeline', true);
 
        /**
         * Star/unstar an item
@@ -1900,49 +2063,59 @@ use \Friendica\Core\Config;
         *
         * api v1 : https://web.archive.org/web/20131019055350/https://dev.twitter.com/docs/api/1/post/favorites/create/%3Aid
         */
-       function api_favorites_create_destroy($type){
+       function api_favorites_create_destroy($type) {
 
                $a = get_app();
 
-               if (api_user()===false) throw new ForbiddenException();
+               if (api_user() === false) {
+                       throw new ForbiddenException();
+               }
 
                // for versioned api.
                /// @TODO We need a better global soluton
-               $action_argv_id=2;
-               if ($a->argv[1]=="1.1") $action_argv_id=3;
+               $action_argv_id = 2;
+               if ($a->argv[1] == "1.1") {
+                       $action_argv_id = 3;
+               }
 
-               if ($a->argc<=$action_argv_id) throw new BadRequestException("Invalid request.");
-               $action = str_replace(".".$type,"",$a->argv[$action_argv_id]);
-               if ($a->argc==$action_argv_id+2) {
-                       $itemid = intval($a->argv[$action_argv_id+1]);
+               if ($a->argc <= $action_argv_id) {
+                       throw new BadRequestException("Invalid request.");
+               }
+               $action = str_replace("." . $type, "", $a->argv[$action_argv_id]);
+               if ($a->argc == $action_argv_id + 2) {
+                       $itemid = intval($a->argv[$action_argv_id + 1]);
                } else {
+                       ///  @TODO use x() to check if _REQUEST contains 'id'
                        $itemid = intval($_REQUEST['id']);
                }
 
-               $item = q("SELECT * FROM item WHERE id=%d AND uid=%d",
+               $item = q("SELECT * FROM `item` WHERE `id`=%d AND `uid`=%d LIMIT 1",
                                $itemid, api_user());
 
-               if ($item===false || count($item)==0)
+               if (!dbm::is_result($item) || count($item) == 0) {
                        throw new BadRequestException("Invalid item.");
+               }
 
-               switch($action){
+               switch ($action) {
                        case "create":
-                               $item[0]['starred']=1;
+                               $item[0]['starred'] = 1;
                                break;
                        case "destroy":
-                               $item[0]['starred']=0;
+                               $item[0]['starred'] = 0;
                                break;
                        default:
                                throw new BadRequestException("Invalid action ".$action);
                }
+
                $r = q("UPDATE item SET starred=%d WHERE id=%d AND uid=%d",
                                $item[0]['starred'], $itemid, api_user());
 
                q("UPDATE thread SET starred=%d WHERE iid=%d AND uid=%d",
                        $item[0]['starred'], $itemid, api_user());
 
-               if ($r===false)
-                       throw InternalServerErrorException("DB error");
+               if ($r === false) {
+                       throw new InternalServerErrorException("DB error");
+               }
 
 
                $user_info = api_get_user($a);
@@ -1950,7 +2123,7 @@ use \Friendica\Core\Config;
                $ret = $rets[0];
 
                $data = array('status' => $ret);
-               switch($type){
+               switch ($type) {
                        case "atom":
                        case "rss":
                                $data = api_rss_extra($a, $data, $user_info);
@@ -1958,17 +2131,21 @@ use \Friendica\Core\Config;
 
                return api_format_data("status", $type, $data);
        }
+
+       /// @TODO move to top of file or somwhere better
        api_register_func('api/favorites/create', 'api_favorites_create_destroy', true, API_METHOD_POST);
        api_register_func('api/favorites/destroy', 'api_favorites_create_destroy', true, API_METHOD_DELETE);
 
-       function api_favorites($type){
+       function api_favorites($type) {
                global $called_api;
 
                $a = get_app();
 
-               if (api_user()===false) throw new ForbiddenException();
+               if (api_user() === false) {
+                       throw new ForbiddenException();
+               }
 
-               $called_api= array();
+               $called_api = array();
 
                $user_info = api_get_user($a);
 
@@ -1976,22 +2153,25 @@ use \Friendica\Core\Config;
                // return favorites only for self
                logger('api_favorites: self:' . $user_info['self']);
 
-               if ($user_info['self']==0) {
+               if ($user_info['self'] == 0) {
                        $ret = array();
                } else {
                        $sql_extra = "";
 
                        // params
-                       $since_id = (x($_REQUEST,'since_id')?$_REQUEST['since_id']:0);
-                       $max_id = (x($_REQUEST,'max_id')?$_REQUEST['max_id']:0);
-                       $count = (x($_GET,'count')?$_GET['count']:20);
-                       $page = (x($_REQUEST,'page')?$_REQUEST['page']-1:0);
-                       if ($page<0) $page=0;
+                       $since_id = (x($_REQUEST, 'since_id') ? $_REQUEST['since_id'] : 0);
+                       $max_id = (x($_REQUEST, 'max_id') ? $_REQUEST['max_id'] : 0);
+                       $count = (x($_GET, 'count') ? $_GET['count'] : 20);
+                       $page = (x($_REQUEST, 'page') ? $_REQUEST['page'] -1 : 0);
+                       if ($page < 0) {
+                               $page = 0;
+                       }
 
                        $start = $page*$count;
 
-                       if ($max_id > 0)
-                               $sql_extra .= ' AND `item`.`id` <= '.intval($max_id);
+                       if ($max_id > 0) {
+                               $sql_extra .= ' AND `item`.`id` <= ' . intval($max_id);
+                       }
 
                        $r = q("SELECT `item`.*, `item`.`id` AS `item_id`, `item`.`network` AS `item_network`,
                                `contact`.`name`, `contact`.`photo`, `contact`.`url`, `contact`.`rel`,
@@ -1999,7 +2179,7 @@ use \Friendica\Core\Config;
                                `contact`.`id` AS `cid`
                                FROM `item`, `contact`
                                WHERE `item`.`uid` = %d
-                               AND `item`.`visible` = 1 and `item`.`moderated` = 0 AND `item`.`deleted` = 0
+                               AND `item`.`visible` = 1 AND `item`.`moderated` = 0 AND `item`.`deleted` = 0
                                AND `item`.`starred` = 1
                                AND `contact`.`id` = `item`.`contact-id`
                                AND (NOT `contact`.`blocked` OR `contact`.`pending`)
@@ -2016,14 +2196,16 @@ use \Friendica\Core\Config;
                }
 
                $data = array('status' => $ret);
-               switch($type){
+               switch ($type) {
                        case "atom":
                        case "rss":
                                $data = api_rss_extra($a, $data, $user_info);
                }
 
-               return  api_format_data("statuses", $type, $data);
+               return api_format_data("statuses", $type, $data);
        }
+
+       /// @TODO move to top of file or somwhere better
        api_register_func('api/favorites','api_favorites', true);
 
        function api_format_messages($item, $recipient, $sender) {
@@ -2038,9 +2220,9 @@ use \Friendica\Core\Config;
                                'recipient_screen_name' => $recipient['screen_name'],
                                'sender'                => $sender,
                                'recipient'             => $recipient,
-                               'title'                 => "",
-                               'friendica_seen'        => $item['seen'],
-                               'friendica_parent_uri'  => $item['parent-uri'],
+                               'title'                 => "",
+                               'friendica_seen'        => $item['seen'],
+                               'friendica_parent_uri'  => $item['parent-uri'],
                );
 
                // "uid" and "self" are only needed for some internal stuff, so remove it from here
@@ -2051,19 +2233,17 @@ use \Friendica\Core\Config;
 
                //don't send title to regular StatusNET requests to avoid confusing these apps
                if (x($_GET, 'getText')) {
-                       $ret['title'] = $item['title'] ;
-                       if ($_GET["getText"] == "html") {
+                       $ret['title'] = $item['title'];
+                       if ($_GET['getText'] == 'html') {
                                $ret['text'] = bbcode($item['body'], false, false);
-                       }
-                       elseif ($_GET["getText"] == "plain") {
+                       } elseif ($_GET['getText'] == 'plain') {
                                //$ret['text'] = html2plain(bbcode($item['body'], false, false, true), 0);
                                $ret['text'] = trim(html2plain(bbcode(api_clean_plain_items($item['body']), false, false, 2, true), 0));
                        }
+               } else {
+                       $ret['text'] = $item['title'] . "\n" . html2plain(bbcode(api_clean_plain_items($item['body']), false, false, 2, true), 0);
                }
-               else {
-                       $ret['text'] = $item['title']."\n".html2plain(bbcode(api_clean_plain_items($item['body']), false, false, 2, true), 0);
-               }
-               if (isset($_GET["getUserObjects"]) && $_GET["getUserObjects"] == "false") {
+               if (x($_GET, 'getUserObjects') && $_GET['getUserObjects'] == 'false') {
                        unset($ret['sender']);
                        unset($ret['recipient']);
                }
@@ -2084,28 +2264,50 @@ use \Friendica\Core\Config;
 
                $statustitle = trim($item['title']);
 
-               if (($statustitle != '') and (strpos($statusbody, $statustitle) !== false))
+               if (($statustitle != '') && (strpos($statusbody, $statustitle) !== false)) {
                        $statustext = trim($statusbody);
-               else
+               } else {
                        $statustext = trim($statustitle."\n\n".$statusbody);
+               }
 
-               if (($item["network"] == NETWORK_FEED) and (strlen($statustext)> 1000))
+               if (($item["network"] == NETWORK_FEED) && (strlen($statustext)> 1000)) {
                        $statustext = substr($statustext, 0, 1000)."... \n".$item["plink"];
+               }
 
                $statushtml = trim(bbcode($body, false, false));
 
+               // Workaround for clients with limited HTML parser functionality
                $search = array("<br>", "<blockquote>", "</blockquote>",
                                "<h1>", "</h1>", "<h2>", "</h2>",
                                "<h3>", "</h3>", "<h4>", "</h4>",
                                "<h5>", "</h5>", "<h6>", "</h6>");
-               $replace = array("<br>\n", "\n<blockquote>", "</blockquote>\n",
-                               "\n<h1>", "</h1>\n", "\n<h2>", "</h2>\n",
-                               "\n<h3>", "</h3>\n", "\n<h4>", "</h4>\n",
-                               "\n<h5>", "</h5>\n", "\n<h6>", "</h6>\n");
+               $replace = array("<br>", "<br><blockquote>", "</blockquote><br>",
+                               "<br><h1>", "</h1><br>", "<br><h2>", "</h2><br>",
+                               "<br><h3>", "</h3><br>", "<br><h4>", "</h4><br>",
+                               "<br><h5>", "</h5><br>", "<br><h6>", "</h6><br>");
                $statushtml = str_replace($search, $replace, $statushtml);
 
-               if ($item['title'] != "")
-                       $statushtml = "<h4>".bbcode($item['title'])."</h4>\n".$statushtml;
+               if ($item['title'] != "") {
+                       $statushtml = "<br><h4>" . bbcode($item['title']) . "</h4><br>" . $statushtml;
+               }
+
+               do {
+                       $oldtext = $statushtml;
+                       $statushtml = str_replace("<br><br>", "<br>", $statushtml);
+               } while ($oldtext != $statushtml);
+
+               if (substr($statushtml, 0, 4) == '<br>') {
+                       $statushtml = substr($statushtml, 4);
+               }
+
+               if (substr($statushtml, 0, -4) == '<br>') {
+                       $statushtml = substr($statushtml, -4);
+               }
+
+               // feeds without body should contain the link
+               if (($item['network'] == NETWORK_FEED) && (strlen($item['body']) == 0)) {
+                       $statushtml .= bbcode($item['plink']);
+               }
 
                $entities = api_get_entitities($statustext, $body);
 
@@ -2125,21 +2327,25 @@ use \Friendica\Core\Config;
                $URLSearchString = "^\[\]";
                $ret = preg_match_all("/\[img\]([$URLSearchString]*)\[\/img\]/ism", $text, $images);
 
-               if (!$ret)
+               if (!$ret) {
                        return false;
+               }
 
                $attachments = array();
 
                foreach ($images[1] AS $image) {
                        $imagedata = get_photo_info($image);
 
-                       if ($imagedata)
+                       if ($imagedata) {
                                $attachments[] = array("url" => $image, "mimetype" => $imagedata["mime"], "size" => $imagedata["size"]);
+                       }
                }
 
-               if (strstr($_SERVER['HTTP_USER_AGENT'], "AndStatus"))
-                       foreach ($images[0] AS $orig)
+               if (strstr($_SERVER['HTTP_USER_AGENT'], "AndStatus")) {
+                       foreach ($images[0] AS $orig) {
                                $body = str_replace($orig, "", $body);
+                       }
+               }
 
                return $attachments;
        }
@@ -2152,7 +2358,7 @@ use \Friendica\Core\Config;
 
                $a = get_app();
 
-               $include_entities = strtolower(x($_REQUEST,'include_entities')?$_REQUEST['include_entities']:"false");
+               $include_entities = strtolower(x($_REQUEST, 'include_entities') ? $_REQUEST['include_entities'] : "false");
 
                if ($include_entities != "true") {
 
@@ -2198,11 +2404,12 @@ use \Friendica\Core\Config;
                preg_match_all("/\[url\=([$URLSearchString]*)\](.*?)\[\/url\]/ism", $bbcode, $urls);
 
                $ordered_urls = array();
-               foreach ($urls[1] AS $id=>$url) {
+               foreach ($urls[1] AS $id => $url) {
                        //$start = strpos($text, $url, $offset);
                        $start = iconv_strpos($text, $url, 0, "UTF-8");
-                       if (!($start === false))
+                       if (!($start === false)) {
                                $ordered_urls[$start] = array("url" => $url, "title" => $urls[2][$id]);
+                       }
                }
 
                ksort($ordered_urls);
@@ -2210,8 +2417,8 @@ use \Friendica\Core\Config;
                $offset = 0;
                //foreach ($urls[1] AS $id=>$url) {
                foreach ($ordered_urls AS $url) {
-                       if ((substr($url["title"], 0, 7) != "http://") AND (substr($url["title"], 0, 8) != "https://") AND
-                               !strpos($url["title"], "http://") AND !strpos($url["title"], "https://"))
+                       if ((substr($url["title"], 0, 7) != "http://") && (substr($url["title"], 0, 8) != "https://") &&
+                               !strpos($url["title"], "http://") && !strpos($url["title"], "https://"))
                                $display_url = $url["title"];
                        else {
                                $display_url = str_replace(array("http://www.", "https://www."), array("", ""), $url["url"]);
@@ -2263,7 +2470,7 @@ use \Friendica\Core\Config;
                                                $scale = scale_image($image[0], $image[1], 150);
                                                $sizes["thumb"] = array("w" => $scale["width"], "h" => $scale["height"], "resize" => "fit");
 
-                                               if (($image[0] > 150) OR ($image[1] > 150)) {
+                                               if (($image[0] > 150) || ($image[1] > 150)) {
                                                        $scale = scale_image($image[0], $image[1], 340);
                                                        $sizes["small"] = array("w" => $scale["width"], "h" => $scale["height"], "resize" => "fit");
                                                }
@@ -2271,7 +2478,7 @@ use \Friendica\Core\Config;
                                                $scale = scale_image($image[0], $image[1], 600);
                                                $sizes["medium"] = array("w" => $scale["width"], "h" => $scale["height"], "resize" => "fit");
 
-                                               if (($image[0] > 600) OR ($image[1] > 600)) {
+                                               if (($image[0] > 600) || ($image[1] > 600)) {
                                                        $scale = scale_image($image[0], $image[1], 1024);
                                                        $sizes["large"] = array("w" => $scale["width"], "h" => $scale["height"], "resize" => "fit");
                                                }
@@ -2296,13 +2503,13 @@ use \Friendica\Core\Config;
                        }
                }
 
-               return($entities);
+               return $entities;
        }
-       function api_format_items_embeded_images(&$item, $text){
+       function api_format_items_embeded_images(&$item, $text) {
                $text = preg_replace_callback(
                                "|data:image/([^;]+)[^=]+=*|m",
                                function($match) use ($item) {
-                                       return App::get_baseurl()."/display/".$item['guid'];
+                                       return System::baseUrl()."/display/".$item['guid'];
                                },
                                $text);
                return $text;
@@ -2352,7 +2559,7 @@ use \Friendica\Core\Config;
                        'dislike' => array(),
                        'attendyes' => array(),
                        'attendno' => array(),
-                       'attendmaybe' => array()
+                       'attendmaybe' => array(),
                );
 
                $items = q('SELECT * FROM item
@@ -2360,13 +2567,13 @@ use \Friendica\Core\Config;
                                        intval($item['uid']),
                                        dbesc($item['uri']));
 
-               foreach ($items as $i){
+               foreach ($items as $i) {
                        // not used as result should be structured like other user data
                        //builtin_activity_puller($i, $activities);
 
                        // get user data and add it to the array of the activity
                        $user = api_get_user($a, $i['author-link']);
-                       switch($i['verb']) {
+                       switch ($i['verb']) {
                                case ACTIVITY_LIKE:
                                        $activities['like'][] = $user;
                                        break;
@@ -2468,25 +2675,27 @@ use \Friendica\Core\Config;
 
                $a = get_app();
 
-               $ret = Array();
+               $ret = array();
 
-               foreach($r as $item) {
+               foreach ($r as $item) {
 
                        localize_item($item);
-                       list($status_user, $owner_user) = api_item_get_user($a,$item);
+                       list($status_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 AND ($status_user["id"] != $user_info["id"]))
+                       if ($filter_user && ($status_user["id"] != $user_info["id"])) {
                                continue;
+                       }
 
                        $in_reply_to = api_in_reply_to($item);
 
                        $converted = api_convert_item($item);
 
-                       if ($type == "xml")
+                       if ($type == "xml") {
                                $geo = "georss:point";
-                       else
+                       } else {
                                $geo = "geo";
+                       }
 
                        $status = array(
                                'text'          => $converted["text"],
@@ -2510,23 +2719,26 @@ use \Friendica\Core\Config;
                                'friendica_activities' => api_format_items_activities($item, $type),
                        );
 
-                       if (count($converted["attachments"]) > 0)
+                       if (count($converted["attachments"]) > 0) {
                                $status["attachments"] = $converted["attachments"];
+                       }
 
-                       if (count($converted["entities"]) > 0)
+                       if (count($converted["entities"]) > 0) {
                                $status["entities"] = $converted["entities"];
+                       }
 
-                       if (($item['item_network'] != "") AND ($status["source"] == 'web'))
+                       if (($item['item_network'] != "") && ($status["source"] == 'web')) {
                                $status["source"] = network_to_name($item['item_network'], $user_info['url']);
-                       else if (($item['item_network'] != "") AND (network_to_name($item['item_network'], $user_info['url']) != $status["source"]))
+                       } elseif (($item['item_network'] != "") && (network_to_name($item['item_network'], $user_info['url']) != $status["source"])) {
                                $status["source"] = trim($status["source"].' ('.network_to_name($item['item_network'], $user_info['url']).')');
+                       }
 
 
                        // Retweets are only valid for top postings
                        // It doesn't work reliable with the link if its a feed
                        //$IsRetweet = ($item['owner-link'] != $item['author-link']);
                        //if ($IsRetweet)
-                       //      $IsRetweet = (($item['owner-name'] != $item['author-name']) OR ($item['owner-avatar'] != $item['author-avatar']));
+                       //      $IsRetweet = (($item['owner-name'] != $item['author-name']) || ($item['owner-avatar'] != $item['author-avatar']));
 
 
                        if ($item["id"] == $item["parent"]) {
@@ -2534,7 +2746,7 @@ use \Friendica\Core\Config;
                                if ($retweeted_item !== false) {
                                        $retweeted_status = $status;
                                        try {
-                                               $retweeted_status["user"] = api_get_user($a,$retweeted_item["author-link"]);
+                                               $retweeted_status["user"] = api_get_user($a, $retweeted_item["author-link"]);
                                        } catch( BadRequestException $e ) {
                                                // user not found. should be found?
                                                /// @todo check if the user should be always found
@@ -2571,67 +2783,81 @@ use \Friendica\Core\Config;
                return $ret;
        }
 
-
        function api_account_rate_limit_status($type) {
 
-               if ($type == "xml")
+               if ($type == "xml") {
                        $hash = array(
-                                       'remaining-hits' => (string) 150,
+                                       'remaining-hits' => '150',
                                        '@attributes' => array("type" => "integer"),
-                                       'hourly-limit' => (string) 150,
+                                       'hourly-limit' => '150',
                                        '@attributes2' => array("type" => "integer"),
-                                       'reset-time' => datetime_convert('UTC','UTC','now + 1 hour',ATOM_TIME),
+                                       'reset-time' => datetime_convert('UTC', 'UTC','now + 1 hour',ATOM_TIME),
                                        '@attributes3' => array("type" => "datetime"),
                                        'reset_time_in_seconds' => strtotime('now + 1 hour'),
                                        '@attributes4' => array("type" => "integer"),
                                );
-               else
+               } else {
                        $hash = array(
                                        'reset_time_in_seconds' => strtotime('now + 1 hour'),
-                                       'remaining_hits' => (string) 150,
-                                       'hourly_limit' => (string) 150,
-                                       'reset_time' => api_date(datetime_convert('UTC','UTC','now + 1 hour',ATOM_TIME)),
+                                       'remaining_hits' => '150',
+                                       'hourly_limit' => '150',
+                                       'reset_time' => api_date(datetime_convert('UTC', 'UTC','now + 1 hour',ATOM_TIME)),
                                );
+               }
 
                return api_format_data('hash', $type, array('hash' => $hash));
        }
+
+       /// @TODO move to top of file or somwhere better
        api_register_func('api/account/rate_limit_status','api_account_rate_limit_status',true);
 
        function api_help_test($type) {
-               if ($type == 'xml')
+               if ($type == 'xml') {
                        $ok = "true";
-               else
+               } else {
                        $ok = "ok";
+               }
 
                return api_format_data('ok', $type, array("ok" => $ok));
        }
-       api_register_func('api/help/test','api_help_test',false);
+
+       /// @TODO move to top of file or somwhere better
+       api_register_func('api/help/test','api_help_test', false);
 
        function api_lists($type) {
                $ret = array();
+               /// @TODO $ret is not filled here?
                return api_format_data('lists', $type, array("lists_list" => $ret));
        }
+
+       /// @TODO move to top of file or somwhere better
        api_register_func('api/lists','api_lists',true);
 
        function api_lists_list($type) {
                $ret = array();
+               /// @TODO $ret is not filled here?
                return api_format_data('lists', $type, array("lists_list" => $ret));
        }
+
+       /// @TODO move to top of file or somwhere better
        api_register_func('api/lists/list','api_lists_list',true);
 
        /**
-        *  https://dev.twitter.com/docs/api/1/get/statuses/friends
-        *  This function is deprecated by Twitter
-        *  returns: json, xml
-        **/
+        * https://dev.twitter.com/docs/api/1/get/statuses/friends
+        * This function is deprecated by Twitter
+        * returns: json, xml
+        */
        function api_statuses_f($type, $qtype) {
 
                $a = get_app();
 
-               if (api_user()===false) throw new ForbiddenException();
+               if (api_user() === false) {
+                       throw new ForbiddenException();
+               }
+
                $user_info = api_get_user($a);
 
-               if (x($_GET,'cursor') && $_GET['cursor']=='undefined'){
+               if (x($_GET, 'cursor') && $_GET['cursor']=='undefined') {
                        /* this is to stop Hotot to load friends multiple times
                        *  I'm not sure if I'm missing return something or
                        *  is a bug in hotot. Workaround, meantime
@@ -2642,66 +2868,74 @@ use \Friendica\Core\Config;
                        return false;
                }
 
-               if($qtype == 'friends')
+               if ($qtype == 'friends') {
                        $sql_extra = sprintf(" AND ( `rel` = %d OR `rel` = %d ) ", intval(CONTACT_IS_SHARING), intval(CONTACT_IS_FRIEND));
-               if($qtype == 'followers')
+               }
+               if ($qtype == 'followers') {
                        $sql_extra = sprintf(" AND ( `rel` = %d OR `rel` = %d ) ", intval(CONTACT_IS_FOLLOWER), intval(CONTACT_IS_FRIEND));
+               }
 
                // friends and followers only for self
-               if ($user_info['self'] == 0)
+               if ($user_info['self'] == 0) {
                        $sql_extra = " AND false ";
+               }
 
-               $r = q("SELECT `nurl` FROM `contact` WHERE `uid` = %d AND NOT `self` AND (NOT `blocked` OR `pending`) $sql_extra",
+               $r = q("SELECT `nurl` FROM `contact` WHERE `uid` = %d AND NOT `self` AND (NOT `blocked` OR `pending`) $sql_extra ORDER BY `nick`",
                        intval(api_user())
                );
 
                $ret = array();
-               foreach($r as $cid){
+               foreach ($r as $cid) {
                        $user = api_get_user($a, $cid['nurl']);
                        // "uid" and "self" are only needed for some internal stuff, so remove it from here
                        unset($user["uid"]);
                        unset($user["self"]);
 
-                       if ($user)
+                       if ($user) {
                                $ret[] = $user;
+                       }
                }
 
                return array('user' => $ret);
 
        }
-       function api_statuses_friends($type){
+
+       function api_statuses_friends($type) {
                $data =  api_statuses_f($type, "friends");
-               if ($data===false) return false;
-               return  api_format_data("users", $type, $data);
+               if ($data === false) {
+                       return false;
+               }
+               return api_format_data("users", $type, $data);
        }
-       function api_statuses_followers($type){
+
+       function api_statuses_followers($type) {
                $data = api_statuses_f($type, "followers");
-               if ($data===false) return false;
-               return  api_format_data("users", $type, $data);
+               if ($data === false) {
+                       return false;
+               }
+               return api_format_data("users", $type, $data);
        }
+
+       /// @TODO move to top of file or somewhere better
        api_register_func('api/statuses/friends','api_statuses_friends',true);
        api_register_func('api/statuses/followers','api_statuses_followers',true);
 
-
-
-
-
-
        function api_statusnet_config($type) {
 
                $a = get_app();
 
                $name = $a->config['sitename'];
                $server = $a->get_hostname();
-               $logo = App::get_baseurl() . '/images/friendica-64.png';
+               $logo = System::baseUrl() . '/images/friendica-64.png';
                $email = $a->config['admin_email'];
                $closed = (($a->config['register_policy'] == REGISTER_CLOSED) ? 'true' : 'false');
                $private = ((Config::get('system', 'block_public')) ? 'true' : 'false');
                $textlimit = (string) (($a->config['max_import_size']) ? $a->config['max_import_size'] : 200000);
-               if($a->config['api_import_size'])
+               if ($a->config['api_import_size']) {
                        $texlimit = string($a->config['api_import_size']);
+               }
                $ssl = ((Config::get('system', 'have_ssl')) ? 'true' : 'false');
-               $sslserver = (($ssl === 'true') ? str_replace('http:','https:',App::get_baseurl()) : '');
+               $sslserver = (($ssl === 'true') ? str_replace('http:','https:',System::baseUrl()) : '');
 
                $config = array(
                        'site' => array('name' => $name,'server' => $server, 'theme' => 'default', 'path' => '',
@@ -2721,8 +2955,10 @@ use \Friendica\Core\Config;
                return api_format_data('config', $type, array('config' => $config));
 
        }
-       api_register_func('api/gnusocial/config','api_statusnet_config',false);
-       api_register_func('api/statusnet/config','api_statusnet_config',false);
+
+       /// @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);
 
        function api_statusnet_version($type) {
                // liar
@@ -2730,8 +2966,10 @@ use \Friendica\Core\Config;
 
                return api_format_data('version', $type, array('version' => $fake_statusnet_version));
        }
-       api_register_func('api/gnusocial/version','api_statusnet_version',false);
-       api_register_func('api/statusnet/version','api_statusnet_version',false);
+
+       /// @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);
 
        /**
         * @todo use api_format_data() to return data
@@ -2740,19 +2978,24 @@ use \Friendica\Core\Config;
 
                $a = get_app();
 
-               if(! api_user()) throw new ForbiddenException();
+               if (! api_user()) {
+                       throw new ForbiddenException();
+               }
 
                $user_info = api_get_user($a);
 
-               if($qtype == 'friends')
+               if ($qtype == 'friends') {
                        $sql_extra = sprintf(" AND ( `rel` = %d OR `rel` = %d ) ", intval(CONTACT_IS_SHARING), intval(CONTACT_IS_FRIEND));
-               if($qtype == 'followers')
+               }
+               if ($qtype == 'followers') {
                        $sql_extra = sprintf(" AND ( `rel` = %d OR `rel` = %d ) ", intval(CONTACT_IS_FOLLOWER), intval(CONTACT_IS_FRIEND));
+               }
 
-               if (!$user_info["self"])
+               if (!$user_info["self"]) {
                        $sql_extra = " AND false ";
+               }
 
-               $stringify_ids = (x($_REQUEST,'stringify_ids')?$_REQUEST['stringify_ids']:false);
+               $stringify_ids = (x($_REQUEST, 'stringify_ids') ? $_REQUEST['stringify_ids'] : false);
 
                $r = q("SELECT `pcontact`.`id` FROM `contact`
                                INNER JOIN `contact` AS `pcontact` ON `contact`.`nurl` = `pcontact`.`nurl` AND `pcontact`.`uid` = 0
@@ -2760,15 +3003,18 @@ use \Friendica\Core\Config;
                        intval(api_user())
                );
 
-               if (!dbm::is_result($r))
+               if (!dbm::is_result($r)) {
                        return;
+               }
 
                $ids = array();
-               foreach($r as $rr)
-                       if ($stringify_ids)
+               foreach ($r as $rr) {
+                       if ($stringify_ids) {
                                $ids[] = $rr['id'];
-                       else
+                       } else {
                                $ids[] = intval($rr['id']);
+                       }
+               }
 
                return api_format_data("ids", $type, array('id' => $ids));
        }
@@ -2776,20 +3022,22 @@ use \Friendica\Core\Config;
        function api_friends_ids($type) {
                return api_ff_ids($type,'friends');
        }
+
        function api_followers_ids($type) {
                return api_ff_ids($type,'followers');
        }
+
+       /// @TODO move to top of file or somewhere better
        api_register_func('api/friends/ids','api_friends_ids',true);
        api_register_func('api/followers/ids','api_followers_ids',true);
 
-
        function api_direct_messages_new($type) {
 
                $a = get_app();
 
-               if (api_user()===false) throw new ForbiddenException();
+               if (api_user() === false) throw new ForbiddenException();
 
-               if (!x($_POST, "text") OR (!x($_POST,"screen_name") AND !x($_POST,"user_id"))) return;
+               if (!x($_POST, "text") || (!x($_POST,"screen_name") && !x($_POST,"user_id"))) return;
 
                $sender = api_get_user($a);
 
@@ -2802,50 +3050,49 @@ use \Friendica\Core\Config;
                        api_best_nickname($r);
 
                        $recipient = api_get_user($a, $r[0]['nurl']);
-               } else
+               } else {
                        $recipient = api_get_user($a, $_POST['user_id']);
+               }
 
                $replyto = '';
                $sub     = '';
-               if (x($_REQUEST,'replyto')) {
+               if (x($_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'];
-               }
-               else {
-                       if (x($_REQUEST,'title')) {
+               } else {
+                       if (x($_REQUEST, 'title')) {
                                $sub = $_REQUEST['title'];
-                       }
-                       else {
-                               $sub = ((strlen($_POST['text'])>10)?substr($_POST['text'],0,10)."...":$_POST['text']);
+                       } else {
+                               $sub = ((strlen($_POST['text'])>10) ? substr($_POST['text'],0,10)."...":$_POST['text']);
                        }
                }
 
                $id = send_message($recipient['cid'], $_POST['text'], $sub, $replyto);
 
-               if ($id>-1) {
+               if ($id > -1) {
                        $r = q("SELECT * FROM `mail` WHERE id=%d", intval($id));
                        $ret = api_format_messages($r[0], $recipient, $sender);
-
                } else {
                        $ret = array("error"=>$id);
                }
 
-               $data = Array('direct_message'=>$ret);
+               $data = array('direct_message'=>$ret);
 
-               switch($type){
+               switch ($type) {
                        case "atom":
                        case "rss":
                                $data = api_rss_extra($a, $data, $user_info);
                }
 
-               return  api_format_data("direct-messages", $type, $data);
+               return api_format_data("direct-messages", $type, $data);
 
        }
-       api_register_func('api/direct_messages/new','api_direct_messages_new',true, API_METHOD_POST);
 
+       /// @TODO move to top of file or somewhere better
+       api_register_func('api/direct_messages/new','api_direct_messages_new',true, API_METHOD_POST);
 
        /**
         * @brief delete a direct_message from mail table through api
@@ -2853,31 +3100,33 @@ use \Friendica\Core\Config;
         * @param string $type Known types are 'atom', 'rss', 'xml' and 'json'
         * @return string
         */
-       function api_direct_messages_destroy($type){
+       function api_direct_messages_destroy($type) {
                $a = get_app();
 
-               if (api_user()===false) throw new ForbiddenException();
+               if (api_user() === false) {
+                       throw new ForbiddenException();
+               }
 
                // params
                $user_info = api_get_user($a);
                //required
-               $id = (x($_REQUEST,'id') ? $_REQUEST['id'] : 0);
+               $id = (x($_REQUEST, 'id') ? $_REQUEST['id'] : 0);
                // optional
                $parenturi = (x($_REQUEST, 'friendica_parenturi') ? $_REQUEST['friendica_parenturi'] : "");
-               $verbose = (x($_GET,'friendica_verbose')?strtolower($_GET['friendica_verbose']):"false");
+               $verbose = (x($_GET, 'friendica_verbose') ? strtolower($_GET['friendica_verbose']) : "false");
                /// @todo optional parameter 'include_entities' from Twitter API not yet implemented
 
                $uid = $user_info['uid'];
                // error if no id or parenturi specified (for clients posting parent-uri as well)
-               if ($verbose == "true") {
-                       if ($id == 0 || $parenturi == "") {
-                               $answer = array('result' => 'error', 'message' => 'message id or parenturi not specified');
-                               return api_format_data("direct_messages_delete", $type, array('$result' => $answer));
-                       }
+               if ($verbose == "true" && ($id == 0 || $parenturi == "")) {
+                       $answer = array('result' => 'error', 'message' => 'message id or parenturi not specified');
+                       return api_format_data("direct_messages_delete", $type, array('$result' => $answer));
                }
 
                // BadRequestException if no id specified (for clients using Twitter API)
-               if ($id == 0) throw new BadRequestException('Message id not specified');
+               if ($id == 0) {
+                       throw new BadRequestException('Message id not specified');
+               }
 
                // add parent-uri to sql command if specified by calling app
                $sql_extra = ($parenturi != "" ? " AND `parent-uri` = '" . dbesc($parenturi) . "'" : "");
@@ -2907,8 +3156,7 @@ use \Friendica\Core\Config;
                                // return success
                                $answer = array('result' => 'ok', 'message' => 'message deleted');
                                return api_format_data("direct_message_delete", $type, array('$result' => $answer));
-                       }
-                       else {
+                       } else {
                                $answer = array('result' => 'error', 'message' => 'unknown error');
                                return api_format_data("direct_messages_delete", $type, array('$result' => $answer));
                        }
@@ -2916,25 +3164,30 @@ use \Friendica\Core\Config;
                /// @todo return JSON data like Twitter API not yet implemented
 
        }
-       api_register_func('api/direct_messages/destroy', 'api_direct_messages_destroy', true, API_METHOD_DELETE);
 
+       /// @TODO move to top of file or somewhere better
+       api_register_func('api/direct_messages/destroy', 'api_direct_messages_destroy', true, API_METHOD_DELETE);
 
        function api_direct_messages_box($type, $box, $verbose) {
 
                $a = get_app();
 
-               if (api_user()===false) throw new ForbiddenException();
+               if (api_user() === false) {
+                       throw new ForbiddenException();
+               }
 
                // params
-               $count = (x($_GET,'count')?$_GET['count']:20);
-               $page = (x($_REQUEST,'page')?$_REQUEST['page']-1:0);
-               if ($page<0) $page=0;
+               $count = (x($_GET, 'count') ? $_GET['count'] : 20);
+               $page = (x($_REQUEST, 'page') ? $_REQUEST['page'] -1 : 0);
+               if ($page < 0) {
+                       $page = 0;
+               }
 
-               $since_id = (x($_REQUEST,'since_id')?$_REQUEST['since_id']:0);
-               $max_id = (x($_REQUEST,'max_id')?$_REQUEST['max_id']:0);
+               $since_id = (x($_REQUEST, 'since_id') ? $_REQUEST['since_id'] : 0);
+               $max_id = (x($_REQUEST, 'max_id') ? $_REQUEST['max_id'] : 0);
 
-               $user_id = (x($_REQUEST,'user_id')?$_REQUEST['user_id']:"");
-               $screen_name = (x($_REQUEST,'screen_name')?$_REQUEST['screen_name']:"");
+               $user_id = (x($_REQUEST, 'user_id') ? $_REQUEST['user_id'] : "");
+               $screen_name = (x($_REQUEST, 'screen_name') ? $_REQUEST['screen_name'] : "");
 
                //  caller user info
                unset($_REQUEST["user_id"]);
@@ -2946,31 +3199,27 @@ use \Friendica\Core\Config;
                $user_info = api_get_user($a);
                $profile_url = $user_info["url"];
 
-
                // pagination
-               $start = $page*$count;
+               $start = $page * $count;
 
                // filters
                if ($box=="sentbox") {
-                       $sql_extra = "`mail`.`from-url`='".dbesc( $profile_url )."'";
-               }
-               elseif ($box=="conversation") {
-                       $sql_extra = "`mail`.`parent-uri`='".dbesc( $_GET["uri"] )  ."'";
-               }
-               elseif ($box=="all") {
+                       $sql_extra = "`mail`.`from-url`='" . dbesc( $profile_url ) . "'";
+               } elseif ($box == "conversation") {
+                       $sql_extra = "`mail`.`parent-uri`='" . dbesc( $_GET["uri"] )  . "'";
+               } elseif ($box == "all") {
                        $sql_extra = "true";
-               }
-               elseif ($box=="inbox") {
-                       $sql_extra = "`mail`.`from-url`!='".dbesc( $profile_url )."'";
+               } elseif ($box == "inbox") {
+                       $sql_extra = "`mail`.`from-url`!='" . dbesc( $profile_url ) . "'";
                }
 
-               if ($max_id > 0)
-                       $sql_extra .= ' AND `mail`.`id` <= '.intval($max_id);
+               if ($max_id > 0) {
+                       $sql_extra .= ' AND `mail`.`id` <= ' . intval($max_id);
+               }
 
-               if ($user_id !="") {
+               if ($user_id != "") {
                        $sql_extra .= ' AND `mail`.`contact-id` = ' . intval($user_id);
-               }
-               elseif($screen_name !=""){
+               } elseif ($screen_name !="") {
                        $sql_extra .= " AND `contact`.`nick` = '" . dbesc($screen_name). "'";
                }
 
@@ -2979,147 +3228,798 @@ use \Friendica\Core\Config;
                                intval($since_id),
                                intval($start), intval($count)
                );
-               if ($verbose == "true") {
-                       // stop execution and return error message if no mails available
-                       if($r == null) {
-                               $answer = array('result' => 'error', 'message' => 'no mails available');
-                               return api_format_data("direct_messages_all", $type, array('$result' => $answer));
-                       }
+               if ($verbose == "true" && !dbm::is_result($r)) {
+                       $answer = array('result' => 'error', 'message' => 'no mails available');
+                       return api_format_data("direct_messages_all", $type, array('$result' => $answer));
                }
 
-               $ret = Array();
-               foreach($r as $item) {
-                       if ($box == "inbox" || $item['from-url'] != $profile_url){
+               $ret = array();
+               foreach ($r as $item) {
+                       if ($box == "inbox" || $item['from-url'] != $profile_url) {
                                $recipient = $user_info;
                                $sender = api_get_user($a,normalise_link($item['contact-url']));
-                       }
-                       elseif ($box == "sentbox" || $item['from-url'] == $profile_url){
+                       } elseif ($box == "sentbox" || $item['from-url'] == $profile_url) {
                                $recipient = api_get_user($a,normalise_link($item['contact-url']));
                                $sender = $user_info;
-
                        }
-                       $ret[]=api_format_messages($item, $recipient, $sender);
+
+                       $ret[] = api_format_messages($item, $recipient, $sender);
                }
 
 
                $data = array('direct_message' => $ret);
-               switch($type){
+               switch ($type) {
                        case "atom":
                        case "rss":
                                $data = api_rss_extra($a, $data, $user_info);
                }
 
-               return  api_format_data("direct-messages", $type, $data);
+               return api_format_data("direct-messages", $type, $data);
 
        }
 
-       function api_direct_messages_sentbox($type){
-               $verbose = (x($_GET,'friendica_verbose')?strtolower($_GET['friendica_verbose']):"false");
+       function api_direct_messages_sentbox($type) {
+               $verbose = (x($_GET, 'friendica_verbose') ? strtolower($_GET['friendica_verbose']) : "false");
                return api_direct_messages_box($type, "sentbox", $verbose);
        }
-       function api_direct_messages_inbox($type){
-               $verbose = (x($_GET,'friendica_verbose')?strtolower($_GET['friendica_verbose']):"false");
+
+       function api_direct_messages_inbox($type) {
+               $verbose = (x($_GET, 'friendica_verbose') ? strtolower($_GET['friendica_verbose']) : "false");
                return api_direct_messages_box($type, "inbox", $verbose);
        }
-       function api_direct_messages_all($type){
-               $verbose = (x($_GET,'friendica_verbose')?strtolower($_GET['friendica_verbose']):"false");
+
+       function api_direct_messages_all($type) {
+               $verbose = (x($_GET, 'friendica_verbose') ? strtolower($_GET['friendica_verbose']) : "false");
                return api_direct_messages_box($type, "all", $verbose);
        }
-       function api_direct_messages_conversation($type){
-               $verbose = (x($_GET,'friendica_verbose')?strtolower($_GET['friendica_verbose']):"false");
+
+       function api_direct_messages_conversation($type) {
+               $verbose = (x($_GET, 'friendica_verbose') ? strtolower($_GET['friendica_verbose']) : "false");
                return api_direct_messages_box($type, "conversation", $verbose);
        }
+
+       /// @TODO move to top of file or somewhere better
        api_register_func('api/direct_messages/conversation','api_direct_messages_conversation',true);
        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);
 
-
-
-       function api_oauth_request_token($type){
-               try{
+       function api_oauth_request_token($type) {
+               try {
                        $oauth = new FKOAuth1();
                        $r = $oauth->fetch_request_token(OAuthRequest::from_request());
-               }catch(Exception $e){
-                       echo "error=". OAuthUtil::urlencode_rfc3986($e->getMessage()); killme();
+               } catch (Exception $e) {
+                       echo "error=" . OAuthUtil::urlencode_rfc3986($e->getMessage());
+                       killme();
                }
                echo $r;
                killme();
        }
-       function api_oauth_access_token($type){
-               try{
+
+       function api_oauth_access_token($type) {
+               try {
                        $oauth = new FKOAuth1();
                        $r = $oauth->fetch_access_token(OAuthRequest::from_request());
-               }catch(Exception $e){
+               } catch (Exception $e) {
                        echo "error=". OAuthUtil::urlencode_rfc3986($e->getMessage()); killme();
                }
                echo $r;
                killme();
        }
 
+       /// @TODO move to top of file or somewhere better
        api_register_func('api/oauth/request_token', 'api_oauth_request_token', false);
        api_register_func('api/oauth/access_token', 'api_oauth_access_token', false);
 
 
+       /**
+        * @brief 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
+        */
+       function api_fr_photoalbum_delete($type) {
+               if (api_user() === false) {
+                       throw new ForbiddenException();
+               }
+               // input params
+               $album = (x($_REQUEST,'album') ? $_REQUEST['album'] : "");
+
+               // we do not allow calls without album string
+               if ($album == "") {
+                       throw new BadRequestException("no albumname specified");
+               }
+               // check if album is existing
+               $r = q("SELECT DISTINCT `resource-id` FROM `photo` WHERE `uid` = %d AND `album` = '%s'",
+                               intval(api_user()),
+                               dbesc($album));
+               if (!dbm::is_result($r))
+                       throw new BadRequestException("album not available");
+
+               // 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
+               foreach ($r as $rr) {
+                       $photo_item = q("SELECT `id` FROM `item` WHERE `uid` = %d AND `resource-id` = '%s' AND `type` = 'photo'",
+                                               intval(local_user()),
+                                               dbesc($rr['resource-id'])
+                                               );
+
+                       if (!dbm::is_result($photo_item)) {
+                               throw new InternalServerErrorException("problem with deleting items occured");
+                       }
+                       drop_item($photo_item[0]['id'],false);
+               }
+
+               // now let's delete all photos from the album
+               $result = q("DELETE FROM `photo` WHERE `uid` = %d AND `album` = '%s'",
+                               intval(api_user()),
+                               dbesc($album));
+
+               // return success of deletion or error message
+               if ($result) {
+                       $answer = array('result' => 'deleted', 'message' => 'album `' . $album . '` with all containing photos has been deleted.');
+                       return api_format_data("photoalbum_delete", $type, array('$result' => $answer));
+               } else {
+                       throw new InternalServerErrorException("unknown error - deleting from database failed");
+               }
+
+       }
+
+       /**
+        * @brief 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
+        */
+       function api_fr_photoalbum_update($type) {
+               if (api_user() === false) {
+                       throw new ForbiddenException();
+               }
+               // input params
+               $album = (x($_REQUEST,'album') ? $_REQUEST['album'] : "");
+               $album_new = (x($_REQUEST,'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
+               $r = q("SELECT `id` FROM `photo` WHERE `uid` = %d AND `album` = '%s'",
+                               intval(api_user()),
+                               dbesc($album));
+               if (!dbm::is_result($r)) {
+                       throw new BadRequestException("album not available");
+               }
+               // now let's update all photos to the albumname
+               $result = q("UPDATE `photo` SET `album` = '%s' WHERE `uid` = %d AND `album` = '%s'",
+                               dbesc($album_new),
+                               intval(api_user()),
+                               dbesc($album));
+
+               // return success of updating or error message
+               if ($result) {
+                       $answer = array('result' => 'updated', 'message' => 'album `' . $album . '` with all containing photos has been renamed to `' . $album_new . '`.');
+                       return api_format_data("photoalbum_update", $type, array('$result' => $answer));
+               } else {
+                       throw new InternalServerErrorException("unknown error - updating in database failed");
+               }
+       }
+
+
+       /**
+        * @brief list all photos of the authenticated user
+        *
+        * @param string $type Known types are 'atom', 'rss', 'xml' and 'json'
+        * @return string
+        */
        function api_fr_photos_list($type) {
-               if (api_user()===false) throw new ForbiddenException();
-               $r = q("select `resource-id`, max(scale) as scale, album, filename, type from photo
-                               where uid = %d and album != 'Contact Photos' group by `resource-id`, album, filename, type",
+               if (api_user() === false) {
+                       throw new ForbiddenException();
+               }
+               $r = q("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`",
                        intval(local_user())
                );
                $typetoext = array(
-               'image/jpeg' => 'jpg',
-               'image/png' => 'png',
-               'image/gif' => 'gif'
+                       'image/jpeg' => 'jpg',
+                       'image/png' => 'png',
+                       'image/gif' => 'gif'
                );
                $data = array('photo'=>array());
-               if ($r) {
+               if (dbm::is_result($r)) {
                        foreach ($r as $rr) {
                                $photo = array();
                                $photo['id'] = $rr['resource-id'];
                                $photo['album'] = $rr['album'];
                                $photo['filename'] = $rr['filename'];
                                $photo['type'] = $rr['type'];
-                               $thumb = App::get_baseurl()."/photo/".$rr['resource-id']."-".$rr['scale'].".".$typetoext[$rr['type']];
+                               $thumb = System::baseUrl() . "/photo/" . $rr['resource-id'] . "-" . $rr['scale'] . "." . $typetoext[$rr['type']];
+                               $photo['created'] = $rr['created'];
+                               $photo['edited'] = $rr['edited'];
+                               $photo['desc'] = $rr['desc'];
 
-                               if ($type == "xml")
+                               if ($type == "xml") {
                                        $data['photo'][] = array("@attributes" => $photo, "1" => $thumb);
-                               else {
+                               else {
                                        $photo['thumb'] = $thumb;
                                        $data['photo'][] = $photo;
                                }
                        }
                }
-               return  api_format_data("photos", $type, $data);
+               return api_format_data("photos", $type, $data);
+       }
+
+       /**
+        * @brief upload a new photo or change an existing photo
+        *
+        * @param string $type Known types are 'atom', 'rss', 'xml' and 'json'
+        * @return string
+        */
+       function api_fr_photo_create_update($type) {
+               if (api_user() === false) {
+                       throw new ForbiddenException();
+               }
+               // input params
+               $photo_id = (x($_REQUEST, 'photo_id') ? $_REQUEST['photo_id'] : null);
+               $desc = (x($_REQUEST, 'desc') ? $_REQUEST['desc'] : (array_key_exists('desc', $_REQUEST) ? "" : null)); // extra check necessary to distinguish between 'not provided' and 'empty string'
+               $album = (x($_REQUEST,'album') ? $_REQUEST['album'] : null);
+               $album_new = (x($_REQUEST,'album_new') ? $_REQUEST['album_new'] : null);
+               $allow_cid = (x($_REQUEST, 'allow_cid') ? $_REQUEST['allow_cid'] : (array_key_exists('allow_cid', $_REQUEST) ? " " : null));
+               $deny_cid = (x($_REQUEST, 'deny_cid') ? $_REQUEST['deny_cid'] : (array_key_exists('deny_cid', $_REQUEST) ? " " : null));
+               $allow_gid = (x($_REQUEST, 'allow_gid') ? $_REQUEST['allow_gid'] : (array_key_exists('allow_gid', $_REQUEST) ? " " : null));
+               $deny_gid = (x($_REQUEST, 'deny_gid') ? $_REQUEST['deny_gid'] : (array_key_exists('deny_gid', $_REQUEST) ? " " : null));
+               $visibility = (x($_REQUEST, 'visibility') ? (($_REQUEST['visibility'] == "true" || $_REQUEST['visibility'] == 1) ? true : false) : false);
+
+               // do several checks on input parameters
+               // we do not allow calls without album string
+               if ($album == null) {
+                       throw new BadRequestException("no albumname specified");
+               }
+               // if photo_id == null --> we are uploading a new photo
+               if ($photo_id == null) {
+                       $mode = "create";
+
+                       // error if no media posted in create-mode
+                       if (!x($_FILES,'media')) {
+                               // Output error
+                               throw new BadRequestException("no media data submitted");
+                       }
+
+                       // album_new will be ignored in create-mode
+                       $album_new = "";
+               } else {
+                       $mode = "update";
+
+                       // check if photo is existing in database
+                       $r = q("SELECT `id` FROM `photo` WHERE `uid` = %d AND `resource-id` = '%s' AND `album` = '%s'",
+                                       intval(api_user()),
+                                       dbesc($photo_id),
+                                       dbesc($album));
+                       if (!dbm::is_result($r)) {
+                               throw new BadRequestException("photo not available");
+                       }
+               }
+
+               // checks on acl strings provided by clients
+               $acl_input_error = false;
+               $acl_input_error |= check_acl_input($allow_cid);
+               $acl_input_error |= check_acl_input($deny_cid);
+               $acl_input_error |= check_acl_input($allow_gid);
+               $acl_input_error |= check_acl_input($deny_gid);
+               if ($acl_input_error) {
+                       throw new BadRequestException("acl data invalid");
+               }
+               // 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);
+
+                       // return success of updating or error message
+                       if (!is_null($data)) {
+                               return api_format_data("photo_create", $type, $data);
+                       } else {
+                               throw new InternalServerErrorException("unknown error - uploading photo failed, see Friendica log for more information");
+                       }
+               }
+
+               // now let's do the changes in update-mode
+               if ($mode == "update") {
+                       $sql_extra = "";
+
+                       if (!is_null($desc)) {
+                               $sql_extra .= (($sql_extra != "") ? " ," : "") . "`desc` = '$desc'";
+                       }
+
+                       if (!is_null($album_new)) {
+                               $sql_extra .= (($sql_extra != "") ? " ," : "") . "`album` = '$album_new'";
+                       }
+
+                       if (!is_null($allow_cid)) {
+                               $allow_cid = trim($allow_cid);
+                               $sql_extra .= (($sql_extra != "") ? " ," : "") . "`allow_cid` = '$allow_cid'";
+                       }
+
+                       if (!is_null($deny_cid)) {
+                               $deny_cid = trim($deny_cid);
+                               $sql_extra .= (($sql_extra != "") ? " ," : "") . "`deny_cid` = '$deny_cid'";
+                       }
+
+                       if (!is_null($allow_gid)) {
+                               $allow_gid = trim($allow_gid);
+                               $sql_extra .= (($sql_extra != "") ? " ," : "") . "`allow_gid` = '$allow_gid'";
+                       }
+
+                       if (!is_null($deny_gid)) {
+                               $deny_gid = trim($deny_gid);
+                               $sql_extra .= (($sql_extra != "") ? " ," : "") . "`deny_gid` = '$deny_gid'";
+                       }
+
+                       $result = false;
+                       if ($sql_extra != "") {
+                               $nothingtodo = false;
+                               $result = q("UPDATE `photo` SET %s, `edited`='%s' WHERE `uid` = %d AND `resource-id` = '%s' AND `album` = '%s'",
+                                               $sql_extra,
+                                               datetime_convert(),   // update edited timestamp
+                                               intval(api_user()),
+                                               dbesc($photo_id),
+                                               dbesc($album));
+                       } else {
+                               $nothingtodo = true;
+                       }
+
+                       if (x($_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);
+                               if (!is_null($data)) {
+                                       return api_format_data("photo_update", $type, $data);
+                               }
+                       }
+
+                       // return success of updating or error message
+                       if ($result) {
+                               $answer = array('result' => 'updated', 'message' => 'Image id `' . $photo_id . '` has been updated.');
+                               return api_format_data("photo_update", $type, array('$result' => $answer));
+                       } else {
+                               if ($nothingtodo) {
+                                       $answer = array('result' => 'cancelled', 'message' => 'Nothing to update for image id `' . $photo_id . '`.');
+                                       return api_format_data("photo_update", $type, array('$result' => $answer));
+                               }
+                               throw new InternalServerErrorException("unknown error - update photo entry in database failed");
+                       }
+               }
+               throw new InternalServerErrorException("unknown error - this error on uploading or updating a photo should never happen");
+       }
+
+
+       /**
+        * @brief delete a single photo from the database through api
+        *
+        * @param string $type Known types are 'atom', 'rss', 'xml' and 'json'
+        * @return string
+        */
+       function api_fr_photo_delete($type) {
+               if (api_user() === false) {
+                       throw new ForbiddenException();
+               }
+               // input params
+               $photo_id = (x($_REQUEST, '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
+               $r = q("SELECT `id` FROM `photo` WHERE `uid` = %d AND `resource-id` = '%s'",
+                               intval(api_user()),
+                               dbesc($photo_id)
+                       );
+               if (!dbm::is_result($r)) {
+                       throw new BadRequestException("photo not available");
+               }
+               // now we can perform on the deletion of the photo
+               $result = q("DELETE FROM `photo` WHERE `uid` = %d AND `resource-id` = '%s'",
+                               intval(api_user()),
+                               dbesc($photo_id));
+
+               // return success of deletion or error message
+               if ($result) {
+                       // retrieve the id of the parent element (the photo element)
+                       $photo_item = q("SELECT `id` FROM `item` WHERE `uid` = %d AND `resource-id` = '%s' AND `type` = 'photo'",
+                               intval(local_user()),
+                               dbesc($photo_id)
+                               );
+
+                       if (!dbm::is_result($photo_item)) {
+                               throw new InternalServerErrorException("problem with deleting items occured");
+                       }
+                       // 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)
+                       drop_item($photo_item[0]['id'], false);
+
+                       $answer = array('result' => 'deleted', 'message' => 'photo with id `' . $photo_id . '` has been deleted from server.');
+                       return api_format_data("photo_delete", $type, array('$result' => $answer));
+               } else {
+                       throw new InternalServerErrorException("unknown error on deleting photo from database table");
+               }
        }
 
+
+       /**
+        * @brief returns the details of a specified photo id, if scale is given, returns the photo data in base 64
+        *
+        * @param string $type Known types are 'atom', 'rss', 'xml' and 'json'
+        * @return string
+        */
        function api_fr_photo_detail($type) {
-               if (api_user()===false) throw new ForbiddenException();
-               if(!x($_REQUEST,'photo_id')) throw new BadRequestException("No photo id.");
+               if (api_user() === false) {
+                       throw new ForbiddenException();
+               }
+               if (!x($_REQUEST, 'photo_id')) {
+                       throw new BadRequestException("No photo id.");
+               }
 
                $scale = (x($_REQUEST, 'scale') ? intval($_REQUEST['scale']) : false);
-               $scale_sql = ($scale === false ? "" : sprintf("and scale=%d",intval($scale)));
-               $data_sql = ($scale === false ? "" : "ANY_VALUE(data) AS data,");
-
-               $r = q("select %s ANY_VALUE(`resource-id`) AS `resource-id`, ANY_VALUE(`created`) AS `created`,
-                               ANY_VALUE(`edited`) AS `edited`, ANY_VALUE(`title`) AS `title`, ANY_VALUE(`desc`) AS `desc`,
-                               ANY_VALUE(`album`) AS `album`, ANY_VALUE(`filename`) AS `filename`, ANY_VALUE(`type`) AS `type`,
-                               ANY_VALUE(`height`) AS `height`, ANY_VALUE(`width`) AS `width`, ANY_VALUE(`datasize`) AS `datasize`,
-                               ANY_VALUE(`profile`) AS `profile`, min(`scale`) as minscale, max(`scale`) as maxscale
-                               from photo where `uid` = %d and `resource-id` = '%s' %s",
+               $photo_id = $_REQUEST['photo_id'];
+
+               // 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);
+       }
+
+
+       /**
+        * @brief updates the profile image for the user (either a specified profile or the default profile)
+        *
+        * @param string $type Known types are 'atom', 'rss', 'xml' and 'json'
+        * @return string
+        */
+       function api_account_update_profile_image($type) {
+               if (api_user() === false) {
+                       throw new ForbiddenException();
+               }
+               // input params
+               $profileid = (x($_REQUEST, 'profile_id') ? $_REQUEST['profile_id'] : 0);
+
+               // error if image data is missing
+               if (!x($_FILES, 'image')) {
+                       throw new BadRequestException("no media data submitted");
+               }
+
+               // check if specified profile id is valid
+               if ($profileid != 0) {
+                       $r = q("SELECT `id` FROM `profile` WHERE `uid` = %d AND `id` = %d",
+                               intval(api_user()),
+                               intval($profileid));
+                       // error message if specified profile id is not in database
+                       if (!dbm::is_result($r)) {
+                               throw new BadRequestException("profile_id not available");
+                       }
+                       $is_default_profile = $r['profile'];
+               } else {
+                       $is_default_profile = 1;
+               }
+
+               // get mediadata from image or media (Twitter call api/account/update_profile_image provides image)
+               $media = null;
+               if (x($_FILES, 'image')) {
+                       $media = $_FILES['image'];
+               } elseif (x($_FILES, 'media')) {
+                       $media = $_FILES['media'];
+               }
+               // save new profile image
+               $data = save_media_to_database("profileimage", $media, $type, t('Profile Photos'), "", "", "", "", "", $is_default_profile);
+
+               // get filetype
+               if (is_array($media['type'])) {
+                       $filetype = $media['type'][0];
+               } else {
+                       $filetype = $media['type'];
+               }
+               if ($filetype == "image/jpeg") {
+                       $fileext = "jpg";
+               } elseif ($filetype == "image/png") {
+                       $fileext = "png";
+               }
+               // change specified profile or all profiles to the new resource-id
+               if ($is_default_profile) {
+                       $r = q("UPDATE `photo` SET `profile` = 0 WHERE `profile` = 1 AND `resource-id` != '%s' AND `uid` = %d",
+                               dbesc($data['photo']['id']),
+                               intval(local_user())
+                       );
+
+                       $r = q("UPDATE `contact` SET `photo` = '%s', `thumb` = '%s', `micro` = '%s'  WHERE `self` AND `uid` = %d",
+                               dbesc(System::baseUrl() . '/photo/' . $data['photo']['id'] . '-4.' . $fileext),
+                               dbesc(System::baseUrl() . '/photo/' . $data['photo']['id'] . '-5.' . $fileext),
+                               dbesc(System::baseUrl() . '/photo/' . $data['photo']['id'] . '-6.' . $fileext),
+                               intval(local_user())
+                       );
+               } else {
+                       $r = q("UPDATE `profile` SET `photo` = '%s', `thumb` = '%s' WHERE `id` = %d AND `uid` = %d",
+                               dbesc(System::baseUrl() . '/photo/' . $data['photo']['id'] . '-4.' . $filetype),
+                               dbesc(System::baseUrl() . '/photo/' . $data['photo']['id'] . '-5.' . $filetype),
+                               intval($_REQUEST['profile']),
+                               intval(local_user())
+                                       );
+               }
+
+               // we'll set the updated profile-photo timestamp even if it isn't the default profile,
+               // so that browsers will do a cache update unconditionally
+
+               $r = q("UPDATE `contact` SET `avatar-date` = '%s' WHERE `self` = 1 AND `uid` = %d",
+                       dbesc(datetime_convert()),
+                       intval(local_user())
+               );
+
+               // Update global directory in background
+               //$user = api_get_user(get_app());
+               $url = System::baseUrl() . '/profile/' . get_app()->user['nickname'];
+               if ($url && strlen(get_config('system', 'directory'))) {
+                       proc_run(PRIORITY_LOW, "include/directory.php", $url);
+               }
+
+               proc_run(PRIORITY_LOW, 'include/profile_update.php', api_user());
+
+               // output for client
+               if ($data) {
+                       return api_account_verify_credentials($type);
+               } else {
+                       // SaveMediaToDatabase failed for some reason
+                       throw new InternalServerErrorException("image upload failed");
+               }
+       }
+
+       // 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);
+
+
+       function check_acl_input($acl_string) {
+               if ($acl_string == null || $acl_string == " ") {
+                       return false;
+               }
+               $contact_not_found = false;
+
+               // split <x><y><z> into array of cid's
+               preg_match_all("/<[A-Za-z0-9]+>/", $acl_string, $array);
+
+               // check for each cid if it is available on server
+               $cid_array = $array[0];
+               foreach ($cid_array as $cid) {
+                       $cid = str_replace("<", "", $cid);
+                       $cid = str_replace(">", "", $cid);
+                       $contact = q("SELECT * FROM `contact` WHERE `id` = %d AND `uid` = %d",
+                                                       intval($cid),
+                                                       intval(api_user()));
+                       $contact_not_found |= !dbm::is_result($contact);
+               }
+               return $contact_not_found;
+       }
+
+       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) {
+               $visitor   = 0;
+               $src = "";
+               $filetype = "";
+               $filename = "";
+               $filesize = 0;
+
+               if (is_array($media)) {
+                       if (is_array($media['tmp_name'])) {
+                               $src = $media['tmp_name'][0];
+                       } else {
+                               $src = $media['tmp_name'];
+                       }
+                       if (is_array($media['name'])) {
+                               $filename = basename($media['name'][0]);
+                       } else {
+                               $filename = basename($media['name']);
+                       }
+                       if (is_array($media['size'])) {
+                               $filesize = intval($media['size'][0]);
+                       } else {
+                               $filesize = intval($media['size']);
+                       }
+                       if (is_array($media['type'])) {
+                               $filetype = $media['type'][0];
+                       } else {
+                               $filetype = $media['type'];
+                       }
+               }
+
+               if ($filetype == "") {
+                       $filetype=guess_image_type($filename);
+               }
+               $imagedata = getimagesize($src);
+               if ($imagedata) {
+                       $filetype = $imagedata['mime'];
+               }
+               logger("File upload src: " . $src . " - filename: " . $filename .
+                       " - size: " . $filesize . " - type: " . $filetype, LOGGER_DEBUG);
+
+               // check if there was a php upload error
+               if ($filesize == 0 && $media['error'] == 1) {
+                       throw new InternalServerErrorException("image size exceeds PHP config settings, file was rejected by server");
+               }
+               // check against max upload size within Friendica instance
+               $maximagesize = get_config('system', 'maximagesize');
+               if (($maximagesize) && ($filesize > $maximagesize)) {
+                       $formattedBytes = formatBytes($maximagesize);
+                       throw new InternalServerErrorException("image size exceeds Friendica config setting (uploaded size: $formattedBytes)");
+               }
+
+               // create Photo instance with the data of the image
+               $imagedata = @file_get_contents($src);
+               $ph = new Photo($imagedata, $filetype);
+               if (! $ph->is_valid()) {
+                       throw new InternalServerErrorException("unable to process image data");
+               }
+
+               // check orientation of image
+               $ph->orient($src);
+               @unlink($src);
+
+               // check max length of images on server
+               $max_length = get_config('system', 'max_image_length');
+               if (! $max_length) {
+                       $max_length = MAX_IMAGE_LENGTH;
+               }
+               if ($max_length > 0) {
+                       $ph->scaleImage($max_length);
+                       logger("File upload: Scaling picture to new size " . $max_length, LOGGER_DEBUG);
+               }
+               $width = $ph->getWidth();
+               $height = $ph->getHeight();
+
+               // create a new resource-id if not already provided
+               $hash = ($photo_id == null) ? photo_new_resource() : $photo_id;
+
+               if ($mediatype == "photo") {
+                       // upload normal image (scales 0, 1, 2)
+                       logger("photo upload: starting new photo upload", LOGGER_DEBUG);
+
+                       $r =$ph->store(local_user(), $visitor, $hash, $filename, $album, 0, 0, $allow_cid, $allow_gid, $deny_cid, $deny_gid, $desc);
+                       if (! $r) {
+                               logger("photo upload: image upload with scale 0 (original size) failed");
+                       }
+                       if($width > 640 || $height > 640) {
+                               $ph->scaleImage(640);
+                               $r = $ph->store(local_user(),$visitor, $hash, $filename, $album, 1, 0, $allow_cid, $allow_gid, $deny_cid, $deny_gid, $desc);
+                               if (! $r) {
+                                       logger("photo upload: image upload with scale 1 (640x640) failed");
+                               }
+                       }
+
+                       if ($width > 320 || $height > 320) {
+                               $ph->scaleImage(320);
+                               $r = $ph->store(local_user(), $visitor, $hash, $filename, $album, 2, 0, $allow_cid, $allow_gid, $deny_cid, $deny_gid, $desc);
+                               if (! $r) {
+                                       logger("photo upload: image upload with scale 2 (320x320) failed");
+                               }
+                       }
+                       logger("photo upload: new photo upload ended", LOGGER_DEBUG);
+               } elseif ($mediatype == "profileimage") {
+                       // upload profile image (scales 4, 5, 6)
+                       logger("photo upload: starting new profile image upload", LOGGER_DEBUG);
+
+                       if ($width > 175 || $height > 175) {
+                               $ph->scaleImage(175);
+                               $r = $ph->store(local_user(),$visitor, $hash, $filename, $album, 4, $profile, $allow_cid, $allow_gid, $deny_cid, $deny_gid, $desc);
+                               if (! $r) {
+                                       logger("photo upload: profile image upload with scale 4 (175x175) failed");
+                               }
+                       }
+
+                       if ($width > 80 || $height > 80) {
+                               $ph->scaleImage(80);
+                               $r = $ph->store(local_user(),$visitor, $hash, $filename, $album, 5, $profile, $allow_cid, $allow_gid, $deny_cid, $deny_gid, $desc);
+                               if (! $r) {
+                                       logger("photo upload: profile image upload with scale 5 (80x80) failed");
+                               }
+                       }
+
+                       if ($width > 48 || $height > 48) {
+                               $ph->scaleImage(48);
+                               $r = $ph->store(local_user(), $visitor, $hash, $filename, $album, 6, $profile, $allow_cid, $allow_gid, $deny_cid, $deny_gid, $desc);
+                               if (! $r) {
+                                       logger("photo upload: profile image upload with scale 6 (48x48) failed");
+                               }
+                       }
+                       $ph->__destruct();
+                       logger("photo upload: new profile image upload ended", LOGGER_DEBUG);
+               }
+
+               if ($r) {
+                       // create entry in 'item'-table on new uploads to enable users to comment/like/dislike the photo
+                       if ($photo_id == null && $mediatype == "photo") {
+                               post_photo_item($hash, $allow_cid, $deny_cid, $allow_gid, $deny_gid, $filetype, $visibility);
+                       }
+                       // on success return image data in json/xml format (like /api/friendica/photo does when no scale is given)
+                       return prepare_photo_data($type, false, $hash);
+               } else {
+                       throw new InternalServerErrorException("image upload failed");
+               }
+       }
+
+       function post_photo_item($hash, $allow_cid, $deny_cid, $allow_gid, $deny_gid, $filetype, $visibility = false) {
+               // get data about the api authenticated user
+               $uri = item_new_uri(get_app()->get_hostname(), intval(api_user()));
+               $owner_record = q("SELECT * FROM `contact` WHERE `uid`= %d AND `self` LIMIT 1", intval(api_user()));
+
+               $arr = array();
+               $arr['guid']          = get_guid(32);
+               $arr['uid']           = intval(api_user());
+               $arr['uri']           = $uri;
+               $arr['parent-uri']    = $uri;
+               $arr['type']          = 'photo';
+               $arr['wall']          = 1;
+               $arr['resource-id']   = $hash;
+               $arr['contact-id']    = $owner_record[0]['id'];
+               $arr['owner-name']    = $owner_record[0]['name'];
+               $arr['owner-link']    = $owner_record[0]['url'];
+               $arr['owner-avatar']  = $owner_record[0]['thumb'];
+               $arr['author-name']   = $owner_record[0]['name'];
+               $arr['author-link']   = $owner_record[0]['url'];
+               $arr['author-avatar'] = $owner_record[0]['thumb'];
+               $arr['title']         = "";
+               $arr['allow_cid']     = $allow_cid;
+               $arr['allow_gid']     = $allow_gid;
+               $arr['deny_cid']      = $deny_cid;
+               $arr['deny_gid']      = $deny_gid;
+               $arr['last-child']    = 1;
+               $arr['visible']       = $visibility;
+               $arr['origin']        = 1;
+
+               $typetoext = array(
+                               'image/jpeg' => 'jpg',
+                               'image/png' => 'png',
+                               'image/gif' => 'gif'
+                               );
+
+               // adds link to the thumbnail scale photo
+               $arr['body'] = '[url=' . System::baseUrl() . '/photos/' . $owner_record[0]['nick'] . '/image/' . $hash . ']'
+                                       . '[img]' . System::baseUrl() . '/photo/' . $hash . '-' . "2" . '.'. $typetoext[$filetype] . '[/img]'
+                                       . '[/url]';
+
+               // do the magic for storing the item in the database and trigger the federation to other contacts
+               item_store($arr);
+       }
+
+       function prepare_photo_data($type, $scale, $photo_id) {
+               $scale_sql = ($scale === false ? "" : sprintf("AND scale=%d", intval($scale)));
+               $data_sql = ($scale === false ? "" : "data, ");
+
+               // 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`,
+                                               `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`",
                        $data_sql,
                        intval(local_user()),
-                       dbesc($_REQUEST['photo_id']),
+                       dbesc($photo_id),
                        $scale_sql
                );
 
                $typetoext = array(
-               'image/jpeg' => 'jpg',
-               'image/png' => 'png',
-               'image/gif' => 'gif'
+                       'image/jpeg' => 'jpg',
+                       'image/png' => 'png',
+                       'image/gif' => 'gif'
                );
 
-               if ($r) {
+               // prepare output data for photo
+               if (dbm::is_result($r)) {
                        $data = array('photo' => $r[0]);
                        $data['photo']['id'] = $data['photo']['resource-id'];
                        if ($scale !== false) {
@@ -3129,14 +4029,18 @@ use \Friendica\Core\Config;
                        }
                        if ($type == "xml") {
                                $data['photo']['links'] = array();
-                               for ($k=intval($data['photo']['minscale']); $k<=intval($data['photo']['maxscale']); $k++)
-                                       $data['photo']['links'][$k.":link"]["@attributes"] = array("type" => $data['photo']['type'],
+                               for ($k = intval($data['photo']['minscale']); $k <= intval($data['photo']['maxscale']); $k++) {
+                                       $data['photo']['links'][$k . ":link"]["@attributes"] = array("type" => $data['photo']['type'],
                                                                                        "scale" => $k,
-                                                                                       "href" => App::get_baseurl()."/photo/".$data['photo']['resource-id']."-".$k.".".$typetoext[$data['photo']['type']]);
+                                                                                       "href" => System::baseUrl() . "/photo/" . $data['photo']['resource-id'] . "-" . $k . "." . $typetoext[$data['photo']['type']]);
+                               }
                        } else {
                                $data['photo']['link'] = array();
-                               for ($k=intval($data['photo']['minscale']); $k<=intval($data['photo']['maxscale']); $k++) {
-                                       $data['photo']['link'][$k] = App::get_baseurl()."/photo/".$data['photo']['resource-id']."-".$k.".".$typetoext[$data['photo']['type']];
+                               // when we have profile images we could have only scales from 4 to 6, but index of array always needs to start with 0
+                               $i = 0;
+                               for ($k = intval($data['photo']['minscale']); $k <= intval($data['photo']['maxscale']); $k++) {
+                                       $data['photo']['link'][$i] = System::baseUrl() . "/photo/" . $data['photo']['resource-id'] . "-" . $k . "." . $typetoext[$data['photo']['type']];
+                                       $i++;
                                }
                        }
                        unset($data['photo']['resource-id']);
@@ -3147,12 +4051,53 @@ use \Friendica\Core\Config;
                        throw new NotFoundException();
                }
 
-               return api_format_data("photo_detail", $type, $data);
-       }
+               // retrieve item element for getting activities (like, dislike etc.) related to photo
+               $item = q("SELECT * FROM `item` WHERE `uid` = %d AND `resource-id` = '%s' AND `type` = 'photo'",
+                       intval(local_user()),
+                       dbesc($photo_id)
+               );
+               $data['photo']['friendica_activities'] = api_format_items_activities($item[0], $type);
 
-       api_register_func('api/friendica/photos/list', 'api_fr_photos_list', true);
-       api_register_func('api/friendica/photo', 'api_fr_photo_detail', true);
+               // retrieve comments on photo
+               $r = q("SELECT `item`.*, `item`.`id` AS `item_id`, `item`.`network` AS `item_network`,
+                       `contact`.`name`, `contact`.`photo`, `contact`.`url`, `contact`.`rel`,
+                       `contact`.`network`, `contact`.`thumb`, `contact`.`dfrn-id`, `contact`.`self`,
+                       `contact`.`id` AS `cid`
+                       FROM `item`
+                       STRAIGHT_JOIN `contact` ON `contact`.`id` = `item`.`contact-id` AND `contact`.`uid` = `item`.`uid`
+                               AND (NOT `contact`.`blocked` OR `contact`.`pending`)
+                       WHERE `item`.`parent` = %d AND `item`.`visible`
+                       AND NOT `item`.`moderated` AND NOT `item`.`deleted`
+                       AND `item`.`uid` = %d AND (`item`.`verb`='%s' OR `type`='photo')",
+                       intval($item[0]['parent']),
+                       intval(api_user()),
+                       dbesc(ACTIVITY_POST)
+               );
 
+               // prepare output of comments
+               $commentData = api_format_items($r, api_get_user(get_app()), false, $type);
+               $comments = array();
+               if ($type == "xml") {
+                       $k = 0;
+                       foreach ($commentData as $comment) {
+                               $comments[$k++ . ":comment"] = $comment;
+                       }
+               } else {
+                       foreach ($commentData as $comment) {
+                               $comments[] = $comment;
+                       }
+               }
+               $data['photo']['friendica_comments'] = $comments;
+
+               // include info if rights on photo and rights on item are mismatching
+               $rights_mismatch = $data['photo']['allow_cid'] != $item[0]['allow_cid'] ||
+                       $data['photo']['deny_cid'] != $item[0]['deny_cid'] ||
+                       $data['photo']['allow_gid'] != $item[0]['allow_gid'] ||
+                       $data['photo']['deny_cid'] != $item[0]['deny_cid'];
+               $data['photo']['rights_mismatch'] = $rights_mismatch;
+
+               return $data;
+       }
 
 
        /**
@@ -3167,11 +4112,12 @@ use \Friendica\Core\Config;
         *              url: string, url to redirect after auth
         */
        function api_friendica_remoteauth() {
-               $url = ((x($_GET,'url')) ? $_GET['url'] : '');
-               $c_url = ((x($_GET,'c_url')) ? $_GET['c_url'] : '');
+               $url = ((x($_GET, 'url')) ? $_GET['url'] : '');
+               $c_url = ((x($_GET, 'c_url')) ? $_GET['c_url'] : '');
 
-               if ($url === '' || $c_url === '')
+               if ($url === '' || $c_url === '') {
                        throw new BadRequestException("Wrong parameters.");
+               }
 
                $c_url = normalise_link($c_url);
 
@@ -3182,18 +4128,19 @@ use \Friendica\Core\Config;
                        intval(api_user())
                );
 
-               if ((! dbm::is_result($r)) || ($r[0]['network'] !== NETWORK_DFRN))
+               if ((! dbm::is_result($r)) || ($r[0]['network'] !== NETWORK_DFRN)) {
                        throw new BadRequestException("Unknown contact");
+               }
 
                $cid = $r[0]['id'];
 
                $dfrn_id = $orig_id = (($r[0]['issued-id']) ? $r[0]['issued-id'] : $r[0]['dfrn-id']);
 
-               if($r[0]['duplex'] && $r[0]['issued-id']) {
+               if ($r[0]['duplex'] && $r[0]['issued-id']) {
                        $orig_id = $r[0]['issued-id'];
                        $dfrn_id = '1:' . $orig_id;
                }
-               if($r[0]['duplex'] && $r[0]['dfrn-id']) {
+               if ($r[0]['duplex'] && $r[0]['dfrn-id']) {
                        $orig_id = $r[0]['dfrn-id'];
                        $dfrn_id = '0:' . $orig_id;
                }
@@ -3230,12 +4177,16 @@ use \Friendica\Core\Config;
                        return false;
                }
 
-               $attributes = preg_replace("/\[share(.*?)\]\s?(.*?)\s?\[\/share\]\s?/ism","$1",$body);
-               // Skip if there is no shared message in there
-               // we already checked this in diaspora::is_reshare()
-               // but better one more than one less...
-               if ($body == $attributes)
+               /// @TODO "$1" should maybe mean '$1' ?
+               $attributes = preg_replace("/\[share(.*?)\]\s?(.*?)\s?\[\/share\]\s?/ism", "$1", $body);
+               /*
+                * Skip if there is no shared message in there
+                * we already checked this in diaspora::is_reshare()
+                * but better one more than one less...
+                */
+               if ($body == $attributes) {
                        return false;
+               }
 
 
                // build the fake reshared item
@@ -3243,55 +4194,63 @@ use \Friendica\Core\Config;
 
                $author = "";
                preg_match("/author='(.*?)'/ism", $attributes, $matches);
-               if ($matches[1] != "")
-                       $author = html_entity_decode($matches[1],ENT_QUOTES,'UTF-8');
+               if ($matches[1] != "") {
+                       $author = html_entity_decode($matches[1], ENT_QUOTES, 'UTF-8');
+               }
 
                preg_match('/author="(.*?)"/ism', $attributes, $matches);
-               if ($matches[1] != "")
+               if ($matches[1] != "") {
                        $author = $matches[1];
+               }
 
                $profile = "";
                preg_match("/profile='(.*?)'/ism", $attributes, $matches);
-               if ($matches[1] != "")
+               if ($matches[1] != "") {
                        $profile = $matches[1];
+               }
 
                preg_match('/profile="(.*?)"/ism', $attributes, $matches);
-               if ($matches[1] != "")
+               if ($matches[1] != "") {
                        $profile = $matches[1];
+               }
 
                $avatar = "";
                preg_match("/avatar='(.*?)'/ism", $attributes, $matches);
-               if ($matches[1] != "")
+               if ($matches[1] != "") {
                        $avatar = $matches[1];
+               }
 
                preg_match('/avatar="(.*?)"/ism', $attributes, $matches);
-               if ($matches[1] != "")
+               if ($matches[1] != "") {
                        $avatar = $matches[1];
+               }
 
                $link = "";
                preg_match("/link='(.*?)'/ism", $attributes, $matches);
-               if ($matches[1] != "")
+               if ($matches[1] != "") {
                        $link = $matches[1];
+               }
 
                preg_match('/link="(.*?)"/ism', $attributes, $matches);
-               if ($matches[1] != "")
+               if ($matches[1] != "") {
                        $link = $matches[1];
+               }
 
                $posted = "";
                preg_match("/posted='(.*?)'/ism", $attributes, $matches);
                if ($matches[1] != "")
-                       $posted= $matches[1];
+                       $posted = $matches[1];
 
                preg_match('/posted="(.*?)"/ism', $attributes, $matches);
-               if ($matches[1] != "")
+               if ($matches[1] != "") {
                        $posted = $matches[1];
+               }
 
                $shared_body = preg_replace("/\[share(.*?)\]\s?(.*?)\s?\[\/share\]\s?/ism","$2",$body);
 
-               if (($shared_body == "") || ($profile == "") || ($author == "") || ($avatar == "") || ($posted == ""))
+               if (($shared_body == "") || ($profile == "") || ($author == "") || ($avatar == "") || ($posted == "")) {
                        return false;
-
-
+               }
 
                $reshared_item["body"] = $shared_body;
                $reshared_item["author-name"] = $author;
@@ -3315,32 +4274,39 @@ use \Friendica\Core\Config;
 
                $r = q("SELECT `nick` FROM `contact` WHERE `uid` = 0 AND `nurl` = '%s'",
                        dbesc(normalise_link($profile)));
-               if ($r)
+
+               if (dbm::is_result($r)) {
                        $nick = $r[0]["nick"];
+               }
 
                if (!$nick == "") {
                        $r = q("SELECT `nick` FROM `contact` WHERE `uid` = 0 AND `nurl` = '%s'",
                                dbesc(normalise_link($profile)));
-                       if ($r)
+
+                       if (dbm::is_result($r)) {
                                $nick = $r[0]["nick"];
+                       }
                }
 
                if (!$nick == "") {
                        $friendica = preg_replace("=https?://(.*)/profile/(.*)=ism", "$2", $profile);
-                       if ($friendica != $profile)
+                       if ($friendica != $profile) {
                                $nick = $friendica;
+                       }
                }
 
                if (!$nick == "") {
                        $diaspora = preg_replace("=https?://(.*)/u/(.*)=ism", "$2", $profile);
-                       if ($diaspora != $profile)
+                       if ($diaspora != $profile) {
                                $nick = $diaspora;
+                       }
                }
 
                if (!$nick == "") {
                        $twitter = preg_replace("=https?://twitter.com/(.*)=ism", "$1", $profile);
-                       if ($twitter != $profile)
+                       if ($twitter != $profile) {
                                $nick = $twitter;
+                       }
                }
 
 
@@ -3351,8 +4317,9 @@ use \Friendica\Core\Config;
                                if ($StatusnetUser != $profile) {
                                        $UserData = fetch_url("http://".$StatusnetHost."/api/users/show.json?user_id=".$StatusnetUser);
                                        $user = json_decode($UserData);
-                                       if ($user)
+                                       if ($user) {
                                                $nick = $user->screen_name;
+                                       }
                                }
                        }
                }
@@ -3366,10 +4333,11 @@ use \Friendica\Core\Config;
 
                //}
 
-               if ($nick != "")
-                       return($nick);
+               if ($nick != "") {
+                       return $nick;
+               }
 
-               return(false);
+               return false;
        }
 
        function api_in_reply_to($item) {
@@ -3381,7 +4349,7 @@ use \Friendica\Core\Config;
                $in_reply_to['user_id_str'] = NULL;
                $in_reply_to['screen_name'] = NULL;
 
-               if (($item['thr-parent'] != $item['uri']) AND (intval($item['parent']) != intval($item['id']))) {
+               if (($item['thr-parent'] != $item['uri']) && (intval($item['parent']) != intval($item['id']))) {
                        $r = q("SELECT `id` FROM `item` WHERE `uid` = %d AND `uri` = '%s' LIMIT 1",
                                intval($item['uid']),
                                dbesc($item['thr-parent']));
@@ -3427,7 +4395,7 @@ use \Friendica\Core\Config;
        }
 
        function api_clean_plain_items($Text) {
-               $include_entities = strtolower(x($_REQUEST,'include_entities')?$_REQUEST['include_entities']:"false");
+               $include_entities = strtolower(x($_REQUEST, 'include_entities') ? $_REQUEST['include_entities'] : "false");
 
                $Text = bb_CleanPictureLinks($Text);
                $URLSearchString = "^\[\]";
@@ -3462,7 +4430,7 @@ use \Friendica\Core\Config;
                if (isset($data["text"]))
                        $body = $data["text"];
 
-               if (($body == "") AND (isset($data["title"])))
+               if (($body == "") && (isset($data["title"])))
                        $body = $data["title"];
 
                if (isset($data["url"]))
@@ -3521,11 +4489,11 @@ use \Friendica\Core\Config;
 
                $a = get_app();
 
-               if (api_user()===false) throw new ForbiddenException();
+               if (api_user() === false) throw new ForbiddenException();
 
                // params
                $user_info = api_get_user($a);
-               $gid = (x($_REQUEST,'gid') ? $_REQUEST['gid'] : 0);
+               $gid = (x($_REQUEST, 'gid') ? $_REQUEST['gid'] : 0);
                $uid = $user_info['uid'];
 
                // get data of the specified group id or all groups if not specified
@@ -3572,11 +4540,11 @@ use \Friendica\Core\Config;
 
                $a = get_app();
 
-               if (api_user()===false) throw new ForbiddenException();
+               if (api_user() === false) throw new ForbiddenException();
 
                // params
                $user_info = api_get_user($a);
-               $gid = (x($_REQUEST,'gid') ? $_REQUEST['gid'] : 0);
+               $gid = (x($_REQUEST, 'gid') ? $_REQUEST['gid'] : 0);
                $name = (x($_REQUEST, 'name') ? $_REQUEST['name'] : "");
                $uid = $user_info['uid'];
 
@@ -3619,7 +4587,7 @@ use \Friendica\Core\Config;
 
                $a = get_app();
 
-               if (api_user()===false) throw new ForbiddenException();
+               if (api_user() === false) throw new ForbiddenException();
 
                // params
                $user_info = api_get_user($a);
@@ -3685,7 +4653,7 @@ use \Friendica\Core\Config;
 
                $a = get_app();
 
-               if (api_user()===false) throw new ForbiddenException();
+               if (api_user() === false) throw new ForbiddenException();
 
                // params
                $user_info = api_get_user($a);
@@ -3737,14 +4705,14 @@ use \Friendica\Core\Config;
                $success = array('success' => true, 'gid' => $gid, 'name' => $name, 'status' => $status, 'wrong users' => $errorusers);
                return api_format_data("group_update", $type, array('result' => $success));
        }
-       api_register_func('api/friendica/group_update', 'api_friendica_group_update', true, API_METHOD_POST);
 
+       api_register_func('api/friendica/group_update', 'api_friendica_group_update', true, API_METHOD_POST);
 
        function api_friendica_activity($type) {
 
                $a = get_app();
 
-               if (api_user()===false) throw new ForbiddenException();
+               if (api_user() === false) throw new ForbiddenException();
                $verb = strtolower($a->argv[3]);
                $verb = preg_replace("|\..*$|", "", $verb);
 
@@ -3763,6 +4731,8 @@ use \Friendica\Core\Config;
                }
 
        }
+
+       /// @TODO move to top of file or somwhere 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);
@@ -3784,7 +4754,7 @@ use \Friendica\Core\Config;
 
                $a = get_app();
 
-               if (api_user()===false) throw new ForbiddenException();
+               if (api_user() === false) throw new ForbiddenException();
                if ($a->argc!==3) throw new BadRequestException("Invalid argument count");
                $nm = new NotificationsManager();
 
@@ -3809,11 +4779,11 @@ use \Friendica\Core\Config;
         * @param string $type Known types are 'atom', 'rss', 'xml' and 'json'
         * @return string
         */
-       function api_friendica_notification_seen($type){
+       function api_friendica_notification_seen($type) {
 
                $a = get_app();
 
-               if (api_user()===false) throw new ForbiddenException();
+               if (api_user() === false) throw new ForbiddenException();
                if ($a->argc!==4) throw new BadRequestException("Invalid argument count");
 
                $id = (x($_REQUEST, 'id') ? intval($_REQUEST['id']) : 0);
@@ -3841,19 +4811,21 @@ use \Friendica\Core\Config;
                return api_format_data('result', $type, array('result' => "success"));
        }
 
+       /// @TODO move to top of file or somwhere 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);
 
-
        /**
         * @brief update a direct_message to seen state
         *
         * @param string $type Known types are 'atom', 'rss', 'xml' and 'json'
         * @return string (success result=ok, error result=error with error message)
         */
-       function api_friendica_direct_messages_setseen($type){
+       function api_friendica_direct_messages_setseen($type) {
                $a = get_app();
-               if (api_user()===false) throw new ForbiddenException();
+               if (api_user() === false) {
+                       throw new ForbiddenException();
+               }
 
                // params
                $user_info = api_get_user($a);
@@ -3870,6 +4842,7 @@ use \Friendica\Core\Config;
                $r = q("SELECT `id` FROM `mail` WHERE `id` = %d AND `uid` = %d",
                        intval($id),
                        intval($uid));
+
                // error message if specified id is not in database
                if (!dbm::is_result($r)) {
                        $answer = array('result' => 'error', 'message' => 'message id not in database');
@@ -3890,10 +4863,9 @@ use \Friendica\Core\Config;
                        return api_format_data("direct_messages_setseen", $type, array('$result' => $answer));
                }
        }
-       api_register_func('api/friendica/direct_messages_setseen', 'api_friendica_direct_messages_setseen', true);
-
-
 
+       /// @TODO move to top of file or somwhere better
+       api_register_func('api/friendica/direct_messages_setseen', 'api_friendica_direct_messages_setseen', true);
 
        /**
         * @brief search for direct_messages containing a searchstring through api
@@ -3903,14 +4875,16 @@ use \Friendica\Core\Config;
         *                          success=false if nothing was found, search_result='nothing found',
         *                 error: result=error with error message)
         */
-       function api_friendica_direct_messages_search($type){
+       function api_friendica_direct_messages_search($type) {
                $a = get_app();
 
-               if (api_user()===false) throw new ForbiddenException();
+               if (api_user() === false) {
+                       throw new ForbiddenException();
+               }
 
                // params
                $user_info = api_get_user($a);
-               $searchstring = (x($_REQUEST,'searchstring') ? $_REQUEST['searchstring'] : "");
+               $searchstring = (x($_REQUEST, 'searchstring') ? $_REQUEST['searchstring'] : "");
                $uid = $user_info['uid'];
 
                // error if no searchstring specified
@@ -3926,29 +4900,32 @@ use \Friendica\Core\Config;
                );
 
                $profile_url = $user_info["url"];
+
                // message if nothing was found
-               if (!dbm::is_result($r))
+               if (!dbm::is_result($r)) {
                        $success = array('success' => false, 'search_results' => 'problem with query');
-               else if (count($r) == 0)
+               } elseif (count($r) == 0) {
                        $success = array('success' => false, 'search_results' => 'nothing found');
-               else {
-                       $ret = Array();
-                       foreach($r as $item) {
-                               if ($box == "inbox" || $item['from-url'] != $profile_url){
+               else {
+                       $ret = array();
+                       foreach ($r as $item) {
+                               if ($box == "inbox" || $item['from-url'] != $profile_url) {
                                        $recipient = $user_info;
                                        $sender = api_get_user($a,normalise_link($item['contact-url']));
-                               }
-                               elseif ($box == "sentbox" || $item['from-url'] == $profile_url){
+                               } elseif ($box == "sentbox" || $item['from-url'] == $profile_url) {
                                        $recipient = api_get_user($a,normalise_link($item['contact-url']));
                                        $sender = $user_info;
                                }
-                               $ret[]=api_format_messages($item, $recipient, $sender);
+
+                               $ret[] = api_format_messages($item, $recipient, $sender);
                        }
                        $success = array('success' => true, 'search_results' => $ret);
                }
 
                return api_format_data("direct_message_search", $type, array('$result' => $success));
        }
+
+       /// @TODO move to top of file or somwhere better
        api_register_func('api/friendica/direct_messages_search', 'api_friendica_direct_messages_search', true);
 
        /**
@@ -3957,13 +4934,15 @@ use \Friendica\Core\Config;
         * @param string $type Known types are 'atom', 'rss', 'xml' and 'json'
         * @return string
         */
-       function api_friendica_profile_show($type){
+       function api_friendica_profile_show($type) {
                $a = get_app();
 
-               if (api_user()===false) throw new ForbiddenException();
+               if (api_user() === false) {
+                       throw new ForbiddenException();
+               }
 
                // input params
-               $profileid = (x($_REQUEST,'profile_id') ? $_REQUEST['profile_id'] : 0);
+               $profileid = (x($_REQUEST, 'profile_id') ? $_REQUEST['profile_id'] : 0);
 
                // retrieve general information about profiles for user
                $multi_profiles = feature_enabled(api_user(),'multi_profiles');
@@ -3974,9 +4953,11 @@ use \Friendica\Core\Config;
                        $r = q("SELECT * FROM `profile` WHERE `uid` = %d AND `id` = %d",
                                intval(api_user()),
                                intval($profileid));
+
                        // error message if specified gid is not in database
-                       if (!dbm::is_result($r))
+                       if (!dbm::is_result($r)) {
                                throw new BadRequestException("profile_id not available");
+                       }
                } else {
                        $r = q("SELECT * FROM `profile` WHERE `uid` = %d",
                                intval(api_user()));
@@ -3994,13 +4975,13 @@ use \Friendica\Core\Config;
 
                        foreach ($r as $rr) {
                                $user = api_get_user($a, $rr['nurl']);
-                               ($type == "xml") ? $users[$k++.":user"] = $user : $users[] = $user;
+                               ($type == "xml") ? $users[$k++ . ":user"] = $user : $users[] = $user;
                        }
                        $profile['users'] = $users;
 
                        // add prepared profile data to array for final return
                        if ($type == "xml") {
-                               $profiles[$k++.":profile"] = $profile;
+                               $profiles[$k++ . ":profile"] = $profile;
                        } else {
                                $profiles[] = $profile;
                        }
@@ -4018,6 +4999,7 @@ use \Friendica\Core\Config;
        api_register_func('api/friendica/profile/show', 'api_friendica_profile_show', true, API_METHOD_GET);
 
 /*
+@TODO Maybe open to implement?
 To.Do:
     [pagename] => api/1.1/statuses/lookup.json
     [id] => 605138389168451584
@@ -4039,7 +5021,6 @@ friendships/exists
 friendships/show
 account/update_location
 account/update_profile_background_image
-account/update_profile_image
 blocks/create
 blocks/destroy
 friendica/profile/update