Android: App allows to go out of ../emulated , but not to come back

Created on 13 Jan 2019 · 13Comments · Source: nextcloud/android

When selecting files, it is possible to go to /. However once being there, it shows no files found and you cannot go back in the tree anymore

approved bug pr exists

Source

telmich

👍1

Most helpful comment

Well, We probably should have a chooser for the storage paths like sd-card, internal, etc. (e.g. like Amaze File Manager), no?

AndyScherzinger on 14 Jan 2019

👍3

All 13 comments

GitMate.io thinks possibly related issues are https://github.com/nextcloud/android/issues/145 (Upload -> no possibility to go back), https://github.com/nextcloud/android/issues/2184 (App crashes on back arrow and menu.), https://github.com/nextcloud/android/issues/163 (Changing back to "all files" should go to latest folder), https://github.com/nextcloud/android/issues/2765 (Social Login App not allowing to log in), and https://github.com/nextcloud/android/issues/971 (Allow to click the entries in Activity to show them in the app).

nextcloud-android-bot on 13 Jan 2019

Maybe we should check if we have read permission on the upper/parent folder and only then allow going to it.

tobiasKaminsky on 14 Jan 2019

Well, We probably should have a chooser for the storage paths like sd-card, internal, etc. (e.g. like Amaze File Manager), no?

AndyScherzinger on 14 Jan 2019

👍3

Maybe also related to https://github.com/nextcloud/android/issues/3419

DecaTec on 14 Jan 2019

Good catch @DecaTec - they probably share the exact same root cause, yes.

AndyScherzinger on 14 Jan 2019

Well, We probably should have a chooser for the storage paths like sd-card, internal, etc. (e.g. like Amaze File Manager), no?

This seems to be the only possible solution, as e.g. on emulator:
/storage/emulated/0/ is internal storage
/storage/0EF5-160/ is sdcard
/storage/emulated/ is permission denied

So going from /storage/emulated/0/ to /storage/ to /storage/0EF5-160/ works, but then you can never get back to internal storage.

Maybe we should check if we have read permission on the upper/parent folder and only then allow going to it.

This will therefore do not work, as then we would be locked in /storage/emulated/0/

@AndyScherzinger do you have an idea where/how this chooser should look like?

tobiasKaminsky on 6 Feb 2019

Good question, we can't use the drawer menu like Amaze but we could use the chooser in the toolbar for that and drop its current functionality?

AndyScherzinger on 6 Feb 2019

Hm. Actually I do like the hierarchically view of folders, as you can then jump easily 2-3 folders up. This would also be nice to have in the regular file listing, in my opinion.

We could have some sort of button that opens a dialog with these "special" folders, also like Camera, etc.
But this feels a bit out of place then…

tobiasKaminsky on 6 Feb 2019

Where would you put this button, also calling @nextcloud/designers for feedback :)

AndyScherzinger on 6 Feb 2019

No real idea…on header next to overflow menu?

tobiasKaminsky on 6 Feb 2019

No real idea…on header next to overflow menu?

Thant would work and likely be the easiest way to implement it (technically speaking)

AndyScherzinger on 6 Feb 2019

Well, We probably should have a chooser for the storage paths like sd-card, internal, etc. (e.g. like Amaze File Manager), no?

Yeees please. :D All this /0/emulated is soo confusing. ;)

Couldn’t this be the very top level, with a listing of "Storage" and "SD card"? If there is only one storage, the view is not needed and the last readable root is "Storage".