Don't use caption field

pull/1/head
Michael Kirk 6 years ago
parent d0f68177f1
commit cf48fb3076

@ -2815,7 +2815,7 @@ typedef enum : NSUInteger {
}
}
- (void)sendMessageAttachments:(NSArray<SignalAttachment *> *)attachments
- (void)sendMessageAttachments:(NSArray<SignalAttachment *> *)attachments messageText:(nullable NSString *)messageText
{
OWSAssertIsOnMainThread();
for (SignalAttachment *attachment in attachments) {
@ -2824,8 +2824,9 @@ typedef enum : NSUInteger {
}
BOOL didAddToProfileWhitelist = [ThreadUtil addThreadToProfileWhitelistIfEmptyContactThread:self.thread];
TSOutgoingMessage *message = [ThreadUtil enqueueMessageWithAttachments:attachments
messageBody:nil
messageBody:messageText
inThread:self.thread
quotedReplyModel:self.inputToolbar.quotedReply];
@ -3546,7 +3547,7 @@ typedef enum : NSUInteger {
}
if (skipApprovalDialog) {
[self sendMessageAttachments:attachments];
[self sendMessageAttachments:attachments messageText:nil];
} else {
OWSNavigationController *modal =
[AttachmentApprovalViewController wrappedInNavControllerWithAttachments:attachments
@ -3662,8 +3663,9 @@ typedef enum : NSUInteger {
- (void)attachmentApproval:(AttachmentApprovalViewController *)attachmentApproval
didApproveAttachments:(NSArray<SignalAttachment *> *)attachments
messageText:(nullable NSString *)messageText
{
[self sendMessageAttachments:attachments];
[self sendMessageAttachments:attachments messageText:messageText];
[self dismissViewControllerAnimated:YES completion:nil];
// We always want to scroll to the bottom of the conversation after the local user
// sends a message. Normally, this is taken care of in yapDatabaseModified:, but

@ -8,7 +8,7 @@ import MediaPlayer
@objc
public protocol AttachmentApprovalViewControllerDelegate: class {
func attachmentApproval(_ attachmentApproval: AttachmentApprovalViewController, didApproveAttachments attachments: [SignalAttachment])
func attachmentApproval(_ attachmentApproval: AttachmentApprovalViewController, didApproveAttachments attachments: [SignalAttachment], messageText: String?)
func attachmentApproval(_ attachmentApproval: AttachmentApprovalViewController, didCancelAttachments attachments: [SignalAttachment])
}
@ -326,11 +326,14 @@ public class AttachmentApprovalViewController: UIPageViewController, UIPageViewC
captioningToolbar.isUserInteractionEnabled = false
captioningToolbar.isHidden = true
approvalDelegate?.attachmentApproval(self, didApproveAttachments: attachments)
approvalDelegate?.attachmentApproval(self, didApproveAttachments: attachments, messageText: captioningToolbar.captionText)
}
func captioningToolbar(_ captioningToolbar: CaptioningToolbar, textViewDidChange textView: UITextView) {
currentItem.attachment.captionText = textView.text
// For 2.32.0 we only have one input bar - and it's for mesageBody text, not for caption text.
// For 2.33.0 there are potentially two input bars - one for message body and one for caption text.
// Commenting this out for now, but we'll have to resolve this merge conflict when RI'ing to 2.33
// currentItem.attachment.captionText = textView.text
}
}

Loading…
Cancel
Save