Fixes #23

Also fix when no structures loaded on the server the client can't render anything

diff --git a/java/com/irtimaled/bbor/ClientProxy.java b/java/com/irtimaled/bbor/ClientProxy.java
index 720c1b7a21e4433059b6c916effd198c353ba5b5..9e70369265a9eeab971d24796f797c4da81f0ddd 100644 (file)
--- a/java/com/irtimaled/bbor/ClientProxy.java
+++ b/java/com/irtimaled/bbor/ClientProxy.java
@@ -81,9 +81,11 @@ public class ClientProxy extends CommonProxy {
 
         if (this.active) {
             DimensionType dimensionType = entityPlayer.worldObj.provider.getDimensionType();
+            Map<BoundingBox, Set<BoundingBox>> boundingBoxes = null;
             if (boundingBoxCacheMap.containsKey(dimensionType)) {
-                renderBoundingBoxes(boundingBoxCacheMap.get(dimensionType).getBoundingBoxes());
+                boundingBoxes = boundingBoxCacheMap.get(dimensionType).getBoundingBoxes();
             }
+            renderBoundingBoxes(boundingBoxes);
         }
     }
 
@@ -276,6 +278,7 @@ public class ClientProxy extends CommonProxy {
             GL11.glClear(GL11.GL_DEPTH_BUFFER_BIT);
         }
 
+        if(map != null)
         for (BoundingBox bb : map.keySet()) {
             if (outerBoxOnly) {
                 renderBoundingBoxSet(map.get(bb));

diff --git a/java/com/irtimaled/bbor/forge/ForgeCommonProxy.java b/java/com/irtimaled/bbor/forge/ForgeCommonProxy.java
index fd6be0d78518d3249bce0fd303bba6e375259d5c..2d43d874ab528431416adf261b1848551befa648 100644 (file)
--- a/java/com/irtimaled/bbor/forge/ForgeCommonProxy.java
+++ b/java/com/irtimaled/bbor/forge/ForgeCommonProxy.java
@@ -24,7 +24,7 @@ import java.util.concurrent.ConcurrentHashMap;
 
 public class ForgeCommonProxy implements IEventHandler {
 
-    public Map<EntityPlayerMP, Integer> playerDimensions = new ConcurrentHashMap<EntityPlayerMP, Integer>();
+    public Map<EntityPlayerMP, DimensionType> playerDimensions = new ConcurrentHashMap<EntityPlayerMP, DimensionType>();
     private Map<EntityPlayerMP, Set<BoundingBox>> playerBoundingBoxesCache = new HashMap<EntityPlayerMP, Set<BoundingBox>>();
 
     protected CommonProxy getProxy() {
@@ -60,10 +60,10 @@ public class ForgeCommonProxy implements IEventHandler {
     public void playerChangedDimensionEvent(PlayerEvent.PlayerChangedDimensionEvent evt) {
         if (playerDimensions.containsKey(evt.player)) {
             EntityPlayerMP player = (EntityPlayerMP) evt.player;
-            int dimension = player.dimension;
-            playerDimensions.put(player, dimension);
+            DimensionType dimensionType = DimensionType.getById(player.dimension);
+            playerDimensions.put(player, dimensionType);
 
-            sendToPlayer(player, getProxy().boundingBoxCacheMap.get(dimension));
+            sendToPlayer(player, getProxy().boundingBoxCacheMap.get(dimensionType));
         }
     }
 
@@ -77,9 +77,9 @@ public class ForgeCommonProxy implements IEventHandler {
                 isRemotePlayer(evt.player)) {
             EntityPlayerMP player = (EntityPlayerMP) evt.player;
             initializeClient(player);
-            int dimension = player.dimension;
-            playerDimensions.put(player, dimension);
-            sendToPlayer(player, getProxy().boundingBoxCacheMap.get(dimension));
+            DimensionType dimensionType = DimensionType.getById(player.dimension);
+            playerDimensions.put(player, dimensionType);
+            sendToPlayer(player, getProxy().boundingBoxCacheMap.get(dimensionType));
         }
     }
 
@@ -98,9 +98,9 @@ public class ForgeCommonProxy implements IEventHandler {
             if(!mc.getPlayerList().getPlayerList().contains(player)) {
                 playerDimensions.remove(player);
             } else {
-                int dimension = playerDimensions.get(player);
-                if (getProxy().boundingBoxCacheMap.containsKey(dimension)) {
-                    sendToPlayer(player, getProxy().boundingBoxCacheMap.get(dimension));
+                DimensionType dimensionType = playerDimensions.get(player);
+                if (getProxy().boundingBoxCacheMap.containsKey(dimensionType)) {
+                    sendToPlayer(player, getProxy().boundingBoxCacheMap.get(dimensionType));
                 }
             }
         }

diff --git a/java/com/irtimaled/bbor/forge/ForgeMod.java b/java/com/irtimaled/bbor/forge/ForgeMod.java
index 8d6a98f075d64b426697dc36fc6015af410f9ca1..26d6b2aaac9e1c2576be0032be8c498d21766390 100644 (file)
--- a/java/com/irtimaled/bbor/forge/ForgeMod.java
+++ b/java/com/irtimaled/bbor/forge/ForgeMod.java
@@ -14,7 +14,7 @@ public class ForgeMod {
 
     public static final String MODID = "bbor";
     public static final String NAME = "Bounding Box Outline Reloaded";
-    public static final String VERSION = "1.0.0-beta12";
+    public static final String VERSION = "1.0.0-beta13";
     public static final String MCVERSION = "[1.10]";
 
     private ConfigManager configManager;
@@ -35,7 +35,6 @@ public class ForgeMod {
     @Mod.EventHandler
     public void load(FMLInitializationEvent evt) {
         MinecraftForge.EVENT_BUS.register(proxy);
-        FMLCommonHandler.instance().bus().register(proxy);
 
         proxy.init(configManager);
     }