[GH-ISSUE #33] Naming of playlist-methods #16

Closed
opened 2026-02-27 19:25:38 +03:00 by kerem · 1 comment
Owner

Originally created by @jwilsson on GitHub (May 2, 2015).
Original GitHub issue: https://github.com/jwilsson/spotify-web-api-php/issues/33

We've got a bunch of playlist related methods:
addUserPlaylistTracks()
createUserPlaylist()
deletePlaylistTracks()
followPlaylist()
getCategoryPlaylists()
getFeaturedPlaylists()
getUserPlaylist()
getUserPlaylists()
getUserPlaylistTracks()
reorderPlaylistTracks()
replacePlaylistTracks()
unfollowPlaylist()
updateUserPlaylist()

Some include "user", some don't. Some shouldn't, like getCategoryPlaylists() and getFeaturedPlaylists() but I've included them for the sake of completeness.

Now, I think the methods should have a consistent naming. I.e. either all of them should include "user" or none of them should include "user". Perhaps it's best to include "user" in all of them since they operate on a user's playlist but what do others think?

Let the discussion begin!

Originally created by @jwilsson on GitHub (May 2, 2015). Original GitHub issue: https://github.com/jwilsson/spotify-web-api-php/issues/33 We've got a bunch of playlist related methods: `addUserPlaylistTracks()` `createUserPlaylist()` `deletePlaylistTracks()` `followPlaylist()` `getCategoryPlaylists()` `getFeaturedPlaylists()` `getUserPlaylist()` `getUserPlaylists()` `getUserPlaylistTracks()` `reorderPlaylistTracks()` `replacePlaylistTracks()` `unfollowPlaylist()` `updateUserPlaylist()` Some include "user", some don't. Some shouldn't, like `getCategoryPlaylists()` and `getFeaturedPlaylists()` but I've included them for the sake of completeness. Now, I think the methods should have a consistent naming. I.e. either all of them should include "user" or none of them should include "user". Perhaps it's best to include "user" in all of them since they operate on a user's playlist but what do others think? Let the discussion begin!
kerem 2026-02-27 19:25:38 +03:00
Author
Owner

@jwilsson commented on GitHub (Jul 6, 2015):

deletePlaylistTracks(), reorderPlaylistTracks(), and replacePlaylistTracks() have been renamed to include "User" in their names. See 8768328aee, 2ce8fc51cc, and 6362510344, respectively.

followPlaylist() and unfollowPlaylist() will be left as-is for now.

<!-- gh-comment-id:118850929 --> @jwilsson commented on GitHub (Jul 6, 2015): `deletePlaylistTracks()`, `reorderPlaylistTracks()`, and `replacePlaylistTracks()` have been renamed to include "User" in their names. See 8768328aeeca1a82ebf652ad0ee557329ded6783, 2ce8fc51cc2a42d6b9055bc6ced1a0f777400486, and 6362510344f746a37a75612d3f41030a60d81f2d, respectively. `followPlaylist()` and `unfollowPlaylist()` will be left as-is for now.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/spotify-web-api-php#16
No description provided.