vanilla-music / vanilla-music-tag-editor

Vanilla Music player Tag Editor plugin
GNU General Public License v3.0
13 stars 6 forks source link

Unable to save new metadata of files on SD card #16

Closed airon90 closed 7 years ago

airon90 commented 7 years ago

The title self-explains the problem. You cannot save new metadata of files on SD card because an exception is risen: screenshot_2017-10-23-23-35-38

Kaned1as commented 7 years ago

How is this different from #3? Which version are you using?

airon90 commented 7 years ago

Ok, didn't see that closed issue. Sorry. I'm using latest F-Droid version of Vanilla Music (1.0.56) and tag editor (1.2.4) on Android 5.1.1

Kaned1as commented 7 years ago

Will take a look this week, thanks

Kaned1as commented 7 years ago

@airon90, can't seem to reproduce the issue. Does Tag Editor ask for SD Card permission? Try clearing Tag Editor app data.

Here's how it goes for me: image

image

image

image

image

image

image

airon90 commented 7 years ago

Thank you for your screenshots! By now it never happened that I had to allow SD access

Kaned1as commented 7 years ago

So, did clearing app data help?

airon90 commented 7 years ago

(I didn't want to comment LOL :D) I'm trying doing it but the options "Clear cache" and "Clear data" are unavailable yet. "Calculating"... I uninstall and reinstall it

airon90 commented 7 years ago

Even uninstalling and reinstalling the app, permission is not asked

Kaned1as commented 7 years ago

OK, I'll try to replicate this exact version of Android.

Kaned1as commented 7 years ago

5.1 doesn't trigger this behaviour, I can write to sdcard without any need to approve SAF storage. Can't find any images of Android 5.1.1 in Google-provided virtual device set.

airon90 commented 7 years ago

Write me if I should do something for you

Kaned1as commented 7 years ago

Does any file browser like Amaze work with these files? Can you write anything with it?

airon90 commented 7 years ago

I can do that with Amaze and I succeded in renaming a file. I tried again with Vanilla but an error appeared again

Kaned1as commented 7 years ago

Took SAF detection from Amaze, pls try this release

airon90 commented 7 years ago

Yeah! It works! :) Thank you very much!

Kaned1as commented 7 years ago

Good :+1: now let's hope it works for everyone else after F-Droid will update!