Fixed skill display stuff #1058

1 files changed, 25 insertions, 6 deletions

diff --git a/src/main/java/at/hannibal2/skyhanni/features/skillprogress/SkillProgress.kt b/src/main/java/at/hannibal2/skyhanni/features/skillprogress/SkillProgress.kt
index 0c88e7633..684d65c82 100644
--- a/src/main/java/at/hannibal2/skyhanni/features/skillprogress/SkillProgress.kt
+++ b/src/main/java/at/hannibal2/skyhanni/features/skillprogress/SkillProgress.kt
@@ -115,8 +115,9 @@ object SkillProgress {
 
         } else {
             maxWidth = barConfig.regularBar.width
+            val factor = skillExpPercentage.coerceAtMost(1.0)
             Renderable.progressBar(
-                percent = skillExpPercentage,
+                percent = factor,
                 startColor = Color(SpecialColour.specialToChromaRGB(barConfig.barStartColor)),
                 endColor = Color(SpecialColour.specialToChromaRGB(barConfig.barStartColor)),
                 width = maxWidth,
@@ -312,13 +313,18 @@ object SkillProgress {
         val useCustomGoalLevel = skillInfo.customGoalLevel != 0 && skillInfo.customGoalLevel > skillInfo.overflowLevel && customGoalConfig.enableInETADisplay
         var targetLevel = if (useCustomGoalLevel) skillInfo.customGoalLevel else level + 1
         if (targetLevel <= level || targetLevel > 400) targetLevel = (level + 1)
-        val currentLevelNeededXp = SkillUtil.xpRequiredForLevel(level.toDouble()) + skillInfo.overflowCurrentXp
+
+        val need = skillInfo.overflowCurrentXpMax
+        val have = skillInfo.overflowCurrentXp
+
+        val currentLevelNeededXp = SkillUtil.xpRequiredForLevel(level.toDouble()) + have
         val targetNeededXp = SkillUtil.xpRequiredForLevel(targetLevel.toDouble())
-        var remaining = if (useCustomGoalLevel) targetNeededXp - currentLevelNeededXp else skillInfo.overflowCurrentXpMax - skillInfo.overflowCurrentXp
 
-        if (!useCustomGoalLevel) {
+        var remaining = if (useCustomGoalLevel) targetNeededXp - currentLevelNeededXp else need - have
+
+        if (!useCustomGoalLevel && have < need) {
             if (skillInfo.overflowCurrentXpMax == skillInfoLast.overflowCurrentXpMax) {
-                remaining = interpolate(remaining.toFloat(), (skillInfoLast.overflowCurrentXpMax - skillInfoLast.overflowCurrentXp).toFloat(), lastGainUpdate.toMillis()).toLong()
+                remaining = interpolate(remaining.toFloat(), (need - have).toFloat(), lastGainUpdate.toMillis()).toLong()
             }
         }
 
@@ -328,7 +334,10 @@ object SkillProgress {
             add(Renderable.string("§7Needed XP: §e${remaining.addSeparators()}"))
 
         var xpInterp = xpInfo.xpGainHour
-        if (xpInfo.xpGainHour < 1000) {
+
+        if (have > need){
+            add(Renderable.string("§7In §cIncrease level cap!"))
+        } else if (xpInfo.xpGainHour < 1000) {
             add(Renderable.string("§7In §cN/A"))
         } else {
             val duration = ((remaining) * 1000 * 60 * 60 / xpInterp.toLong()).milliseconds
@@ -441,6 +450,16 @@ object SkillProgress {
         if (xpInfo.lastTotalXp > 0) {
             val delta = totalXp - xpInfo.lastTotalXp
             if (delta > 0 && delta < 1000) {
+
+                xpInfo.timer = when (SkillAPI.activeSkill) {
+                    SkillType.FARMING -> etaConfig.farmingPauseTime
+                    SkillType.MINING -> etaConfig.miningPauseTime
+                    SkillType.COMBAT -> etaConfig.combatPauseTime
+                    SkillType.FORAGING -> etaConfig.foragingPauseTime
+                    SkillType.FISHING -> etaConfig.fishingPauseTime
+                    else -> 3
+                }
+
                 xpInfo.xpGainQueue.add(0, delta)
 
                 calculateXPHour(xpInfo)