/issues/4753 - Prevent unnecessary audio file conversions if final fi…

…le already exists on disk.
element-hq · Dec 2, 2021 · bb1ad41 · bb1ad41
1 parent eb8d782
commit bb1ad41
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/Riot/Modules/Room/VoiceMessages/VoiceMessageAttachmentCacheManager.swift b/Riot/Modules/Room/VoiceMessages/VoiceMessageAttachmentCacheManager.swift
@@ -208,9 +208,9 @@ class VoiceMessageAttachmentCacheManager {
             return
         }
 
-        let newURL = temporaryFilesFolderURL.appendingPathComponent(ProcessInfo().globallyUniqueString).appendingPathExtension("m4a")
+        let newURL = temporaryFilesFolderURL.appendingPathComponent(identifier).appendingPathExtension("m4a")
 
-        VoiceMessageAudioConverter.convertToMPEG4AAC(sourceURL: URL(fileURLWithPath: filePath), destinationURL: newURL) { result in
+        let conversionCompletion: (Result<Void, VoiceMessageAudioConverterError>) -> Void = { result in
             self.workQueue.async {
                 switch result {
                 case .success:
@@ -245,6 +245,12 @@ class VoiceMessageAttachmentCacheManager {
                 }
             }
         }
+
+        if FileManager.default.fileExists(atPath: newURL.path) {
+            conversionCompletion(Result.success(()))
+        } else {
+            VoiceMessageAudioConverter.convertToMPEG4AAC(sourceURL: URL(fileURLWithPath: filePath), destinationURL: newURL, completion: conversionCompletion)
+        }
     }
 
     private func sampleFileAtURL(_ url: URL, duration: TimeInterval, numberOfSamples: Int, identifier: String, semaphore: DispatchSemaphore) {

diff --git a/changelog.d/4753.feature b/changelog.d/4753.feature
@@ -1 +1 @@
-Allow audio file attachments to be played back inline by reusing the existing voice message UI.
+Allow audio file attachments to be played back inline by reusing the existing voice message UI. Prevent unnecessary conversions if final file already exists on disk.