From 1fb5aa97801dcfb28cd1acbc459025fcff804c89 Mon Sep 17 00:00:00 2001
From: Brettflan <dsr@fenrisoft.com>
Date: Wed, 14 Dec 2011 06:03:00 -0600
Subject: [PATCH] Added 1 more "safe" and 1 more "painful" block type for safe
 location determination from newer blocks Added in check to make sure target
 chunk is loaded before trying to find safe location in it

---
 src/com/wimbli/WorldBorder/BorderData.java | 25 ++++++++++++++++------
 src/com/wimbli/WorldBorder/Config.java     |  1 +
 2 files changed, 19 insertions(+), 7 deletions(-)

diff --git a/src/com/wimbli/WorldBorder/BorderData.java b/src/com/wimbli/WorldBorder/BorderData.java
index 2776c7f..43fae97 100644
--- a/src/com/wimbli/WorldBorder/BorderData.java
+++ b/src/com/wimbli/WorldBorder/BorderData.java
@@ -3,6 +3,7 @@ package com.wimbli.WorldBorder;
 import java.util.Arrays;
 import java.util.LinkedHashSet;
 
+import org.bukkit.Chunk;
 import org.bukkit.Location;
 import org.bukkit.World;
 
@@ -165,7 +166,15 @@ public class BorderData
 			zLoc = z + vZ / magV * (radius - Config.KnockBack());
 		}
 
-		yLoc = getSafeY(loc.getWorld(), Location.locToBlock(xLoc), Location.locToBlock(yLoc), Location.locToBlock(zLoc));
+		int ixLoc = Location.locToBlock(xLoc);
+		int izLoc = Location.locToBlock(zLoc);
+
+		// Make sure the chunk we're checking in is actually loaded
+		Chunk tChunk = loc.getWorld().getChunkAt(CoordXZ.blockToChunk(ixLoc), CoordXZ.blockToChunk(izLoc));
+		if (!tChunk.isLoaded())
+			tChunk.load();
+
+		yLoc = getSafeY(loc.getWorld(), ixLoc, Location.locToBlock(yLoc), izLoc);
 		if (yLoc == -1)
 			return null;
 
@@ -178,12 +187,12 @@ public class BorderData
 
 	//these material IDs are acceptable for places to teleport player; breathable blocks and water
 	private static final LinkedHashSet<Integer> safeOpenBlocks = new LinkedHashSet<Integer>(Arrays.asList(
-		 new Integer[] {0, 6, 8, 9, 27, 28, 30, 31, 32, 37, 38, 39, 40, 50, 55, 59, 63, 64, 65, 66, 68, 69, 70, 71, 72, 75, 76, 77, 78, 83, 90, 93, 94, 96, 104, 105, 106}
+		 new Integer[] {0, 6, 8, 9, 27, 28, 30, 31, 32, 37, 38, 39, 40, 50, 55, 59, 63, 64, 65, 66, 68, 69, 70, 71, 72, 75, 76, 77, 78, 83, 90, 93, 94, 96, 104, 105, 106, 115}
 	));
 
-	//these material IDs are ones we don't want to drop the player onto, like cactus or lava or fire
+	//these material IDs are ones we don't want to drop the player onto, like cactus or lava or fire or activated Ender portal
 	private static final LinkedHashSet<Integer> painfulBlocks = new LinkedHashSet<Integer>(Arrays.asList(
-		 new Integer[] {10, 11, 51, 81}
+		 new Integer[] {10, 11, 51, 81, 119}
 	));
 
 	// check if a particular spot consists of 2 breathable blocks over something relatively solid
@@ -197,7 +206,7 @@ public class BorderData
 			);
 	}
 
-	private static final int limTop = 120, limBot = 1;
+	private static final int limTop = 126, limBot = 1;
 
 	// find closest safe Y position from the starting position
 	private double getSafeY(World world, int X, int Y, int Z)
@@ -206,13 +215,15 @@ public class BorderData
 
 		for(int y1 = Y, y2 = Y; (y1 > limBot) || (y2 < limTop); y1--, y2++){
 			// Look below.
-			if(y1 > limBot){
+			if(y1 > limBot)
+			{
 				if (isSafeSpot(world, X, y1, Z))
 					return (double)y1;
 			}
 
 			// Look above.
-			if(y2 < limTop && y2 != y1){
+			if(y2 < limTop && y2 != y1)
+			{
 				if (isSafeSpot(world, X, y2, Z))
 					return (double)y2;
 			}
diff --git a/src/com/wimbli/WorldBorder/Config.java b/src/com/wimbli/WorldBorder/Config.java
index 3f86eda..9c9b896 100644
--- a/src/com/wimbli/WorldBorder/Config.java
+++ b/src/com/wimbli/WorldBorder/Config.java
@@ -439,6 +439,7 @@ public class Config
 			LogConfig("Configuration saved.");
 	}
 
+	// needed in case config.yml has any old "¨" compatibility characters in it, since they cause parse errors in Bukkit's new configuration handler
 	public static void compat_load()
 	{ // load config from file
 		Configuration old_cfg = plugin.getConfiguration();