From f569ccbbc2a615d2aedf85754bb39cfbb050066c Mon Sep 17 00:00:00 2001
From: Michael <heluecht@pirati.ca>
Date: Sat, 29 Oct 2022 22:55:39 +0000
Subject: [PATCH] Code is simplyfied

---
 src/Content/Item.php |  3 +--
 src/Model/Item.php   | 31 ++++++++++++++++---------------
 2 files changed, 17 insertions(+), 17 deletions(-)

diff --git a/src/Content/Item.php b/src/Content/Item.php
index f226867ec..3d6443fe3 100644
--- a/src/Content/Item.php
+++ b/src/Content/Item.php
@@ -748,8 +748,7 @@ class Item
 			return $body;
 		}
 
-		$link = $post['plink'] ?: $post['uri'];
-		$body .= "\n♲ " . $link;
+		$body .= "\n♲ " . ($post['plink'] ?: $post['uri']);
 
 		return $body;
 	}
diff --git a/src/Model/Item.php b/src/Model/Item.php
index 5f2f51834..f95b22e27 100644
--- a/src/Model/Item.php
+++ b/src/Model/Item.php
@@ -3662,38 +3662,39 @@ class Item
 			$shared_item = Post::selectFirst(['uri-id'], ['guid' => $shared['guid'], 'uid' => [0, $uid]]);
 			if (!empty($shared_item['uri-id'])) {
 				Logger::debug('Found post by guid', ['guid' => $shared['guid'], 'uid' => $uid]);
+				return $shared_item['uri-id'];
 			}
 		}
 
-		if (empty($shared_item['uri-id']) && !empty($shared['message_id'])) {
+		if (!empty($shared['message_id'])) {
 			$shared_item = Post::selectFirst(['uri-id'], ['uri' => $shared['message_id'], 'uid' => [0, $uid]]);
 			if (!empty($shared_item['uri-id'])) {
 				Logger::debug('Found post by message_id', ['message_id' => $shared['message_id'], 'uid' => $uid]);
+				return $shared_item['uri-id'];
 			}
 		}
 
-		if (empty($shared_item['uri-id']) && !empty($shared['link'])) {
+		if (!empty($shared['link'])) {
 			$shared_item = Post::selectFirst(['uri-id'], ['plink' => $shared['link'], 'uid' => [0, $uid]]);
 			if (!empty($shared_item['uri-id'])) {
 				Logger::debug('Found post by link', ['link' => $shared['link'], 'uid' => $uid]);
+				return $shared_item['uri-id'];
 			}
 		}
 
-		if (empty($shared_item['uri-id'])) {
-			$url = $shared['message_id'] ?: $shared['link'];
-			$id = self::fetchByLink($url);
-			if (!$id) {
-				Logger::notice('Post could not be fetched.', ['url' => $url, 'uid' => $uid]);
-				return 0;
-			}
+		$url = $shared['message_id'] ?: $shared['link'];
+		$id = self::fetchByLink($url);
+		if (!$id) {
+			Logger::notice('Post could not be fetched.', ['url' => $url, 'uid' => $uid]);
+			return 0;
+		}
 
+		$shared_item = Post::selectFirst(['uri-id'], ['id' => $id]);
+		if (!DBA::isResult($shared_item)) {
+			Logger::warning('Post does not exist.', ['id' => $id, 'url' => $url, 'uid' => $uid]);
+			return 0;
+		} else {
 			Logger::debug('Fetched shared post', ['id' => $id, 'url' => $url, 'uid' => $uid]);
-
-			$shared_item = Post::selectFirst(['uri-id'], ['id' => $id]);
-			if (!DBA::isResult($shared_item)) {
-				Logger::warning('Post does not exist.', ['id' => $id, 'url' => $url, 'uid' => $uid]);
-				return 0;
-			}
 		}
 
 		return $shared_item['uri-id'];