lostecho/ncspot
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix slicing of playlists when saving large lists

Browse Source 
playlist updates need to be submitted in batches of max 100 previously, it would
submit the entire rest after submitting the first 100 via
user_playlist_replace_tracks, which was rejected by the server.

fixes #99
This commit is contained in:
Henrik Friedrichsen
2019-10-12 19:28:18 +02:00
parent d01dad9a57
commit 4defded546
1 changed files with 23 additions and 21 deletions
Download Patch File Download Diff File Expand all files Collapse all files

44
src/spotify.rs
View File

@@ -430,30 +430,32 @@ impl Spotify {
None
};
match self.api_with_retry(|api| api.user_playlist_replace_tracks(&self.user, id, &tracks)) {
Some(()) => {
debug!("saved {} tracks to playlist {}", tracks.len(), id);
if let Some(()) = self.api_with_retry(|api| api.user_playlist_replace_tracks(&self.user, id, &tracks)) {
debug!("saved {} tracks to playlist {}", tracks.len(), id);
while let Some(ref mut tracks) = remainder.clone() {
// grab the next set of 100 tracks
remainder = if tracks.len() > 100 {
Some(tracks.split_off(100))
} else {
None
};
// send the remaining tracks in batches of max 100
while let Some(ref mut tracks) = remainder.clone() {
let result = self.api_with_retry(|api| {
api.user_playlist_add_tracks(&self.user, id, &tracks, None)
});
if result.is_some() {
// grab the next set of tracks
remainder = if tracks.len() > 100 {
Some(tracks.split_off(100))
} else {
None
};
} else {
error!("error saving tracks to playlists {}", id);
}
debug!("adding another {} tracks to playlist", tracks.len());
let result = self.api_with_retry(|api| {
api.user_playlist_add_tracks(&self.user, id, &tracks, None)
});
if result.is_some() {
debug!("{} tracks successfully added", tracks.len());
} else {
error!("error saving tracks to playlists {}", id);
return;
}
}
None => {
error!("error saving tracks to playlist {}", id);
}
}
else {
error!("error saving tracks to playlist {}", id);
}
}