From f045562a3332b0b1454eaf33dbfcfe81fda791fa Mon Sep 17 00:00:00 2001
From: William Grant <willmgrant@gmail.com>
Date: Thu, 21 Sep 2023 11:39:58 +1000
Subject: [PATCH] fix: use updateExpireTimer for UserProfile

before we were setting the convo values directly
---
 ts/receiver/configMessage.ts | 37 +++++++++++++++++++-----------------
 1 file changed, 20 insertions(+), 17 deletions(-)

diff --git a/ts/receiver/configMessage.ts b/ts/receiver/configMessage.ts
index 53e82fba1..f7fc30684 100644
--- a/ts/receiver/configMessage.ts
+++ b/ts/receiver/configMessage.ts
@@ -228,26 +228,26 @@ async function handleUserProfileUpdate(result: IncomingConfResult): Promise<Inco
     const expireTimer = ourConvo.get('expireTimer');
     const wrapperNoteToSelfExpirySeconds = await UserConfigWrapperActions.getNoteToSelfExpiry();
 
-    // TODO Should use updateExpireTimer instead
     if (wrapperNoteToSelfExpirySeconds !== expireTimer) {
-      // we trust the wrapper more than the DB, so let's update the DB but we don't show it in the UI
-      ourConvo.set('expireTimer', wrapperNoteToSelfExpirySeconds);
-
-      // NOTE Can only be 'off' or 'deleteAfterSend' so we don't need to check the expirationType we can just override it
-      ourConvo.set(
-        'expirationType',
-        wrapperNoteToSelfExpirySeconds && wrapperNoteToSelfExpirySeconds > 0
-          ? 'deleteAfterSend'
-          : 'off'
-      );
+      await ourConvo.updateExpireTimer({
+        providedExpirationType:
+          wrapperNoteToSelfExpirySeconds && wrapperNoteToSelfExpirySeconds > 0
+            ? 'deleteAfterSend'
+            : 'off',
+        providedExpireTimer: wrapperNoteToSelfExpirySeconds,
+        providedChangeTimestamp: result.latestEnvelopeTimestamp,
+        providedSource: ourConvo.id,
+        receivedAt: result.latestEnvelopeTimestamp,
+        fromSync: true,
+        shouldCommit: false,
+      });
       changes = true;
       window.log.debug(
-        `WIP: [userProfileWrapper] updating disappearing messages to`,
-        wrapperNoteToSelfExpirySeconds && wrapperNoteToSelfExpirySeconds > 0
-          ? 'deleteAfterSend'
-          : 'off',
-        ' ',
-        wrapperNoteToSelfExpirySeconds
+        `WIP: [userProfileWrapper] updating disappearing messages to expiratonMode: ${
+          wrapperNoteToSelfExpirySeconds && wrapperNoteToSelfExpirySeconds > 0
+            ? 'deleteAfterSend'
+            : 'off'
+        } wrapperNoteToSelfExpirySeconds: ${wrapperNoteToSelfExpirySeconds}`
       );
     }
 
@@ -395,6 +395,9 @@ async function handleContactsUpdate(result: IncomingConfResult): Promise<Incomin
           shouldCommit: false,
         });
         changes = true;
+        window.log.debug(
+          `WIP: [contactsWrapper] updating disappearing messages to expirationMode: ${wrapperConvo.expirationMode} expirationTimerSeconds: ${wrapperConvo.expirationTimerSeconds}`
+        );
       }
 
       // we want to set the active_at to the created_at timestamp if active_at is unset, so that it shows up in our list.