Merge pull request #16133 from nextcloud/fix/cancel-gallery-thumbnail-generation-when-leave-media-tab

fix: stop running gallery background job when user leave

Author: alperozturk96

app/src/main/java/com/nextcloud/client/jobs/gallery/GalleryImageGenerationJob.kt

@@ -40,6 +40,13 @@ class GalleryImageGenerationJob(private val user: User, private val storageManag
         )
         private val activeJobs = WeakHashMap<ImageView, Job>()
 
+        fun cancelAllActiveJobs() {
+            for ((_, job) in activeJobs) {
+                job.cancel()
+            }
+            activeJobs.clear()
+        }
+
         fun removeActiveJob(imageView: ImageView, job: CoroutineScope) {
             if (isActiveJob(imageView, job)) {
                 removeJob(imageView)

app/src/main/java/com/owncloud/android/ui/adapter/GalleryAdapter.kt

@@ -372,4 +372,8 @@ class GalleryAdapter(
     override fun setHighlightedItem(file: OCFile) = Unit
 
     override fun setSortOrder(mFile: OCFile, sortOrder: FileSortOrder) = Unit
+
+    fun cleanup() {
+        ocFileListDelegate.cleanup()
+    }
 }

app/src/main/java/com/owncloud/android/ui/adapter/OCFileListDelegate.kt

@@ -413,6 +413,13 @@ class OCFileListDelegate(
 
     fun cleanup() {
         ioScope.cancel()
+
+        GalleryImageGenerationJob.cancelAllActiveJobs()
+
+        // cancel async tasks from ThumbnailsCacheManager
+        cancelAllPendingTasks()
+
+        Log_OC.d(TAG, "background jobs cancelled")
     }
 
     companion object {

app/src/main/java/com/owncloud/android/ui/fragment/GalleryFragment.java

@@ -120,6 +120,7 @@ public static void setLastMediaItemPosition(Integer position) {
     public void onDestroyView() {
         LocalBroadcastManager.getInstance(requireContext()).unregisterReceiver(refreshSearchEventReceiver);
         setLastMediaItemPosition(null);
+        mAdapter.cleanup();
         super.onDestroyView();
     }