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

Explorer navigation bar not handling symlink properly #416

Closed
ghost opened this Issue Jun 15, 2016 · 4 comments

Comments

1 participant
@ghost

ghost commented Jun 15, 2016

root: yes. Rooted with SuperSU v2.74
SD Maid: 4.2.5

Let's say you have an app like smallbrowser (I'm starting hating this app), in AppControl the path points to /data/user/0/com.sonymobile.smallbrowser. If you open the directory in Explorer (by clicking on it from the AppControl) you will have /data/user/0/com.sonymobile.smallbrowser in the navigation bar. This allow you to access /data/user/0 (by clicking 0 in the navigation bar) which returns "0 items" as 0 is a symlink in /data/user which points to /data/data

@d4rken

This comment has been minimized.

Owner

d4rken commented Jun 15, 2016

After clicking the path in AppControl and the Explorer loading, the initial folder /data/user/0/com.sonymobile.smallbrowser shows correctly though, right?
What happens if you navigate down in that folder e.g. into /files/, does that work?

@randomname1234 Loving your reports, thanks!

@ghost

This comment has been minimized.

ghost commented Jun 15, 2016

After clicking the path in AppControl and the Explorer loading, the initial folder /data/user/0/com.sonymobile.smallbrowser shows correctly though, right?

yes

What happens if you navigate down in that folder e.g. into /files/, does that work?

yes

The problem is when I try to access the symlink path, it doesn't redirect me to the path the symlink points to

@d4rken

This comment has been minimized.

Owner

d4rken commented Jun 15, 2016

Maybe it's as simple as if(file.isSymlink()), I hope... Thanks should be enough to get me started on this.

@d4rken

This comment has been minimized.

Owner

d4rken commented Jun 24, 2016

Wasn't as easy as I thought, but could improve the whole logic a bit in this change and it seems to also have improved navigation around /storage/*

Committed for next v4.2.7

@d4rken d4rken closed this Jun 24, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment