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

wrong album artist shown when different track-artists available #13

Closed
Rello opened this issue Jul 20, 2016 · 3 comments
Closed

wrong album artist shown when different track-artists available #13

Rello opened this issue Jul 20, 2016 · 3 comments
Assignees
Milestone

Comments

@Rello
Copy link
Owner

Rello commented Jul 20, 2016

when scanning several files with different artists - but of the same album - the last one is used as album artist.
in this case, the "Various" should be used

@Rello Rello added the bug label Jul 20, 2016
@Rello Rello self-assigned this Jul 20, 2016
Rello pushed a commit that referenced this issue Jul 20, 2016
@Rello Rello added this to the 1.0.1 milestone Jul 20, 2016
@Rello Rello closed this as completed Jul 20, 2016
@Rello Rello reopened this Jul 20, 2016
Rello pushed a commit that referenced this issue Jul 20, 2016
@Rello
Copy link
Owner Author

Rello commented Jul 20, 2016

1st during scanner:

  • when an album-artist is maintained on a track, it is stored as album artist
  • when no album-artist is maintained, the track-artist is stored as album artist

2nd during display:

  • when all album-artists are the same, it is displayed in the Album display
  • when there are different album-artists, "various" is displayed

@youphyun
Copy link

youphyun commented Feb 6, 2017

I am still able to repro this. I have two Artists with the same album named "Greatest Hits".
They did ended up in the same album in the web UI (the last artist in alphabetical order was used for the displayed cover art). I confirm both Artist + Album Artists fields do contain different names for these two albums. As a workaround I added the artist name between brackets to the 2nd Album name. Can this issue be re-opened?

@ghost ghost reopened this Feb 7, 2017
@ghost ghost removed this from the 1.0.1 milestone Feb 7, 2017
@Rello
Copy link
Owner Author

Rello commented Mar 17, 2017

The Album Artist is coming from ID3v2 TPE2 (band).
I can confirm your finding. not all combinations were caught correctly.

it is corrected now - part of next release
screen shot 2017-03-17 at 06 31 16

@xraMsamohT

@Rello Rello added this to the 1.5.0 milestone Mar 17, 2017
Rello added a commit that referenced this issue Mar 17, 2017
ghost pushed a commit that referenced this issue Mar 17, 2017
ghost pushed a commit that referenced this issue Mar 17, 2017
Rello added a commit that referenced this issue Mar 31, 2017
@ghost ghost closed this as completed Mar 31, 2017
@ghost ghost added metadata and removed pending release labels Mar 31, 2017
This issue was closed.
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

2 participants