wip

ConversationsClassic/AppData/Model/Message.swift

@@ -37,6 +37,13 @@ enum MessageContentType: Codable & Equatable, DatabaseValueConvertible {
     case typing
     case invite
     case attachment(Attachment)
+
+    var isAttachment: Bool {
+        if case .attachment = self {
+            return true
+        }
+        return false
+    }
 }
 
 enum MessageStatus: Int, Codable, DatabaseValueConvertible {

ConversationsClassic/AppData/Store/AttachmentsStore.swift

@@ -1,5 +1,6 @@
 import Combine
 import Foundation
+import GRDB
 import Photos
 import SwiftUI
 
@@ -12,11 +13,13 @@ final class AttachmentsStore: ObservableObject {
     private let client: Client
     private let roster: Roster
 
+    private var messagesCancellable: AnyCancellable?
     private var processing: Set<String> = []
 
     init(roster: Roster, client: Client) {
         self.client = client
         self.roster = roster
+        subscribe()
     }
 }
 
@@ -191,38 +194,54 @@ extension AttachmentsStore {
     }
 }
 
-// MARK: - Uploadings/Downloadings
+// MARK: - Processing attachments
-extension AttachmentsStore {
+private extension AttachmentsStore {
-    func processAttachment(_ message: Message) {
+    func subscribe() {
-        // Prevent multiple processing
+        messagesCancellable = ValueObservation.tracking(Message
-        if processing.contains(message.id) {
+            .filter(
-            return
+                (Column("to") == roster.bareJid && Column("from") == roster.contactBareJid) ||
-        }
+                    (Column("from") == roster.bareJid && Column("to") == roster.contactBareJid)
-
+            )
-        // Process in background
+            .order(Column("date").desc)
-        Task(priority: .background) {
+            .fetchAll
-            // Do needed processing
+        )
-            if case .attachment(let attachment) = message.contentType {
+        .publisher(in: Database.shared.dbQueue, scheduling: .immediate)
-                if attachment.localPath != nil, attachment.remotePath == nil {
+        .receive(on: DispatchQueue.main)
-                    // Uploading
+        .sink { _ in
-                    processing.insert(message.id)
+        } receiveValue: { [weak self] messages in
-                    await uploadAttachment(message)
+            let forProcessing = messages
-                    processing.remove(message.id)
+                // .filter { $0.status == .pending }
-                } else if attachment.localPath == nil, attachment.remotePath != nil {
+                .filter { self?.processing.contains($0.id) == false }
-                    // Downloading
+                .filter { $0.contentType.isAttachment }
-                    processing.insert(message.id)
+            for message in forProcessing {
-                    await downloadAttachment(message)
+                if case .attachment(let attachment) = message.contentType {
-                    processing.remove(message.id)
+                    if attachment.localPath != nil, attachment.remotePath == nil {
-                } else if attachment.localPath != nil, attachment.remotePath != nil, attachment.thumbnailName == nil, attachment.type == .image {
+                        // Uploading
-                    // Generate thumbnail
+                        self?.processing.insert(message.id)
-                    processing.insert(message.id)
+                        Task(priority: .background) {
-                    await generateThumbnail(message)
+                            await self?.uploadAttachment(message)
-                    processing.remove(message.id)
+                        }
+                    } else if attachment.localPath == nil, attachment.remotePath != nil {
+                        // Downloading
+                        self?.processing.insert(message.id)
+                        Task(priority: .background) {
+                            await self?.downloadAttachment(message)
+                        }
+                    } else if attachment.localPath != nil, attachment.remotePath != nil, attachment.thumbnailName == nil, attachment.type == .image {
+                        // Generate thumbnail
+                        self?.processing.insert(message.id)
+                        Task(priority: .background) {
+                            await self?.generateThumbnail(message)
+                        }
+                    }
                 }
             }
         }
     }
+}
 
+// MARK: - Uploadings/Downloadings
+extension AttachmentsStore {
     private func uploadAttachment(_ message: Message) async {
         do {
             try await message.setStatus(.pending)
@@ -246,8 +265,10 @@ extension AttachmentsStore {
             message.oobUrl = remotePath
             try await message.save()
             try await client.sendMessage(message)
+            processing.remove(message.id)
             try await message.setStatus(.sent)
         } catch {
+            processing.remove(message.id)
             try? await message.setStatus(.error)
         }
     }
@@ -276,6 +297,7 @@ extension AttachmentsStore {
                     remotePath: remotePath
                 )
             )
+            processing.remove(message.id)
             try await message.save()
         } catch {
             logIt(.error, "Can't download attachment: \(error)")
@@ -324,6 +346,7 @@ extension AttachmentsStore {
                 remotePath: attachment.remotePath
             )
         )
+        processing.remove(message.id)
         try? await message.save()
     }
 }

ConversationsClassic/View/Main/Conversation/ConversationMessageContainer.swift

@@ -150,15 +150,12 @@ private struct AttachmentView: View {
                     ProgressView()
                         .scaleEffect(1.5)
                         .progressViewStyle(CircularProgressViewStyle(tint: .Material.Elements.active))
-                    let imageName = progressImageName(attachment.type ?? .file)
+                    let imageName = progressImageName(attachment.type)
                     Image(systemName: imageName)
                         .font(.body1)
                         .foregroundColor(.Material.Elements.active)
                 }
             }
-            .onAppear {
-                attachments.processAttachment(message)
-            }
     }
 
     @ViewBuilder private var failed: some View {
@@ -178,7 +175,9 @@ private struct AttachmentView: View {
                 }
             }
             .onTapGesture {
-                attachments.processAttachment(message)
+                Task {
+                    try? await message.setStatus(.pending)
+                }
             }
     }