From e025b86e75f2d231b9c52cc5b65718dc6e102a51 Mon Sep 17 00:00:00 2001 From: Matthew Chen Date: Wed, 12 Apr 2017 17:30:16 -0400 Subject: [PATCH] Rework outgoing message state. // FREEBIE --- .../ViewControllers/MessagesViewController.m | 16 -------- Signal/src/util/ThreadUtil.m | 39 ++++++------------- 2 files changed, 12 insertions(+), 43 deletions(-) diff --git a/Signal/src/ViewControllers/MessagesViewController.m b/Signal/src/ViewControllers/MessagesViewController.m index 0795c8835..482577a52 100644 --- a/Signal/src/ViewControllers/MessagesViewController.m +++ b/Signal/src/ViewControllers/MessagesViewController.m @@ -2319,22 +2319,6 @@ typedef enum : NSUInteger { (unsigned long)attachment.data.length, [attachment mimeType]); [ThreadUtil sendMessageWithAttachment:attachment inThread:self.thread messageSender:self.messageSender]; - - TSOutgoingMessage *message; - OWSDisappearingMessagesConfiguration *configuration = - [OWSDisappearingMessagesConfiguration fetchObjectWithUniqueID:self.thread.uniqueId]; - if (configuration.isEnabled) { - message = [[TSOutgoingMessage alloc] initWithTimestamp:[NSDate ows_millisecondTimeStamp] - inThread:self.thread - messageBody:nil - attachmentIds:[NSMutableArray new] - expiresInSeconds:configuration.durationSeconds]; - } else { - message = [[TSOutgoingMessage alloc] initWithTimestamp:[NSDate ows_millisecondTimeStamp] - inThread:self.thread - messageBody:nil - attachmentIds:[NSMutableArray new]]; - } } - (NSURL *)videoTempFolder { diff --git a/Signal/src/util/ThreadUtil.m b/Signal/src/util/ThreadUtil.m index ab623bee3..2eca11f87 100644 --- a/Signal/src/util/ThreadUtil.m +++ b/Signal/src/util/ThreadUtil.m @@ -25,21 +25,14 @@ NS_ASSUME_NONNULL_BEGIN OWSAssert(thread); OWSAssert(messageSender); - TSOutgoingMessage *message; OWSDisappearingMessagesConfiguration *configuration = [OWSDisappearingMessagesConfiguration fetchObjectWithUniqueID:thread.uniqueId]; - if (configuration.isEnabled) { - message = [[TSOutgoingMessage alloc] initWithTimestamp:[NSDate ows_millisecondTimeStamp] - inThread:thread - messageBody:text - attachmentIds:[NSMutableArray new] - expiresInSeconds:configuration.durationSeconds]; - } else { - message = [[TSOutgoingMessage alloc] initWithTimestamp:[NSDate ows_millisecondTimeStamp] - inThread:thread - messageBody:text]; - } - + TSOutgoingMessage *message = + [[TSOutgoingMessage alloc] initWithTimestamp:[NSDate ows_millisecondTimeStamp] + inThread:thread + messageBody:text + attachmentIds:[NSMutableArray new] + expiresInSeconds:(configuration.isEnabled ? configuration.durationSeconds : 0)]; [messageSender sendMessage:message success:^{ DDLogInfo(@"%@ Successfully sent message.", self.tag); @@ -61,22 +54,14 @@ NS_ASSUME_NONNULL_BEGIN OWSAssert(thread); OWSAssert(messageSender); - TSOutgoingMessage *message; OWSDisappearingMessagesConfiguration *configuration = [OWSDisappearingMessagesConfiguration fetchObjectWithUniqueID:thread.uniqueId]; - if (configuration.isEnabled) { - message = [[TSOutgoingMessage alloc] initWithTimestamp:[NSDate ows_millisecondTimeStamp] - inThread:thread - messageBody:nil - attachmentIds:[NSMutableArray new] - expiresInSeconds:configuration.durationSeconds]; - } else { - message = [[TSOutgoingMessage alloc] initWithTimestamp:[NSDate ows_millisecondTimeStamp] - inThread:thread - messageBody:nil - attachmentIds:[NSMutableArray new]]; - } - + TSOutgoingMessage *message = + [[TSOutgoingMessage alloc] initWithTimestamp:[NSDate ows_millisecondTimeStamp] + inThread:thread + messageBody:nil + attachmentIds:[NSMutableArray new] + expiresInSeconds:(configuration.isEnabled ? configuration.durationSeconds : 0)]; [messageSender sendAttachmentData:attachment.data contentType:attachment.mimeType filename:attachment.filename