swiftp icon indicating copy to clipboard operation
swiftp copied to clipboard

easy fix to allow selecting the root SD card volume

Open constambeys opened this issue 1 year ago • 5 comments

constambeys avatar May 27 '23 08:05 constambeys

I'm confused. Did this not work on older Android versions? I see you changed some of the recent scoped storage code and Android 13 (or more correctly scoped storage on recent Android versions) done properly will not allow sd card root to be used.

I didn't test your changes here though and there's too many to know what the end result is which is why I'm asking what was going on here.

Xavron avatar Jul 14 '23 20:07 Xavron

Oh I see. You're trying to get ACTION_MANAGE_APP_ALL_FILES_ACCESS_PERMISSION into here. That's something Pieter will want to figure out. I'm not getting involved in this one lol. (Yeah, I know, already am a little :)P

Intent intent = new Intent(Settings.ACTION_MANAGE_APP_ALL_FILES_ACCESS_PERMISSION);

See here why its not allowed by the Play Store: https://support.google.com/googleplay/android-developer/answer/10467955?hl=en

A separate branch would probably be better. Google is very likely to deny approval of it for the app. And scoped storage is already implemented and working as its intended to work by Android. Of course if Pieter can get approval for it that would be cool. That's not my experience with it however lol.

Xavron avatar Jul 14 '23 20:07 Xavron

Seems like the perfect functionality for the f-droid version. The play-store version cannot have this.

ppareit avatar Jul 18 '24 14:07 ppareit

Ok maybe is safer for the user to grant specific directory permissions !!!

constambeys avatar Jul 18 '24 14:07 constambeys

I'll keep this open. I do like this modification. But only for f-droid version.

Your fix allows SwiFTP to be used in the spirit that it was written for. Access to all files on the device.

ppareit avatar Jul 19 '24 06:07 ppareit