From d800b225b6648e17981311169784e45cc4db57c0 Mon Sep 17 00:00:00 2001
From: Benoit Marty <benoit@matrix.org>
Date: Fri, 15 Dec 2023 15:16:23 +0100
Subject: [PATCH] Consider the returned value of `LinkifyCompat.addLinks`

---
 .../factories/event/TimelineItemContentMessageFactory.kt   | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/features/messages/impl/src/main/kotlin/io/element/android/features/messages/impl/timeline/factories/event/TimelineItemContentMessageFactory.kt b/features/messages/impl/src/main/kotlin/io/element/android/features/messages/impl/timeline/factories/event/TimelineItemContentMessageFactory.kt
index 2e6445deda..7aa821d5c4 100644
--- a/features/messages/impl/src/main/kotlin/io/element/android/features/messages/impl/timeline/factories/event/TimelineItemContentMessageFactory.kt
+++ b/features/messages/impl/src/main/kotlin/io/element/android/features/messages/impl/timeline/factories/event/TimelineItemContentMessageFactory.kt
@@ -244,9 +244,6 @@ class TimelineItemContentMessageFactory @Inject constructor(
 
 private fun String.withLinks(): CharSequence? {
     val spannable = toSpannable()
-    LinkifyCompat.addLinks(spannable, Linkify.WEB_URLS or Linkify.PHONE_NUMBERS)
-    if (spannable.getSpans<URLSpan>().isEmpty()) {
-        return null
-    }
-    return spannable
+    val addedLinks = LinkifyCompat.addLinks(spannable, Linkify.WEB_URLS or Linkify.PHONE_NUMBERS)
+    return spannable.takeIf { addedLinks }
 }