Skip to content
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

[BUG] No covers are displayed #37

Closed
Porkepix opened this issue Nov 6, 2023 · 6 comments
Closed

[BUG] No covers are displayed #37

Porkepix opened this issue Nov 6, 2023 · 6 comments
Assignees
Labels
bug Something isn't working

Comments

@Porkepix
Copy link

Porkepix commented Nov 6, 2023

Describe the bug
Gramophone currently shows no covers at all

To Reproduce
Just install Gramophone, probably on older Android (see below) and have songs that should have covers whether it is in metadata or a cover picture in the folder and see that none are displayed

Expected behavior
Covers should be loaded

Smartphone (please complete the following information):

  • Device: Xperia XZ
  • OS: Android Oreo (8.0)
  • Version 1.0.1

Additional context
This might be because of some changes in behavior along Android versions, and maybe some information can be found there: chr56/Phonograph_Plus#65 (comment)

These logs seems related:

Logs
11-06 17:41:03.324  3226  3426 E DatabaseUtils: Writing exception to parcel
11-06 17:41:03.324  3226  3426 E DatabaseUtils: java.lang.SecurityException: External path: /storage/emulated/0/Android/data/com.android.providers.media/albumthumbs/1620562783893: Neither user 10248 nor current process has android.permission.WRITE_EXTERNAL_STORAGE.
11-06 17:41:03.324  3226  3426 E DatabaseUtils: 	at android.app.ContextImpl.enforce(ContextImpl.java:1735)
11-06 17:41:03.324  3226  3426 E DatabaseUtils: 	at android.app.ContextImpl.enforceCallingOrSelfPermission(ContextImpl.java:1767)
11-06 17:41:03.324  3226  3426 E DatabaseUtils: 	at android.content.ContextWrapper.enforceCallingOrSelfPermission(ContextWrapper.java:756)
11-06 17:41:03.324  3226  3426 E DatabaseUtils: 	at com.android.providers.media.MediaProvider.enforceCallingOrSelfPermissionAndAppOps(MediaProvider.java:5680)
11-06 17:41:03.324  3226  3426 E DatabaseUtils: 	at com.android.providers.media.MediaProvider.checkAccess(MediaProvider.java:4604)
11-06 17:41:03.324  3226  3426 E DatabaseUtils: 	at com.android.providers.media.MediaProvider.ensureFileExists(MediaProvider.java:3473)
11-06 17:41:03.324  3226  3426 E DatabaseUtils: 	at com.android.providers.media.MediaProvider.getAlbumArtOutputUri(MediaProvider.java:4887)
11-06 17:41:03.324  3226  3426 E DatabaseUtils: 	at com.android.providers.media.MediaProvider.makeThumbInternal(MediaProvider.java:5020)
11-06 17:41:03.324  3226  3426 E DatabaseUtils: 	at com.android.providers.media.MediaProvider.getThumb(MediaProvider.java:4950)
11-06 17:41:03.324  3226  3426 E DatabaseUtils: 	at com.android.providers.media.MediaProvider.openFile(MediaProvider.java:4503)
11-06 17:41:03.324  3226  3426 E DatabaseUtils: 	at android.content.ContentProvider.openAssetFile(ContentProvider.java:1545)
11-06 17:41:03.324  3226  3426 E DatabaseUtils: 	at android.content.ContentProvider.openTypedAssetFile(ContentProvider.java:1725)
11-06 17:41:03.324  3226  3426 E DatabaseUtils: 	at android.content.ContentProvider.openTypedAssetFile(ContentProvider.java:1791)
11-06 17:41:03.324  3226  3426 E DatabaseUtils: 	at android.content.ContentProvider$Transport.openTypedAssetFile(ContentProvider.java:425)
11-06 17:41:03.324  3226  3426 E DatabaseUtils: 	at android.content.ContentProviderNative.onTransact(ContentProviderNative.java:308)
11-06 17:41:03.324  3226  3426 E DatabaseUtils: 	at android.os.Binder.execTransact(Binder.java:674)
11-06 17:41:03.327  3226  3704 E DatabaseUtils: Writing exception to parcel
11-06 17:41:03.327  3226  3704 E DatabaseUtils: java.lang.SecurityException: External path: /storage/emulated/0/Android/data/com.android.providers.media/albumthumbs/1620562614225: Neither user 10248 nor current process has android.permission.WRITE_EXTERNAL_STORAGE.
11-06 17:41:03.327  3226  3704 E DatabaseUtils: 	at android.app.ContextImpl.enforce(ContextImpl.java:1735)
11-06 17:41:03.327  3226  3704 E DatabaseUtils: 	at android.app.ContextImpl.enforceCallingOrSelfPermission(ContextImpl.java:1767)
11-06 17:41:03.327  3226  3704 E DatabaseUtils: 	at android.content.ContextWrapper.enforceCallingOrSelfPermission(ContextWrapper.java:756)
11-06 17:41:03.327  3226  3704 E DatabaseUtils: 	at com.android.providers.media.MediaProvider.enforceCallingOrSelfPermissionAndAppOps(MediaProvider.java:5680)
11-06 17:41:03.327  3226  3704 E DatabaseUtils: 	at com.android.providers.media.MediaProvider.checkAccess(MediaProvider.java:4604)
11-06 17:41:03.327  3226  3704 E DatabaseUtils: 	at com.android.providers.media.MediaProvider.ensureFileExists(MediaProvider.java:3473)
11-06 17:41:03.327  3226  3704 E DatabaseUtils: 	at com.android.providers.media.MediaProvider.getAlbumArtOutputUri(MediaProvider.java:4887)
11-06 17:41:03.327  3226  3704 E DatabaseUtils: 	at com.android.providers.media.MediaProvider.makeThumbInternal(MediaProvider.java:5020)
11-06 17:41:03.327  3226  3704 E DatabaseUtils: 	at com.android.providers.media.MediaProvider.getThumb(MediaProvider.java:4950)
11-06 17:41:03.327  3226  3704 E DatabaseUtils: 	at com.android.providers.media.MediaProvider.openFile(MediaProvider.java:4503)
11-06 17:41:03.327  3226  3704 E DatabaseUtils: 	at android.content.ContentProvider.openAssetFile(ContentProvider.java:1545)
11-06 17:41:03.327  3226  3704 E DatabaseUtils: 	at android.content.ContentProvider.openTypedAssetFile(ContentProvider.java:1725)
11-06 17:41:03.327  3226  3704 E DatabaseUtils: 	at android.content.ContentProvider.openTypedAssetFile(ContentProvider.java:1791)
11-06 17:41:03.327  3226  3704 E DatabaseUtils: 	at android.content.ContentProvider$Transport.openTypedAssetFile(ContentProvider.java:425)
11-06 17:41:03.327  3226  3704 E DatabaseUtils: 	at android.content.ContentProviderNative.onTransact(ContentProviderNative.java:308)
11-06 17:41:03.327  3226  3704 E DatabaseUtils: 	at android.os.Binder.execTransact(Binder.java:674)
11-06 17:41:03.340  3226  3240 E DatabaseUtils: Writing exception to parcel
11-06 17:41:03.340  3226  3240 E DatabaseUtils: java.lang.SecurityException: External path: /storage/emulated/0/Android/data/com.android.providers.media/albumthumbs/1611423074907: Neither user 10248 nor current process has android.permission.WRITE_EXTERNAL_STORAGE.
11-06 17:41:03.340  3226  3240 E DatabaseUtils: 	at android.app.ContextImpl.enforce(ContextImpl.java:1735)
11-06 17:41:03.340  3226  3240 E DatabaseUtils: 	at android.app.ContextImpl.enforceCallingOrSelfPermission(ContextImpl.java:1767)
11-06 17:41:03.340  3226  3240 E DatabaseUtils: 	at android.content.ContextWrapper.enforceCallingOrSelfPermission(ContextWrapper.java:756)
11-06 17:41:03.340  3226  3240 E DatabaseUtils: 	at com.android.providers.media.MediaProvider.enforceCallingOrSelfPermissionAndAppOps(MediaProvider.java:5680)
11-06 17:41:03.340  3226  3240 E DatabaseUtils: 	at com.android.providers.media.MediaProvider.checkAccess(MediaProvider.java:4604)
11-06 17:41:03.340  3226  3240 E DatabaseUtils: 	at com.android.providers.media.MediaProvider.ensureFileExists(MediaProvider.java:3473)
11-06 17:41:03.340  3226  3240 E DatabaseUtils: 	at com.android.providers.media.MediaProvider.getAlbumArtOutputUri(MediaProvider.java:4887)
11-06 17:41:03.340  3226  3240 E DatabaseUtils: 	at com.android.providers.media.MediaProvider.makeThumbInternal(MediaProvider.java:5020)
11-06 17:41:03.340  3226  3240 E DatabaseUtils: 	at com.android.providers.media.MediaProvider.getThumb(MediaProvider.java:4950)
11-06 17:41:03.340  3226  3240 E DatabaseUtils: 	at com.android.providers.media.MediaProvider.openFile(MediaProvider.java:4503)
11-06 17:41:03.340  3226  3240 E DatabaseUtils: 	at android.content.ContentProvider.openAssetFile(ContentProvider.java:1545)
11-06 17:41:03.340  3226  3240 E DatabaseUtils: 	at android.content.ContentProvider.openTypedAssetFile(ContentProvider.java:1725)
11-06 17:41:03.340  3226  3240 E DatabaseUtils: 	at android.content.ContentProvider.openTypedAssetFile(ContentProvider.java:1791)
11-06 17:41:03.340  3226  3240 E DatabaseUtils: 	at android.content.ContentProvider$Transport.openTypedAssetFile(ContentProvider.java:425)
11-06 17:41:03.340  3226  3240 E DatabaseUtils: 	at android.content.ContentProviderNative.onTransact(ContentProviderNative.java:308)
11-06 17:41:03.340  3226  3240 E DatabaseUtils: 	at android.os.Binder.execTransact(Binder.java:674)

What's strange though is that the application does have the storage permission though.

@AkaneTan AkaneTan added bug Something isn't working and removed need confirm labels Nov 12, 2023
@nift4
Copy link
Collaborator

nift4 commented Dec 3, 2023

@Porkepix
Copy link
Author

Porkepix commented Dec 3, 2023

hi, please try: AkaneTan/Gramophone/files/13538499/Gramophone-1.0.2.bd8ec5-release.apk.zip

I can confirm covers now displays.

Notes on that version though:

  • Curiously by updating with this apk the icon that was placed on homescreen was removed in the process (dunno why?)
  • Most settings menu can be slided too far while being very short but some of them such as Player UI can't be enough and therefore the end of it can't be completely read; I don't remember if it was the case before.
  • And little personal preference for the previous icon where the vinyl player was more obvious I think :p (but maybe there's some consistency choice behind it, or it's to match Google's style guides).

@nift4
Copy link
Collaborator

nift4 commented Dec 3, 2023

please create new issue for settings including screenshots, they have been redesigned recently. the icon was changed because most people did not recognize it as gramophone - the homescreen icon disappearing can't be controlled by the app, it may be related to your launcher settings.

@nift4 nift4 closed this as completed Dec 3, 2023
@Porkepix
Copy link
Author

Porkepix commented Dec 3, 2023

please create new issue for settings including screenshots, they have been redesigned recently. the icon was changed because most people did not recognize it as gramophone - the homescreen icon disappearing can't be controlled by the app, it may be related to your launcher settings.

Well, my launcher is the default one and have no setting for this. I've already seen this before but only once on an application update, so I've a hunch it's a slight difference in a signature or something which leads the launcher to think the application related to the previous icon doesn't exist anymore, but I'm not really sure about this.
I could try to see if I can reproduce it and if there's something in logs, but I've the feeling I'm hoping for too much here.
And… I guess icons aren't recognized as well depending on the people, then.

@nift4
Copy link
Collaborator

nift4 commented Dec 3, 2023

seeing how major the changes from 1.0.1 to the current development version were, I'll write this down as a one-off issue; let me know if it happens again

@Porkepix
Copy link
Author

Porkepix commented Dec 3, 2023

Curiously, I went back to previous release after uninstalling the zip from here to check on other issues, and it asked for permission and did show me the covers. Pretty curious.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants