From: Michael Date: Sun, 21 Apr 2019 17:09:01 +0000 (+0000) Subject: Improved reshare check X-Git-Url: https://git.mxchange.org/?a=commitdiff_plain;h=d8ed5482aa2aad3cf72ac9cfa0f431e3dca1856f;p=friendica.git Improved reshare check --- diff --git a/include/conversation.php b/include/conversation.php index ac2c963036..85938ca0f0 100644 --- a/include/conversation.php +++ b/include/conversation.php @@ -800,20 +800,16 @@ function conversation_fetch_comments($thread_items) { $lineno = 0; $actor = []; $created = ''; - $knownauthor = false; while ($row = Item::fetch($thread_items)) { if (($row['verb'] == ACTIVITY2_ANNOUNCE) && !empty($row['contact-uid']) && ($row['created'] > $created) && ($row['thr-parent'] == $row['parent-uri'])) { $actor = ['link' => $row['author-link'], 'avatar' => $row['author-avatar'], 'name' => $row['author-name']]; $created = $row['created']; } - if ($row['gravity'] == GRAVITY_PARENT) { - $parentlines[] = $lineno; - // We could have several parents, so it has to be done this way. - if (!empty($row['contact-uid']) && in_array($row['network'], Protocol::NATIVE_SUPPORT)) { - $knownauthor = true; - } + if ((($row['gravity'] == GRAVITY_PARENT) && !$row['origin'] && !in_array($row['network'], [Protocol::DIASPORA])) && + (empty($row['contact-uid']) || !in_array($row['network'], Protocol::NATIVE_SUPPORT))) { + $parentlines[] = $lineno; } $comments[] = $row; @@ -822,13 +818,11 @@ function conversation_fetch_comments($thread_items) { DBA::close($thread_items); - if (!$knownauthor && !empty($actor)) { + if (!empty($actor)) { foreach ($parentlines as $line) { - if (!in_array($comments[$line]['network'], [Protocol::DIASPORA]) && !$comments[$line]['origin']) { - $comments[$line]['owner-link'] = $actor['link']; - $comments[$line]['owner-avatar'] = $actor['avatar']; - $comments[$line]['owner-name'] = $actor['name']; - } + $comments[$line]['owner-link'] = $actor['link']; + $comments[$line]['owner-avatar'] = $actor['avatar']; + $comments[$line]['owner-name'] = $actor['name']; } } return $comments;