[Android] Remove useless permission livedata from the PermissionListDialog

Nicolas Pomepuy git at videolan.org
Mon Feb 10 13:38:07 UTC 2025


vlc-android | branch: master | Nicolas Pomepuy <nicolas at videolabs.io> | Mon Feb 10 10:40:03 2025 +0100| [c2f2ba49bb677ab802468b6e613d8955f7d1dc5a] | committer: Nicolas Pomepuy

Remove useless permission livedata from the PermissionListDialog

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

 .../org/videolan/vlc/gui/browser/MainBrowserFragment.kt  |  6 ------
 .../org/videolan/vlc/gui/dialogs/PermissionListDialog.kt | 16 ----------------
 2 files changed, 22 deletions(-)

diff --git a/application/vlc-android/src/org/videolan/vlc/gui/browser/MainBrowserFragment.kt b/application/vlc-android/src/org/videolan/vlc/gui/browser/MainBrowserFragment.kt
index 6740d3736a..12bc04ddd7 100644
--- a/application/vlc-android/src/org/videolan/vlc/gui/browser/MainBrowserFragment.kt
+++ b/application/vlc-android/src/org/videolan/vlc/gui/browser/MainBrowserFragment.kt
@@ -190,12 +190,6 @@ class MainBrowserFragment : BaseFragment(), View.OnClickListener, CtxActionRecei
         val storageBrowserAdapter = BaseBrowserAdapter(storageBrowserContainer)
         localEntry.list.adapter = storageBrowserAdapter
         containerAdapterAssociation[storageBrowserContainer] = Pair(storageBrowserAdapter, localViewModel)
-        PermissionListDialog.hasAnyPermission.observe(viewLifecycleOwner) {
-            if (it) {
-                localViewModel.provider.refresh()
-                favoritesViewModel.provider.refresh()
-            }
-        }
         localViewModel.dataset.observe(viewLifecycleOwner) { list ->
             list?.let {
                 if (Permissions.canReadStorage(requireActivity())) storageBrowserAdapter.update(it)
diff --git a/application/vlc-android/src/org/videolan/vlc/gui/dialogs/PermissionListDialog.kt b/application/vlc-android/src/org/videolan/vlc/gui/dialogs/PermissionListDialog.kt
index d3fd596f19..241924f15d 100644
--- a/application/vlc-android/src/org/videolan/vlc/gui/dialogs/PermissionListDialog.kt
+++ b/application/vlc-android/src/org/videolan/vlc/gui/dialogs/PermissionListDialog.kt
@@ -70,8 +70,6 @@ class PermissionListDialog : VLCBottomSheetDialogFragment() {
         }
 
     companion object {
-        val hasAnyPermission = MutableLiveData<Boolean>().apply { value = false }
-
         fun newInstance(): PermissionListDialog {
             return PermissionListDialog()
         }
@@ -87,15 +85,6 @@ class PermissionListDialog : VLCBottomSheetDialogFragment() {
 
     override fun initialFocusedView(): View = binding.permissionTitle
 
-    override fun onCreate(savedInstanceState: Bundle?) {
-        refreshPermissionLiveData()
-        super.onCreate(savedInstanceState)
-    }
-
-    private fun refreshPermissionLiveData() {
-        hasAnyPermission.postValue(Permissions.hasAllAccess(requireActivity()) || Permissions.hasAnyFileFineAccess(requireActivity()))
-    }
-
     override fun onCreateView(
         inflater: LayoutInflater, container: ViewGroup?,
         savedInstanceState: Bundle?
@@ -115,11 +104,6 @@ class PermissionListDialog : VLCBottomSheetDialogFragment() {
     }
 
     private fun updateStorageState() {
-
-        refreshPermissionLiveData()
-
-        //Notification
-
         binding.notificationPermissionCheck.setImageDrawable(ContextCompat.getDrawable(requireActivity(), if (Permissions.canSendNotifications(requireActivity())) R.drawable.ic_permission_check_checked else R.drawable.ic_permission_check_unchecked))
         binding.notificationPermissionContainer.setOnClickListener {
             if (Permissions.canSendNotifications(requireActivity())) {



More information about the Android mailing list