Skip to content

Fix/dir launcher #14

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
VictorModi merged 2 commits into from
Jan 5, 2026
Merged

Fix/dir launcher #14

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
6dbeb01
Fix openDirLauncher being opened multiple times in some cases
VictorModi Jan 5, 2026
6af929e
Fix repeated OpenDocumentTree launches in NDEF file picker
VictorModi Jan 5, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 11 additions & 2 deletions app/src/main/java/mba/vm/onhit/ui/FragmentNdefFilePicker.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -31,8 +31,11 @@ class FragmentNdefFilePicker : Fragment() {
private var _binding: FragmentNdefFilePickerBinding? = null
private val binding get() = _binding!!

private var isDirLauncherOpened: Boolean = false

private val openDirLauncher =
registerForActivityResult(ActivityResultContracts.OpenDocumentTree()) { uri: Uri? ->
isDirLauncherOpened = false
uri ?: return@registerForActivityResult
val flags = Intent.FLAG_GRANT_READ_URI_PERMISSION or
Intent.FLAG_GRANT_WRITE_URI_PERMISSION
Expand Down Expand Up @@ -80,7 +83,7 @@ class FragmentNdefFilePicker : Fragment() {
refreshFileList()
} ?: run {
Toast.makeText(requireContext(), R.string.toast_select_folder_saving_ndef_files, Toast.LENGTH_SHORT).show()
openDirLauncher.launch(null)
launchOpenDirOnce()
}
}

Expand All @@ -105,7 +108,7 @@ class FragmentNdefFilePicker : Fragment() {
R.string.toast_selected_folder_unavailable,
Toast.LENGTH_SHORT
).show()
openDirLauncher.launch(null)
launchOpenDirOnce()
return
}
val fileList: List<DocumentFile> = dir.listFiles()
Expand Down Expand Up @@ -199,6 +202,12 @@ class FragmentNdefFilePicker : Fragment() {
}
}

private fun launchOpenDirOnce() {
if (isDirLauncherOpened) return
isDirLauncherOpened = true
openDirLauncher.launch(null)
}

override fun onResume() {
super.onResume()
getChosenFolderUri(requireContext())?.let {
Expand Down