From 12dcb104854cdb68b669a5ce97ccd4b491f04acc Mon Sep 17 00:00:00 2001
From: Sylvain Berfini <sylvain.berfini@belledonne-communications.com>
Date: Tue, 13 Jul 2021 15:10:28 +0200
Subject: [PATCH] Fixed encrypted file export to external app

---
 .../main/chat/data/ChatMessageContentData.kt  |  3 +-
 .../chat/fragments/DetailChatRoomFragment.kt  | 16 ++++++---
 .../main/java/org/linphone/utils/FileUtils.kt | 33 +++++++++++++++++++
 app/src/main/res/xml/provider_paths.xml       |  1 +
 4 files changed, 48 insertions(+), 5 deletions(-)

diff --git a/app/src/main/java/org/linphone/activities/main/chat/data/ChatMessageContentData.kt b/app/src/main/java/org/linphone/activities/main/chat/data/ChatMessageContentData.kt
index 487f3cd9e..a9f390771 100644
--- a/app/src/main/java/org/linphone/activities/main/chat/data/ChatMessageContentData.kt
+++ b/app/src/main/java/org/linphone/activities/main/chat/data/ChatMessageContentData.kt
@@ -199,7 +199,8 @@ class ChatMessageContentData(
         downloadLabel.value = spannable
 
         if (content.isFile || (content.isFileTransfer && chatMessage.isOutgoing)) {
-            val path = if (content.isFileEncrypted) content.plainFilePath else content.filePath ?: ""
+            Log.i("[Content] Is content encrypted ? $isFileEncrypted")
+            val path = if (isFileEncrypted) content.plainFilePath else content.filePath ?: ""
             downloadable.value = content.filePath.orEmpty().isEmpty()
 
             if (path.isNotEmpty()) {
diff --git a/app/src/main/java/org/linphone/activities/main/chat/fragments/DetailChatRoomFragment.kt b/app/src/main/java/org/linphone/activities/main/chat/fragments/DetailChatRoomFragment.kt
index c79a1b74f..b9c13a7df 100644
--- a/app/src/main/java/org/linphone/activities/main/chat/fragments/DetailChatRoomFragment.kt
+++ b/app/src/main/java/org/linphone/activities/main/chat/fragments/DetailChatRoomFragment.kt
@@ -298,7 +298,7 @@ class DetailChatRoomFragment : MasterFragment<ChatRoomDetailFragmentBinding, Cha
                             else -> {
                                 if (content.isFileEncrypted) {
                                     Log.w("[Chat Message] File is encrypted and can't be opened in one of our viewers...")
-                                    showDialogForUserConsentBeforeExportingFileInThirdPartyApp(path)
+                                    showDialogForUserConsentBeforeExportingFileInThirdPartyApp(content)
                                 } else if (!FileUtils.openFileInThirdPartyApp(requireActivity(), path)) {
                                     showDialogToSuggestOpeningFileAsText()
                                 }
@@ -675,7 +675,7 @@ class DetailChatRoomFragment : MasterFragment<ChatRoomDetailFragmentBinding, Cha
         dialog.show()
     }
 
-    private fun showDialogForUserConsentBeforeExportingFileInThirdPartyApp(path: String) {
+    private fun showDialogForUserConsentBeforeExportingFileInThirdPartyApp(content: Content) {
         val dialogViewModel = DialogViewModel(
             getString(R.string.chat_message_cant_open_file_in_app_dialog_message),
             getString(R.string.chat_message_cant_open_file_in_app_dialog_title)
@@ -684,8 +684,16 @@ class DetailChatRoomFragment : MasterFragment<ChatRoomDetailFragmentBinding, Cha
 
         dialogViewModel.showDeleteButton({
             dialog.dismiss()
-            if (!FileUtils.openFileInThirdPartyApp(requireActivity(), path)) {
-                showDialogToSuggestOpeningFileAsText()
+            lifecycleScope.launch {
+                val plainFilePath = content.plainFilePath
+                Log.i("[Cht Room] Making a copy of [$plainFilePath] to the cache directory before exporting it")
+                val cacheCopyPath = FileUtils.copyFileToCache(plainFilePath)
+                if (cacheCopyPath != null) {
+                    Log.i("[Cht Room] Cache copy has been made: $cacheCopyPath")
+                    if (!FileUtils.openFileInThirdPartyApp(requireActivity(), cacheCopyPath)) {
+                        showDialogToSuggestOpeningFileAsText()
+                    }
+                }
             }
         }, getString(R.string.chat_message_cant_open_file_in_app_dialog_export_button))
 
diff --git a/app/src/main/java/org/linphone/utils/FileUtils.kt b/app/src/main/java/org/linphone/utils/FileUtils.kt
index 8e5aa6671..00a512070 100644
--- a/app/src/main/java/org/linphone/utils/FileUtils.kt
+++ b/app/src/main/java/org/linphone/utils/FileUtils.kt
@@ -118,6 +118,24 @@ class FileUtils {
             return returnPath
         }
 
+        private fun getFileStorageCacheDir(fileName: String): File {
+            val path = coreContext.context.cacheDir
+            Log.i("[File Utils] Cache directory is: $path")
+
+            val realFileName = if (fileName.endsWith(VFS_PLAIN_FILE_EXTENSION)) {
+                fileName.substring(0, fileName.length - VFS_PLAIN_FILE_EXTENSION.length)
+            } else fileName
+            var file = File(path, realFileName)
+
+            var prefix = 1
+            while (file.exists()) {
+                file = File(path, prefix.toString() + "_" + realFileName)
+                Log.w("[File Utils] File with that name already exists, renamed to ${file.name}")
+                prefix += 1
+            }
+            return file
+        }
+
         fun getFileStoragePath(fileName: String): File {
             val path = getFileStorageDir(isExtensionImage(fileName))
             var file = File(path, fileName)
@@ -251,6 +269,21 @@ class FileUtils {
             return false
         }
 
+        suspend fun copyFileToCache(plainFilePath: String): String? {
+            val cacheFile = getFileStorageCacheDir(getNameFromFilePath(plainFilePath))
+            try {
+                withContext(Dispatchers.IO) {
+                    FileOutputStream(cacheFile).use { out ->
+                        copyFileTo(plainFilePath, out)
+                    }
+                }
+                return cacheFile.absolutePath
+            } catch (e: IOException) {
+                Log.e("[File Utils] copyFileToCache exception: $e")
+            }
+            return null
+        }
+
         private fun createFile(file: String): File {
             var fileName = file
 
diff --git a/app/src/main/res/xml/provider_paths.xml b/app/src/main/res/xml/provider_paths.xml
index 5932c8903..07009997a 100644
--- a/app/src/main/res/xml/provider_paths.xml
+++ b/app/src/main/res/xml/provider_paths.xml
@@ -4,4 +4,5 @@
     <external-files-path name="pictures" path="Pictures/" />
     <external-files-path name="downloads" path="Download/" />
     <external-files-path name="files" path="." />
+    <cache-path name="cache" path="."/>
 </paths>
\ No newline at end of file