From 6d7d8118144e79e3556916c7a54955e1b69abd4a Mon Sep 17 00:00:00 2001
From: TfT_02 <tft_02@hotmail.com>
Date: Mon, 24 Jun 2013 16:25:49 +0200
Subject: [PATCH] Fixed bug with Ice Fishing in the Nether

Fixes  #1218
---
 Changelog.txt                                                | 1 +
 .../com/gmail/nossr50/skills/fishing/FishingManager.java     | 5 +++++
 2 files changed, 6 insertions(+)

diff --git a/Changelog.txt b/Changelog.txt
index 9da24ce04..725f14d45 100644
--- a/Changelog.txt
+++ b/Changelog.txt
@@ -51,6 +51,7 @@ Version 1.4.06-dev
  = Fixed bug with Blast Mining where the Ability refreshed message was being send too early
  = Fixed bug where the chance of a successful Gracefull Roll was twice as high as displayed
  = Fixed bug where lucky perks where not working
+ = Fixed bug with Ice Fishing in the Nether
  ! Changed Spout notification tiers to be stored in SpoutConfig instead of AdvancedConfig
  ! Changed Berserk to add items to inventory rather than denying pickup
  ! Changed Call of the Wild, newly summoned pet's will have a custom name. (added permission node to disable this)
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 220a86c38..53ab8d0c4 100644
--- a/src/main/java/com/gmail/nossr50/skills/fishing/FishingManager.java
+++ b/src/main/java/com/gmail/nossr50/skills/fishing/FishingManager.java
@@ -12,6 +12,7 @@ import org.bukkit.Material;
 import org.bukkit.Sound;
 import org.bukkit.WeatherType;
 import org.bukkit.World;
+import org.bukkit.World.Environment;
 import org.bukkit.block.Biome;
 import org.bukkit.block.Block;
 import org.bukkit.block.BlockFace;
@@ -176,6 +177,10 @@ public class FishingManager extends SkillManager {
             return false;
         }
 
+        if (block.getWorld().getEnvironment() == Environment.NETHER) {
+            return false;
+        }
+
         if (block.getType() != Material.ICE) {
             return false;
         }