-
Notifications
You must be signed in to change notification settings - Fork 27
Use Android 5 Storage Access Framework to select root folder #96
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
Use Android 5 Storage Access Framework to select root folder #96
Conversation
|
Works well on my Nexus 4 with Android 5.1 |
|
IMO IFF the user does not select a remote storage provider nothing should become problematic. When I was using the SAF I don't see anything except my internal storage. |
|
I have to think a little bit more about the content provider problem. We might need to find out if many apps have problems accessing files via the content provider. I only noticed VLC so far, but this might be a widespread problem. |
bf0c32b to
77833f4
Compare
|
I'm bumping this up. Since it loses the not ready tag does it mean that this PR is ready for merge? I'd most welcome a word from @StelianMorariu and @laura-almieda on this |
|
I, personally, wouldn't have added the option to change the vaults directory. I would have just saved them directly on the external storage I think disabling this feature is the fastest way to achieve a stable version of the app. Yes, I understand that some users may complain about it (how many ?) . |
|
I don't think that we should remove existing features. From the Android documentation:
And as far as I remember, the problem with a stable and unified API to get the external storrage has only been added by Android 5. |
|
I know it's not guaranteed to be SD Card (and for ensuring that data is not saved on the internal storage it was enough for me). If freezing a feature is not an option, maybe we could look at some file managers and see how they get around the problem. Unfortunately I don't have any phones with SD cards, I'll try with Genymotion to see if it works. |
|
Its not that removing a feature is generally not an option, but I think that users want to be able to select where their files are stored. I also don't have a device with an SD card to test, only the android emulator. But as far as I know, Samsung devices are problematic, but I currently don't know the details. |
Use Android 5 Storage Access Framework to select root folder. Use old method on devices without Lollipop SDK.
77833f4 to
5bff4de
Compare
Use Android 5 Storage Access Framework to select root folder
Use Android 5 Storage Access Framework to select root folder. Use old
method on devices without Lollipop SDK.
This is an untested attempt to close #94. It uses the Storage Access Framework to select any folder as the secrecy root folder. The storage is not accessed by the Storage Access Framework but by the legacy way. I'm not sure if the app really has write right in that folder if it is accessed the legacy way. (Works on my OPO with CM12) If not, we might have to adapt all file operations to the new SAF, if available.