]> git.mxchange.org Git - friendica.git/blobdiff - include/conversation.php
Merge remote-tracking branch 'upstream/develop' into more-abstraction
[friendica.git] / include / conversation.php
index 5f56723e2e361e47b84b15e7dc10439fc3e043ba..97be7e9aa984436725c748be807553e14a06e574 100644 (file)
@@ -15,6 +15,7 @@ use Friendica\Core\System;
 use Friendica\Database\DBM;
 use Friendica\Model\Contact;
 use Friendica\Model\Profile;
+use Friendica\Model\Item;
 use Friendica\Object\Post;
 use Friendica\Object\Thread;
 use Friendica\Util\DateTimeFormat;
@@ -197,10 +198,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;
@@ -234,10 +235,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]';
                }
 
                /*
@@ -269,8 +270,8 @@ function localize_item(&$item) {
 
                $obj = $r[0];
 
-               $author  = '[url=' . Profile::zrl($item['author-link']) . ']' . $item['author-name'] . '[/url]';
-               $objauthor =  '[url=' . Profile::zrl($obj['author-link']) . ']' . $obj['author-name'] . '[/url]';
+               $author  = '[url=' . Contact::magicLinkById($item['author-id']) . ']' . $item['author-name'] . '[/url]';
+               $objauthor =  '[url=' . Contact::magicLinkById($obj['author-id']) . ']' . $obj['author-name'] . '[/url]';
 
                switch ($obj['verb']) {
                        case ACTIVITY_POST:
@@ -323,8 +324,8 @@ function localize_item(&$item) {
                                $target = $r[0];
                                $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";
                        }
@@ -334,7 +335,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']);
                        }
                }
        }
@@ -347,7 +348,7 @@ function localize_item(&$item) {
        }
 
        // add sparkle links to appropriate permalinks
-       $item['plink'] = Contact::magicLink($item['author-link'], $item['plink']);
+       $item['plink'] = Contact::magicLinkById($item['author-id'], $item['plink']);
 }
 
 /**
@@ -390,95 +391,6 @@ function visible_activity($item) {
        return true;
 }
 
-/**
- * @brief SQL query for items
- *
- * @param int $uid user id
- */
-function item_query($uid = 0) {
-       return "SELECT " . item_fieldlists() . " FROM `item` " .
-               item_joins($uid) . " WHERE " . item_condition();
-}
-
-/**
- * @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`.`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`.`origin`,
-               `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`";
-}
-
-/**
- * @brief SQL join for contacts that are needed for displaying items
- *
- * @param int $uid user id
- */
-function item_joins($uid = 0) {
-       return sprintf("STRAIGHT_JOIN `contact` ON `contact`.`id` = `item`.`contact-id`
-               AND NOT `contact`.`blocked`
-               AND ((NOT `contact`.`readonly` AND NOT `contact`.`pending` AND (`contact`.`rel` IN (%s, %s)))
-               OR `contact`.`self` OR (`item`.`id` != `item`.`parent`) OR `contact`.`uid` = 0)
-               INNER JOIN `contact` AS `author` ON `author`.`id`=`item`.`author-id` AND NOT `author`.`blocked`
-               INNER JOIN `contact` AS `owner` ON `owner`.`id`=`item`.`owner-id` AND NOT `owner`.`blocked`
-               LEFT JOIN `user-item` ON `user-item`.`iid` = `item`.`id` AND `user-item`.`uid` = %d
-               LEFT JOIN `event` ON `event-id` = `event`.`id`",
-               CONTACT_IS_SHARING, CONTACT_IS_FRIEND, intval($uid)
-       );
-}
-
-/**
- * @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` AND (`user-item`.`hidden` IS NULL OR NOT `user-item`.`hidden`) ";
-}
-
 /**
  * "Render" a conversation or list of items for HTML display.
  * There are two major forms of display:
@@ -532,7 +444,6 @@ 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']   : '')
@@ -658,6 +569,13 @@ function conversation(App $a, $items, $mode, $update, $preview = false, $order =
                                        continue;
                                }
 
+                               if ($item['network'] == NETWORK_FEED) {
+                                       $item['author-avatar'] = $item['contact-avatar'];
+                                       $item['author-name'] = $item['contact-name'];
+                                       $item['owner-avatar'] = $item['contact-avatar'];
+                                       $item['owner-name'] = $item['contact-name'];
+                               }
+
                                $profile_name = (strlen($item['author-name']) ? $item['author-name'] : $item['name']);
                                if ($item['author-link'] && !$item['author-name']) {
                                        $profile_name = $item['author-link'];
@@ -665,30 +583,12 @@ function conversation(App $a, $items, $mode, $update, $preview = false, $order =
 
                                $tags = \Friendica\Model\Term::populateTagsFromItem($item);
 
-                               $profile_link = Contact::magicLink($item['author-link']);
+                               $profile_link = Contact::magicLinkbyId($item['author-id']);
 
                                if (strpos($profile_link, 'redir/') === 0) {
                                        $sparkle = ' sparkle';
                                }
 
-                               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' => ''];
                                Addon::callHooks('render_location',$locate);
 
@@ -743,13 +643,13 @@ function conversation(App $a, $items, $mode, $update, $preview = false, $order =
                                        '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'])),
+                                       '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(proxy_url($item['author-avatar'], false, PROXY_SIZE_THUMB)),
                                        'title' => $title_e,
                                        'body' => $body_e,
                                        'tags' => $tags_e,
@@ -768,7 +668,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(proxy_url($item['owner-avatar'], false, PROXY_SIZE_THUMB)),
                                        'plink' => get_plink($item),
                                        'edpost' => false,
                                        'isstarred' => $isstarred,
@@ -874,21 +774,21 @@ function conversation(App $a, $items, $mode, $update, $preview = false, $order =
 function conversation_add_children($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(local_user())."AND `item`.`parent-uri` = ?
-                       AND `item`.`uid` IN (0, ?) $block_sql
-                       ORDER BY `item`.`uid` ASC, `item`.`commented` DESC" . $limit,
-                       $parent['uri'], local_user());
+               $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);
 
                $comments = dba::inArray($thread_items);
 
@@ -908,48 +808,6 @@ function conversation_add_children($parents, $block_authors, $order, $uid) {
        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 = '';
@@ -963,7 +821,7 @@ function item_photo_menu($item) {
                $sub_link = 'javascript:dosubthread(' . $item['id'] . '); return false;';
        }
 
-       $profile_link = Contact::magicLink($item['author-link']);
+       $profile_link = Contact::magicLinkById($item['author-id']);
        $sparkle = (strpos($profile_link, 'redir/') === 0);
 
        $cid = 0;
@@ -981,8 +839,6 @@ 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']) {
@@ -1070,7 +926,7 @@ function builtin_activity_puller($item, &$conv_responses) {
                }
 
                if (activity_match($item['verb'], $verb) && ($item['id'] != $item['parent'])) {
-                       $url = Contact::MagicLink($item['author-link']);
+                       $url = Contact::MagicLinkbyId($item['author-id']);
                        if (strpos($url, 'redir/') === 0) {
                                $sparkle = ' class="sparkle" ';
                        }