Added method in RemotefileDownloadWorker with the implementation details of files

Author: jbsession

app/src/main/java/org/session/libsession/avatars/AvatarCacheCleaner.kt

@@ -51,8 +51,7 @@ class AvatarCacheCleaner @Inject constructor(
             .toSet()
 
         // 5) Delete everything not wanted in cache/remote_files
-        val dir = File(application.cacheDir, "remote_files")
-        val files = dir.listFiles().orEmpty()
+        val files = RemoteFileDownloadWorker.listDownloadedFiles(application)
         var deleted = 0
         for (file in files) {
             if (file !in wantedFiles && file.delete()) deleted++

app/src/main/java/org/thoughtcrime/securesms/attachments/RemoteFileDownloadWorker.kt

@@ -261,6 +261,8 @@ class RemoteFileDownloadWorker @AssistedInject constructor(
         private const val ARG_COMMUNITY_ROOM_ID = "community_room_id"
         private const val ARG_COMMUNITY_FILE_ID = "community_file_id"
 
+        private const val SUBDIRECTORY = "remote_files"
+
         private fun RemoteFile.sha256Hash(): String {
             val hash = MessageDigest.getInstance("SHA-256")
             when (this) {
@@ -278,9 +280,12 @@ class RemoteFileDownloadWorker @AssistedInject constructor(
             return hash.digest().toHexString()
         }
 
+        private fun downloadsDirectory(context: Context): File =
+            File(context.cacheDir, SUBDIRECTORY)
+
         // Deterministically get the file path for the given remote file.
         fun computeFileName(context: Context, remote: RemoteFile): File {
-            return File(context.cacheDir, "remote_files/${remote.sha256Hash()}")
+            return File(downloadsDirectory(context), remote.sha256Hash())
         }
 
         fun cancelAll(context: Context) {
@@ -297,6 +302,12 @@ class RemoteFileDownloadWorker @AssistedInject constructor(
             )
         }
 
+        /** Returns all currently downloaded files (may be empty). */
+        fun listDownloadedFiles(context: Context): List<File> {
+            val directory = downloadsDirectory(context)
+            return directory.listFiles()?.toList().orEmpty()
+        }
+
         /**
          * @param isOldAvatarOf used to indicate that this file is an avatar of a specific address. This
          * information is optional and only used for migration purposes (to move the avatar from