fix: original asset objects are now cast during first load to avoid l…

…eaking originals where we shouldnt, #977
AlmasB · Feb 28, 2023 · 6ba4715 · 6ba4715
1 parent a65770d
commit 6ba4715
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/fxgl/src/main/kotlin/com/almasb/fxgl/app/services/FXGLAssetLoaderService.kt b/fxgl/src/main/kotlin/com/almasb/fxgl/app/services/FXGLAssetLoaderService.kt
@@ -533,7 +533,7 @@ class FXGLAssetLoaderService : AssetLoaderService() {
                 cachedAssets[cacheKey] = loaded as Any
             }
 
-            loaded
+            data.cast(loaded as Any)
         } catch (e: Exception) {
             log.warning("Failed to load ${loadParams.url}", e)
             data.getDummy()
@@ -696,6 +696,12 @@ private class TextAssetLoader : AssetLoader<List<*>>(
         List::class.java,
         TEXT_DIR
 ) {
+    override fun cast(obj: Any): List<String> {
+        val list = obj as List<String>
+
+        return list.toList()
+    }
+
     override fun load(url: URL): List<String> = url.openStream().bufferedReader().readLines()
 
     override fun getDummy(): List<String> = emptyList()

diff --git a/fxgl/src/test/kotlin/com/almasb/fxgl/app/AssetLoaderServiceTest.kt b/fxgl/src/test/kotlin/com/almasb/fxgl/app/AssetLoaderServiceTest.kt
@@ -185,7 +185,8 @@ class AssetLoaderServiceTest {
 
         assertThat(lines.size, `is`(0))
 
-        assertTrue(assetLoader.loadText("test1.txt") === assetLoader.loadText("test1.txt"))
+        // ensure a new copy of the list is loaded so that original cannot be modified
+        assertTrue(assetLoader.loadText("test1.txt") !== assetLoader.loadText("test1.txt"))
     }
 
     @Test