X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;ds=sidebyside;f=include%2Fapi.php;h=8325e39b71442f6518054ac1293ea3657ae666ab;hb=1b5fa55bdcc6d0d62bc5ee2a6182027b9d552318;hp=23afa72de9c3c2edbdd0a812ac86089f133753cc;hpb=eefcf99af47013a7837a00910569b2b964e2bf36;p=friendica.git diff --git a/include/api.php b/include/api.php index 23afa72de9..8325e39b71 100644 --- a/include/api.php +++ b/include/api.php @@ -7,6 +7,7 @@ */ use Friendica\App; +use Friendica\BaseObject; use Friendica\Content\ContactSelector; use Friendica\Content\Feature; use Friendica\Content\Text\BBCode; @@ -15,7 +16,6 @@ use Friendica\Core\Config; use Friendica\Core\Hook; use Friendica\Core\L10n; use Friendica\Core\Logger; -use Friendica\Core\NotificationsManager; use Friendica\Core\PConfig; use Friendica\Core\Protocol; use Friendica\Core\Session; @@ -26,7 +26,9 @@ use Friendica\Model\Contact; use Friendica\Model\Group; use Friendica\Model\Item; use Friendica\Model\Mail; +use Friendica\Model\Notify; use Friendica\Model\Photo; +use Friendica\Model\Profile; use Friendica\Model\User; use Friendica\Network\FKOAuth1; use Friendica\Network\HTTPException; @@ -40,16 +42,18 @@ use Friendica\Network\HTTPException\NotImplementedException; use Friendica\Network\HTTPException\TooManyRequestsException; use Friendica\Network\HTTPException\UnauthorizedException; use Friendica\Object\Image; +use Friendica\Protocol\Activity; use Friendica\Protocol\Diaspora; use Friendica\Util\DateTimeFormat; +use Friendica\Util\Images; use Friendica\Util\Network; use Friendica\Util\Proxy as ProxyUtils; use Friendica\Util\Strings; use Friendica\Util\XML; -require_once 'mod/share.php'; -require_once 'mod/item.php'; -require_once 'mod/wall_upload.php'; +require_once __DIR__ . '/../mod/share.php'; +require_once __DIR__ . '/../mod/item.php'; +require_once __DIR__ . '/../mod/wall_upload.php'; define('API_METHOD_ANY', '*'); define('API_METHOD_GET', 'GET'); @@ -161,6 +165,7 @@ function api_register_func($path, $func, $auth = false, $method = API_METHOD_ANY * @brief Login API user * * @param App $a App + * @throws ForbiddenException * @throws InternalServerErrorException * @throws UnauthorizedException * @hook 'authenticate' @@ -169,8 +174,6 @@ function api_register_func($path, $func, $auth = false, $method = API_METHOD_ANY * 'password' => password from login form * 'authenticated' => return status, * 'user_record' => return authenticated user record - * @hook 'logged_in' - * array $user logged user record */ function api_login(App $a) { @@ -181,7 +184,7 @@ function api_login(App $a) list($consumer, $token) = $oauth1->verify_request($request); if (!is_null($token)) { $oauth1->loginUser($token->uid); - Hook::callAll('logged_in', $a->user); + Session::set('allow_api', true); return; } echo __FILE__.__LINE__.__FUNCTION__ . "
"; @@ -207,8 +210,8 @@ function api_login(App $a) throw new UnauthorizedException("This API requires login"); } - $user = defaults($_SERVER, 'PHP_AUTH_USER', ''); - $password = defaults($_SERVER, 'PHP_AUTH_PW', ''); + $user = $_SERVER['PHP_AUTH_USER'] ?? ''; + $password = $_SERVER['PHP_AUTH_PW'] ?? ''; // allow "user@server" login (but ignore 'server' part) $at = strstr($user, "@", true); @@ -272,7 +275,7 @@ function api_check_method($method) if ($method == "*") { return true; } - return (stripos($method, defaults($_SERVER, 'REQUEST_METHOD', 'GET')) !== false); + return (stripos($method, $_SERVER['REQUEST_METHOD'] ?? 'GET') !== false); } /** @@ -322,7 +325,7 @@ function api_call(App $a) $stamp = microtime(true); $return = call_user_func($info['func'], $type); - $duration = (float) (microtime(true) - $stamp); + $duration = floatval(microtime(true) - $stamp); Logger::info(API_LOG_PREFIX . 'username {username}', ['module' => 'api', 'action' => 'call', 'username' => $a->user['username'], 'duration' => round($duration, 2)]); @@ -342,11 +345,13 @@ function api_call(App $a) break; case "json": header("Content-Type: application/json"); - $json = json_encode(end($return)); - if (!empty($_GET['callback'])) { - $json = $_GET['callback'] . "(" . $json . ")"; + if (!empty($return)) { + $json = json_encode(end($return)); + if (!empty($_GET['callback'])) { + $json = $_GET['callback'] . "(" . $json . ")"; + } + $return = $json; } - $return = $json; break; case "rss": header("Content-Type: application/rss+xml"); @@ -611,7 +616,7 @@ function api_get_user(App $a, $contact_id = null) 'name' => $contact["name"], 'screen_name' => (($contact['nick']) ? $contact['nick'] : $contact['name']), 'location' => ($contact["location"] != "") ? $contact["location"] : ContactSelector::networkToName($contact['network'], $contact['url']), - 'description' => HTML::toPlaintext(BBCode::toPlaintext($contact["about"])), + 'description' => BBCode::toPlaintext($contact["about"]), 'profile_image_url' => $contact["micro"], 'profile_image_url_https' => $contact["micro"], 'profile_image_url_profile_size' => $contact["thumb"], @@ -690,7 +695,7 @@ function api_get_user(App $a, $contact_id = null) 'name' => (($uinfo[0]['name']) ? $uinfo[0]['name'] : $uinfo[0]['nick']), 'screen_name' => (($uinfo[0]['nick']) ? $uinfo[0]['nick'] : $uinfo[0]['name']), 'location' => $location, - 'description' => HTML::toPlaintext(BBCode::toPlaintext($description)), + 'description' => BBCode::toPlaintext($description), 'profile_image_url' => $uinfo[0]['micro'], 'profile_image_url_https' => $uinfo[0]['micro'], 'profile_image_url_profile_size' => $uinfo[0]["thumb"], @@ -773,14 +778,14 @@ function api_get_user(App $a, $contact_id = null) */ function api_item_get_user(App $a, $item) { - $status_user = api_get_user($a, defaults($item, 'author-id', null)); + $status_user = api_get_user($a, $item['author-id'] ?? null); $author_user = $status_user; - $status_user["protected"] = defaults($item, 'private', 0); + $status_user["protected"] = $item['private'] ?? 0; - if (defaults($item, 'thr-parent', '') == defaults($item, 'uri', '')) { - $owner_user = api_get_user($a, defaults($item, 'owner-id', null)); + if (($item['thr-parent'] ?? '') == ($item['uri'] ?? '')) { + $owner_user = api_get_user($a, $item['owner-id'] ?? null); } else { $owner_user = $author_user; } @@ -944,7 +949,7 @@ function api_account_verify_credentials($type) unset($_REQUEST["screen_name"]); unset($_GET["screen_name"]); - $skip_status = defaults($_REQUEST, 'skip_status', false); + $skip_status = $_REQUEST['skip_status'] ?? false; $user_info = api_get_user($a); @@ -1154,18 +1159,21 @@ function api_statuses_update($type) } } - // 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 (DBA::isResult($r)) { - $phototypes = Image::supportedTypes(); - $ext = $phototypes[$r[0]['type']]; - $_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]'; + $ids = explode(',', requestdata('media_ids')); + foreach ($ids as $id) { + $r = q( + "SELECT `resource-id`, `scale`, `nickname`, `type`, `desc` FROM `photo` INNER JOIN `user` ON `user`.`uid` = `photo`.`uid` WHERE `resource-id` IN (SELECT `resource-id` FROM `photo` WHERE `id` = %d) AND `scale` > 0 AND `photo`.`uid` = %d ORDER BY `photo`.`width` DESC LIMIT 1", + intval($id), + api_user() + ); + if (DBA::isResult($r)) { + $phototypes = Images::supportedTypes(); + $ext = $phototypes[$r[0]['type']]; + $description = $r[0]['desc'] ?? ''; + $_REQUEST['body'] .= "\n\n" . '[url=' . System::baseUrl() . '/photos/' . $r[0]['nickname'] . '/image/' . $r[0]['resource-id'] . ']'; + $_REQUEST['body'] .= '[img=' . System::baseUrl() . '/photo/' . $r[0]['resource-id'] . '-' . $r[0]['scale'] . '.' . $ext . ']' . $description . '[/img][/url]'; + } } } @@ -1238,6 +1246,65 @@ function api_media_upload() /// @TODO move to top of file or somewhere better api_register_func('api/media/upload', 'api_media_upload', true, API_METHOD_POST); +/** + * Updates media meta data (picture descriptions) + * + * @param string $type Return type (atom, rss, xml, json) + * + * @return array|string + * @throws BadRequestException + * @throws ForbiddenException + * @throws ImagickException + * @throws InternalServerErrorException + * @throws TooManyRequestsException + * @throws UnauthorizedException + * @see https://developer.twitter.com/en/docs/tweets/post-and-engage/api-reference/post-statuses-update + * + * @todo Compare the corresponding Twitter function for correct return values + */ +function api_media_metadata_create($type) +{ + $a = \get_app(); + + if (api_user() === false) { + Logger::info('no user'); + throw new ForbiddenException(); + } + + api_get_user($a); + + $postdata = Network::postdata(); + + if (empty($postdata)) { + throw new BadRequestException("No post data"); + } + + $data = json_decode($postdata, true); + if (empty($data)) { + throw new BadRequestException("Invalid post data"); + } + + if (empty($data['media_id']) || empty($data['alt_text'])) { + throw new BadRequestException("Missing post data values"); + } + + if (empty($data['alt_text']['text'])) { + throw new BadRequestException("No alt text."); + } + + Logger::info('Updating metadata', ['media_id' => $data['media_id']]); + + $condition = ['id' => $data['media_id'], 'uid' => api_user()]; + $photo = DBA::selectFirst('photo', ['resource-id'], $condition); + if (!DBA::isResult($photo)) { + throw new BadRequestException("Metadata not found."); + } + + DBA::update('photo', ['desc' => $data['alt_text']['text']], ['resource-id' => $photo['resource-id']]); +} + +api_register_func('api/media/metadata/create', 'api_media_metadata_create', true, API_METHOD_POST); + /** * @param string $type Return format (atom, rss, xml, json) * @param int $item_id @@ -1310,7 +1377,7 @@ function api_get_item(array $condition) */ function api_users_show($type) { - $a = \Friendica\BaseObject::getApp(); + $a = BaseObject::getApp(); $user_info = api_get_user($a); @@ -1453,10 +1520,12 @@ function api_search($type) $count = $_REQUEST['count']; } - $since_id = defaults($_REQUEST, 'since_id', 0); - $max_id = defaults($_REQUEST, 'max_id', 0); - $page = (!empty($_REQUEST['page']) ? $_REQUEST['page'] - 1 : 0); - $start = $page * $count; + $since_id = $_REQUEST['since_id'] ?? 0; + $max_id = $_REQUEST['max_id'] ?? 0; + $page = $_REQUEST['page'] ?? 1; + + $start = max(0, ($page - 1) * $count); + $params = ['order' => ['id' => true], 'limit' => [$start, $count]]; if (preg_match('/^#(\w+)$/', $searchTerm, $matches) === 1 && isset($matches[1])) { $searchTerm = $matches[1]; @@ -1544,17 +1613,14 @@ function api_statuses_home_timeline($type) // get last network messages // params - $count = defaults($_REQUEST, 'count', 20); - $page = (!empty($_REQUEST['page']) ? $_REQUEST['page'] - 1 : 0); - if ($page < 0) { - $page = 0; - } - $since_id = defaults($_REQUEST, 'since_id', 0); - $max_id = defaults($_REQUEST, 'max_id', 0); + $count = $_REQUEST['count'] ?? 20; + $page = $_REQUEST['page']?? 0; + $since_id = $_REQUEST['since_id'] ?? 0; + $max_id = $_REQUEST['max_id'] ?? 0; $exclude_replies = !empty($_REQUEST['exclude_replies']); - $conversation_id = defaults($_REQUEST, 'conversation_id', 0); + $conversation_id = $_REQUEST['conversation_id'] ?? 0; - $start = $page * $count; + $start = max(0, ($page - 1) * $count); $condition = ["`uid` = ? AND `gravity` IN (?, ?) AND `item`.`id` > ?", api_user(), GRAVITY_PARENT, GRAVITY_COMMENT, $since_id]; @@ -1634,17 +1700,14 @@ function api_statuses_public_timeline($type) // get last network messages // params - $count = defaults($_REQUEST, 'count', 20); - $page = (!empty($_REQUEST['page']) ? $_REQUEST['page'] -1 : 0); - if ($page < 0) { - $page = 0; - } - $since_id = defaults($_REQUEST, 'since_id', 0); - $max_id = defaults($_REQUEST, 'max_id', 0); + $count = $_REQUEST['count'] ?? 20; + $page = $_REQUEST['page'] ?? 1; + $since_id = $_REQUEST['since_id'] ?? 0; + $max_id = $_REQUEST['max_id'] ?? 0; $exclude_replies = (!empty($_REQUEST['exclude_replies']) ? 1 : 0); - $conversation_id = defaults($_REQUEST, 'conversation_id', 0); + $conversation_id = $_REQUEST['conversation_id'] ?? 0; - $start = $page * $count; + $start = max(0, ($page - 1) * $count); if ($exclude_replies && !$conversation_id) { $condition = ["`gravity` IN (?, ?) AND `iid` > ? AND NOT `private` AND `wall` AND NOT `user`.`hidewall` AND NOT `author`.`hidden`", @@ -1719,16 +1782,14 @@ function api_statuses_networkpublic_timeline($type) throw new ForbiddenException(); } - $since_id = defaults($_REQUEST, 'since_id', 0); - $max_id = defaults($_REQUEST, 'max_id', 0); + $since_id = $_REQUEST['since_id'] ?? 0; + $max_id = $_REQUEST['max_id'] ?? 0; // pagination - $count = defaults($_REQUEST, 'count', 20); - $page = defaults($_REQUEST, 'page', 1); - if ($page < 1) { - $page = 1; - } - $start = ($page - 1) * $count; + $count = $_REQUEST['count'] ?? 20; + $page = $_REQUEST['page'] ?? 1; + + $start = max(0, ($page - 1) * $count); $condition = ["`uid` = 0 AND `gravity` IN (?, ?) AND `thread`.`iid` > ? AND NOT `private`", GRAVITY_PARENT, GRAVITY_COMMENT, $since_id]; @@ -1783,15 +1844,15 @@ function api_statuses_show($type) } // params - $id = intval(defaults($a->argv, 3, 0)); + $id = intval($a->argv[3] ?? 0); if ($id == 0) { - $id = intval(defaults($_REQUEST, 'id', 0)); + $id = intval($_REQUEST['id'] ?? 0); } // Hotot workaround if ($id == 0) { - $id = intval(defaults($a->argv, 4, 0)); + $id = intval($a->argv[4] ?? 0); } Logger::log('API: api_statuses_show: ' . $id); @@ -1862,24 +1923,21 @@ function api_conversation_show($type) } // params - $id = intval(defaults($a->argv , 3 , 0)); - $since_id = intval(defaults($_REQUEST, 'since_id', 0)); - $max_id = intval(defaults($_REQUEST, 'max_id' , 0)); - $count = intval(defaults($_REQUEST, 'count' , 20)); - $page = intval(defaults($_REQUEST, 'page' , 1)) - 1; - if ($page < 0) { - $page = 0; - } + $id = intval($a->argv[3] ?? 0); + $since_id = intval($_REQUEST['since_id'] ?? 0); + $max_id = intval($_REQUEST['max_id'] ?? 0); + $count = intval($_REQUEST['count'] ?? 20); + $page = intval($_REQUEST['page'] ?? 1); - $start = $page * $count; + $start = max(0, ($page - 1) * $count); if ($id == 0) { - $id = intval(defaults($_REQUEST, 'id', 0)); + $id = intval($_REQUEST['id'] ?? 0); } // Hotot workaround if ($id == 0) { - $id = intval(defaults($a->argv, 4, 0)); + $id = intval($a->argv[4] ?? 0); } Logger::info(API_LOG_PREFIX . '{subaction}', ['module' => 'api', 'action' => 'conversation', 'subaction' => 'show', 'id' => $id]); @@ -1948,15 +2006,15 @@ function api_statuses_repeat($type) api_get_user($a); // params - $id = intval(defaults($a->argv, 3, 0)); + $id = intval($a->argv[3] ?? 0); if ($id == 0) { - $id = intval(defaults($_REQUEST, 'id', 0)); + $id = intval($_REQUEST['id'] ?? 0); } // Hotot workaround if ($id == 0) { - $id = intval(defaults($a->argv, 4, 0)); + $id = intval($a->argv[4] ?? 0); } Logger::log('API: api_statuses_repeat: '.$id); @@ -2019,15 +2077,15 @@ function api_statuses_destroy($type) api_get_user($a); // params - $id = intval(defaults($a->argv, 3, 0)); + $id = intval($a->argv[3] ?? 0); if ($id == 0) { - $id = intval(defaults($_REQUEST, 'id', 0)); + $id = intval($_REQUEST['id'] ?? 0); } // Hotot workaround if ($id == 0) { - $id = intval(defaults($a->argv, 4, 0)); + $id = intval($a->argv[4] ?? 0); } Logger::log('API: api_statuses_destroy: '.$id); @@ -2073,15 +2131,12 @@ function api_statuses_mentions($type) // get last network messages // params - $since_id = defaults($_REQUEST, 'since_id', 0); - $max_id = defaults($_REQUEST, 'max_id' , 0); - $count = defaults($_REQUEST, 'count' , 20); - $page = defaults($_REQUEST, 'page' , 1); - if ($page < 1) { - $page = 1; - } + $since_id = $_REQUEST['since_id'] ?? 0; + $max_id = $_REQUEST['max_id'] ?? 0; + $count = $_REQUEST['count'] ?? 20; + $page = $_REQUEST['page'] ?? 1; - $start = ($page - 1) * $count; + $start = max(0, ($page - 1) * $count); $condition = ["`uid` = ? AND `gravity` IN (?, ?) AND `item`.`id` > ? AND `author-id` != ? AND `item`.`parent` IN (SELECT `iid` FROM `thread` WHERE `thread`.`uid` = ? AND `thread`.`mention` AND NOT `thread`.`ignored`)", @@ -2143,18 +2198,16 @@ function api_statuses_user_timeline($type) Logger::DEBUG ); - $since_id = defaults($_REQUEST, 'since_id', 0); - $max_id = defaults($_REQUEST, 'max_id', 0); + $since_id = $_REQUEST['since_id'] ?? 0; + $max_id = $_REQUEST['max_id'] ?? 0; $exclude_replies = !empty($_REQUEST['exclude_replies']); - $conversation_id = defaults($_REQUEST, 'conversation_id', 0); + $conversation_id = $_REQUEST['conversation_id'] ?? 0; // pagination - $count = defaults($_REQUEST, 'count', 20); - $page = defaults($_REQUEST, 'page', 1); - if ($page < 1) { - $page = 1; - } - $start = ($page - 1) * $count; + $count = $_REQUEST['count'] ?? 20; + $page = $_REQUEST['page'] ?? 1; + + $start = max(0, ($page - 1) * $count); $condition = ["`uid` = ? AND `gravity` IN (?, ?) AND `item`.`id` > ? AND `item`.`contact-id` = ?", api_user(), GRAVITY_PARENT, GRAVITY_COMMENT, $since_id, $user_info['cid']]; @@ -2233,9 +2286,9 @@ function api_favorites_create_destroy($type) } $action = str_replace("." . $type, "", $a->argv[$action_argv_id]); if ($a->argc == $action_argv_id + 2) { - $itemid = intval(defaults($a->argv, $action_argv_id + 1, 0)); + $itemid = intval($a->argv[$action_argv_id + 1] ?? 0); } else { - $itemid = intval(defaults($_REQUEST, 'id', 0)); + $itemid = intval($_REQUEST['id'] ?? 0); } $item = Item::selectFirstForUser(api_user(), [], ['id' => $itemid, 'uid' => api_user()]); @@ -2315,15 +2368,12 @@ function api_favorites($type) $ret = []; } else { // params - $since_id = defaults($_REQUEST, 'since_id', 0); - $max_id = defaults($_REQUEST, 'max_id', 0); - $count = defaults($_GET, 'count', 20); - $page = (!empty($_REQUEST['page']) ? $_REQUEST['page'] -1 : 0); - if ($page < 0) { - $page = 0; - } + $since_id = $_REQUEST['since_id'] ?? 0; + $max_id = $_REQUEST['max_id'] ?? 0; + $count = $_GET['count'] ?? 20; + $page = $_REQUEST['page'] ?? 1; - $start = $page*$count; + $start = max(0, ($page - 1) * $count); $condition = ["`uid` = ? AND `gravity` IN (?, ?) AND `id` > ? AND `starred`", api_user(), GRAVITY_PARENT, GRAVITY_COMMENT, $since_id]; @@ -2374,14 +2424,14 @@ function api_format_messages($item, $recipient, $sender) 'sender_id' => $sender['id'], 'text' => "", 'recipient_id' => $recipient['id'], - 'created_at' => api_date(defaults($item, 'created', DateTimeFormat::utcNow())), + 'created_at' => api_date($item['created'] ?? DateTimeFormat::utcNow()), 'sender_screen_name' => $sender['screen_name'], 'recipient_screen_name' => $recipient['screen_name'], 'sender' => $sender, 'recipient' => $recipient, 'title' => "", - 'friendica_seen' => defaults($item, 'seen', 0), - 'friendica_parent_uri' => defaults($item, 'parent-uri', ''), + 'friendica_seen' => $item['seen'] ?? 0, + 'friendica_parent_uri' => $item['parent-uri'] ?? '', ]; // "uid" and "self" are only needed for some internal stuff, so remove it from here @@ -2444,8 +2494,8 @@ function api_convert_item($item) $statustext = trim($statustitle."\n\n".$statusbody); } - if ((defaults($item, 'network', Protocol::PHANTOM) == Protocol::FEED) && (mb_strlen($statustext)> 1000)) { - $statustext = mb_substr($statustext, 0, 1000) . "... \n" . defaults($item, 'plink', ''); + if ((($item['network'] ?? Protocol::PHANTOM) == Protocol::FEED) && (mb_strlen($statustext)> 1000)) { + $statustext = mb_substr($statustext, 0, 1000) . "... \n" . ($item['plink'] ?? ''); } $statushtml = BBCode::convert(api_clean_attachments($body), false); @@ -2479,7 +2529,7 @@ function api_convert_item($item) } // feeds without body should contain the link - if ((defaults($item, 'network', Protocol::PHANTOM) == Protocol::FEED) && (strlen($item['body']) == 0)) { + if ((($item['network'] ?? Protocol::PHANTOM) == Protocol::FEED) && (strlen($item['body']) == 0)) { $statushtml .= BBCode::convert($item['plink']); } @@ -2504,6 +2554,7 @@ function api_get_attachments(&$body) { $text = $body; $text = preg_replace("/\[img\=([0-9]*)x([0-9]*)\](.*?)\[\/img\]/ism", '[img]$3[/img]', $text); + $text = preg_replace("/\[img\=(.*?)\](.*?)\[\/img\]/ism", '[img]$1[/img]', $text); $URLSearchString = "^\[\]"; $ret = preg_match_all("/\[img\]([$URLSearchString]*)\[\/img\]/ism", $text, $images); @@ -2515,14 +2566,14 @@ function api_get_attachments(&$body) $attachments = []; foreach ($images[1] as $image) { - $imagedata = Image::getInfoFromURL($image); + $imagedata = Images::getInfoFromURLCached($image); if ($imagedata) { $attachments[] = ["url" => $image, "mimetype" => $imagedata["mime"], "size" => $imagedata["size"]]; } } - if (strstr(defaults($_SERVER, 'HTTP_USER_AGENT', ''), "AndStatus")) { + if (strstr($_SERVER['HTTP_USER_AGENT'] ?? '', 'AndStatus')) { foreach ($images[0] as $orig) { $body = str_replace($orig, "", $body); } @@ -2542,7 +2593,7 @@ function api_get_attachments(&$body) */ function api_get_entitities(&$text, $bbcode) { - $include_entities = strtolower(defaults($_REQUEST, 'include_entities', "false")); + $include_entities = strtolower($_REQUEST['include_entities'] ?? 'false'); if ($include_entities != "true") { preg_match_all("/\[img](.*?)\[\/img\]/ism", $bbcode, $images); @@ -2630,19 +2681,29 @@ function api_get_entitities(&$text, $bbcode) } } - preg_match_all("/\[img](.*?)\[\/img\]/ism", $bbcode, $images); + preg_match_all("/\[img\=(.*?)\](.*?)\[\/img\]/ism", $bbcode, $images, PREG_SET_ORDER); $ordered_images = []; + foreach ($images as $image) { + $start = iconv_strpos($text, $image[1], 0, "UTF-8"); + if (!($start === false)) { + $ordered_images[$start] = ['url' => $image[1], 'alt' => $image[2]]; + } + } + + preg_match_all("/\[img](.*?)\[\/img\]/ism", $bbcode, $images); foreach ($images[1] as $image) { - //$start = strpos($text, $url, $offset); $start = iconv_strpos($text, $image, 0, "UTF-8"); if (!($start === false)) { - $ordered_images[$start] = $image; + $ordered_images[$start] = ['url' => $image, 'alt' => '']; } } //$entities["media"] = array(); $offset = 0; - foreach ($ordered_images as $url) { + foreach ($ordered_images as $image) { + $url = $image['url']; + $ext_alt_text = $image['alt']; + $display_url = str_replace(["http://www.", "https://www."], ["", ""], $url); $display_url = str_replace(["http://", "https://"], ["", ""], $display_url); @@ -2652,7 +2713,7 @@ function api_get_entitities(&$text, $bbcode) $start = iconv_strpos($text, $url, $offset, "UTF-8"); if (!($start === false)) { - $image = Image::getInfoFromURL($url); + $image = Images::getInfoFromURLCached($url); if ($image) { // If image cache is activated, then use the following sizes: // thumb (150), small (340), medium (600) and large (1024) @@ -2660,19 +2721,19 @@ function api_get_entitities(&$text, $bbcode) $media_url = ProxyUtils::proxifyUrl($url); $sizes = []; - $scale = Image::getScalingDimensions($image[0], $image[1], 150); + $scale = Images::getScalingDimensions($image[0], $image[1], 150); $sizes["thumb"] = ["w" => $scale["width"], "h" => $scale["height"], "resize" => "fit"]; if (($image[0] > 150) || ($image[1] > 150)) { - $scale = Image::getScalingDimensions($image[0], $image[1], 340); + $scale = Images::getScalingDimensions($image[0], $image[1], 340); $sizes["small"] = ["w" => $scale["width"], "h" => $scale["height"], "resize" => "fit"]; } - $scale = Image::getScalingDimensions($image[0], $image[1], 600); + $scale = Images::getScalingDimensions($image[0], $image[1], 600); $sizes["medium"] = ["w" => $scale["width"], "h" => $scale["height"], "resize" => "fit"]; if (($image[0] > 600) || ($image[1] > 600)) { - $scale = Image::getScalingDimensions($image[0], $image[1], 1024); + $scale = Images::getScalingDimensions($image[0], $image[1], 1024); $sizes["large"] = ["w" => $scale["width"], "h" => $scale["height"], "resize" => "fit"]; } } else { @@ -2689,6 +2750,7 @@ function api_get_entitities(&$text, $bbcode) "url" => $url, "display_url" => $display_url, "expanded_url" => $url, + "ext_alt_text" => $ext_alt_text, "type" => "photo", "sizes" => $sizes]; } @@ -2781,19 +2843,19 @@ function api_format_items_activities($item, $type = "json") // get user data and add it to the array of the activity $user = api_get_user($a, $parent_item['author-id']); switch ($parent_item['verb']) { - case ACTIVITY_LIKE: + case Activity::LIKE: $activities['like'][] = $user; break; - case ACTIVITY_DISLIKE: + case Activity::DISLIKE: $activities['dislike'][] = $user; break; - case ACTIVITY_ATTEND: + case Activity::ATTEND: $activities['attendyes'][] = $user; break; - case ACTIVITY_ATTENDNO: + case Activity::ATTENDNO: $activities['attendno'][] = $user; break; - case ACTIVITY_ATTENDMAYBE: + case Activity::ATTENDMAYBE: $activities['attendmaybe'][] = $user; break; default: @@ -2889,7 +2951,7 @@ function api_format_items_profiles($profile_row) */ function api_format_items($items, $user_info, $filter_user = false, $type = "json") { - $a = \Friendica\BaseObject::getApp(); + $a = BaseObject::getApp(); $ret = []; @@ -2923,7 +2985,7 @@ function api_format_items($items, $user_info, $filter_user = false, $type = "jso */ function api_format_item($item, $type = "json", $status_user = null, $author_user = null, $owner_user = null) { - $a = \Friendica\BaseObject::getApp(); + $a = BaseObject::getApp(); if (empty($status_user) || empty($author_user) || empty($owner_user)) { list($status_user, $author_user, $owner_user) = api_item_get_user($a, $item); @@ -2964,6 +3026,8 @@ function api_format_item($item, $type = "json", $status_user = null, $author_use 'statusnet_conversation_id' => $item['parent'], 'external_url' => System::baseUrl() . "/display/" . $item['guid'], 'friendica_activities' => api_format_items_activities($item, $type), + 'friendica_title' => $item['title'], + 'friendica_html' => BBCode::convert($item['body'], false) ]; if (count($converted["attachments"]) > 0) { @@ -3234,17 +3298,14 @@ function api_lists_statuses($type) } // params - $count = defaults($_REQUEST, 'count', 20); - $page = (!empty($_REQUEST['page']) ? $_REQUEST['page'] - 1 : 0); - if ($page < 0) { - $page = 0; - } - $since_id = defaults($_REQUEST, 'since_id', 0); - $max_id = defaults($_REQUEST, 'max_id', 0); + $count = $_REQUEST['count'] ?? 20; + $page = $_REQUEST['page'] ?? 1; + $since_id = $_REQUEST['since_id'] ?? 0; + $max_id = $_REQUEST['max_id'] ?? 0; $exclude_replies = (!empty($_REQUEST['exclude_replies']) ? 1 : 0); - $conversation_id = defaults($_REQUEST, 'conversation_id', 0); + $conversation_id = $_REQUEST['conversation_id'] ?? 0; - $start = $page * $count; + $start = max(0, ($page - 1) * $count); $condition = ["`uid` = ? AND `gravity` IN (?, ?) AND `id` > ? AND `group_member`.`gid` = ?", api_user(), GRAVITY_PARENT, GRAVITY_COMMENT, $since_id, $_REQUEST['list_id']]; @@ -3304,12 +3365,10 @@ function api_statuses_f($qtype) } // pagination - $count = defaults($_GET, 'count', 20); - $page = defaults($_GET, 'page', 1); - if ($page < 1) { - $page = 1; - } - $start = ($page - 1) * $count; + $count = $_GET['count'] ?? 20; + $page = $_GET['page'] ?? 1; + + $start = max(0, ($page - 1) * $count); $user_info = api_get_user($a); @@ -3556,7 +3615,7 @@ function api_ff_ids($type) api_get_user($a); - $stringify_ids = defaults($_REQUEST, 'stringify_ids', false); + $stringify_ids = $_REQUEST['stringify_ids'] ?? false; $r = q( "SELECT `pcontact`.`id` FROM `contact` @@ -3731,9 +3790,9 @@ function api_direct_messages_destroy($type) // params $user_info = api_get_user($a); //required - $id = defaults($_REQUEST, 'id', 0); + $id = $_REQUEST['id'] ?? 0; // optional - $parenturi = defaults($_REQUEST, 'friendica_parenturi', ""); + $parenturi = $_REQUEST['friendica_parenturi'] ?? ''; $verbose = (!empty($_GET['friendica_verbose']) ? strtolower($_GET['friendica_verbose']) : "false"); /// @todo optional parameter 'include_entities' from Twitter API not yet implemented @@ -3814,7 +3873,7 @@ function api_friendships_destroy($type) throw new ForbiddenException(); } - $contact_id = defaults($_REQUEST, 'user_id'); + $contact_id = $_REQUEST['user_id'] ?? 0; if (empty($contact_id)) { Logger::notice(API_LOG_PREFIX . 'No user_id specified', ['module' => 'api', 'action' => 'friendships_destroy']); @@ -3895,17 +3954,14 @@ function api_direct_messages_box($type, $box, $verbose) throw new ForbiddenException(); } // params - $count = defaults($_GET, 'count', 20); - $page = defaults($_REQUEST, 'page', 1) - 1; - if ($page < 0) { - $page = 0; - } + $count = $_GET['count'] ?? 20; + $page = $_REQUEST['page'] ?? 1; - $since_id = defaults($_REQUEST, 'since_id', 0); - $max_id = defaults($_REQUEST, 'max_id', 0); + $since_id = $_REQUEST['since_id'] ?? 0; + $max_id = $_REQUEST['max_id'] ?? 0; - $user_id = defaults($_REQUEST, 'user_id', ''); - $screen_name = defaults($_REQUEST, 'screen_name', ''); + $user_id = $_REQUEST['user_id'] ?? ''; + $screen_name = $_REQUEST['screen_name'] ?? ''; // caller user info unset($_REQUEST["user_id"]); @@ -3921,7 +3977,7 @@ function api_direct_messages_box($type, $box, $verbose) $profile_url = $user_info["url"]; // pagination - $start = $page * $count; + $start = max(0, ($page - 1) * $count); $sql_extra = ""; @@ -3929,7 +3985,7 @@ function api_direct_messages_box($type, $box, $verbose) if ($box=="sentbox") { $sql_extra = "`mail`.`from-url`='" . DBA::escape($profile_url) . "'"; } elseif ($box == "conversation") { - $sql_extra = "`mail`.`parent-uri`='" . DBA::escape(defaults($_GET, 'uri', '')) . "'"; + $sql_extra = "`mail`.`parent-uri`='" . DBA::escape($_GET['uri'] ?? '') . "'"; } elseif ($box == "all") { $sql_extra = "true"; } elseif ($box == "inbox") { @@ -4109,7 +4165,7 @@ function api_fr_photoalbum_delete($type) throw new ForbiddenException(); } // input params - $album = defaults($_REQUEST, 'album', ""); + $album = $_REQUEST['album'] ?? ''; // we do not allow calls without album string if ($album == "") { @@ -4164,8 +4220,8 @@ function api_fr_photoalbum_update($type) throw new ForbiddenException(); } // input params - $album = defaults($_REQUEST, 'album', ""); - $album_new = defaults($_REQUEST, 'album_new', ""); + $album = $_REQUEST['album'] ?? ''; + $album_new = $_REQUEST['album_new'] ?? ''; // we do not allow calls without album string if ($album == "") { @@ -4256,14 +4312,14 @@ function api_fr_photo_create_update($type) throw new ForbiddenException(); } // input params - $photo_id = defaults($_REQUEST, 'photo_id', null); - $desc = defaults($_REQUEST, 'desc', (array_key_exists('desc', $_REQUEST) ? "" : null)) ; // extra check necessary to distinguish between 'not provided' and 'empty string' - $album = defaults($_REQUEST, 'album', null); - $album_new = defaults($_REQUEST, 'album_new', null); - $allow_cid = defaults($_REQUEST, 'allow_cid', (array_key_exists('allow_cid', $_REQUEST) ? " " : null)); - $deny_cid = defaults($_REQUEST, 'deny_cid' , (array_key_exists('deny_cid' , $_REQUEST) ? " " : null)); - $allow_gid = defaults($_REQUEST, 'allow_gid', (array_key_exists('allow_gid', $_REQUEST) ? " " : null)); - $deny_gid = defaults($_REQUEST, 'deny_gid' , (array_key_exists('deny_gid' , $_REQUEST) ? " " : null)); + $photo_id = $_REQUEST['photo_id'] ?? null; + $desc = $_REQUEST['desc'] ?? null; + $album = $_REQUEST['album'] ?? null; + $album_new = $_REQUEST['album_new'] ?? null; + $allow_cid = $_REQUEST['allow_cid'] ?? null; + $deny_cid = $_REQUEST['deny_cid' ] ?? null; + $allow_gid = $_REQUEST['allow_gid'] ?? null; + $deny_gid = $_REQUEST['deny_gid' ] ?? null; $visibility = !empty($_REQUEST['visibility']) && $_REQUEST['visibility'] !== "false"; // do several checks on input parameters @@ -4394,7 +4450,7 @@ function api_fr_photo_delete($type) } // input params - $photo_id = defaults($_REQUEST, 'photo_id', null); + $photo_id = $_REQUEST['photo_id'] ?? null; // do several checks on input parameters // we do not allow calls without photo id @@ -4481,7 +4537,7 @@ function api_account_update_profile_image($type) throw new ForbiddenException(); } // input params - $profile_id = defaults($_REQUEST, 'profile_id', 0); + $profile_id = $_REQUEST['profile_id'] ?? 0; // error if image data is missing if (empty($_FILES['image'])) { @@ -4613,9 +4669,10 @@ api_register_func('api/account/update_profile', 'api_account_update_profile', tr */ function check_acl_input($acl_string) { - if ($acl_string == null || $acl_string == " ") { + if (empty($acl_string)) { return false; } + $contact_not_found = false; // splitinto array of cid's @@ -4633,7 +4690,6 @@ function check_acl_input($acl_string) } /** - * * @param string $mediatype * @param array $media * @param string $type @@ -4652,6 +4708,7 @@ function check_acl_input($acl_string) * @throws ImagickException * @throws InternalServerErrorException * @throws NotFoundException + * @throws UnauthorizedException */ function save_media_to_database($mediatype, $media, $type, $album, $allow_cid, $deny_cid, $allow_gid, $deny_gid, $desc, $profile = 0, $visibility = false, $photo_id = null) { @@ -4685,7 +4742,7 @@ function save_media_to_database($mediatype, $media, $type, $album, $allow_cid, $ } if ($filetype == "") { - $filetype=Image::guessType($filename); + $filetype = Images::guessType($filename); } $imagedata = @getimagesize($src); if ($imagedata) { @@ -4732,19 +4789,19 @@ function save_media_to_database($mediatype, $media, $type, $album, $allow_cid, $ $height = $Image->getHeight(); // create a new resource-id if not already provided - $hash = ($photo_id == null) ? Photo::newResource() : $photo_id; + $resource_id = ($photo_id == null) ? Photo::newResource() : $photo_id; if ($mediatype == "photo") { // upload normal image (scales 0, 1, 2) Logger::log("photo upload: starting new photo upload", Logger::DEBUG); - $r = Photo::store($Image, local_user(), $visitor, $hash, $filename, $album, 0, 0, $allow_cid, $allow_gid, $deny_cid, $deny_gid, $desc); + $r = Photo::store($Image, local_user(), $visitor, $resource_id, $filename, $album, 0, 0, $allow_cid, $allow_gid, $deny_cid, $deny_gid, $desc); if (!$r) { Logger::log("photo upload: image upload with scale 0 (original size) failed"); } if ($width > 640 || $height > 640) { $Image->scaleDown(640); - $r = Photo::store($Image, local_user(), $visitor, $hash, $filename, $album, 1, 0, $allow_cid, $allow_gid, $deny_cid, $deny_gid, $desc); + $r = Photo::store($Image, local_user(), $visitor, $resource_id, $filename, $album, 1, 0, $allow_cid, $allow_gid, $deny_cid, $deny_gid, $desc); if (!$r) { Logger::log("photo upload: image upload with scale 1 (640x640) failed"); } @@ -4752,7 +4809,7 @@ function save_media_to_database($mediatype, $media, $type, $album, $allow_cid, $ if ($width > 320 || $height > 320) { $Image->scaleDown(320); - $r = Photo::store($Image, local_user(), $visitor, $hash, $filename, $album, 2, 0, $allow_cid, $allow_gid, $deny_cid, $deny_gid, $desc); + $r = Photo::store($Image, local_user(), $visitor, $resource_id, $filename, $album, 2, 0, $allow_cid, $allow_gid, $deny_cid, $deny_gid, $desc); if (!$r) { Logger::log("photo upload: image upload with scale 2 (320x320) failed"); } @@ -4764,7 +4821,7 @@ function save_media_to_database($mediatype, $media, $type, $album, $allow_cid, $ if ($width > 300 || $height > 300) { $Image->scaleDown(300); - $r = Photo::store($Image, local_user(), $visitor, $hash, $filename, $album, 4, $profile, $allow_cid, $allow_gid, $deny_cid, $deny_gid, $desc); + $r = Photo::store($Image, local_user(), $visitor, $resource_id, $filename, $album, 4, $profile, $allow_cid, $allow_gid, $deny_cid, $deny_gid, $desc); if (!$r) { Logger::log("photo upload: profile image upload with scale 4 (300x300) failed"); } @@ -4772,7 +4829,7 @@ function save_media_to_database($mediatype, $media, $type, $album, $allow_cid, $ if ($width > 80 || $height > 80) { $Image->scaleDown(80); - $r = Photo::store($Image, local_user(), $visitor, $hash, $filename, $album, 5, $profile, $allow_cid, $allow_gid, $deny_cid, $deny_gid, $desc); + $r = Photo::store($Image, local_user(), $visitor, $resource_id, $filename, $album, 5, $profile, $allow_cid, $allow_gid, $deny_cid, $deny_gid, $desc); if (!$r) { Logger::log("photo upload: profile image upload with scale 5 (80x80) failed"); } @@ -4780,7 +4837,7 @@ function save_media_to_database($mediatype, $media, $type, $album, $allow_cid, $ if ($width > 48 || $height > 48) { $Image->scaleDown(48); - $r = Photo::store($Image, local_user(), $visitor, $hash, $filename, $album, 6, $profile, $allow_cid, $allow_gid, $deny_cid, $deny_gid, $desc); + $r = Photo::store($Image, local_user(), $visitor, $resource_id, $filename, $album, 6, $profile, $allow_cid, $allow_gid, $deny_cid, $deny_gid, $desc); if (!$r) { Logger::log("photo upload: profile image upload with scale 6 (48x48) failed"); } @@ -4792,10 +4849,10 @@ function save_media_to_database($mediatype, $media, $type, $album, $allow_cid, $ if (isset($r) && $r) { // create entry in 'item'-table on new uploads to enable users to comment/like/dislike the photo if ($photo_id == null && $mediatype == "photo") { - post_photo_item($hash, $allow_cid, $deny_cid, $allow_gid, $deny_gid, $filetype, $visibility); + post_photo_item($resource_id, $allow_cid, $deny_cid, $allow_gid, $deny_gid, $filetype, $visibility); } // on success return image data in json/xml format (like /api/friendica/photo does when no scale is given) - return prepare_photo_data($type, false, $hash); + return prepare_photo_data($type, false, $resource_id); } else { throw new InternalServerErrorException("image upload failed"); } @@ -4986,8 +5043,8 @@ function prepare_photo_data($type, $scale, $photo_id) */ function api_friendica_remoteauth() { - $url = defaults($_GET, 'url', ''); - $c_url = defaults($_GET, 'c_url', ''); + $url = $_GET['url'] ?? ''; + $c_url = $_GET['c_url'] ?? ''; if ($url === '' || $c_url === '') { throw new BadRequestException("Wrong parameters."); @@ -5005,7 +5062,7 @@ function api_friendica_remoteauth() $cid = $contact['id']; - $dfrn_id = defaults($contact, 'issued-id', $contact['dfrn-id']); + $dfrn_id = $contact['issued-id'] ?? $contact['dfrn-id']; if ($contact['duplex'] && $contact['issued-id']) { $orig_id = $contact['issued-id']; @@ -5057,7 +5114,7 @@ function api_get_announce($item) } $fields = ['author-id', 'author-name', 'author-link', 'author-avatar']; - $activity = Item::activityToIndex(ACTIVITY2_ANNOUNCE); + $activity = Item::activityToIndex(Activity::ANNOUNCE); $condition = ['parent-uri' => $item['uri'], 'gravity' => GRAVITY_ACTIVITY, 'uid' => [0, $item['uid']], 'activity' => $activity]; $announce = Item::selectFirstForUser($item['uid'], $fields, $condition, ['order' => ['received' => true]]); if (!DBA::isResult($announce)) { @@ -5340,7 +5397,7 @@ function api_in_reply_to($item) */ function api_clean_plain_items($text) { - $include_entities = strtolower(defaults($_REQUEST, 'include_entities', "false")); + $include_entities = strtolower($_REQUEST['include_entities'] ?? 'false'); $text = BBCode::cleanPictureLinks($text); $URLSearchString = "^\[\]"; @@ -5478,7 +5535,7 @@ function api_friendica_group_show($type) // params $user_info = api_get_user($a); - $gid = defaults($_REQUEST, 'gid', 0); + $gid = $_REQUEST['gid'] ?? 0; $uid = $user_info['uid']; // get data of the specified group id or all groups if not specified @@ -5548,8 +5605,8 @@ function api_friendica_group_delete($type) // params $user_info = api_get_user($a); - $gid = defaults($_REQUEST, 'gid', 0); - $name = defaults($_REQUEST, 'name', ""); + $gid = $_REQUEST['gid'] ?? 0; + $name = $_REQUEST['name'] ?? ''; $uid = $user_info['uid']; // error if no gid specified @@ -5615,7 +5672,7 @@ function api_lists_destroy($type) // params $user_info = api_get_user($a); - $gid = defaults($_REQUEST, 'list_id', 0); + $gid = $_REQUEST['list_id'] ?? 0; $uid = $user_info['uid']; // error if no gid specified @@ -5737,7 +5794,7 @@ function api_friendica_group_create($type) // params $user_info = api_get_user($a); - $name = defaults($_REQUEST, 'name', ""); + $name = $_REQUEST['name'] ?? ''; $uid = $user_info['uid']; $json = json_decode($_POST['json'], true); $users = $json['user']; @@ -5771,7 +5828,7 @@ function api_lists_create($type) // params $user_info = api_get_user($a); - $name = defaults($_REQUEST, 'name', ""); + $name = $_REQUEST['name'] ?? ''; $uid = $user_info['uid']; $success = group_create($name, $uid); @@ -5811,8 +5868,8 @@ function api_friendica_group_update($type) // params $user_info = api_get_user($a); $uid = $user_info['uid']; - $gid = defaults($_REQUEST, 'gid', 0); - $name = defaults($_REQUEST, 'name', ""); + $gid = $_REQUEST['gid'] ?? 0; + $name = $_REQUEST['name'] ?? ''; $json = json_decode($_POST['json'], true); $users = $json['user']; @@ -5889,8 +5946,8 @@ function api_lists_update($type) // params $user_info = api_get_user($a); - $gid = defaults($_REQUEST, 'list_id', 0); - $name = defaults($_REQUEST, 'name', ""); + $gid = $_REQUEST['list_id'] ?? 0; + $name = $_REQUEST['name'] ?? ''; $uid = $user_info['uid']; // error if no gid specified @@ -5939,7 +5996,7 @@ function api_friendica_activity($type) $verb = strtolower($a->argv[3]); $verb = preg_replace("|\..*$|", "", $verb); - $id = defaults($_REQUEST, 'id', 0); + $id = $_REQUEST['id'] ?? 0; $res = Item::performLike($id, $verb); @@ -5986,7 +6043,8 @@ function api_friendica_notification($type) if ($a->argc!==3) { throw new BadRequestException("Invalid argument count"); } - $nm = new NotificationsManager(); + /** @var Notify $nm */ + $nm = BaseObject::getClass(Notify::class); $notes = $nm->getAll([], ['seen' => 'ASC', 'date' => 'DESC'], 50); @@ -6030,7 +6088,8 @@ function api_friendica_notification_seen($type) $id = (!empty($_REQUEST['id']) ? intval($_REQUEST['id']) : 0); - $nm = new NotificationsManager(); + /** @var Notify $nm */ + $nm = BaseObject::getClass(Notify::class); $note = $nm->getByID($id); if (is_null($note)) { throw new BadRequestException("Invalid argument"); @@ -6076,7 +6135,7 @@ function api_friendica_direct_messages_setseen($type) // params $user_info = api_get_user($a); $uid = $user_info['uid']; - $id = defaults($_REQUEST, 'id', 0); + $id = $_REQUEST['id'] ?? 0; // return error if id is zero if ($id == "") { @@ -6130,7 +6189,7 @@ function api_friendica_direct_messages_search($type, $box = "") // params $user_info = api_get_user($a); - $searchstring = defaults($_REQUEST, 'searchstring', ""); + $searchstring = $_REQUEST['searchstring'] ?? ''; $uid = $user_info['uid']; // error if no searchstring specified @@ -6197,7 +6256,7 @@ function api_friendica_profile_show($type) } // input params - $profile_id = defaults($_REQUEST, 'profile_id', 0); + $profile_id = $_REQUEST['profile_id'] ?? 0; // retrieve general information about profiles for user $multi_profiles = Feature::isEnabled(api_user(), 'multi_profiles'); @@ -6205,47 +6264,36 @@ function api_friendica_profile_show($type) // get data of the specified profile id or all profiles of the user if not specified if ($profile_id != 0) { - $r = q( - "SELECT * FROM `profile` WHERE `uid` = %d AND `id` = %d", - intval(api_user()), - intval($profile_id) - ); - + $r = Profile::getById(api_user(), $profile_id); // error message if specified gid is not in database if (!DBA::isResult($r)) { throw new BadRequestException("profile_id not available"); } } else { - $r = q( - "SELECT * FROM `profile` WHERE `uid` = %d", - intval(api_user()) - ); + $r = Profile::getListByUser(api_user()); } // loop through all returned profiles and retrieve data and users $k = 0; $profiles = []; - foreach ($r as $rr) { - $profile = api_format_items_profiles($rr); - - // select all users from contact table, loop and prepare standard return for user data - $users = []; - $nurls = q( - "SELECT `id`, `nurl` FROM `contact` WHERE `uid`= %d AND `profile-id` = %d", - intval(api_user()), - intval($rr['id']) - ); - - foreach ($nurls as $nurl) { - $user = api_get_user($a, $nurl['nurl']); - ($type == "xml") ? $users[$k++ . ":user"] = $user : $users[] = $user; - } - $profile['users'] = $users; + if (DBA::isResult($r)) { + foreach ($r as $rr) { + $profile = api_format_items_profiles($rr); + + // select all users from contact table, loop and prepare standard return for user data + $users = []; + $nurls = Contact::selectToArray(['id', 'nurl'], ['uid' => api_user(), 'profile-id' => $rr['id']]); + foreach ($nurls as $nurl) { + $user = api_get_user($a, $nurl['nurl']); + ($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; - } else { - $profiles[] = $profile; + // add prepared profile data to array for final return + if ($type == "xml") { + $profiles[$k++ . ":profile"] = $profile; + } else { + $profiles[] = $profile; + } } } @@ -6275,7 +6323,7 @@ function api_saved_searches_list($type) $terms = DBA::select('search', ['id', 'term'], ['uid' => local_user()]); $result = []; - while ($term = $terms->fetch()) { + while ($term = DBA::fetch($terms)) { $result[] = [ 'created_at' => api_date(time()), 'id' => intval($term['id']),