Filter out empty voice states when transferring

Author: duncte123

src/main/kotlin/dev/arbjerg/lavalink/client/LavalinkClient.kt

@@ -6,6 +6,7 @@ import dev.arbjerg.lavalink.client.loadbalancing.builtin.DefaultLoadBalancer
 import dev.arbjerg.lavalink.client.event.ClientEvent
 import dev.arbjerg.lavalink.client.player.LavalinkPlayer
 import dev.arbjerg.lavalink.internal.ReconnectTask
+import dev.arbjerg.lavalink.protocol.v4.VoiceState
 import reactor.core.Disposable
 import reactor.core.publisher.Flux
 import reactor.core.publisher.Sinks
@@ -128,6 +129,7 @@ class LavalinkClient(val userId: Long) : Closeable, Disposable {
         val orphans = findOrphanedPlayers()
 
         orphans.mapNotNull { linkMap[it.guildId] }
+            .filter { !it.cachedPlayer?.voiceState.isEmpty() } // TODO: do some logging to mark them as skipped?
             .forEach { link ->
                 link.transferNode(node)
             }
@@ -225,4 +227,12 @@ class LavalinkClient(val userId: Long) : Closeable, Disposable {
                 }
             }
     }
+
+    private fun VoiceState?.isEmpty(): Boolean {
+        if (this == null) {
+            return true
+        }
+
+        return this.token.isBlank() || this.endpoint.isBlank() || this.sessionId.isBlank();
+    }
 }