Should not try to use the cached state flow as they can be out of date just following config updates

Author: SessionHero01

app/src/main/java/org/thoughtcrime/securesms/database/MmsDatabase.kt

@@ -654,10 +654,8 @@ class MmsDatabase @Inject constructor(
             linkPreviews = retrieved.linkPreviews,
             contentValues = contentValues,
         )
-        if (retrieved.messageContent !is DisappearingMessageUpdate) {
-            if (runThreadUpdate) {
-                threadDatabase.update(threadId, true)
-            }
+        if (runThreadUpdate) {
+            threadDatabase.update(threadId, true)
         }
         return Optional.of(InsertResult(messageId, threadId))
     }

app/src/main/java/org/thoughtcrime/securesms/database/RecipientRepository.kt

@@ -369,13 +369,6 @@ class RecipientRepository @Inject constructor(
      */
     @DelicateCoroutinesApi
     fun getRecipientSync(address: Address): Recipient {
-        // If we have a cached flow, we can try to grab its current value.
-        val cached = recipientFlowCache[address]?.get()?.replayCache?.firstOrNull()
-        if (cached != null) {
-            return cached
-        }
-
-        // Otherwise, we might have to go to the database to get the recipient..
         return fetchRecipient(
             address = address,
             settingsFetcher = recipientSettingsDatabase::getSettings,