fixup java access for plugin info deserialization

Co-authored-by: viztea <[email protected]>
lavalink-devs · Oct 12, 2023 · 5781161 · 5781161
1 parent 0300521
commit 5781161
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 12 deletions.
diff --git a/protocol/src/commonMain/kotlin/dev/arbjerg/lavalink/protocol/v4/loadResult.kt b/protocol/src/commonMain/kotlin/dev/arbjerg/lavalink/protocol/v4/loadResult.kt
@@ -96,7 +96,7 @@ sealed interface LoadResult {
     companion object {
         fun trackLoaded(track: Track) = TrackLoaded(track)
 
-        fun playlistLoaded(playlistInfo: PlaylistInfo, pluginInfo: PluginData, tracks: List<Track>) =
+        fun playlistLoaded(playlistInfo: PlaylistInfo, pluginInfo: JsonObject, tracks: List<Track>) =
             PlaylistLoaded(
                 Playlist(
                     playlistInfo,
@@ -133,9 +133,11 @@ data class PlaylistInfo(
 @Serializable
 data class Playlist(
     val info: PlaylistInfo,
-    val pluginInfo: PluginData,
+    val pluginInfo: JsonObject,
     val tracks: List<Track>
-) : LoadResult.Data
+) : LoadResult.Data {
+    fun <T> deserialize(deserializer: DeserializationStrategy<T>): T = pluginInfo.deserialize(deserializer)
+}
 
 @Serializable
 data class Exception(

diff --git a/protocol/src/commonMain/kotlin/dev/arbjerg/lavalink/protocol/v4/player.kt b/protocol/src/commonMain/kotlin/dev/arbjerg/lavalink/protocol/v4/player.kt
@@ -6,12 +6,11 @@ import kotlinx.serialization.json.JsonObject
 import kotlinx.serialization.serializer
 import kotlin.jvm.JvmInline
 
-@Serializable
-@JvmInline
-value class PluginData(private val obj: JsonObject){
-    inline fun <reified T> deserialize(): T = deserialize(json.serializersModule.serializer<T>())
-    fun <T> deserialize(deserializer: DeserializationStrategy<T>): T = json.decodeFromJsonElement(deserializer, obj)
-}
+inline fun <reified T> JsonObject.deserialize(): T =
+    deserialize(json.serializersModule.serializer<T>())
+
+fun <T> JsonObject.deserialize(deserializer: DeserializationStrategy<T>): T =
+    json.decodeFromJsonElement(deserializer, this)
 
 @Serializable
 @JvmInline
@@ -32,8 +31,10 @@ data class Player(
 data class Track(
     val encoded: String,
     val info: TrackInfo,
-    val pluginInfo: PluginData
-) : LoadResult.Data
+    val pluginInfo: JsonObject
+) : LoadResult.Data {
+    fun <T> deserialize(deserializer: DeserializationStrategy<T>): T = pluginInfo.deserialize(deserializer)
+}
 
 @Serializable
 @JvmInline

diff --git a/protocol/src/commonTest/kotlin/LoadResultSerializerTest.kt b/protocol/src/commonTest/kotlin/LoadResultSerializerTest.kt
@@ -151,7 +151,7 @@ class LoadResultSerializerTest {
             loadType shouldBe ResultStatus.PLAYLIST
             assertIs<LoadResult.PlaylistLoaded>(this)
             data {
-                pluginInfo shouldBe JsonObject(emptyMap())
+                pluginInfo shouldBe emptyMap()
                 info {
                     name shouldBe "Example YouTube Playlist"
                     selectedTrack shouldBe 3