From f590fde7a437ae17d1858bb24191bed04491fb79 Mon Sep 17 00:00:00 2001
From: FineFindus <FineFindus@proton.me>
Date: Sat, 29 Jun 2024 11:33:37 +0200
Subject: [PATCH] feat(LinkCard): skip redirects to accounts

---
 .../LinkCardStatusDisplayItem.java            | 41 +++++++++++--------
 .../android/ui/utils/UiUtils.java             |  2 +-
 2 files changed, 25 insertions(+), 18 deletions(-)

diff --git a/mastodon/src/main/java/org/joinmastodon/android/ui/displayitems/LinkCardStatusDisplayItem.java b/mastodon/src/main/java/org/joinmastodon/android/ui/displayitems/LinkCardStatusDisplayItem.java
index 97afd612f..831e10dd4 100644
--- a/mastodon/src/main/java/org/joinmastodon/android/ui/displayitems/LinkCardStatusDisplayItem.java
+++ b/mastodon/src/main/java/org/joinmastodon/android/ui/displayitems/LinkCardStatusDisplayItem.java
@@ -20,6 +20,7 @@ import org.joinmastodon.android.ui.drawables.BlurhashCrossfadeDrawable;
 import org.joinmastodon.android.ui.text.HtmlParser;
 import org.joinmastodon.android.ui.utils.UiUtils;
 
+import java.util.Optional;
 import java.util.regex.Matcher;
 
 import me.grishka.appkit.imageloader.ImageLoaderViewHolder;
@@ -147,25 +148,31 @@ public class LinkCardStatusDisplayItem extends StatusDisplayItem{
 			// Mastodon.social sometimes adds an additional redirect page
 			// this is really disruptive on mobile, especially since it breaks the loopUp/openURL functionality
 			Uri parsedURL=Uri.parse(url);
-			if(parsedURL.getPath()!=null && parsedURL.getPath().startsWith("/redirect/statuses/")){
-				// find actually linked url in status content
-				Matcher matcher=HtmlParser.URL_PATTERN.matcher(item.status.content);
-				String foundURL;
-				while(matcher.find()){
-					foundURL=matcher.group(3);
-					if(TextUtils.isEmpty(matcher.group(4)))
-						foundURL="http://"+foundURL;
-					// SAFETY: Cannot be null, as otherwise the matcher wouldn't find it
-					// also, group is marked as non-null
-					assert foundURL!=null && parsedURL.getLastPathSegment()!=null;
-					if(foundURL.endsWith(parsedURL.getLastPathSegment())) {
-						// found correct URL
-						url=foundURL;
-						break;
-					}
-				}
+			if(parsedURL.getPath()!=null && parsedURL.getPath().startsWith("/redirect/")){
+				url=findRedirectedURL(parsedURL).orElse(url);
 			}
 			UiUtils.openURL(itemView.getContext(), item.parentFragment.getAccountID(), url);
 		}
+
+		private Optional<String> findRedirectedURL(Uri url){
+			// find actually linked url in status content
+			Matcher matcher=HtmlParser.URL_PATTERN.matcher(item.status.content);
+			boolean isAccountRedirect=url.getPath().startsWith("/redirect/accounts");
+			String foundURL;
+			while(matcher.find()){
+				foundURL=matcher.group(3);
+				if(TextUtils.isEmpty(matcher.group(4)))
+					foundURL="http://"+foundURL;
+				// SAFETY: Cannot be null, as otherwise the matcher wouldn't find it
+				// also, group is marked as non-null
+				assert foundURL!=null && url.getLastPathSegment()!=null;
+				if(foundURL.endsWith(url.getLastPathSegment()) ||
+						(isAccountRedirect && foundURL.matches("https://"+url.getHost()+"/@[a-zA-Z0-9_]+@[a-zA-Z0-9._]+$"))){
+					// found correct URL
+					return Optional.of(foundURL);
+				}
+			}
+			return Optional.empty();
+		}
 	}
 }
diff --git a/mastodon/src/main/java/org/joinmastodon/android/ui/utils/UiUtils.java b/mastodon/src/main/java/org/joinmastodon/android/ui/utils/UiUtils.java
index c8473bf60..b3a6c50c3 100644
--- a/mastodon/src/main/java/org/joinmastodon/android/ui/utils/UiUtils.java
+++ b/mastodon/src/main/java/org/joinmastodon/android/ui/utils/UiUtils.java
@@ -1482,7 +1482,7 @@ public class UiUtils {
 									return;
 								}
 								Optional<Account> account = results.accounts.stream()
-										.filter(a -> uri.equals(Uri.parse(a.url))).findAny();
+										.filter(a -> uri.getPath().contains(a.getFullyQualifiedName())).findAny();
 								if (account.isPresent()) {
 									args.putParcelable("profileAccount", Parcels.wrap(account.get()));
 									go.accept(ProfileFragment.class, args);