Mirrors/webamp
1
0
Fork 0
mirror of https://github.com/captbaritone/webamp.git synced 2026-07-21 09:09:01 +00:00
Code Issues Projects Releases Wiki Activity

Don't repeat tracks when going to next random track

Browse source
This commit is contained in:
Jordan Eldredge 2017-11-29 19:35:02 -08:00
parent 3bbf3016ac
commit 567a40ef50
1 changed files with 8 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

12
js/actionCreators.js
View file

@ -49,10 +49,14 @@ import {
function playRandomTrack() {
return (dispatch, getState) => {
const { playlist: { trackOrder } } = getState();
const nextIndex = Math.floor(trackOrder.length * Math.random());
// TODO: Ensure we don't repeat the same track.
dispatch({ type: PLAY_TRACK, id: trackOrder[nextIndex] });
const { playlist: { trackOrder, currentTrack } } = getState();
let nextId;
do {
nextId = trackOrder[Math.floor(trackOrder.length * Math.random())];
} while (nextId === currentTrack && trackOrder.length > 1);
// TODO: Sigh... Technically, we should detect if we are looping only repeat if we are.
// I think this would require pre-computing the "random" order of a playlist.
dispatch({ type: PLAY_TRACK, id: nextId });
};
}