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

saveFile method implemented for android platform #1396

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

vinxxe
Copy link

@vinxxe vinxxe commented Oct 26, 2023

I implemented a saveFile method for the android platform because I needed it.
I don't know if it is interesting, I think that you already know how to do it but maybe violates some android security rule and that's the reason why you didn't add it.
If this is the case just ignore it.

The only way I found to implement the method is by removing the file that the saveFile dialog creates because that file cannot be written by the calling thread. By removing the 0 length created file and returning the path, the calling thread is able to open the file, provided that the file is in an accessible folder.

now the saveFile method for android uses the fileName as
input file from where it reads the content to be written
in the file that the user is going to choose. this way
there is no need to ask for permission and no need to
delete the 0 byte file created by the save filePicker
window. From the android app ONLY you need to write
the file in a temporary file before calling the saveFile()
method. then you pass the temp file to the saveMethod(),
the user choose the file where the content of the temp
file will be saved and the temp file is written to the
file chosen by the user
@miguelpruivo miguelpruivo added improvement Improvement needed on a specific feature android Issue applies to Android platform labels Mar 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
android Issue applies to Android platform improvement Improvement needed on a specific feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants