fix(player): "Now Playing" notification could be sent multiple times in some cases

simonzsh · simonzsh · commit 12c34069d460 · 2024-06-28T01:09:36.000+02:00
diff --git a/player/playSong.js b/player/playSong.js
@@ -27,4 +27,5 @@ module.exports = async (client, queue, song) => {
 	// Send advanced notification if queue is not already playing before
 	if (!playing && (!song.playlist || song == song.playlist.songs[0]))
 		client.sendAdvancedNotification(destination, `${lang.MESSAGE_NOW_PLAYING}`, `${song.name}`, song.thumbnail, { editReply: true });
+	song.metadata.playing = true;
 }

Original file line number	Diff line number	Diff line change
`@@ -27,4 +27,5 @@ module.exports = async (client, queue, song) => {`
`27`	`27`	`// Send advanced notification if queue is not already playing before`
`28`	`28`	`if (!playing && (!song.playlist \|\| song == song.playlist.songs[0]))`
`29`	`29`	client.sendAdvancedNotification(destination, `${lang.MESSAGE_NOW_PLAYING}`, `${song.name}`, song.thumbnail, { editReply: true });
	`30`	`+ song.metadata.playing = true;`
`30`	`31`	`}`