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

Display ALBUM_ARTIST in Album list view #652

Open
4 tasks
telematoxx opened this issue Aug 17, 2017 · 8 comments
Open
4 tasks

Display ALBUM_ARTIST in Album list view #652

telematoxx opened this issue Aug 17, 2017 · 8 comments

Comments

@telematoxx
Copy link

  • Vanilla Music Version (Settings -> About): 1.0.53
  • Android Version: 6.0.1
  • Phone vendor / model: Fairphone 2
  • Application was downloaded from (use [x]):
    • Play Store
    • [ x] F-Droid
    • Other (Nightly Build)
  • Did this work in a previous version?
    • Yes
    • No
    • [ x] NA / Don't know.

Description

In Albums list view a user would expect Album related information. Currently it seems that always the ARTIST information of one the albums tracks is displayed. This information is track related - thats fine in case that only one artist contributed to the album but for compilations this is misleading. If ALBUM_ARTIST information is available, it should be used for display/sorting inside Albums list view.

Steps to reproduce the issue

Add an album with different artists to the library, where each track's ALBUM_ARTIST tag information is set to value "Various Artists (test)". Switch to Album list view: Album title will be displayed along with a single track related Artist. "Various Artists (test)" is not displayed.

Screenshot / Video (If applicable)

...

@adrian-bl
Copy link
Member

Every album has a 'primary artist' which is what we display. The primary artist is just choosen from whatever we see as artist while creating the album entry.

I wonder if it would make sense to use Album_artist in this case (and fall back to artist if not present) or to have an option in the media library settings to choose the behavior.

@telematoxx
Copy link
Author

What is the different meaning of 'primary artist' compared to ALBUM_ARTIST tag field?

@adrian-bl
Copy link
Member

primary artist is an 'internal' concept of our database schema:
It is used to link an album entry to an artist entry

@Kaned1as
Copy link
Member

Do we need media-db schema update to support this?

@adrian-bl
Copy link
Member

Probably not:

We join PRIMARY_ARTIST_ID with TABLE_CONTRIBUTORS, which holds artists, album artists and composers, so it doesn't really matter.

However: there might be some side effects: queries operating on albums (eg: 'More from Artist' in the Albums tab) would suddenly link to album_artists which may not be listed in the artists tab.

@telematoxx
Copy link
Author

Regarding "Show more from": If this operation does not switch the tab and the result list contains at least the list entry from that the operation has been invoked, i would not be confused.

@ddv239
Copy link
Contributor

ddv239 commented Jan 28, 2018

Regarding the "primary artist" field: my recent change that is not merged yet (#721) makes that field always contain the "album artist" value which is also always populated form the "artist" field if not present in the file's tags. This may already have fixed this issue completely, let me know if you think this is the case or not.

@telematoxx
Copy link
Author

It sounds to me that this change would solve this issue. Thanks.
(Maybe the product owner can have a look at it - there are some possible side effects and related issues in this Diskussion).

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

No branches or pull requests

4 participants