fix: reverted setToExpire timer,

fixed incorrect expirationLength value in getPropsForExpiringMessage

ts/models/conversation.ts

@@ -2010,6 +2010,8 @@ export class ConversationModel extends Backbone.Model<ConversationAttributes> {
     const messageId = await model.commit(false);
     model.set({ id: messageId });
 
+    await model.setToExpire();
+
     const messageModelProps = model.getMessageModelProps();
     window.inboxStore?.dispatch(conversationActions.messagesChanged([messageModelProps]));
     const unreadCount = await this.getUnreadCount();

ts/models/message.ts

@@ -280,13 +280,15 @@ export class MessageModel extends Backbone.Model<MessageAttributes> {
 
   public getPropsForExpiringMessage(): PropsForExpiringMessage | { direction: MessageModelType } {
     const expirationType = this.get('expirationType');
-    const expirationLength = this.get('expireTimer') || null;
+    const expirationLength = this.get('expireTimer')
+      ? this.get('expireTimer') * DURATION.SECONDS
+      : null;
 
     const expireTimerStart = this.get('expirationStartTimestamp') || null;
 
     const expirationTimestamp =
       expirationType && expireTimerStart && expirationLength
-        ? expireTimerStart + expirationLength * DURATION.SECONDS
+        ? expireTimerStart + expirationLength
         : null;
 
     const direction =
@@ -492,10 +494,12 @@ export class MessageModel extends Backbone.Model<MessageAttributes> {
   public getPropsForMessage(options: any = {}): PropsForMessageWithoutConvoProps {
     const sender = this.getSource();
     const expirationType = this.get('expirationType');
-    const expirationLength = this.get('expireTimer') * 1000;
+    const expirationLength = this.get('expireTimer') * DURATION.SECONDS;
     const expireTimerStart = this.get('expirationStartTimestamp');
     const expirationTimestamp =
-      expirationLength && expireTimerStart ? expireTimerStart + expirationLength : null;
+      expirationType && expirationLength && expireTimerStart
+        ? expireTimerStart + expirationLength
+        : null;
 
     const attachments = this.get('attachments') || [];
     const isTrustedForAttachmentDownload = this.isTrustedForAttachmentDownload();

ts/receiver/contentMessage.ts

@@ -781,7 +781,7 @@ export async function handleDataExtractionNotification(
     // TODO legacy messages support will be removed in a future release
     const isLegacyMode = convo && convo.isPrivate() && expirationType === 'legacy';
 
-    const msgModel = await convo.addSingleIncomingMessage({
+    await convo.addSingleIncomingMessage({
       source,
       sent_at: envelopeTimestamp,
       dataExtractionNotification: {
@@ -799,9 +799,6 @@ export async function handleDataExtractionNotification(
           : undefined,
     });
 
-    if (isLegacyMode || expirationType === 'deleteAfterSend') {
-      await msgModel.setToExpire();
-    }
     convo.updateLastMessage();
   }
 }

ts/receiver/queuedJob.ts

@@ -350,7 +350,6 @@ export async function handleMessageJob(
           messageModel.get('expirationType') === 'legacy'
         ),
       });
-      messageModel.setToExpire();
     }
 
     if (messageModel.isExpirationTimerUpdate()) {

ts/session/group/closed-group.ts

@@ -188,10 +188,6 @@ export async function addUpdateMessage(
 
   if (UserUtils.isUsFromCache(sender)) {
     const outgoingMessage = await convo.addSingleOutgoingMessage(msgModel);
-    // NOTE will always be delete after send
-    if (expirationType) {
-      outgoingMessage.setToExpire();
-    }
     return outgoingMessage;
   }
 
@@ -208,9 +204,6 @@ export async function addUpdateMessage(
 
   await convo.commit();
 
-  if (expirationType) {
-    incomingMessage.setToExpire();
-  }
   return incomingMessage;
 }
 

ts/session/sending/MessageSentHandler.ts

@@ -151,7 +151,6 @@ async function handleMessageSentSuccess(
           isLegacySentMode
         ),
       });
-      fetchedMessage.setToExpire();
     }
   }