Skip to content

Commit acb8b98

Browse files
committed
Switch more checks to the 200-299 success checks
1 parent f7ba9c6 commit acb8b98

File tree

3 files changed

+14
-14
lines changed

3 files changed

+14
-14
lines changed

SpotifyAPI.Web/Clients/FollowClient.cs

+3-3
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ public async Task<bool> FollowPlaylist(string playlistId, CancellationToken canc
4343
var statusCode = await API
4444
.Put(URLs.PlaylistFollowers(playlistId), null, null, cancel)
4545
.ConfigureAwait(false);
46-
return statusCode == HttpStatusCode.OK;
46+
return HTTPUtil.StatusCodeIsSuccess(statusCode);
4747
}
4848

4949
public async Task<bool> FollowPlaylist(string playlistId, FollowPlaylistRequest request, CancellationToken cancel = default)
@@ -54,7 +54,7 @@ public async Task<bool> FollowPlaylist(string playlistId, FollowPlaylistRequest
5454
var statusCode = await API
5555
.Put(URLs.PlaylistFollowers(playlistId), null, request.BuildBodyParams(), cancel)
5656
.ConfigureAwait(false);
57-
return statusCode == HttpStatusCode.OK;
57+
return HTTPUtil.StatusCodeIsSuccess(statusCode);
5858
}
5959

6060
public Task<FollowedArtistsResponse> OfCurrentUser(CancellationToken cancel = default)
@@ -88,7 +88,7 @@ public async Task<bool> UnfollowPlaylist(string playlistId, CancellationToken ca
8888
var statusCode = await API
8989
.Delete(URLs.PlaylistFollowers(playlistId), null, null, cancel)
9090
.ConfigureAwait(false);
91-
return statusCode == HttpStatusCode.OK;
91+
return HTTPUtil.StatusCodeIsSuccess(statusCode);
9292
}
9393
}
9494
}

SpotifyAPI.Web/Clients/LibraryClient.cs

+10-10
Original file line numberDiff line numberDiff line change
@@ -98,79 +98,79 @@ public async Task<bool> RemoveAlbums(LibraryRemoveAlbumsRequest request, Cancell
9898
Ensure.ArgumentNotNull(request, nameof(request));
9999

100100
var statusCode = await API.Delete(SpotifyUrls.LibraryAlbums(), request.BuildQueryParams(), null, cancel).ConfigureAwait(false);
101-
return statusCode == HttpStatusCode.OK;
101+
return HTTPUtil.StatusCodeIsSuccess(statusCode);
102102
}
103103

104104
public async Task<bool> RemoveShows(LibraryRemoveShowsRequest request, CancellationToken cancel = default)
105105
{
106106
Ensure.ArgumentNotNull(request, nameof(request));
107107

108108
var statusCode = await API.Delete(SpotifyUrls.LibraryShows(), request.BuildQueryParams(), null, cancel).ConfigureAwait(false);
109-
return statusCode == HttpStatusCode.OK;
109+
return HTTPUtil.StatusCodeIsSuccess(statusCode);
110110
}
111111

112112
public async Task<bool> RemoveTracks(LibraryRemoveTracksRequest request, CancellationToken cancel = default)
113113
{
114114
Ensure.ArgumentNotNull(request, nameof(request));
115115

116116
var statusCode = await API.Delete(SpotifyUrls.LibraryTracks(), request.BuildQueryParams(), null, cancel).ConfigureAwait(false);
117-
return statusCode == HttpStatusCode.OK;
117+
return HTTPUtil.StatusCodeIsSuccess(statusCode);
118118
}
119119

120120
public async Task<bool> RemoveEpisodes(LibraryRemoveEpisodesRequest request, CancellationToken cancel = default)
121121
{
122122
Ensure.ArgumentNotNull(request, nameof(request));
123123

124124
var statusCode = await API.Delete(SpotifyUrls.LibraryEpisodes(), request.BuildQueryParams(), null, cancel).ConfigureAwait(false);
125-
return statusCode == HttpStatusCode.OK;
125+
return HTTPUtil.StatusCodeIsSuccess(statusCode);
126126
}
127127

128128
public async Task<bool> SaveAlbums(LibrarySaveAlbumsRequest request, CancellationToken cancel = default)
129129
{
130130
Ensure.ArgumentNotNull(request, nameof(request));
131131

132132
var statusCode = await API.Put(SpotifyUrls.LibraryAlbums(), request.BuildQueryParams(), null, cancel).ConfigureAwait(false);
133-
return statusCode == HttpStatusCode.OK;
133+
return HTTPUtil.StatusCodeIsSuccess(statusCode);
134134
}
135135

136136
public async Task<bool> SaveShows(LibrarySaveShowsRequest request, CancellationToken cancel = default)
137137
{
138138
Ensure.ArgumentNotNull(request, nameof(request));
139139

140140
var statusCode = await API.Put(SpotifyUrls.LibraryShows(), request.BuildQueryParams(), null, cancel).ConfigureAwait(false);
141-
return statusCode == HttpStatusCode.OK;
141+
return HTTPUtil.StatusCodeIsSuccess(statusCode);
142142
}
143143

144144
public async Task<bool> SaveTracks(LibrarySaveTracksRequest request, CancellationToken cancel = default)
145145
{
146146
Ensure.ArgumentNotNull(request, nameof(request));
147147

148148
var statusCode = await API.Put(SpotifyUrls.LibraryTracks(), request.BuildQueryParams(), null, cancel).ConfigureAwait(false);
149-
return statusCode == HttpStatusCode.OK;
149+
return HTTPUtil.StatusCodeIsSuccess(statusCode);
150150
}
151151

152152
public async Task<bool> SaveEpisodes(LibrarySaveEpisodesRequest request, CancellationToken cancel = default)
153153
{
154154
Ensure.ArgumentNotNull(request, nameof(request));
155155

156156
var statusCode = await API.Put(SpotifyUrls.LibraryEpisodes(), request.BuildQueryParams(), null, cancel).ConfigureAwait(false);
157-
return statusCode == HttpStatusCode.OK;
157+
return HTTPUtil.StatusCodeIsSuccess(statusCode);
158158
}
159159

160160
public async Task<bool> SaveAudiobooks(LibrarySaveAudiobooksRequest request, CancellationToken cancel = default)
161161
{
162162
Ensure.ArgumentNotNull(request, nameof(request));
163163

164164
var statusCode = await API.Put(SpotifyUrls.LibraryAudiobooks(), request.BuildQueryParams(), null, cancel).ConfigureAwait(false);
165-
return statusCode == HttpStatusCode.OK;
165+
return HTTPUtil.StatusCodeIsSuccess(statusCode);
166166
}
167167

168168
public async Task<bool> RemoveAudiobooks(LibraryRemoveAudiobooksRequest request, CancellationToken cancel = default)
169169
{
170170
Ensure.ArgumentNotNull(request, nameof(request));
171171

172172
var statusCode = await API.Delete(SpotifyUrls.LibraryAudiobooks(), request.BuildQueryParams(), null, cancel).ConfigureAwait(false);
173-
return statusCode == HttpStatusCode.OK;
173+
return HTTPUtil.StatusCodeIsSuccess(statusCode);
174174
}
175175

176176
public Task<List<bool>> CheckAudiobooks(LibraryCheckAudiobooksRequest request, CancellationToken cancel = default)

SpotifyAPI.Web/Clients/PlaylistsClient.cs

+1-1
Original file line numberDiff line numberDiff line change
@@ -122,7 +122,7 @@ public async Task<bool> ChangeDetails(string playlistId, PlaylistChangeDetailsRe
122122
Ensure.ArgumentNotNull(request, nameof(request));
123123

124124
var statusCode = await API.Put(URLs.Playlist(playlistId), null, request.BuildBodyParams(), cancel).ConfigureAwait(false);
125-
return statusCode == HttpStatusCode.OK;
125+
return HTTPUtil.StatusCodeIsSuccess(statusCode);
126126
}
127127

128128
public Task<SnapshotResponse> ReorderItems(string playlistId, PlaylistReorderItemsRequest request, CancellationToken cancel = default)

0 commit comments

Comments
 (0)