Use Documents directory for generic files on Android 9- to maintain consistency with existing behavior

Verryx-02 · Verryx-02 · commit c480b9a09682 · 2025-08-04T22:12:57.000+02:00
diff --git a/app/src/main/java/org/thoughtcrime/securesms/util/SaveAttachmentUtil.kt b/app/src/main/java/org/thoughtcrime/securesms/util/SaveAttachmentUtil.kt
@@ -226,7 +226,7 @@ object SaveAttachmentUtil {
       contentType.startsWith("video/") -> File(Environment.getExternalStoragePublicDirectory(Environment.DIRECTORY_MOVIES), "Signal")
       contentType.startsWith("audio/") -> File(Environment.getExternalStoragePublicDirectory(Environment.DIRECTORY_MUSIC), "Signal")
       contentType.startsWith("image/") -> File(Environment.getExternalStoragePublicDirectory(Environment.DIRECTORY_PICTURES), "Signal")
-      else -> File(Environment.getExternalStoragePublicDirectory(Environment.DIRECTORY_DOWNLOADS), "Signal")
+      else -> File(Environment.getExternalStoragePublicDirectory(Environment.DIRECTORY_DOCUMENTS), "Signal")
     }
 
     return storage?.let { ensureExternalPath(storage) }?.absolutePath

Original file line number	Diff line number	Diff line change
`@@ -226,7 +226,7 @@ object SaveAttachmentUtil {`
`226`	`226`	`contentType.startsWith("video/") -> File(Environment.getExternalStoragePublicDirectory(Environment.DIRECTORY_MOVIES), "Signal")`
`227`	`227`	`contentType.startsWith("audio/") -> File(Environment.getExternalStoragePublicDirectory(Environment.DIRECTORY_MUSIC), "Signal")`
`228`	`228`	`contentType.startsWith("image/") -> File(Environment.getExternalStoragePublicDirectory(Environment.DIRECTORY_PICTURES), "Signal")`
`229`		`- else -> File(Environment.getExternalStoragePublicDirectory(Environment.DIRECTORY_DOWNLOADS), "Signal")`
	`229`	`+ else -> File(Environment.getExternalStoragePublicDirectory(Environment.DIRECTORY_DOCUMENTS), "Signal")`
`230`	`230`	`}`
`231`	`231`
`232`	`232`	`return storage?.let { ensureExternalPath(storage) }?.absolutePath`