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

Uniquify spotify playlist names #351

Closed
wants to merge 2 commits into from
Closed

Uniquify spotify playlist names #351

wants to merge 2 commits into from

Conversation

kingosticks
Copy link
Member

Implements #114 (comment)

  • Appends the playlist owner to those playlists owned by other users i.e. followed playlists
    • Left playlists owned by the current Spotify user alone - presumably they shouldn't appear any different to LocalBackend playlists
  • Appends an incrementing number to playlists with the same name
    • First occurrence of a name is unchanged
    • Looking again I've realised this isn't the place to do this when dealing with multiple backends, perhaps I should remove this bit?

@jodal
Copy link
Member

jodal commented Mar 26, 2013

I also think that identically named playlists isn't the Spotify backend's problem as it identifies playlists uniquely by URI. It's an issue the MPD frontend should solve on its own.

You don't have to do anything to remove the identical name part. I'll just merge the first commit tonight and let the other one be.

@kingosticks
Copy link
Member Author

OK great, thanks!

@jodal
Copy link
Member

jodal commented Mar 30, 2013

Merged in 1c0601e.

@jodal jodal closed this Mar 30, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants