change SaveAsDialog now appends "_{index}" to filename if file already exists

Author: Jan Guegel

CHANGELOG.md

@@ -8,6 +8,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Fixed
 - Fixed files from hidden folders showing up in storage tab browser ([#217])
 
+### Changed
+- SaveAsDialog now appends "_{index}" to filename if file already exists ([#131])
+
 ## [1.3.0] - 2025-09-30
 ### Added
 - Added a separate "Save as" option in the text editor ([#224])
@@ -96,6 +99,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 [#250]: https://github.com/FossifyOrg/File-Manager/issues/250
 [#251]: https://github.com/FossifyOrg/File-Manager/issues/251
 [#267]: https://github.com/FossifyOrg/File-Manager/issues/267
+[#267]: https://github.com/FossifyOrg/File-Manager/issues/131
 
 [Unreleased]: https://github.com/FossifyOrg/File-Manager/compare/1.3.0...HEAD
 [1.3.0]: https://github.com/FossifyOrg/File-Manager/compare/1.2.3...1.3.0

app/src/main/kotlin/org/fossify/filemanager/activities/SaveAsActivity.kt

@@ -58,7 +58,7 @@ class SaveAsActivity : SimpleActivity() {
                                 ?: filename.getMimeType()
                             val inputStream = contentResolver.openInputStream(source)
 
-                            val destinationPath = "$destination/$filename"
+                            val destinationPath = setAvailablePath("$destination/$filename")
                             val outputStream = getFileOutputStreamSync(destinationPath, mimeType, null)!!
                             inputStream!!.copyTo(outputStream)
                             rescanPaths(arrayListOf(destinationPath))
@@ -86,4 +86,32 @@ class SaveAsActivity : SimpleActivity() {
         return filename.replace("[/\\\\<>:\"|?*\u0000-\u001F]".toRegex(), "_")
             .takeIf { it.isNotBlank() } ?: "unnamed_file"
     }
+
+    private fun setAvailablePath(path: String): String {
+        val file = File(path)
+
+        return if (!file.exists()) {
+            path
+        } else {
+            findAvailableNameForExistingFile(file)
+        }
+    }
+
+    private fun findAvailableNameForExistingFile(file: File): String {
+        val parent = file.parent ?: return file.absolutePath
+        val name = file.nameWithoutExtension
+        val ext = if (file.extension.isNotEmpty()) ".${file.extension}" else ""
+
+        var index = 1
+        var newFile: File
+
+        do {
+            val newName = "${name}_${index}$ext"
+            newFile = File(parent, newName)
+            index++
+        } while (newFile.exists())
+
+        return newFile.absolutePath
+    }
+
 }