Fixing Shake showing the wrong chance to succeed

mcMMO-Dev · Apr 5, 2021 · c5f4666 · c5f4666
1 parent 3b7f416
commit c5f4666
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/Changelog.txt b/Changelog.txt
@@ -1,5 +1,6 @@
 Version 2.1.187
     Fixed a ClassCastException error involving Rupture
+    Fixed a bug where /fishing was showing the wrong shake chance
     Default Shake chance increased from 15% to 30% (update advanced.yml manually or delete the file to regenerate it and receive these changes)
     Removed entries for ranks 2-8 of Shake from advanced.yml (Shake only has one rank, these entries were a mistake)
     Modified the warning about UltraPermissions

diff --git a/src/main/java/com/gmail/nossr50/skills/fishing/FishingManager.java b/src/main/java/com/gmail/nossr50/skills/fishing/FishingManager.java
@@ -198,7 +198,7 @@ public int getLootTier() {
     }
 
     public double getShakeChance() {
-        return AdvancedConfig.getInstance().getShakeChance(getLootTier());
+        return AdvancedConfig.getInstance().getShakeChance(RankUtils.getRank(mmoPlayer.getPlayer(), SubSkillType.FISHING_SHAKE));
     }
 
     protected int getVanillaXPBoostModifier() {