dany/clapper
1
0
Fork 0
mirror of https://github.com/Rafostar/clapper.git synced 2025-08-31 08:21:59 +02:00
Code Issues Packages Projects Releases Wiki Activity

Do not check Ids length in shuffle repeat mode

Browse Source 
We always have at least 2 playlist items, thus at least 1 Id in array when we reach this point in code
This commit is contained in:
Rafostar
2021-05-01 12:27:56 +02:00
parent 9bb3f999b1
commit a7288adf4c
1 changed files with 7 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
src/playlist.js
View File

@@ -192,7 +192,8 @@ class ClapperPlaylistWidget extends Gtk.ListBox
index++; index++;
} }
if(playlistIds.length) { /* We always have non-empty array here,
* otherwise seek to beginning is performed */
const randomId = playlistIds[ const randomId = playlistIds[
Math.floor(Math.random() * playlistIds.length) Math.floor(Math.random() * playlistIds.length)
]; ];
@@ -200,13 +201,6 @@ class ClapperPlaylistWidget extends Gtk.ListBox
return this.changeActiveRow(randomId); return this.changeActiveRow(randomId);
} }
else {
debug('only one playlist item, playing again');
player.seek(0);
return true;
}
}
if(this.nextTrack()) if(this.nextTrack())
return true; return true;