From 9e52fc067d97133fcf290a4c5a6e7c7814f3e535 Mon Sep 17 00:00:00 2001 From: Benoit Marty Date: Thu, 15 Jan 2026 17:07:53 +0100 Subject: [PATCH] Avoid creating a new constant for the same goal. --- .../impl/DefaultRoomLatestEventFormatter.kt | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/libraries/eventformatter/impl/src/main/kotlin/io/element/android/libraries/eventformatter/impl/DefaultRoomLatestEventFormatter.kt b/libraries/eventformatter/impl/src/main/kotlin/io/element/android/libraries/eventformatter/impl/DefaultRoomLatestEventFormatter.kt index 7e1ffc3ee6..009547f9eb 100644 --- a/libraries/eventformatter/impl/src/main/kotlin/io/element/android/libraries/eventformatter/impl/DefaultRoomLatestEventFormatter.kt +++ b/libraries/eventformatter/impl/src/main/kotlin/io/element/android/libraries/eventformatter/impl/DefaultRoomLatestEventFormatter.kt @@ -9,6 +9,7 @@ package io.element.android.libraries.eventformatter.impl import dev.zacsweers.metro.ContributesBinding +import io.element.android.libraries.core.extensions.DEFAULT_SAFE_LENGTH import io.element.android.libraries.di.SessionScope import io.element.android.libraries.eventformatter.api.RoomLatestEventFormatter import io.element.android.libraries.eventformatter.impl.mode.RenderingMode @@ -54,11 +55,6 @@ class DefaultRoomLatestEventFormatter( private val stateContentFormatter: StateContentFormatter, private val permalinkParser: PermalinkParser, ) : RoomLatestEventFormatter { - companion object { - // Max characters to display in the last message. This works around https://github.com/element-hq/element-x-android/issues/2105 - private const val MAX_SAFE_LENGTH = 500 - } - override fun format( latestEvent: LatestEventValue.Local, isDmRoom: Boolean, @@ -121,7 +117,7 @@ class DefaultRoomLatestEventFormatter( } is LegacyCallInviteContent -> sp.getString(CommonStrings.common_unsupported_call) is CallNotifyContent -> sp.getString(CommonStrings.common_call_started) - }?.take(MAX_SAFE_LENGTH) + }?.take(DEFAULT_SAFE_LENGTH) } private fun MessageContent.process(