diff --git a/src/main/java/com/gmail/nossr50/config/experience/ExperienceConfig.java b/src/main/java/com/gmail/nossr50/config/experience/ExperienceConfig.java index 7b9e78791..4a0912be0 100644 --- a/src/main/java/com/gmail/nossr50/config/experience/ExperienceConfig.java +++ b/src/main/java/com/gmail/nossr50/config/experience/ExperienceConfig.java @@ -254,9 +254,4 @@ public class ExperienceConfig extends ConfigValidated { public double getRepairXP(ItemMaterialCategory repairItemMaterialCategory) { return getDoubleValue(EXPERIENCE, REPAIR, StringUtils.getCapitalized(repairItemMaterialCategory.toString())); } - - /* Taming */ - public int getTamingXP(EntityType type) { - return getIntValue(EXPERIENCE, TAMING, ANIMAL_TAMING, StringUtils.getEntityConfigName(type)); - } } diff --git a/src/main/java/com/gmail/nossr50/skills/taming/TamingManager.java b/src/main/java/com/gmail/nossr50/skills/taming/TamingManager.java index cf49dc153..d40d36338 100644 --- a/src/main/java/com/gmail/nossr50/skills/taming/TamingManager.java +++ b/src/main/java/com/gmail/nossr50/skills/taming/TamingManager.java @@ -106,7 +106,7 @@ public class TamingManager extends SkillManager { * @param entity The LivingEntity to award XP for */ public void awardTamingXP(LivingEntity entity) { - applyXpGain(ExperienceConfig.getInstance().getTamingXP(entity.getType()), XPGainReason.PVE); + applyXpGain(mcMMO.getConfigManager().getExperienceMapManager().getTamingXp(entity.getType()), XPGainReason.PVE); } /** diff --git a/src/main/java/com/gmail/nossr50/util/experience/ExperienceMapManager.java b/src/main/java/com/gmail/nossr50/util/experience/ExperienceMapManager.java index fab258969..c6cfbb801 100644 --- a/src/main/java/com/gmail/nossr50/util/experience/ExperienceMapManager.java +++ b/src/main/java/com/gmail/nossr50/util/experience/ExperienceMapManager.java @@ -17,7 +17,7 @@ public class ExperienceMapManager implements Unload { private HashMap herbalismFullyQualifiedBlockXpMap; private HashMap woodcuttingFullyQualifiedBlockXpMap; private HashMap excavationFullyQualifiedBlockXpMap; - private HashMap tamingExperienceMap; + private HashMap tamingExperienceMap; private double globalXpMult; @@ -66,7 +66,7 @@ public class ExperienceMapManager implements Unload { { //Match! matchFound = true; - tamingExperienceMap.put(entityType, userTamingConfigMap.get(s)); + tamingExperienceMap.put(entityType, (float) userTamingConfigMap.get(s)); } } if(!matchFound) @@ -155,6 +155,16 @@ public class ExperienceMapManager implements Unload { return globalXpMult; } + /** + * Gets the taming XP for this entity + * @param entityType target entity + * @return value of XP for this entity + */ + public float getTamingXp(EntityType entityType) + { + return tamingExperienceMap.get(entityType); + } + /** * Gets the original value of the global XP multiplier * This is defined by the users config @@ -172,7 +182,7 @@ public class ExperienceMapManager implements Unload { * @return true if the block has valid xp registers */ public boolean hasMiningXp(Material material) { - return miningFullyQualifiedBlockXpMap.get(material.getKey().getKey()) != null; + return miningFullyQualifiedBlockXpMap.get(material.getKey()) != null; } /** @@ -182,7 +192,7 @@ public class ExperienceMapManager implements Unload { * @return true if the block has valid xp registers */ public boolean hasHerbalismXp(Material material) { - return herbalismFullyQualifiedBlockXpMap.get(material) != null; + return herbalismFullyQualifiedBlockXpMap.get(material.getKey()) != null; } /** @@ -192,7 +202,7 @@ public class ExperienceMapManager implements Unload { * @return true if the block has valid xp registers */ public boolean hasWoodcuttingXp(Material material) { - return woodcuttingFullyQualifiedBlockXpMap.get(material) != null; + return woodcuttingFullyQualifiedBlockXpMap.get(material.getKey()) != null; } /** @@ -202,7 +212,7 @@ public class ExperienceMapManager implements Unload { * @return true if the block has valid xp registers */ public boolean hasExcavationXp(Material material) { - return excavationFullyQualifiedBlockXpMap.get(material) != null; + return excavationFullyQualifiedBlockXpMap.get(material.getKey()) != null; } /** @@ -212,7 +222,7 @@ public class ExperienceMapManager implements Unload { * @return the raw XP value before any modifiers are applied */ public int getMiningXp(Material material) { - return miningFullyQualifiedBlockXpMap.get(material); + return miningFullyQualifiedBlockXpMap.get(material.getKey()); } /** @@ -222,7 +232,7 @@ public class ExperienceMapManager implements Unload { * @return the raw XP value before any modifiers are applied */ public int getHerbalismXp(Material material) { - return herbalismFullyQualifiedBlockXpMap.get(material); + return herbalismFullyQualifiedBlockXpMap.get(material.getKey()); } /** @@ -232,7 +242,7 @@ public class ExperienceMapManager implements Unload { * @return the raw XP value before any modifiers are applied */ public int getWoodcuttingXp(Material material) { - return woodcuttingFullyQualifiedBlockXpMap.get(material); + return woodcuttingFullyQualifiedBlockXpMap.get(material.getKey()); } /** @@ -242,7 +252,7 @@ public class ExperienceMapManager implements Unload { * @return the raw XP value before any modifiers are applied */ public int getExcavationXp(Material material) { - return excavationFullyQualifiedBlockXpMap.get(material); + return excavationFullyQualifiedBlockXpMap.get(material.getKey()); } @Override