]> git.mxchange.org Git - friendica.git/blobdiff - include/conversation.php
Missing Include
[friendica.git] / include / conversation.php
index 41f10959b2c9524974046f907b6e969c50f501c3..cd635521e51516545e43e937c4e294a252a4b3e6 100644 (file)
@@ -6,20 +6,28 @@
 use Friendica\App;
 use Friendica\Content\ContactSelector;
 use Friendica\Content\Feature;
+use Friendica\Content\Pager;
 use Friendica\Content\Text\BBCode;
 use Friendica\Core\Addon;
 use Friendica\Core\Config;
 use Friendica\Core\L10n;
+use Friendica\Core\Logger;
 use Friendica\Core\PConfig;
+use Friendica\Core\Protocol;
+use Friendica\Core\Renderer;
 use Friendica\Core\System;
-use Friendica\Database\DBM;
+use Friendica\Database\DBA;
 use Friendica\Model\Contact;
+use Friendica\Model\Item;
 use Friendica\Model\Profile;
+use Friendica\Model\Term;
 use Friendica\Object\Post;
 use Friendica\Object\Thread;
 use Friendica\Util\DateTimeFormat;
+use Friendica\Util\Proxy as ProxyUtils;
 use Friendica\Util\Temporal;
 use Friendica\Util\XML;
+use Friendica\Util\Crypto;
 
 function item_extract_images($body) {
 
@@ -109,14 +117,21 @@ function item_redir_and_replace_images($body, $images, $cid) {
 /**
  * Render actions localized
  */
-function localize_item(&$item) {
-
+function localize_item(&$item)
+{
        $extracted = item_extract_images($item['body']);
        if ($extracted['images']) {
                $item['body'] = item_redir_and_replace_images($extracted['body'], $extracted['images'], $item['contact-id']);
        }
 
-       /// @Separted ???
+       /*
+       heluecht 2018-06-19: from my point of view this whole code part is useless.
+       It just renders the body message of technical posts (Like, dislike, ...).
+       But: The body isn't visible at all. So we do this stuff just because we can.
+       Even if these messages were visible, this would only mean that something went wrong.
+       During the further steps of the database restructuring I would like to address this issue.
+       */
+
        $xmlhead = "<" . "?xml version='1.0' encoding='UTF-8' ?" . ">";
        if (activity_match($item['verb'], ACTIVITY_LIKE)
                || activity_match($item['verb'], ACTIVITY_DISLIKE)
@@ -124,15 +139,11 @@ function localize_item(&$item) {
                || activity_match($item['verb'], ACTIVITY_ATTENDNO)
                || activity_match($item['verb'], ACTIVITY_ATTENDMAYBE)) {
 
-               /// @TODO may hurt performance
-               $r = q("SELECT * FROM `item`, `contact`
-                       WHERE `item`.`contact-id`=`contact`.`id`
-                       AND `item`.`uri`='%s'",
-                       dbesc($item['parent-uri']));
-               if (!DBM::is_result($r)) {
+               $fields = ['author-link', 'author-name', 'verb', 'object-type', 'resource-id', 'body', 'plink'];
+               $obj = Item::selectFirst($fields, ['uri' => $item['parent-uri']]);
+               if (!DBA::isResult($obj)) {
                        return;
                }
-               $obj = $r[0];
 
                $author  = '[url=' . $item['author-link'] . ']' . $item['author-name'] . '[/url]';
                $objauthor =  '[url=' . $obj['author-link'] . ']' . $obj['author-name'] . '[/url]';
@@ -162,22 +173,19 @@ function localize_item(&$item) {
 
                if (activity_match($item['verb'], ACTIVITY_LIKE)) {
                        $bodyverb = L10n::t('%1$s likes %2$s\'s %3$s');
-               }
-               elseif (activity_match($item['verb'], ACTIVITY_DISLIKE)) {
+               } elseif (activity_match($item['verb'], ACTIVITY_DISLIKE)) {
                        $bodyverb = L10n::t('%1$s doesn\'t like %2$s\'s %3$s');
-               }
-               elseif (activity_match($item['verb'], ACTIVITY_ATTEND)) {
+               } elseif (activity_match($item['verb'], ACTIVITY_ATTEND)) {
                        $bodyverb = L10n::t('%1$s attends %2$s\'s %3$s');
-               }
-               elseif (activity_match($item['verb'], ACTIVITY_ATTENDNO)) {
+               } elseif (activity_match($item['verb'], ACTIVITY_ATTENDNO)) {
                        $bodyverb = L10n::t('%1$s doesn\'t attend %2$s\'s %3$s');
-               }
-               elseif (activity_match($item['verb'], ACTIVITY_ATTENDMAYBE)) {
+               } elseif (activity_match($item['verb'], ACTIVITY_ATTENDMAYBE)) {
                        $bodyverb = L10n::t('%1$s attends maybe %2$s\'s %3$s');
                }
-               $item['body'] = sprintf($bodyverb, $author, $objauthor, $plink);
 
+               $item['body'] = sprintf($bodyverb, $author, $objauthor, $plink);
        }
+
        if (activity_match($item['verb'], ACTIVITY_FRIEND)) {
 
                if ($item['object-type']=="" || $item['object-type']!== ACTIVITY_OBJ_PERSON) return;
@@ -188,10 +196,11 @@ function localize_item(&$item) {
                $xmlhead="<"."?xml version='1.0' encoding='UTF-8' ?".">";
 
                $obj = XML::parseString($xmlhead.$item['object']);
-               $links = XML::parseString($xmlhead."<links>".unxmlify($obj->link)."</links>");
+               $links = XML::parseString($xmlhead."<links>".XML::unescape($obj->link)."</links>");
 
                $Bname = $obj->title;
-               $Blink = ""; $Bphoto = "";
+               $Blink = "";
+               $Bphoto = "";
                foreach ($links->link as $l) {
                        $atts = $l->attributes();
                        switch ($atts['rel']) {
@@ -200,10 +209,10 @@ function localize_item(&$item) {
                        }
                }
 
-               $A = '[url=' . Profile::zrl($Alink) . ']' . $Aname . '[/url]';
-               $B = '[url=' . Profile::zrl($Blink) . ']' . $Bname . '[/url]';
+               $A = '[url=' . Contact::magicLink($Alink) . ']' . $Aname . '[/url]';
+               $B = '[url=' . Contact::magicLink($Blink) . ']' . $Bname . '[/url]';
                if ($Bphoto != "") {
-                       $Bphoto = '[url=' . Profile::zrl($Blink) . '][img]' . $Bphoto . '[/img][/url]';
+                       $Bphoto = '[url=' . Contact::magicLink($Blink) . '][img]' . $Bphoto . '[/img][/url]';
                }
 
                $item['body'] = L10n::t('%1$s is now friends with %2$s', $A, $B)."\n\n\n".$Bphoto;
@@ -224,12 +233,12 @@ function localize_item(&$item) {
                $xmlhead = "<" . "?xml version='1.0' encoding='UTF-8' ?" . ">";
 
                $obj = XML::parseString($xmlhead.$item['object']);
-               $links = XML::parseString($xmlhead."<links>".unxmlify($obj->link)."</links>");
 
                $Bname = $obj->title;
-               $Blink = "";
+               $Blink = $obj->id;
                $Bphoto = "";
-               foreach ($links->link as $l) {
+
+               foreach ($obj->link as $l) {
                        $atts = $l->attributes();
                        switch ($atts['rel']) {
                                case "alternate": $Blink = $atts['href'];
@@ -237,10 +246,10 @@ function localize_item(&$item) {
                        }
                }
 
-               $A = '[url=' . Profile::zrl($Alink) . ']' . $Aname . '[/url]';
-               $B = '[url=' . Profile::zrl($Blink) . ']' . $Bname . '[/url]';
+               $A = '[url=' . Contact::magicLink($Alink) . ']' . $Aname . '[/url]';
+               $B = '[url=' . Contact::magicLink($Blink) . ']' . $Bname . '[/url]';
                if ($Bphoto != "") {
-                       $Bphoto = '[url=' . Profile::zrl($Blink) . '][img=80x80]' . $Bphoto . '[/img][/url]';
+                       $Bphoto = '[url=' . Contact::magicLink($Blink) . '][img=80x80]' . $Bphoto . '[/img][/url]';
                }
 
                /*
@@ -251,7 +260,7 @@ function localize_item(&$item) {
 
                // now translate the verb
                $poked_t = trim(sprintf($txt, "", ""));
-               $txt = str_replace( $poked_t, L10n::t($verb), $txt);
+               $txt = str_replace($poked_t, L10n::t($verb), $txt);
 
                // then do the sprintf on the translation string
 
@@ -260,20 +269,20 @@ function localize_item(&$item) {
        }
 
        if (activity_match($item['verb'], ACTIVITY_TAG)) {
-               /// @TODO may hurt performance "joining" two tables + asterisk
-               $r = q("SELECT * FROM `item`, `contact`
-                       WHERE `item`.`contact-id`=`contact`.`id`
-                       AND `item`.`uri`='%s'",
-                       dbesc($item['parent-uri']));
-
-               if (!DBM::is_result($r)) {
+               $fields = ['author-id', 'author-link', 'author-name', 'author-network',
+                       'verb', 'object-type', 'resource-id', 'body', 'plink'];
+               $obj = Item::selectFirst($fields, ['uri' => $item['parent-uri']]);
+               if (!DBA::isResult($obj)) {
                        return;
                }
 
-               $obj = $r[0];
+               $author_arr = ['uid' => 0, 'id' => $item['author-id'],
+                       'network' => $item['author-network'], 'url' => $item['author-link']];
+               $author  = '[url=' . Contact::magicLinkByContact($author_arr) . ']' . $item['author-name'] . '[/url]';
 
-               $author  = '[url=' . Profile::zrl($item['author-link']) . ']' . $item['author-name'] . '[/url]';
-               $objauthor =  '[url=' . Profile::zrl($obj['author-link']) . ']' . $obj['author-name'] . '[/url]';
+               $author_arr = ['uid' => 0, 'id' => $obj['author-id'],
+                       'network' => $obj['author-network'], 'url' => $obj['author-link']];
+               $objauthor  = '[url=' . Contact::magicLinkByContact($author_arr) . ']' . $obj['author-name'] . '[/url]';
 
                switch ($obj['verb']) {
                        case ACTIVITY_POST:
@@ -301,11 +310,10 @@ function localize_item(&$item) {
                $parsedobj = XML::parseString($xmlhead.$item['object']);
 
                $tag = sprintf('#[url=%s]%s[/url]', $parsedobj->id, $parsedobj->content);
-               $item['body'] = L10n::t('%1$s tagged %2$s\'s %3$s with %4$s', $author, $objauthor, $plink, $tag );
-
+               $item['body'] = L10n::t('%1$s tagged %2$s\'s %3$s with %4$s', $author, $objauthor, $plink, $tag);
        }
-       if (activity_match($item['verb'], ACTIVITY_FAVORITE)) {
 
+       if (activity_match($item['verb'], ACTIVITY_FAVORITE)) {
                if ($item['object-type'] == "") {
                        return;
                }
@@ -317,17 +325,13 @@ function localize_item(&$item) {
 
                $obj = XML::parseString($xmlhead.$item['object']);
                if (strlen($obj->id)) {
-                       $r = q("SELECT * FROM `item` WHERE `uri` = '%s' AND `uid` = %d LIMIT 1",
-                                       dbesc($obj->id),
-                                       intval($item['uid'])
-                       );
-
-                       if (DBM::is_result($r) && $r[0]['plink']) {
-                               $target = $r[0];
+                       $fields = ['author-link', 'author-name', 'plink'];
+                       $target = Item::selectFirst($fields, ['uri' => $obj->id, 'uid' => $item['uid']]);
+                       if (DBA::isResult($target) && $target['plink']) {
                                $Bname = $target['author-name'];
                                $Blink = $target['author-link'];
-                               $A = '[url=' . Profile::zrl($Alink) . ']' . $Aname . '[/url]';
-                               $B = '[url=' . Profile::zrl($Blink) . ']' . $Bname . '[/url]';
+                               $A = '[url=' . Contact::magicLink($Alink) . ']' . $Aname . '[/url]';
+                               $B = '[url=' . Contact::magicLink($Blink) . ']' . $Bname . '[/url]';
                                $P = '[url=' . $target['plink'] . ']' . L10n::t('post/item') . '[/url]';
                                $item['body'] = L10n::t('%1$s marked %2$s\'s %3$s as favorite', $A, $B, $P)."\n";
                        }
@@ -337,7 +341,7 @@ function localize_item(&$item) {
        if (preg_match_all('/@\[url=(.*?)\]/is', $item['body'], $matches, PREG_SET_ORDER)) {
                foreach ($matches as $mtch) {
                        if (!strpos($mtch[1], 'zrl=')) {
-                               $item['body'] = str_replace($mtch[0], '@[url=' . Profile::zrl($mtch[1]) . ']', $item['body']);
+                               $item['body'] = str_replace($mtch[0], '@[url=' . Contact::magicLink($mtch[1]) . ']', $item['body']);
                        }
                }
        }
@@ -350,15 +354,12 @@ function localize_item(&$item) {
        }
 
        // add sparkle links to appropriate permalinks
+       $author = ['uid' => 0, 'id' => $item['author-id'],
+               'network' => $item['author-network'], 'url' => $item['author-link']];
 
-       $x = stristr($item['plink'],'/display/');
-       if ($x) {
-               $sparkle = false;
-               $y = best_link_url($item, $sparkle);
-
-               if (strstr($y, '/redir/')) {
-                       $item['plink'] = $y . '?f=&url=' . $item['plink'];
-               }
+       // Only create a redirection to a magic link when logged in
+       if (!empty($item['plink']) && (local_user() || remote_user())) {
+               $item['plink'] = Contact::magicLinkbyContact($author, $item['plink']);
        }
 }
 
@@ -394,94 +395,36 @@ function visible_activity($item) {
                }
        }
 
-       if (activity_match($item['verb'], ACTIVITY_FOLLOW) && $item['object-type'] === ACTIVITY_OBJ_NOTE) {
-               if (!($item['self'] && ($item['uid'] == local_user()))) {
-                       return false;
-               }
+       // @TODO below if() block can be rewritten to a single line: $isVisible = allConditionsHere;
+       if (activity_match($item['verb'], ACTIVITY_FOLLOW) && $item['object-type'] === ACTIVITY_OBJ_NOTE && empty($item['self']) && $item['uid'] == local_user()) {
+               return false;
        }
 
        return true;
 }
 
-/**
- * @brief SQL query for items
- */
-function item_query() {
-       return "SELECT " . item_fieldlists() . " FROM `item` " .
-               item_joins() . " WHERE " . item_condition();
-}
+function conv_get_blocklist()
+{
+       if (!local_user()) {
+               return [];
+       }
 
-/**
- * @brief List of all data fields that are needed for displaying items
- */
-function item_fieldlists() {
-
-/*
-These Fields are not added below (yet). They are here to for bug search.
-`item`.`type`,
-`item`.`extid`,
-`item`.`changed`,
-`item`.`moderated`,
-`item`.`target-type`,
-`item`.`target`,
-`item`.`resource-id`,
-`item`.`tag`,
-`item`.`inform`,
-`item`.`pubmail`,
-`item`.`visible`,
-`item`.`spam`,
-`item`.`bookmark`,
-`item`.`unseen`,
-`item`.`deleted`,
-`item`.`origin`,
-`item`.`forum_mode`,
-`item`.`mention`,
-`item`.`global`,
-`item`.`shadow`,
-*/
-
-       return "`item`.`author-id`, `item`.`author-link`, `item`.`author-name`, `item`.`author-avatar`,
-               `item`.`owner-id`, `item`.`owner-link`, `item`.`owner-name`, `item`.`owner-avatar`,
-               `item`.`contact-id`, `item`.`uid`, `item`.`id`, `item`.`parent`,
-               `item`.`uri`, `item`.`thr-parent`, `item`.`parent-uri`, `item`.`content-warning`,
-               `item`.`commented`, `item`.`created`, `item`.`edited`, `item`.`received`,
-               `item`.`verb`, `item`.`object-type`, `item`.`postopts`, `item`.`plink`,
-               `item`.`guid`, `item`.`wall`, `item`.`private`, `item`.`starred`,
-               `item`.`title`, `item`.`body`, `item`.`file`, `item`.`event-id`,
-               `item`.`location`, `item`.`coord`, `item`.`app`, `item`.`attach`,
-               `item`.`rendered-hash`, `item`.`rendered-html`, `item`.`object`,
-               `item`.`allow_cid`, `item`.`allow_gid`, `item`.`deny_cid`, `item`.`deny_gid`,
-               `item`.`id` AS `item_id`, `item`.`network` AS `item_network`,
-
-               `author`.`thumb` AS `author-thumb`, `owner`.`thumb` AS `owner-thumb`,
-
-               `contact`.`network`, `contact`.`url`, `contact`.`name`, `contact`.`writable`,
-               `contact`.`self`, `contact`.`id` AS `cid`, `contact`.`alias`,
-
-               `event`.`created` AS `event-created`, `event`.`edited` AS `event-edited`,
-               `event`.`start` AS `event-start`,`event`.`finish` AS `event-finish`,
-               `event`.`summary` AS `event-summary`,`event`.`desc` AS `event-desc`,
-               `event`.`location` AS `event-location`, `event`.`type` AS `event-type`,
-               `event`.`nofinish` AS `event-nofinish`,`event`.`adjust` AS `event-adjust`,
-               `event`.`ignore` AS `event-ignore`, `event`.`id` AS `event-id`";
-}
+       $str_blocked = PConfig::get(local_user(), 'system', 'blocked');
+       if (empty($str_blocked)) {
+               return [];
+       }
 
-/**
- * @brief SQL join for contacts that are needed for displaying items
- */
-function item_joins() {
-       return "STRAIGHT_JOIN `contact` ON `contact`.`id` = `item`.`contact-id` AND
-               (NOT `contact`.`blocked` OR `contact`.`pending`)
-               LEFT JOIN `contact` AS `author` ON `author`.`id`=`item`.`author-id`
-               LEFT JOIN `contact` AS `owner` ON `owner`.`id`=`item`.`owner-id`
-               LEFT JOIN `event` ON `event-id` = `event`.`id`";
-}
+       $blocklist = [];
 
-/**
- * @brief SQL condition for items that are needed for displaying items
- */
-function item_condition() {
-       return "`item`.`visible` AND NOT `item`.`deleted` AND NOT `item`.`moderated`";
+       foreach (explode(',', $str_blocked) as $entry) {
+               // The 4th parameter guarantees that there always will be a public contact entry
+               $cid = Contact::getIdForURL(trim($entry), 0, true, ['url' => trim($entry)]);
+               if (!empty($cid)) {
+                       $blocklist[] = $cid;
+               }
+       }
+
+       return $blocklist;
 }
 
 /**
@@ -494,31 +437,19 @@ function item_condition() {
  * that are based on unique features of the calling module.
  *
  */
-function conversation(App $a, $items, $mode, $update, $preview = false, $order = 'commented') {
-       require_once 'mod/proxy.php';
-
-       $ssl_state = ((local_user()) ? true : false);
+function conversation(App $a, array $items, Pager $pager, $mode, $update, $preview = false, $order = 'commented', $uid = 0)
+{
+       $ssl_state = (local_user() ? true : false);
 
        $profile_owner = 0;
        $live_update_div = '';
 
-       $arr_blocked = null;
-
-       if (local_user()) {
-               $str_blocked = PConfig::get(local_user(), 'system', 'blocked');
-               if ($str_blocked) {
-                       $arr_blocked = explode(',', $str_blocked);
-                       for ($x = 0; $x < count($arr_blocked); $x ++) {
-                               $arr_blocked[$x] = trim($arr_blocked[$x]);
-                       }
-               }
-
-       }
+       $blocklist = conv_get_blocklist();
 
        $previewing = (($preview) ? ' preview ' : '');
 
        if ($mode === 'network') {
-               $items = conversation_add_children($items, false, $order);
+               $items = conversation_add_children($items, false, $order, $uid);
                $profile_owner = local_user();
                if (!$update) {
                        /*
@@ -537,15 +468,15 @@ function conversation(App $a, $items, $mode, $update, $preview = false, $order =
                                . ((x($_GET, 'bmark'))  ? '&bmark='  . $_GET['bmark']  : '')
                                . ((x($_GET, 'liked'))  ? '&liked='  . $_GET['liked']  : '')
                                . ((x($_GET, 'conv'))   ? '&conv='   . $_GET['conv']   : '')
-                               . ((x($_GET, 'spam'))   ? '&spam='   . $_GET['spam']   : '')
                                . ((x($_GET, 'nets'))   ? '&nets='   . $_GET['nets']   : '')
                                . ((x($_GET, 'cmin'))   ? '&cmin='   . $_GET['cmin']   : '')
                                . ((x($_GET, 'cmax'))   ? '&cmax='   . $_GET['cmax']   : '')
                                . ((x($_GET, 'file'))   ? '&file='   . $_GET['file']   : '')
 
-                               . "'; var profile_page = " . $a->pager['page'] . "; </script>\r\n";
+                               . "'; var profile_page = " . $pager->getPage() . "; </script>\r\n";
                }
        } elseif ($mode === 'profile') {
+               $items = conversation_add_children($items, false, $order, $uid);
                $profile_owner = $a->profile['profile_uid'];
 
                if (!$update) {
@@ -561,30 +492,44 @@ function conversation(App $a, $items, $mode, $update, $preview = false, $order =
 
                                $live_update_div = '<div id="live-profile"></div>' . "\r\n"
                                        . "<script> var profile_uid = " . $a->profile['profile_uid']
-                                       . "; var netargs = '?f='; var profile_page = " . $a->pager['page'] . "; </script>\r\n";
+                                       . "; var netargs = '?f='; var profile_page = " . $pager->getPage() . "; </script>\r\n";
                        }
                }
        } elseif ($mode === 'notes') {
+               $items = conversation_add_children($items, false, $order, $uid);
                $profile_owner = local_user();
+
                if (!$update) {
                        $live_update_div = '<div id="live-notes"></div>' . "\r\n"
                                . "<script> var profile_uid = " . local_user()
-                               . "; var netargs = '/?f='; var profile_page = " . $a->pager['page'] . "; </script>\r\n";
+                               . "; var netargs = '/?f='; var profile_page = " . $pager->getPage() . "; </script>\r\n";
                }
        } elseif ($mode === 'display') {
+               $items = conversation_add_children($items, false, $order, $uid);
                $profile_owner = $a->profile['uid'];
+
                if (!$update) {
                        $live_update_div = '<div id="live-display"></div>' . "\r\n"
-                               . "<script> var profile_uid = " . $_SESSION['uid'] . ";"
+                               . "<script> var profile_uid = " . defaults($_SESSION, 'uid', 0) . ";"
                                . " var profile_page = 1; </script>";
                }
        } elseif ($mode === 'community') {
-               $items = conversation_add_children($items, true, $order);
+               $items = conversation_add_children($items, true, $order, $uid);
                $profile_owner = 0;
+
                if (!$update) {
                        $live_update_div = '<div id="live-community"></div>' . "\r\n"
                                . "<script> var profile_uid = -1; var netargs = '" . substr($a->cmd, 10)
-                               ."/?f='; var profile_page = " . $a->pager['page'] . "; </script>\r\n";
+                               ."/?f='; var profile_page = " . $pager->getPage() . "; </script>\r\n";
+               }
+       } elseif ($mode === 'contacts') {
+               $items = conversation_add_children($items, true, $order, $uid);
+               $profile_owner = 0;
+
+               if (!$update) {
+                       $live_update_div = '<div id="live-contacts"></div>' . "\r\n"
+                               . "<script> var profile_uid = -1; var netargs = '" . substr($a->cmd, 9)
+                               ."/?f='; var profile_page = " . $pager->getPage() . "; </script>\r\n";
                }
        } elseif ($mode === 'search') {
                $live_update_div = '<div id="live-search"></div>' . "\r\n";
@@ -593,7 +538,7 @@ function conversation(App $a, $items, $mode, $update, $preview = false, $order =
        $page_dropping = ((local_user() && local_user() == $profile_owner) ? true : false);
 
        if (!$update) {
-               $_SESSION['return_url'] = $a->query_string;
+               $_SESSION['return_path'] = $a->query_string;
        }
 
        $cb = ['items' => $items, 'mode' => $mode, 'update' => $update, 'preview' => $preview];
@@ -610,13 +555,13 @@ function conversation(App $a, $items, $mode, $update, $preview = false, $order =
        $threads = [];
        $threadsid = -1;
 
-       $page_template = get_markup_template("conversation.tpl");
+       $page_template = Renderer::getMarkupTemplate("conversation.tpl");
 
-       if ($items && count($items)) {
-               if ($mode === 'community') {
+       if (!empty($items)) {
+               if (in_array($mode, ['community', 'contacts'])) {
                        $writable = true;
                } else {
-                       $writable = ($items[0]['uid'] == 0) && in_array($items[0]['network'], [NETWORK_OSTATUS, NETWORK_DIASPORA, NETWORK_DFRN]);
+                       $writable = ($items[0]['uid'] == 0) && in_array($items[0]['network'], [Protocol::ACTIVITYPUB, Protocol::OSTATUS, Protocol::DIASPORA, Protocol::DFRN]);
                }
 
                if (!local_user()) {
@@ -638,20 +583,10 @@ function conversation(App $a, $items, $mode, $update, $preview = false, $order =
                                        continue;
                                }
 
-                               if ($arr_blocked) {
-                                       $blocked = false;
-                                       foreach ($arr_blocked as $b) {
-                                               if ($b && link_compare($item['author-link'], $b)) {
-                                                       $blocked = true;
-                                                       break;
-                                               }
-                                       }
-                                       if ($blocked) {
-                                               continue;
-                                       }
+                               if (in_array($item['author-id'], $blocklist)) {
+                                       continue;
                                }
 
-
                                $threadsid++;
 
                                $owner_url   = '';
@@ -659,45 +594,23 @@ function conversation(App $a, $items, $mode, $update, $preview = false, $order =
                                $sparkle     = '';
 
                                // prevent private email from leaking.
-                               if ($item['network'] === NETWORK_MAIL && local_user() != $item['uid']) {
+                               if ($item['network'] === Protocol::MAIL && local_user() != $item['uid']) {
                                        continue;
                                }
 
-                               $profile_name = (strlen($item['author-name']) ? $item['author-name'] : $item['name']);
-                               if ($item['author-link'] && !$item['author-name']) {
+                               $profile_name = $item['author-name'];
+                               if (!empty($item['author-link']) && empty($item['author-name'])) {
                                        $profile_name = $item['author-link'];
                                }
 
-                               $tags = \Friendica\Model\Term::populateTagsFromItem($item);
+                               $tags = Term::populateTagsFromItem($item);
 
-                               $sp = false;
-                               $profile_link = best_link_url($item, $sp);
-                               if ($profile_link === 'mailbox') {
-                                       $profile_link = '';
-                               }
+                               $author = ['uid' => 0, 'id' => $item['author-id'],
+                                       'network' => $item['author-network'], 'url' => $item['author-link']];
+                               $profile_link = Contact::magicLinkbyContact($author);
 
-                               if ($sp) {
+                               if (strpos($profile_link, 'redir/') === 0) {
                                        $sparkle = ' sparkle';
-                               } else {
-                                       $profile_link = Profile::zrl($profile_link);
-                               }
-
-                               if (!x($item, 'author-thumb') || ($item['author-thumb'] == "")) {
-                                       $author_contact = Contact::getDetailsByURL($item['author-link'], $profile_owner);
-                                       if ($author_contact["thumb"]) {
-                                               $item['author-thumb'] = $author_contact["thumb"];
-                                       } else {
-                                               $item['author-thumb'] = $item['author-avatar'];
-                                       }
-                               }
-
-                               if (!isset($item['owner-thumb']) || ($item['owner-thumb'] == "")) {
-                                       $owner_contact = Contact::getDetailsByURL($item['owner-link'], $profile_owner);
-                                       if ($owner_contact["thumb"]) {
-                                               $item['owner-thumb'] = $owner_contact["thumb"];
-                                       } else {
-                                               $item['owner-thumb'] = $item['owner-avatar'];
-                                       }
                                }
 
                                $locate = ['location' => $item['location'], 'coord' => $item['coord'], 'html' => ''];
@@ -744,23 +657,19 @@ function conversation(App $a, $items, $mode, $update, $preview = false, $order =
                                $location_e = $location;
                                $owner_name_e = $owner_name;
 
-                               if ($item['item_network'] == "") {
-                                       $item['item_network'] = $item['network'];
-                               }
-
                                $tmp_item = [
                                        'template' => $tpl,
-                                       'id' => (($preview) ? 'P0' : $item['item_id']),
-                                       'guid' => (($preview) ? 'Q0' : $item['guid']),
-                                       'network' => $item['item_network'],
-                                       'network_name' => ContactSelector::networkToName($item['item_network'], $profile_link),
-                                       'linktitle' => L10n::t('View %s\'s profile @ %s', $profile_name, ((strlen($item['author-link'])) ? $item['author-link'] : $item['url'])),
+                                       'id' => ($preview ? 'P0' : $item['id']),
+                                       'guid' => ($preview ? 'Q0' : $item['guid']),
+                                       'network' => $item['network'],
+                                       'network_name' => ContactSelector::networkToName($item['network'], $item['author-link']),
+                                       'linktitle' => L10n::t('View %s\'s profile @ %s', $profile_name, $item['author-link']),
                                        'profile_url' => $profile_link,
                                        'item_photo_menu' => item_photo_menu($item),
                                        'name' => $profile_name_e,
                                        'sparkle' => $sparkle,
                                        'lock' => $lock,
-                                       'thumb' => System::removedBaseUrl(proxy_url($item['author-thumb'], false, PROXY_SIZE_THUMB)),
+                                       'thumb' => System::removedBaseUrl(ProxyUtils::proxifyUrl($item['author-avatar'], false, ProxyUtils::SIZE_THUMB)),
                                        'title' => $title_e,
                                        'body' => $body_e,
                                        'tags' => $tags_e,
@@ -779,7 +688,7 @@ function conversation(App $a, $items, $mode, $update, $preview = false, $order =
                                        'indent' => '',
                                        'owner_name' => $owner_name_e,
                                        'owner_url' => $owner_url,
-                                       'owner_photo' => System::removedBaseUrl(proxy_url($item['owner-thumb'], false, PROXY_SIZE_THUMB)),
+                                       'owner_photo' => System::removedBaseUrl(ProxyUtils::proxifyUrl($item['owner-avatar'], false, ProxyUtils::SIZE_THUMB)),
                                        'plink' => get_plink($item),
                                        'edpost' => false,
                                        'isstarred' => $isstarred,
@@ -798,14 +707,14 @@ function conversation(App $a, $items, $mode, $update, $preview = false, $order =
                                $arr = ['item' => $item, 'output' => $tmp_item];
                                Addon::callHooks('display_item', $arr);
 
-                               $threads[$threadsid]['id'] = $item['item_id'];
-                               $threads[$threadsid]['network'] = $item['item_network'];
+                               $threads[$threadsid]['id'] = $item['id'];
+                               $threads[$threadsid]['network'] = $item['network'];
                                $threads[$threadsid]['items'] = [$arr['output']];
 
                        }
                } else {
                        // Normal View
-                       $page_template = get_markup_template("threaded_conversation.tpl");
+                       $page_template = Renderer::getMarkupTemplate("threaded_conversation.tpl");
 
                        $conv = new Thread($mode, $preview, $writable);
 
@@ -815,24 +724,15 @@ function conversation(App $a, $items, $mode, $update, $preview = false, $order =
                         * But for now, this array respects the old style, just in case
                         */
                        foreach ($items as $item) {
-                               if ($arr_blocked) {
-                                       $blocked = false;
-                                       foreach ($arr_blocked as $b) {
-                                               if ($b && link_compare($item['author-link'], $b)) {
-                                                       $blocked = true;
-                                                       break;
-                                               }
-                                       }
-                                       if ($blocked) {
-                                               continue;
-                                       }
+                               if (in_array($item['author-id'], $blocklist)) {
+                                       continue;
                                }
 
                                // Can we put this after the visibility check?
                                builtin_activity_puller($item, $conv_responses);
 
                                // Only add what is visible
-                               if ($item['network'] === NETWORK_MAIL && local_user() != $item['uid']) {
+                               if ($item['network'] === Protocol::MAIL && local_user() != $item['uid']) {
                                        continue;
                                }
 
@@ -840,6 +740,8 @@ function conversation(App $a, $items, $mode, $update, $preview = false, $order =
                                        continue;
                                }
 
+                               /// @todo Check if this call is needed or not
+                               $arr = ['item' => $item];
                                Addon::callHooks('display_item', $arr);
 
                                $item['pagedrop'] = $page_dropping;
@@ -852,13 +754,13 @@ function conversation(App $a, $items, $mode, $update, $preview = false, $order =
 
                        $threads = $conv->getTemplateData($conv_responses);
                        if (!$threads) {
-                               logger('[ERROR] conversation : Failed to get template data.', LOGGER_DEBUG);
+                               Logger::log('[ERROR] conversation : Failed to get template data.', Logger::DEBUG);
                                $threads = [];
                        }
                }
        }
 
-       $o = replace_macros($page_template, [
+       $o = Renderer::replaceMacros($page_template, [
                '$baseurl' => System::baseUrl($ssl_state),
                '$return_path' => $a->query_string,
                '$live_update' => $live_update_div,
@@ -882,51 +784,26 @@ function conversation(App $a, $items, $mode, $update, $preview = false, $order =
  *
  * @return array items with parents and comments
  */
-function conversation_add_children($parents, $block_authors, $order) {
+function conversation_add_children(array $parents, $block_authors, $order, $uid) {
        $max_comments = Config::get('system', 'max_comments', 100);
 
+       $params = ['order' => ['uid', 'commented' => true]];
+
        if ($max_comments > 0) {
-               $limit = ' LIMIT '.intval($max_comments + 1);
-       } else {
-               $limit = '';
+               $params['limit'] = $max_comments;
        }
 
        $items = [];
 
-       $block_sql = $block_authors ? "AND NOT `author`.`hidden` AND NOT `author`.`blocked`" : "";
-
        foreach ($parents AS $parent) {
-               $thread_items = dba::p(item_query()." AND `item`.`uid` = ?
-                       AND `item`.`parent-uri` = ? $block_sql
-                       ORDER BY `item`.`commented` DESC" . $limit,
-                       local_user(),
-                       $parent['uri']
-               );
-               $comments = dba::inArray($thread_items);
-
-               // Check if the original item is in the result.
-               // When commenting from the community page there can be incomplete threads
-               if (count($comments) > 0) {
-                       $parent_found = false;
-                       foreach ($comments as $comment) {
-                               if ($comment['uri'] == $comment['parent-uri']) {
-                                       $parent_found = true;
-                                       break;
-                               }
-                       }
-                       if (!$parent_found) {
-                               $comments = [];
-                       }
+               $condition = ["`item`.`parent-uri` = ? AND `item`.`uid` IN (0, ?) ",
+                       $parent['uri'], local_user()];
+               if ($block_authors) {
+                       $condition[0] .= "AND NOT `author`.`hidden`";
                }
+               $thread_items = Item::selectForUser(local_user(), [], $condition, $params);
 
-               if (count($comments) == 0) {
-                       $thread_items = dba::p(item_query()." AND `item`.`uid` = 0
-                               AND `item`.`parent-uri` = ?
-                               ORDER BY `item`.`commented` DESC LIMIT ".intval($max_comments + 1),
-                               $parent['uri']
-                       );
-                       $comments = dba::inArray($thread_items);
-               }
+               $comments = Item::inArray($thread_items);
 
                if (count($comments) != 0) {
                        $items = array_merge($items, $comments);
@@ -935,7 +812,7 @@ function conversation_add_children($parents, $block_authors, $order) {
 
        foreach ($items as $index => $item) {
                if ($item['uid'] == 0) {
-                       $items[$index]['writable'] = in_array($item['network'], [NETWORK_OSTATUS, NETWORK_DIASPORA, NETWORK_DFRN]);
+                       $items[$index]['writable'] = in_array($item['network'], [Protocol::ACTIVITYPUB, Protocol::OSTATUS, Protocol::DIASPORA, Protocol::DFRN]);
                }
        }
 
@@ -944,48 +821,6 @@ function conversation_add_children($parents, $block_authors, $order) {
        return $items;
 }
 
-function best_link_url($item, &$sparkle, $url = '') {
-
-       $best_url = '';
-       $sparkle  = false;
-
-       $clean_url = normalise_link($item['author-link']);
-
-       if (local_user()) {
-               $condition = [
-                       'network' => NETWORK_DFRN,
-                       'uid' => local_user(),
-                       'nurl' => normalise_link($clean_url),
-                       'pending' => false
-               ];
-               $contact = dba::selectFirst('contact', ['id'], $condition);
-               if (DBM::is_result($contact)) {
-                       $best_url = 'redir/' . $contact['id'];
-                       $sparkle = true;
-                       if ($url != '') {
-                               $hostname = get_app()->get_hostname();
-                               if (!strstr($url, $hostname)) {
-                                       $best_url .= "?url=".$url;
-                               } else {
-                                       $best_url = $url;
-                               }
-                       }
-               }
-       }
-       if (!$best_url) {
-               if ($url != '') {
-                       $best_url = $url;
-               } elseif (strlen($item['author-link'])) {
-                       $best_url = $item['author-link'];
-               } else {
-                       $best_url = $item['url'];
-               }
-       }
-
-       return $best_url;
-}
-
-
 function item_photo_menu($item) {
        $sub_link = '';
        $poke_link = '';
@@ -999,18 +834,17 @@ function item_photo_menu($item) {
                $sub_link = 'javascript:dosubthread(' . $item['id'] . '); return false;';
        }
 
-       $sparkle = false;
-       $profile_link = best_link_url($item, $sparkle);
-       if ($profile_link === 'mailbox') {
-               $profile_link = '';
-       }
+       $author = ['uid' => 0, 'id' => $item['author-id'],
+               'network' => $item['author-network'], 'url' => $item['author-link']];
+       $profile_link = Contact::magicLinkbyContact($author);
+       $sparkle = (strpos($profile_link, 'redir/') === 0);
 
        $cid = 0;
        $network = '';
        $rel = 0;
        $condition = ['uid' => local_user(), 'nurl' => normalise_link($item['author-link'])];
-       $contact = dba::selectFirst('contact', ['id', 'network', 'rel'], $condition);
-       if (DBM::is_result($contact)) {
+       $contact = DBA::selectFirst('contact', ['id', 'network', 'rel'], $condition);
+       if (DBA::isResult($contact)) {
                $cid = $contact['id'];
                $network = $contact['network'];
                $rel = $contact['rel'];
@@ -1020,16 +854,14 @@ function item_photo_menu($item) {
                $status_link = $profile_link . '?url=status';
                $photos_link = $profile_link . '?url=photos';
                $profile_link = $profile_link . '?url=profile';
-       } else {
-               $profile_link = Profile::zrl($profile_link);
        }
 
        if ($cid && !$item['self']) {
                $poke_link = 'poke/?f=&c=' . $cid;
-               $contact_url = 'contacts/' . $cid;
-               $posts_link = 'contacts/' . $cid . '/posts';
+               $contact_url = 'contact/' . $cid;
+               $posts_link = 'contact/' . $cid . '/posts';
 
-               if (in_array($network, [NETWORK_DFRN, NETWORK_DIASPORA])) {
+               if (in_array($network, [Protocol::DFRN, Protocol::DIASPORA])) {
                        $pm_url = 'message/new/' . $cid;
                }
        }
@@ -1045,12 +877,12 @@ function item_photo_menu($item) {
                        L10n::t('Send PM') => $pm_url
                ];
 
-               if ($network == NETWORK_DFRN) {
+               if ($network == Protocol::DFRN) {
                        $menu[L10n::t("Poke")] = $poke_link;
                }
 
-               if ((($cid == 0) || ($rel == CONTACT_IS_FOLLOWER)) &&
-                       in_array($item['network'], [NETWORK_DFRN, NETWORK_OSTATUS, NETWORK_DIASPORA])) {
+               if ((($cid == 0) || ($rel == Contact::FOLLOWER)) &&
+                       in_array($item['network'], [Protocol::ACTIVITYPUB, Protocol::DFRN, Protocol::OSTATUS, Protocol::DIASPORA])) {
                        $menu[L10n::t('Connect/Follow')] = 'follow?url=' . urlencode($item['author-link']);
                }
        } else {
@@ -1109,17 +941,16 @@ function builtin_activity_puller($item, &$conv_responses) {
                }
 
                if (activity_match($item['verb'], $verb) && ($item['id'] != $item['parent'])) {
-                       $url = $item['author-link'];
-                       if (local_user() && (local_user() == $item['uid']) && ($item['network'] === NETWORK_DFRN) && !$item['self'] && link_compare($item['author-link'], $item['url'])) {
-                               $url = 'redir/' . $item['contact-id'];
+                       $author = ['uid' => 0, 'id' => $item['author-id'],
+                               'network' => $item['author-network'], 'url' => $item['author-link']];
+                       $url = Contact::magicLinkbyContact($author);
+                       if (strpos($url, 'redir/') === 0) {
                                $sparkle = ' class="sparkle" ';
-                       } else {
-                               $url = Profile::zrl($url);
                        }
 
                        $url = '<a href="'. $url . '"'. $sparkle .'>' . htmlentities($item['author-name']) . '</a>';
 
-                       if (!$item['thr-parent']) {
+                       if (!x($item, 'thr-parent')) {
                                $item['thr-parent'] = $item['parent-uri'];
                        }
 
@@ -1199,7 +1030,7 @@ function format_like($cnt, array $arr, $type, $id) {
                }
                if ($total >= MAX_LIKERS) {
                        $str = implode(', ', $arr);
-                       $str .= L10n::t('and %d other people', $total - MAX_LIKERS );
+                       $str .= L10n::t('and %d other people', $total - MAX_LIKERS);
                }
 
                $likers = $str;
@@ -1233,7 +1064,7 @@ function format_like($cnt, array $arr, $type, $id) {
        }
 
        $phrase .= EOL ;
-       $o .= replace_macros(get_markup_template('voting_fakelink.tpl'), [
+       $o .= Renderer::replaceMacros(Renderer::getMarkupTemplate('voting_fakelink.tpl'), [
                '$phrase' => $phrase,
                '$type' => $type,
                '$id' => $id
@@ -1247,39 +1078,22 @@ function status_editor(App $a, $x, $notes_cid = 0, $popup = false)
 {
        $o = '';
 
-       $geotag = x($x, 'allow_location') ? replace_macros(get_markup_template('jot_geotag.tpl'), []) : '';
+       $geotag = x($x, 'allow_location') ? Renderer::replaceMacros(Renderer::getMarkupTemplate('jot_geotag.tpl'), []) : '';
 
-       $tpl = get_markup_template('jot-header.tpl');
-       $a->page['htmlhead'] .= replace_macros($tpl, [
+       $tpl = Renderer::getMarkupTemplate('jot-header.tpl');
+       $a->page['htmlhead'] .= Renderer::replaceMacros($tpl, [
                '$newpost'   => 'true',
                '$baseurl'   => System::baseUrl(true),
                '$geotag'    => $geotag,
                '$nickname'  => $x['nickname'],
                '$ispublic'  => L10n::t('Visible to <strong>everybody</strong>'),
-               '$linkurl'   => L10n::t('Please enter a link URL:'),
-               '$vidurl'    => L10n::t("Please enter a video link/URL:"),
-               '$audurl'    => L10n::t("Please enter an audio link/URL:"),
+               '$linkurl'   => L10n::t('Please enter a image/video/audio/webpage URL:'),
                '$term'      => L10n::t('Tag term:'),
                '$fileas'    => L10n::t('Save to Folder:'),
                '$whereareu' => L10n::t('Where are you right now?'),
                '$delitems'  => L10n::t("Delete item\x28s\x29?")
        ]);
 
-       $tpl = get_markup_template('jot-end.tpl');
-       $a->page['end'] .= replace_macros($tpl, [
-               '$newpost'   => 'true',
-               '$baseurl'   => System::baseUrl(true),
-               '$geotag'    => $geotag,
-               '$nickname'  => $x['nickname'],
-               '$ispublic'  => L10n::t('Visible to <strong>everybody</strong>'),
-               '$linkurl'   => L10n::t('Please enter a link URL:'),
-               '$vidurl'    => L10n::t("Please enter a video link/URL:"),
-               '$audurl'    => L10n::t("Please enter an audio link/URL:"),
-               '$term'      => L10n::t('Tag term:'),
-               '$fileas'    => L10n::t('Save to Folder:'),
-               '$whereareu' => L10n::t('Where are you right now?')
-       ]);
-
        $jotplugins = '';
        Addon::callHooks('jot_tool', $jotplugins);
 
@@ -1305,10 +1119,10 @@ function status_editor(App $a, $x, $notes_cid = 0, $popup = false)
                $public_post_link = '&public=1';
        }
 
-       // $tpl = replace_macros($tpl,array('$jotplugins' => $jotplugins));
-       $tpl = get_markup_template("jot.tpl");
+       // $tpl = Renderer::replaceMacros($tpl,array('$jotplugins' => $jotplugins));
+       $tpl = Renderer::getMarkupTemplate("jot.tpl");
 
-       $o .= replace_macros($tpl,[
+       $o .= Renderer::replaceMacros($tpl,[
                '$new_post' => L10n::t('New Post'),
                '$return_path'  => $query_str,
                '$action'       => 'item',
@@ -1317,12 +1131,14 @@ function status_editor(App $a, $x, $notes_cid = 0, $popup = false)
                '$shortupload'  => L10n::t('upload photo'),
                '$attach'       => L10n::t('Attach file'),
                '$shortattach'  => L10n::t('attach file'),
-               '$weblink'      => L10n::t('Insert web link'),
-               '$shortweblink' => L10n::t('web link'),
-               '$video'        => L10n::t('Insert video link'),
-               '$shortvideo'   => L10n::t('video link'),
-               '$audio'        => L10n::t('Insert audio link'),
-               '$shortaudio'   => L10n::t('audio link'),
+               '$edbold'       => L10n::t('Bold'),
+               '$editalic'     => L10n::t('Italic'),
+               '$eduline'      => L10n::t('Underline'),
+               '$edquote'      => L10n::t('Quote'),
+               '$edcode'       => L10n::t('Code'),
+               '$edimg'        => L10n::t('Image'),
+               '$edurl'        => L10n::t('Link'),
+               '$edattach'     => L10n::t('Link or Media'),
                '$setloc'       => L10n::t('Set your location'),
                '$shortsetloc'  => L10n::t('set location'),
                '$noloc'        => L10n::t('Clear browser location'),
@@ -1334,7 +1150,8 @@ function status_editor(App $a, $x, $notes_cid = 0, $popup = false)
                '$wait'         => L10n::t('Please wait'),
                '$permset'      => L10n::t('Permission settings'),
                '$shortpermset' => L10n::t('permissions'),
-               '$ptyp'         => $notes_cid ? 'note' : 'wall',
+               '$wall'         => $notes_cid ? 0 : 1,
+               '$posttype'     => $notes_cid ? Item::PT_PERSONAL_NOTE : Item::PT_ARTICLE,
                '$content'      => defaults($x, 'content', ''),
                '$post_id'      => defaults($x, 'post_id', ''),
                '$baseurl'      => System::baseUrl(true),
@@ -1350,7 +1167,7 @@ function status_editor(App $a, $x, $notes_cid = 0, $popup = false)
                '$notes_cid'    => $notes_cid,
                '$sourceapp'    => L10n::t($a->sourcename),
                '$cancel'       => L10n::t('Cancel'),
-               '$rand_num'     => random_digits(12),
+               '$rand_num'     => Crypto::randomDigits(12),
 
                // ACL permissions box
                '$acl'           => $x['acl'],
@@ -1520,10 +1337,16 @@ function conv_sort(array $item_list, $order)
                return $parents;
        }
 
+       $blocklist = conv_get_blocklist();
+
        $item_array = [];
 
        // Dedupes the item list on the uri to prevent infinite loops
        foreach ($item_list as $item) {
+               if (in_array($item['author-id'], $blocklist)) {
+                       continue;
+               }
+
                $item_array[$item['uri']] = $item;
        }
 
@@ -1607,23 +1430,21 @@ function sort_thr_commented(array $a, array $b)
        return strcmp($b['commented'], $a['commented']);
 }
 
-/// @TODO Add type-hint
-function render_location_dummy($item) {
-       if ($item['location'] != "") {
+function render_location_dummy(array $item) {
+       if (x($item, 'location') && !empty($item['location'])) {
                return $item['location'];
        }
 
-       if ($item['coord'] != "") {
+       if (x($item, 'coord') && !empty($item['coord'])) {
                return $item['coord'];
        }
 }
 
-/// @TODO Add type-hint
-function get_responses($conv_responses, $response_verbs, $ob, $item) {
+function get_responses(array $conv_responses, array $response_verbs, $ob, array $item) {
        $ret = [];
        foreach ($response_verbs as $v) {
                $ret[$v] = [];
-               $ret[$v]['count'] = defaults($conv_responses[$v], $item['uri'], '');
+               $ret[$v]['count'] = defaults($conv_responses[$v], $item['uri'], 0);
                $ret[$v]['list']  = defaults($conv_responses[$v], $item['uri'] . '-l', []);
                $ret[$v]['self']  = defaults($conv_responses[$v], $item['uri'] . '-self', '0');
                if (count($ret[$v]['list']) > MAX_LIKERS) {