Fixed skill timer. (#1070)

1 files changed, 27 insertions, 25 deletions

diff --git a/src/main/java/at/hannibal2/skyhanni/api/SkillAPI.kt b/src/main/java/at/hannibal2/skyhanni/api/SkillAPI.kt
index 4c25b4c3e..2e7af1836 100644
--- a/src/main/java/at/hannibal2/skyhanni/api/SkillAPI.kt
+++ b/src/main/java/at/hannibal2/skyhanni/api/SkillAPI.kt
@@ -36,7 +36,6 @@ import io.github.moulberry.notenoughupdates.util.Utils
 import net.minecraft.command.CommandBase
 import net.minecraftforge.fml.common.eventhandler.SubscribeEvent
 import java.util.LinkedList
-import java.util.Timer
 import java.util.regex.Matcher
 import kotlin.concurrent.fixedRateTimer
 import kotlin.time.Duration.Companion.seconds
@@ -81,6 +80,33 @@ object SkillAPI {
     var showDisplay = false
     var lastUpdate = SimpleTimeMark.farPast()
 
+    init {
+        fixedRateTimer(name = "skyhanni-skillprogress-timer", initialDelay = 1_000L, period = 1_000L) {
+            tickSkill()
+        }
+    }
+
+    private fun tickSkill() {
+        val activeSkill = activeSkill ?: return
+        val info = skillXPInfoMap[activeSkill] ?: return
+        if (!info.sessionTimerActive) return
+
+        val time = when (activeSkill) {
+            SkillType.FARMING -> SkillProgress.etaConfig.farmingPauseTime
+            SkillType.MINING -> SkillProgress.etaConfig.miningPauseTime
+            SkillType.COMBAT -> SkillProgress.etaConfig.combatPauseTime
+            SkillType.FORAGING -> SkillProgress.etaConfig.foragingPauseTime
+            SkillType.FISHING -> SkillProgress.etaConfig.fishingPauseTime
+            else -> 0
+        }
+        if (info.lastUpdate.passedSince() > time.seconds) {
+            info.sessionTimerActive = false
+        }
+        if (info.sessionTimerActive) {
+            info.timeActive++
+        }
+    }
+
     @SubscribeEvent
     fun onActionBar(event: ActionBarUpdateEvent) {
         val actionBar = event.actionBar.removeColor()
@@ -106,10 +132,6 @@ object SkillAPI {
                 skillXp.lastUpdate = SimpleTimeMark.now()
                 skillXp.sessionTimerActive = true
 
-
-                if (skillType.timer == null) {
-                    skillType.timer = runTimer(skillType, skillXp)
-                }
                 SkillProgress.updateDisplay()
                 SkillProgress.hideInActionBar = listOf(component)
                 return
@@ -244,26 +266,6 @@ object SkillAPI {
         add("-  CustomGoalLevel: ${skillInfo.customGoalLevel}\n")
     }
 
-    // TODO only use one statuc timer for the whole feature. this timer just ticks the currently active skill.
-    private fun runTimer(skillType: SkillType, info: SkillXPInfo): Timer =
-        fixedRateTimer(name = "skyhanni-skillprogress-timer-${skillType.displayName}", initialDelay = 1_000L, period = 1_000L) {
-            if (skillType.timer != this) cancel()
-            val time = when (activeSkill) {
-                SkillType.FARMING -> SkillProgress.etaConfig.farmingPauseTime
-                SkillType.MINING -> SkillProgress.etaConfig.miningPauseTime
-                SkillType.COMBAT -> SkillProgress.etaConfig.combatPauseTime
-                SkillType.FORAGING -> SkillProgress.etaConfig.foragingPauseTime
-                SkillType.FISHING -> SkillProgress.etaConfig.fishingPauseTime
-                else -> 0
-            }
-            if (info.lastUpdate.passedSince() > time.seconds) {
-                info.sessionTimerActive = false
-            }
-            if (info.sessionTimerActive) {
-                info.timeActive++
-            }
-        }
-
     private fun handleSkillPattern(matcher: Matcher, skillType: SkillType, skillInfo: SkillInfo) {
         val currentXp = matcher.group("current").formatNumber()
         val maxXp = matcher.group("needed").formatNumber()