vector-im/element-ios/issues/5298 - Added body message content keys and replaced plain string usages.

This commit is contained in:
Stefan Ceriu 2021-12-13 12:08:52 +02:00 committed by Stefan Ceriu
parent 5a72750f8d
commit 574a155c80
10 changed files with 16 additions and 16 deletions

View file

@ -37,7 +37,7 @@
roomId = event.roomId; roomId = event.roomId;
// Title is here the file name stored in event body // Title is here the file name stored in event body
title = [event.content[@"body"] isKindOfClass:[NSString class]] ? event.content[@"body"] : nil; title = [event.content[kMXMessageBodyKey] isKindOfClass:[NSString class]] ? event.content[kMXMessageBodyKey] : nil;
// Check attachment if any // Check attachment if any
if ([searchDataSource.eventFormatter isSupportedAttachment:event]) if ([searchDataSource.eventFormatter isSupportedAttachment:event])

View file

@ -2184,7 +2184,7 @@
NSString* textMessage; NSString* textMessage;
if ([msgtype isEqualToString:kMXMessageTypeText]) if ([msgtype isEqualToString:kMXMessageTypeText])
{ {
textMessage = event.content[@"body"]; textMessage = event.content[kMXMessageBodyKey];
} }
// Show a confirmation popup to the end user // Show a confirmation popup to the end user

View file

@ -119,7 +119,7 @@ NSString *const kMXKAttachmentFileNameBase = @"attatchment";
} }
} }
MXJSONModelSetString(_originalFileName, eventContent[@"body"]); MXJSONModelSetString(_originalFileName, eventContent[kMXMessageBodyKey]);
MXJSONModelSetDictionary(_contentInfo, eventContent[@"info"]); MXJSONModelSetDictionary(_contentInfo, eventContent[@"info"]);
MXJSONModelSetMXJSONModel(contentFile, MXEncryptedContentFile, eventContent[@"file"]); MXJSONModelSetMXJSONModel(contentFile, MXEncryptedContentFile, eventContent[@"file"]);

View file

@ -3992,7 +3992,7 @@ typedef NS_ENUM (NSUInteger, MXKRoomDataSourceError) {
} }
else else
{ {
editableTextMessage = event.content[@"body"]; editableTextMessage = event.content[kMXMessageBodyKey];
} }
return editableTextMessage; return editableTextMessage;
@ -4109,7 +4109,7 @@ typedef NS_ENUM (NSUInteger, MXKRoomDataSourceError) {
NSString *sanitizedText = [self sanitizedMessageText:text]; NSString *sanitizedText = [self sanitizedMessageText:text];
NSString *formattedText = [self htmlMessageFromSanitizedText:sanitizedText]; NSString *formattedText = [self htmlMessageFromSanitizedText:sanitizedText];
NSString *eventBody = event.content[@"body"]; NSString *eventBody = event.content[kMXMessageBodyKey];
NSString *eventFormattedBody = event.content[@"formatted_body"]; NSString *eventFormattedBody = event.content[@"formatted_body"];
if (![sanitizedText isEqualToString:eventBody] && (!eventFormattedBody || ![formattedText isEqualToString:eventFormattedBody])) if (![sanitizedText isEqualToString:eventBody] && (!eventFormattedBody || ![formattedText isEqualToString:eventFormattedBody]))

View file

@ -56,7 +56,7 @@
date = [searchDataSource.eventFormatter dateStringFromEvent:searchResult.result withTime:YES]; date = [searchDataSource.eventFormatter dateStringFromEvent:searchResult.result withTime:YES];
// Code from [MXEventFormatter stringFromEvent] for the particular case of a text message // Code from [MXEventFormatter stringFromEvent] for the particular case of a text message
message = [searchResult.result.content[@"body"] isKindOfClass:[NSString class]] ? searchResult.result.content[@"body"] : nil; message = [searchResult.result.content[kMXMessageBodyKey] isKindOfClass:[NSString class]] ? searchResult.result.content[kMXMessageBodyKey] : nil;
} }
return self; return self;
} }

View file

@ -139,7 +139,7 @@ static NSString *const kHTMLATagRegexPattern = @"<a href=\"(.*?)\">([^<]*)</a>";
if (event.eventType == MXEventTypeRoomMessage) if (event.eventType == MXEventTypeRoomMessage)
{ {
NSString *msgtype; NSString *msgtype;
MXJSONModelSetString(msgtype, event.content[kMXMessageTypeKey]); MXJSONModelSetString(msgtype, event.content[@"msgtype"]);
NSString *urlField; NSString *urlField;
NSDictionary *fileField; NSDictionary *fileField;
@ -1263,12 +1263,12 @@ static NSString *const kHTMLATagRegexPattern = @"<a href=\"(.*?)\">([^<]*)</a>";
else if (eventThreadIdentifier) else if (eventThreadIdentifier)
{ {
isHTML = YES; isHTML = YES;
MXJSONModelSetString(body, event.content[@"body"]); MXJSONModelSetString(body, event.content[kMXMessageBodyKey]);
MXEvent *threadRootEvent = [mxSession.store eventWithEventId:eventThreadIdentifier MXEvent *threadRootEvent = [mxSession.store eventWithEventId:eventThreadIdentifier
inRoom:event.roomId]; inRoom:event.roomId];
NSString *threadRootEventContent; NSString *threadRootEventContent;
MXJSONModelSetString(threadRootEventContent, threadRootEvent.content[@"body"]); MXJSONModelSetString(threadRootEventContent, threadRootEvent.content[kMXMessageBodyKey]);
body = [NSString stringWithFormat:@"<mx-reply><blockquote><a href=\"%@\">In reply to</a> <a href=\"%@\">%@</a><br>%@</blockquote></mx-reply>%@", body = [NSString stringWithFormat:@"<mx-reply><blockquote><a href=\"%@\">In reply to</a> <a href=\"%@\">%@</a><br>%@</blockquote></mx-reply>%@",
[MXTools permalinkToEvent:eventThreadIdentifier inRoom:event.roomId], [MXTools permalinkToEvent:eventThreadIdentifier inRoom:event.roomId],
[MXTools permalinkToUserWithUserId:threadRootEvent.sender], [MXTools permalinkToUserWithUserId:threadRootEvent.sender],
@ -1279,7 +1279,7 @@ static NSString *const kHTMLATagRegexPattern = @"<a href=\"(.*?)\">([^<]*)</a>";
} }
else else
{ {
MXJSONModelSetString(body, event.content[@"body"]); MXJSONModelSetString(body, event.content[kMXMessageBodyKey]);
} }
if (body) if (body)
@ -1561,7 +1561,7 @@ static NSString *const kHTMLATagRegexPattern = @"<a href=\"(.*?)\">([^<]*)</a>";
else else
{ {
NSString *body; NSString *body;
MXJSONModelSetString(body, event.content[@"body"]); MXJSONModelSetString(body, event.content[kMXMessageBodyKey]);
// Check sticker validity // Check sticker validity
if (![self isSupportedAttachment:event]) if (![self isSupportedAttachment:event])
@ -2100,7 +2100,7 @@ static NSString *const kHTMLATagRegexPattern = @"<a href=\"(.*?)\">([^<]*)</a>";
else if (!_isForSubtitle && event.eventType == MXEventTypeRoomMessage && (_emojiOnlyTextFont || _singleEmojiTextFont)) else if (!_isForSubtitle && event.eventType == MXEventTypeRoomMessage && (_emojiOnlyTextFont || _singleEmojiTextFont))
{ {
NSString *message; NSString *message;
MXJSONModelSetString(message, event.content[@"body"]); MXJSONModelSetString(message, event.content[kMXMessageBodyKey]);
if (_emojiOnlyTextFont && [MXKTools isEmojiOnlyString:message]) if (_emojiOnlyTextFont && [MXKTools isEmojiOnlyString:message])
{ {

View file

@ -1066,7 +1066,7 @@ NSString *const URLPreviewDidUpdateNotification = @"URLPreviewDidUpdateNotificat
{ {
NSString *mediaName = [self accessibilityLabelForAttachmentType:self.attachment.type]; NSString *mediaName = [self accessibilityLabelForAttachmentType:self.attachment.type];
MXJSONModelSetString(accessibilityLabel, self.events.firstObject.content[@"body"]); MXJSONModelSetString(accessibilityLabel, self.events.firstObject.content[kMXMessageBodyKey]);
if (accessibilityLabel) if (accessibilityLabel)
{ {
accessibilityLabel = [NSString stringWithFormat:@"%@ %@", mediaName, accessibilityLabel]; accessibilityLabel = [NSString stringWithFormat:@"%@ %@", mediaName, accessibilityLabel];

View file

@ -386,7 +386,7 @@ class NotificationService: UNNotificationServiceExtension {
} }
let msgType = event.content[kMXMessageTypeKey] as? String let msgType = event.content[kMXMessageTypeKey] as? String
let messageContent = event.content["body"] as? String let messageContent = event.content[kMXMessageBodyKey] as? String
let isReply = event.isReply() let isReply = event.isReply()
if isReply { if isReply {

View file

@ -103,7 +103,7 @@ class TemplateRoomChatService: TemplateRoomChatServiceProtocol {
}) })
.compactMap({ event -> TemplateRoomChatMessage? in .compactMap({ event -> TemplateRoomChatMessage? in
guard let eventId = event.eventId, guard let eventId = event.eventId,
let body = event.content["body"] as? String, let body = event.content[kMXMessageBodyKey] as? String,
let sender = senderForMessage(event: event) let sender = senderForMessage(event: event)
else { return nil } else { return nil }

View file

@ -52,7 +52,7 @@
anEvent.wireType = kMXEventTypeStringRoomMessage; anEvent.wireType = kMXEventTypeStringRoomMessage;
anEvent.originServerTs = (uint64_t) ([[NSDate date] timeIntervalSince1970] * 1000); anEvent.originServerTs = (uint64_t) ([[NSDate date] timeIntervalSince1970] * 1000);
anEvent.wireContent = @{ kMXMessageTypeKey: kMXMessageTypeText, anEvent.wireContent = @{ kMXMessageTypeKey: kMXMessageTypeText,
@"body": @"deded" }; kMXMessageBodyKey: @"deded" };
maxHeaderSize = ceil(eventFormatter.defaultTextFont.pointSize * 1.2); maxHeaderSize = ceil(eventFormatter.defaultTextFont.pointSize * 1.2);
} }