fix #185

Author: Raival-e

app/src/main/java/com/raival/compose/file/explorer/screen/main/MainActivity.kt

@@ -119,7 +119,10 @@ class MainActivity : BaseActivity() {
                         show = mainActivityState.showSaveEditorFilesDialog,
                         isSaving = mainActivityState.isSavingFiles,
                         onDismiss = { mainActivityManager.toggleSaveEditorFilesDialog(false) },
-                        onRequestFinish = { finish() },
+                        onIgnore = {
+                            mainActivityManager.ignoreTextEditorFiles()
+                            finish()
+                        },
                         onSave = { mainActivityManager.saveTextEditorFiles { finish() } }
                     )
 

app/src/main/java/com/raival/compose/file/explorer/screen/main/MainActivityManager.kt

@@ -278,6 +278,10 @@ class MainActivityManager {
         return true
     }
 
+    fun ignoreTextEditorFiles() {
+        globalClass.textEditorManager.fileInstanceList.clear()
+    }
+
     fun saveTextEditorFiles(onFinish: () -> Unit) {
         _state.update {
             it.copy(
@@ -286,10 +290,13 @@ class MainActivityManager {
         }
 
         managerScope.launch {
-            globalClass.textEditorManager.fileInstanceList.forEach {
-                if (it.requireSave) {
-                    it.file.writeText(it.content.toString())
+            globalClass.textEditorManager.fileInstanceList.apply {
+                forEach {
+                    if (it.requireSave) {
+                        it.file.writeText(it.content.toString())
+                    }
                 }
+                clear()
             }
 
             _state.update {

app/src/main/java/com/raival/compose/file/explorer/screen/main/ui/SaveTextEditorFilesDialog.kt

@@ -11,7 +11,7 @@ import com.raival.compose.file.explorer.R
 fun SaveTextEditorFilesDialog(
     show: Boolean,
     onDismiss: () -> Unit,
-    onRequestFinish: () -> Unit,
+    onIgnore: () -> Unit,
     onSave: () -> Unit,
     isSaving: Boolean
 ) {
@@ -31,7 +31,7 @@ fun SaveTextEditorFilesDialog(
                 TextButton(
                     onClick = {
                         onDismiss()
-                        onRequestFinish()
+                        onIgnore()
                     }
                 ) { Text(text = stringResource(R.string.ignore)) }
             },

app/src/main/java/com/raival/compose/file/explorer/screen/viewer/text/TextViewerActivity.kt

@@ -72,6 +72,27 @@ class TextViewerActivity : ViewerActivity() {
                         val symbols = remember { mutableStateListOf<SymbolHolder>() }
                         val activityScope = rememberCoroutineScope()
 
+                        BackHandler(
+                            textViewerInstance.requireSave
+                                    || textViewerInstance.showSearchPanel
+                        ) {
+                            if (textViewerInstance.showSearchPanel) {
+                                textViewerInstance.hideSearchPanel(codeEditor)
+                            } else {
+                                textViewerInstance.showUnsavedChangesWarningDialog(
+                                    onSaved = { finish() },
+                                    onFailed = {
+                                        if (it is SecurityException) {
+                                            globalClass.showMsg(getString(R.string.permission_denied))
+                                        } else {
+                                            globalClass.showMsg(getString(R.string.failed_to_save))
+                                        }
+                                    },
+                                    onIgnore = { finish() }
+                                )
+                            }
+                        }
+
                         LaunchedEffect(Unit) {
                             // Temporary solution to prevent OOM exception. Text Editor needs to be refactored to handle large files.
                             val maxFileSizeBytes = 15 * 1024 * 1024 // 15 MB limit
@@ -133,10 +154,6 @@ class TextViewerActivity : ViewerActivity() {
                             symbols.addAll(textViewerInstance.updateSymbols())
                         }
 
-                        BackHandler(enabled = textViewerInstance.showSearchPanel) {
-                            textViewerInstance.hideSearchPanel(codeEditor)
-                        }
-
                         LifecycleEventEffect(Lifecycle.Event.ON_RESUME) {
                             textViewerInstance.checkActiveFileValidity(
                                 onSourceReload = {

app/src/main/java/com/raival/compose/file/explorer/screen/viewer/text/TextViewerInstance.kt

@@ -246,6 +246,32 @@ class TextViewerInstance(
         }
     }
 
+    fun showUnsavedChangesWarningDialog(
+        onSaved: () -> Unit,
+        onFailed: (error: Exception) -> Unit,
+        onIgnore: () -> Unit
+    ) {
+        warningDialogProperties.apply {
+            title = globalClass.getString(R.string.warning)
+            message = globalClass.getString(R.string.unsaved_changes)
+            confirmText = globalClass.getString(R.string.save)
+            dismissText = globalClass.getString(R.string.ignore_changes)
+            onDismiss = {
+                onIgnore()
+                showWarningDialog = false
+            }
+            warningDialogProperties.onConfirm = {
+                save(
+                    onSaved = onSaved,
+                    onFailed = onFailed
+                )
+                requireSave = false
+                showWarningDialog = false
+            }
+            showWarningDialog = true
+        }
+    }
+
     fun updateSymbols(): List<SymbolHolder> {
         if (!customSymbolsFile.exists()) {
             customSymbolsFile.writeText(
@@ -266,15 +292,20 @@ class TextViewerInstance(
         return customSymbolHolders.ifEmpty { defaultSymbolHolders }
     }
 
-    fun save(onSaved: () -> Unit, onFailed: () -> Unit) {
+    fun save(onSaved: () -> Unit, onFailed: (error: Exception) -> Unit) {
         isSaving = true
         scope.launch {
-            globalClass.contentResolver.openOutputStream(uri, "wt")?.use { outputStream ->
-                BufferedWriter(OutputStreamWriter(outputStream)).use { writer ->
-                    writer.write(content)
+            try {
+                globalClass.contentResolver.openOutputStream(uri, "wt").use { outputStream ->
+                    BufferedWriter(OutputStreamWriter(outputStream)).use { writer ->
+                        writer.write(content)
+                    }
+                    withContext(Dispatchers.Main) { onSaved().also { isSaving = false } }
                 }
-                withContext(Dispatchers.Main) { onSaved().also { isSaving = false } }
-            } ?: withContext(Dispatchers.Main) { onFailed().also { isSaving = false } }
+            } catch (e: Exception) {
+                logger.logError(e)
+                withContext(Dispatchers.Main) { onFailed(e).also { isSaving = false } }
+            }
         }
     }
 

app/src/main/java/com/raival/compose/file/explorer/screen/viewer/text/ui/ToolbarView.kt

@@ -81,7 +81,13 @@ fun ToolbarView(
                         globalClass.showMsg(R.string.saved)
                         textViewerInstance.requireSave = false
                     },
-                    onFailed = { globalClass.showMsg(R.string.failed_to_save) }
+                    onFailed = {
+                        if (it is SecurityException) {
+                            globalClass.showMsg(R.string.permission_denied)
+                        } else {
+                            globalClass.showMsg(R.string.failed_to_save)
+                        }
+                    }
                 )
             }
         ) {

app/src/main/res/values-es/strings.xml

@@ -444,4 +444,6 @@
     <string name="updating">Actualizando</string>
     <string name="minimize">Minimizar</string>
     <string name="remove_from_bookmarks">Eliminar de marcadores</string>
+    <string name="unsaved_changes">Do you want to save the file?</string>
+    <string name="permission_denied">Permission denied</string>
 </resources>

app/src/main/res/values-fa/strings.xml

@@ -444,4 +444,6 @@
     <string name="updating">در حال به‌روزرسانی</string>
     <string name="minimize">کمینه کردن</string>
     <string name="remove_from_bookmarks">حذف از نشانک‌ها</string>
+    <string name="unsaved_changes">Do you want to save the file?</string>
+    <string name="permission_denied">Permission denied</string>
 </resources>

app/src/main/res/values-fr/strings.xml

@@ -444,4 +444,6 @@
     <string name="updating">Mise à jour</string>
     <string name="minimize">Réduire</string>
     <string name="remove_from_bookmarks">Retirer des favoris</string>
+    <string name="unsaved_changes">Do you want to save the file?</string>
+    <string name="permission_denied">Permission denied</string>
 </resources>

app/src/main/res/values-pl/strings.xml

@@ -444,4 +444,6 @@
     <string name="updating">Aktualizowanie</string>
     <string name="minimize">Minimalizuj</string>
     <string name="remove_from_bookmarks">Usuń z zakładek</string>
+    <string name="unsaved_changes">Do you want to save the file?</string>
+    <string name="permission_denied">Permission denied</string>
 </resources>