Plumb guid down for deduplication on the conversation level

kordophone/Bridge/Operations/MBIMMessagesListOperation.m

@@ -43,10 +43,10 @@
                 // Load messages
                 [chat loadMessagesBeforeDate:[NSDate date] limit:50 loadImmediately:YES];
                 
-                for (IMMessageItem *imMessage in [[chat chatItems] messages]) {
-                    NSDictionary *messageDict = [imMessage mbim_dictionaryRepresentation];
+                [[chat chatItems] enumerateMessagesWithOptions:0 usingBlock:^(IMMessage *message, BOOL *stop) {
+                    NSDictionary *messageDict = [message mbim_dictionaryRepresentation];
                     [messages addObject:messageDict];
-                }
+                }];
             }
         });
         

kordophone/Categories/IMMessageItem+Encoded.h

@@ -16,9 +16,4 @@ NS_ASSUME_NONNULL_BEGIN
 
 @end
 
-@interface IMMessageItem (Encoded)
-- (NSDictionary *)mbim_dictionaryRepresentation;
-
-@end
-
 NS_ASSUME_NONNULL_END

kordophone/Categories/IMMessageItem+Encoded.m

@@ -16,21 +16,8 @@
     NSMutableDictionary *messageDict = [NSMutableDictionary dictionary];
     messageDict[@"text"] = [[self text] string];
     messageDict[@"date"] = MBIMWebServerFormatISO8601([self time]);
-    messageDict[@"sender"] = [[self sender] displayID];
-    
-    return messageDict;
-}
-
-@end
-
-@implementation IMMessageItem (Encoded)
-
-- (NSDictionary *)mbim_dictionaryRepresentation
-{
-    NSMutableDictionary *messageDict = [NSMutableDictionary dictionary];
-    messageDict[@"text"] = [[self body] string];
-    messageDict[@"date"] = MBIMWebServerFormatISO8601([self time]);
-    messageDict[@"sender"] = [self sender];
+    messageDict[@"sender"] = ([self isFromMe] ? nil : [[self sender] displayID]); // TODO: nil sender is still a weird way to represent this...
+    messageDict[@"guid"] = [self guid];
     
     if ([self fileTransferGUIDs]) {
         // Support only images right now
@@ -51,3 +38,4 @@
 }
 
 @end
+