fix: make sure DataFetcher only runs once at a time

Skytils · May 16, 2024 · 6735825 · 6735825
1 parent 205edae
commit 6735825
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 3 deletions.
diff --git a/src/main/kotlin/gg/skytils/skytilsmod/core/DataFetcher.kt b/src/main/kotlin/gg/skytils/skytilsmod/core/DataFetcher.kt
@@ -51,6 +51,8 @@ import kotlin.concurrent.fixedRateTimer
 import kotlin.reflect.jvm.jvmName
 
 object DataFetcher {
+    private var job: Job? = null
+
     private fun loadData(): Job {
         return Skytils.IO.launch {
             try {
@@ -213,8 +215,13 @@ object DataFetcher {
         }
 
     @JvmStatic
-    fun reloadData() =
-        loadData()
+    fun reloadData() {
+        if (job?.isActive != true) {
+            job = loadData()
+        } else {
+            UChat.chat("$failPrefix §cData fetch requested while already fetching!")
+        }
+    }
 
     internal fun preload() {}
 

diff --git a/src/main/kotlin/gg/skytils/skytilsmod/features/impl/misc/SummonSkins.kt b/src/main/kotlin/gg/skytils/skytilsmod/features/impl/misc/SummonSkins.kt
@@ -26,11 +26,12 @@ import io.ktor.client.request.*
 import io.ktor.client.statement.*
 import io.ktor.utils.io.jvm.javaio.*
 import kotlinx.coroutines.launch
+import java.util.concurrent.ConcurrentHashMap
 import javax.imageio.ImageIO
 
 object SummonSkins {
     // maps name to url
-    val skinMap = HashMap<String, String>()
+    val skinMap = ConcurrentHashMap<String, String>()
 
     // maps name to dynamic resource
     val skintextureMap = HashMap<String, DynamicResource>()