[Android] Fix the file deletion only deleting the first one in the file browser

Nicolas Pomepuy git at videolan.org
Thu Mar 20 16:54:47 UTC 2025


vlc-android | branch: master | Nicolas Pomepuy <nicolas at videolabs.io> | Thu Mar 20 13:48:14 2025 +0100| [54224c82f169bd89404be51bebf435ece3920ca1] | committer: Nicolas Pomepuy

Fix the file deletion only deleting the first one in the file browser

Fixes #3172

> https://code.videolan.org/videolan/vlc-android/commit/54224c82f169bd89404be51bebf435ece3920ca1
---

 .../src/org/videolan/vlc/gui/browser/BaseBrowserFragment.kt            | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/application/vlc-android/src/org/videolan/vlc/gui/browser/BaseBrowserFragment.kt b/application/vlc-android/src/org/videolan/vlc/gui/browser/BaseBrowserFragment.kt
index 1783180585..c23df8a143 100644
--- a/application/vlc-android/src/org/videolan/vlc/gui/browser/BaseBrowserFragment.kt
+++ b/application/vlc-android/src/org/videolan/vlc/gui/browser/BaseBrowserFragment.kt
@@ -296,8 +296,7 @@ abstract class BaseBrowserFragment : MediaBrowserFragment<BrowserModel>(), IRefr
         }
         requireActivity().supportFragmentManager.setFragmentResultListener(CONFIRM_DELETE_DIALOG_RESULT, viewLifecycleOwner) { requestKey, bundle ->
             val items: List<MediaWrapper> = bundle.parcelableList(CONFIRM_DELETE_DIALOG_MEDIALIST) ?: listOf()
-            if (items.isNotEmpty()) {
-                val mw = items[0]
+            items.forEach { mw ->
                 val deleteAction = Runnable {
                     lifecycleScope.launch {
                         MediaUtils.deleteItem(requireActivity(), mw) { viewModel.refresh() }



More information about the Android mailing list