Skip to content
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.

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

Ensure that there is files available before playing local content #1358

Merged
merged 2 commits into from
Nov 4, 2024

Conversation

ISO-B
Copy link
Contributor

@ISO-B ISO-B commented Oct 29, 2024

This should fix #105 . This wont work 100% with books that have multiple books.

If there is way to know how many files there should be then this can easily be extended for multi file books.

@advplyr
Copy link
Owner

advplyr commented Nov 3, 2024

This only works after re-opening the app because on first load the cleanLocalLibraryItems function gets ran that updates the LocalLibraryItem data removing any files that are no longer there.

When check is triggered code now checks that are files really exists
@ISO-B
Copy link
Contributor Author

ISO-B commented Nov 4, 2024

Added check that files really exists. Now it prevents playing book if file is removed while app is running.

@advplyr
Copy link
Owner

advplyr commented Nov 4, 2024

Thanks! I wanted to push an additional update in this PR but I don't think permission was given.

Typically this is shown.
image

but it wasn't shown on this PR.

@advplyr advplyr merged commit 38bb5af into advplyr:master Nov 4, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Delete Local Audiobook In Use Causes Android App To Be Unusable
2 participants