From 42a30b45219a2d80f2700a008b935ad909e64758 Mon Sep 17 00:00:00 2001 From: GJ Date: Wed, 17 Apr 2013 07:47:43 -0400 Subject: [PATCH] Fixed Fishing treasures always having the same enchants. --- Changelog.txt | 1 + .../java/com/gmail/nossr50/skills/fishing/FishingManager.java | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/Changelog.txt b/Changelog.txt index 992f42112..9d6eed93a 100644 --- a/Changelog.txt +++ b/Changelog.txt @@ -22,6 +22,7 @@ Version 1.4.06-dev = Fixed bug where experience commands were adding levels to all skills when they shouldn't = Fixed mcmmo.commands.ptp.send not being set by default = Fixed NPE when trying to tab-complete /mctop + = Fixed Fishing treasures always having the same enchants ! Changed Berserk to add items to inventory rather than denying pickup ! Changed Chimaera Wing's recipe result to use the ingredient Material ! Players will no longer pickup items to their hotbar while using Unarmed diff --git a/src/main/java/com/gmail/nossr50/skills/fishing/FishingManager.java b/src/main/java/com/gmail/nossr50/skills/fishing/FishingManager.java index aa25b6ec1..bf5b321a5 100644 --- a/src/main/java/com/gmail/nossr50/skills/fishing/FishingManager.java +++ b/src/main/java/com/gmail/nossr50/skills/fishing/FishingManager.java @@ -112,7 +112,7 @@ public class FishingManager extends SkillManager { player.sendMessage(LocaleLoader.getString("Fishing.ItemFound")); treasureXp = treasure.getXp(); - ItemStack treasureDrop = treasure.getDrop(); + ItemStack treasureDrop = treasure.getDrop().clone(); // Not cloning is bad, m'kay? if (Permissions.magicHunter(player) && ItemUtils.isEnchantable(treasureDrop) && handleMagicHunter(treasureDrop)) { player.sendMessage(LocaleLoader.getString("Fishing.MagicFound"));