refactor: Media check dialog

- use ViewModel to handle the result in the dialog

Author: criticalAY

AnkiDroid/src/main/java/com/ichi2/anki/DeckPicker.kt

@@ -130,13 +130,14 @@ import com.ichi2.anki.dialogs.EmptyCardsDialogFragment
 import com.ichi2.anki.dialogs.ImportDialog.ImportDialogListener
 import com.ichi2.anki.dialogs.ImportFileSelectionFragment.ApkgImportResultLauncherProvider
 import com.ichi2.anki.dialogs.ImportFileSelectionFragment.CsvImportResultLauncherProvider
-import com.ichi2.anki.dialogs.MediaCheckDialog
-import com.ichi2.anki.dialogs.MediaCheckDialog.MediaCheckDialogListener
 import com.ichi2.anki.dialogs.SyncErrorDialog
 import com.ichi2.anki.dialogs.SyncErrorDialog.Companion.newInstance
 import com.ichi2.anki.dialogs.SyncErrorDialog.SyncErrorDialogListener
 import com.ichi2.anki.dialogs.customstudy.CustomStudyDialog
 import com.ichi2.anki.dialogs.customstudy.CustomStudyDialog.CustomStudyAction
+import com.ichi2.anki.dialogs.mediacheck.ConfirmMediaCheckDialog
+import com.ichi2.anki.dialogs.mediacheck.MediaCheckResultDialog
+import com.ichi2.anki.dialogs.viewmodel.MediaCheckViewModel
 import com.ichi2.anki.export.ActivityExportingDelegate
 import com.ichi2.anki.export.ExportDialogFragment
 import com.ichi2.anki.export.ExportDialogsFactory
@@ -153,7 +154,6 @@ import com.ichi2.anki.preferences.PreferencesActivity
 import com.ichi2.anki.preferences.sharedPrefs
 import com.ichi2.anki.receiver.SdCardReceiver
 import com.ichi2.anki.servicelayer.ScopedStorageService
-import com.ichi2.anki.servicelayer.checkMedia
 import com.ichi2.anki.settings.Prefs
 import com.ichi2.anki.snackbar.BaseSnackbarBuilderProvider
 import com.ichi2.anki.snackbar.SnackbarBuilder
@@ -165,13 +165,11 @@ import com.ichi2.anki.worker.SyncMediaWorker
 import com.ichi2.anki.worker.SyncWorker
 import com.ichi2.anki.worker.UniqueWorkNames
 import com.ichi2.annotations.NeedsTest
-import com.ichi2.async.deleteMedia
 import com.ichi2.compat.CompatHelper.Companion.getSerializableCompat
 import com.ichi2.compat.CompatHelper.Companion.sdkVersion
 import com.ichi2.libanki.ChangeManager
 import com.ichi2.libanki.DeckId
 import com.ichi2.libanki.Decks
-import com.ichi2.libanki.MediaCheckResult
 import com.ichi2.libanki.exception.ConfirmModSchemaException
 import com.ichi2.libanki.sched.DeckNode
 import com.ichi2.libanki.utils.TimeManager
@@ -244,7 +242,6 @@ open class DeckPicker :
     StudyOptionsListener,
     SyncErrorDialogListener,
     ImportDialogListener,
-    MediaCheckDialogListener,
     OnRequestPermissionsResultCallback,
     ChangeManager.Subscriber,
     SyncCompletionListener,
@@ -256,6 +253,8 @@ open class DeckPicker :
     ExportDialogsFactoryProvider {
     val viewModel: DeckPickerViewModel by viewModels()
 
+    private val mediaCheckViewModel: MediaCheckViewModel by viewModels()
+
     // Short animation duration from system
     private var shortAnimDuration = 0
     private lateinit var deckPickerContent: RelativeLayout
@@ -679,6 +678,10 @@ open class DeckPicker :
         viewModel.deckDeletedNotification.launchCollectionInLifecycleScope(::onDeckDeleted)
         viewModel.emptyCardsNotification.launchCollectionInLifecycleScope(::onCardsEmptied)
         viewModel.flowOfDeckCountsChanged.launchCollectionInLifecycleScope(::onDeckCountsChanged)
+
+        mediaCheckViewModel.startMediaCheck.launchCollectionInLifecycleScope { startMediaCheck() }
+        mediaCheckViewModel.tagMissing.launchCollectionInLifecycleScope { tagMissing() }
+        mediaCheckViewModel.deleteUnused.launchCollectionInLifecycleScope { deleteUnused() }
     }
 
     private val onReceiveContentListener =
@@ -1181,7 +1184,7 @@ open class DeckPicker :
             }
             R.id.action_check_media -> {
                 Timber.i("DeckPicker:: Check media button pressed")
-                showMediaCheckDialog(MediaCheckDialog.Type.DIALOG_CONFIRM_MEDIA_CHECK)
+                ConfirmMediaCheckDialog().show(supportFragmentManager, "ConfirmMediaCheckDialog")
                 return true
             }
             R.id.action_empty_cards -> {
@@ -1495,7 +1498,7 @@ open class DeckPicker :
             }
             KeyEvent.KEYCODE_M -> {
                 Timber.i("Check media from keypress")
-                showMediaCheckDialog(MediaCheckDialog.Type.DIALOG_CONFIRM_MEDIA_CHECK)
+                ConfirmMediaCheckDialog().show(supportFragmentManager, "ConfirmMediaCheckDialog")
                 return true
             }
             KeyEvent.KEYCODE_E -> {
@@ -1812,17 +1815,6 @@ open class DeckPicker :
         }
     }
 
-    override fun showMediaCheckDialog(dialogType: MediaCheckDialog.Type) {
-        showAsyncDialogFragment(MediaCheckDialog.newInstance(dialogType))
-    }
-
-    override fun showMediaCheckDialog(
-        dialogType: MediaCheckDialog.Type,
-        checkList: MediaCheckResult,
-    ) {
-        showAsyncDialogFragment(MediaCheckDialog.newInstance(dialogType, checkList))
-    }
-
     /**
      * Show a specific sync error dialog
      * @param dialogType id of dialog to show
@@ -1895,41 +1887,47 @@ open class DeckPicker :
         handleDatabaseCheck()
     }
 
-    /**
-     * Schedules a background job to find missing, unused and invalid media files.
-     * Shows a progress dialog while operation is running.
-     * When check is finished a dialog box shows number of missing, unused and invalid media files.
-     *
-     * If has the storage permission, job is scheduled, otherwise storage permission is asked first.
-     */
-    override fun mediaCheck() {
+    private fun startMediaCheck() {
         launchCatchingTask {
-            val mediaCheckResult = checkMedia()
-            showMediaCheckDialog(MediaCheckDialog.Type.DIALOG_MEDIA_CHECK_RESULTS, mediaCheckResult)
+            withProgress(R.string.check_media_message) {
+                mediaCheckViewModel.checkMedia().join()
+            }
+
+            MediaCheckResultDialog().show(
+                supportFragmentManager,
+                "MediaCheckResultDialog",
+            )
         }
     }
 
-    override fun deleteUnused(unused: List<String>) {
+    override fun mediaCheck() {
+        ConfirmMediaCheckDialog().show(supportFragmentManager, "ConfirmMediaCheckDialog")
+    }
+
+    private fun deleteUnused() {
+        Timber.d("DeckPicker:: Deleting unused media")
         launchCatchingTask {
-            // Number of deleted files
-            val noOfDeletedFiles =
-                withProgress(resources.getString(R.string.delete_media_message)) {
-                    withCol { deleteMedia(this@withCol, unused) }
-                }
-            showSimpleMessageDialog(
-                title = resources.getString(R.string.delete_media_result_title),
-                message = resources.getQuantityString(R.plurals.delete_media_result_message, noOfDeletedFiles, noOfDeletedFiles),
-            )
+            withProgress(resources.getString(R.string.delete_media_message)) {
+                mediaCheckViewModel.deleteUnusedMedia().join()
+
+                showSimpleMessageDialog(
+                    title = resources.getString(R.string.delete_media_result_title),
+                    message =
+                        resources.getQuantityString(
+                            R.plurals.delete_media_result_message,
+                            mediaCheckViewModel.deletedFiles,
+                            mediaCheckViewModel.deletedFiles,
+                        ),
+                )
+            }
         }
     }
 
-    override fun tagMissing(missingMediaNotes: List<Long>?) {
-        if (missingMediaNotes == null) return
-
+    private fun tagMissing() {
         Timber.d("DeckPicker:: Adding missing media tag")
         launchCatchingTask {
-            withCol {
-                tags.bulkAdd(missingMediaNotes, TR.mediaCheckMissingMediaTag())
+            withProgress(getString(R.string.adding_missing_tag)) {
+                mediaCheckViewModel.tagMissing(TR.mediaCheckMissingMediaTag()).join()
             }
         }
     }

AnkiDroid/src/main/java/com/ichi2/anki/dialogs/DialogHandler.kt

@@ -119,7 +119,6 @@ abstract class DialogHandlerMessage protected constructor(
                 WhichDialogHandler.MSG_SHOW_COLLECTION_IMPORT_REPLACE_DIALOG -> ImportUtils.CollectionImportReplace.fromMessage(message)
                 WhichDialogHandler.MSG_SHOW_COLLECTION_IMPORT_ADD_DIALOG -> ImportUtils.CollectionImportAdd.fromMessage(message)
                 WhichDialogHandler.MSG_SHOW_SYNC_ERROR_DIALOG -> SyncErrorDialog.SyncErrorDialogMessageHandler.fromMessage(message)
-                WhichDialogHandler.MSG_SHOW_MEDIA_CHECK_COMPLETE_DIALOG -> MediaCheckDialog.MediaCheckCompleteDialog.fromMessage(message)
                 WhichDialogHandler.MSG_SHOW_DATABASE_ERROR_DIALOG -> DatabaseErrorDialog.ShowDatabaseErrorDialog.fromMessage(message)
                 WhichDialogHandler.MSG_SHOW_ONE_WAY_SYNC_DIALOG -> OneWaySyncDialog.fromMessage(message)
                 WhichDialogHandler.MSG_DO_SYNC -> IntentHandler.Companion.DoSync()
@@ -136,7 +135,6 @@ abstract class DialogHandlerMessage protected constructor(
         MSG_SHOW_COLLECTION_IMPORT_REPLACE_DIALOG(1),
         MSG_SHOW_COLLECTION_IMPORT_ADD_DIALOG(2),
         MSG_SHOW_SYNC_ERROR_DIALOG(3),
-        MSG_SHOW_MEDIA_CHECK_COMPLETE_DIALOG(5),
         MSG_SHOW_DATABASE_ERROR_DIALOG(6),
         MSG_SHOW_ONE_WAY_SYNC_DIALOG(7),
         MSG_DO_SYNC(8),