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

Add an Album to Favorites goes wrong #785

Open
BeActivated opened this issue Apr 21, 2022 · 15 comments
Open

Add an Album to Favorites goes wrong #785

BeActivated opened this issue Apr 21, 2022 · 15 comments
Assignees

Comments

@BeActivated
Copy link

Hi,

If there are several albums in the LMS collection with the same Title but by a different Artist it adds always the same Album to the favorites, but that is not the right one (except for the one case).
Example:
I add 'Anthology' from 'The Clean' to the favorites, but In the favorites appears 'Anthology' from 'The Babys'

Kind regards

@mherger
Copy link
Contributor

mherger commented Apr 23, 2022

The favoriting system indeed is a bit limited in this regard. It only ever uses an album title to identify the album. I'll see whether this can be changed.

@mherger mherger self-assigned this Apr 23, 2022
@mherger
Copy link
Contributor

mherger commented Apr 24, 2022

Fixed/improved in 8.3

@BeActivated
Copy link
Author

Thanks,
I can add then now on 'Album Artists' and 'All Artists' but not on 'Albums'.
But unfortunately they are not Playable and cannot be Add to the current playlist, only the one with has alphabetically the first artist.

@mherger
Copy link
Contributor

mherger commented Apr 25, 2022

I'm sorry, I don't understand what you're trying to do? What are you not able to add? This issue is about Albums - which I've tested extensively. It's not about adding artists to favorites.

@BeActivated
Copy link
Author

Sorry I didn't explain it properly.
With 'Album Artists' , 'All Artists' and 'Albums' i mean the left side menus under My Music.
I can add ''Albums with the same Title'' to favorites when i choose an album in 'Album Artists' and 'All Artists' but not in 'Albums'.
When added in favorites, the ''Albums with the same Title'' cannot be Played or Add to the current playlist.
If the ''Album with the same Title'' in favorites has alphabetically the first artist it can be played.

@mherger
Copy link
Contributor

mherger commented Apr 25, 2022

Are you using the Default skin, or Material? If the latter, would it work with Default?

Would you mind sharing your favorites.opml file with me?

Do you have multiple artists for those albums (as shown in the left hand side menu)?

@mherger mherger reopened this Apr 25, 2022
@BeActivated
Copy link
Author

I tested it in the default skin.
Hereby the test Favorites.
Uploading Favorites.zip…

@BeActivated
Copy link
Author

favorites.zip
Favorites

@mherger
Copy link
Contributor

mherger commented Apr 25, 2022

Thanks! Can you tell me which one of these works, and which one doesn't?

If you enabled logging for database.sql you'd get the SQL query resulting from the above information when you try to play an item. Would you mind sharing the server.log.zip file with me?

How does favoriting an album from the albums list fail? Do you get any error message?

@BeActivated
Copy link
Author

BeActivated commented Apr 25, 2022 via email

@BeActivated
Copy link
Author

server.log.zip
FavoritesNotPlaying

@michaelherger
Copy link
Member

Thanks for the additional information. I was able to reproduce some of what you reported. Please give the next build another try. I've improved handling of those favorites.

@BeActivated
Copy link
Author

'Albums with the same Title' in Favorites can now be Played or Added to the current playlist.

'Albums with the same Title' can not be added to Favorites from My Music->Albums

Thank you for your work.

@michaelherger
Copy link
Member

Could you please be a little more precise? Favoriting from the "Albums" menu actually is the only way I tested, and it's definitely working here. What exactly are you doing, what happens, what are you expecting to happen? Would you get the heart icon? Can you describe step by step what you are clicking?

@michaelherger
Copy link
Member

Ok, I've found some more funkiness when drilling down to the album details page. And the heart icon on the album page seems to be suffering from some caching issue. It sometimes wouldn't show the correct state. More work to do.

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

3 participants