load mesh data and structure meshes

src/engine/collisionEngine/CollisionManager.java

@@ -0,0 +1,10 @@
+package engine.collisionEngine;
+
+import java.awt.geom.Point2D;
+import java.util.ArrayList;
+import java.util.HashMap;
+
+public class CollisionManager {
+    public static HashMap<Integer, ArrayList<MeshTriangle>> mesh_triangles;
+    public static HashMap<Integer, ArrayList<MeshData>> structure_meshes;
+}

src/engine/collisionEngine/Mesh.java

@@ -0,0 +1,125 @@
+package engine.collisionEngine;
+
+import engine.gameManager.BuildingManager;
+import engine.math.Vector3f;
+import engine.math.Vector3fImmutable;
+import engine.objects.AbstractCharacter;
+import engine.objects.Building;
+
+import java.awt.geom.Line2D;
+import java.awt.geom.Point2D;
+import java.awt.geom.Rectangle2D;
+import java.util.ArrayList;
+
+public class Mesh {
+    public int parentUUID;
+    public MeshData meshData;
+    private int meshID;
+    public Rectangle2D.Float bounds;
+    public ArrayList<MeshTriangle> triangles;
+    public Vector3f mesh_loc;
+    public Vector3f mesh_ref;
+    public Vector3f mesh_end;
+    public float mesh_maxY;
+    public float mesh_minY;
+
+    public Mesh(MeshData data, int parentUUID){
+        this.meshData = data;
+        this.parentUUID = parentUUID;
+        this.meshID = data.meshID;
+        this.BakeTriangles();
+        this.BakeBounds();
+    }
+
+    public void BakeTriangles(){
+
+        if(CollisionManager.mesh_triangles.containsKey(this.meshID) == false)
+            return; // no triangle data to bake
+
+        this.triangles = new ArrayList<>();
+
+        Building building = BuildingManager.getBuilding(this.parentUUID);
+        if(building == null)
+            return; // can't continue without a building to base location offsets from
+
+        Vector3f adjustedBuildingLoc = new Vector3f(building.loc.x,building.loc.y, building.loc.z * -1);
+        int rotDegrees = (int)Math.toDegrees(building.getBounds().getQuaternion().angleY);
+        this.mesh_loc = Vector3f.rotateAroundPoint(adjustedBuildingLoc.add(this.meshData.mesh_loc),adjustedBuildingLoc,rotDegrees);
+        this.mesh_ref = Vector3f.rotateAroundPoint(adjustedBuildingLoc.add(this.meshData.mesh_ref),adjustedBuildingLoc,rotDegrees);
+        this.mesh_end = Vector3f.rotateAroundPoint(adjustedBuildingLoc.add(this.meshData.mesh_end),adjustedBuildingLoc,rotDegrees);
+        this.mesh_minY = adjustedBuildingLoc.y + this.meshData.mesh_minY;
+        this.mesh_maxY = adjustedBuildingLoc.y + this.meshData.mesh_maxY;
+
+        for(MeshTriangle tri : CollisionManager.mesh_triangles.get(this.meshID)){
+
+            Vector3f point1 = this.mesh_loc.add(new Vector3f(tri.point1.x,this.mesh_loc.y,tri.point1.y));
+            Vector3f point2 = this.mesh_loc.add(new Vector3f(tri.point2.x,this.mesh_loc.y,tri.point2.y));
+            Vector3f point3 = this.mesh_loc.add(new Vector3f(tri.point3.x,this.mesh_loc.y,tri.point3.y));
+
+            Vector3f rotatedPoint1 = Vector3f.rotateAroundPoint(point1,this.mesh_loc,rotDegrees);
+            Vector3f rotatedPoint2 = Vector3f.rotateAroundPoint(point2,this.mesh_loc,rotDegrees);
+            Vector3f rotatedPoint3 = Vector3f.rotateAroundPoint(point3,this.mesh_loc,rotDegrees);
+
+            MeshTriangle newTri = new MeshTriangle();
+
+            newTri.point1 = new Point2D.Float(rotatedPoint1.x,rotatedPoint1.z);
+            newTri.point2 = new Point2D.Float(rotatedPoint2.x,rotatedPoint2.z);
+            newTri.point3 = new Point2D.Float(rotatedPoint3.x,rotatedPoint3.z);
+
+            newTri.sides = new ArrayList<>();
+
+            newTri.sides.add(new Line2D.Float(newTri.point1,newTri.point2));
+            newTri.sides.add(new Line2D.Float(newTri.point2,newTri.point3));
+            newTri.sides.add(new Line2D.Float(newTri.point3,newTri.point1));
+
+            this.triangles.add(newTri);
+        }
+    }
+
+    public void BakeBounds(){
+        float width = (this.mesh_ref.x - this.mesh_end.x) * 0.5f;
+        float height = (this.mesh_ref.z - this.mesh_end.z) * 0.5f;
+        this.bounds = new Rectangle2D.Float(this.mesh_ref.x,this.mesh_ref.z,width,height);
+    }
+
+    public boolean collides(AbstractCharacter mover, Vector3fImmutable destination){
+
+        if(mover == null)
+            return false;
+
+        Line2D.Float line = new Line2D.Float(new Point2D.Float(mover.loc.x,mover.loc.z * -1),new Point2D.Float(destination.x,destination.z * -1));
+        float footHeight = mover.loc.y;
+        float headHeight = mover.loc.y + mover.characterHeight;
+
+        if(line.intersects(this.bounds) == false)
+            return false; // character path doesn't cross over this mesh
+
+        if(footHeight > this.mesh_maxY || headHeight < this.mesh_minY)
+            return false; //character is either above or below the bounds of this mesh
+
+        for(MeshTriangle tri : this.triangles)
+            if(tri.collides(line))
+                return true; //character's path directly hits part of this mesh
+
+        return false;
+    }
+
+    public boolean losBlocked(AbstractCharacter looker, Vector3fImmutable target){
+
+        float headHeight = looker.loc.y + looker.characterHeight;
+        float targetAlt = target.y;
+
+        Line2D.Float eyeLine = new Line2D.Float(new Point2D.Float(looker.loc.x,looker.loc.z * -1),new Point2D.Float(target.x,target.z * -1));
+
+        if(eyeLine.intersects(this.bounds) == false)
+            return false; // character eye-line doesn't cross over this mesh
+
+        if(targetAlt > this.mesh_maxY && headHeight > this.mesh_maxY)
+            return false; // both characters are above this mesh
+
+        if(targetAlt < this.mesh_maxY && headHeight < this.mesh_maxY)
+            return false; // both characters are below this mesh
+
+        return true;
+    }
+}

src/engine/collisionEngine/MeshData.java

@@ -0,0 +1,23 @@
+package engine.collisionEngine;
+
+import engine.math.Vector3f;
+
+public class MeshData {
+    public int propID;
+    public int meshID;
+    public Vector3f mesh_loc;
+    public Vector3f mesh_ref;
+    public Vector3f mesh_end;
+    public float mesh_maxY;
+    public float mesh_minY;
+
+    public MeshData(int propID,int meshID, Vector3f meshLoc, Vector3f meshRef, Vector3f meshEnd, float maxY, float minY){
+        this.propID = propID;
+        this.meshID = meshID;
+        this.mesh_loc = meshLoc;
+        this.mesh_ref = meshRef;
+        this.mesh_end = meshEnd;
+        this.mesh_maxY = maxY;
+        this.mesh_minY = minY;
+    }
+}

src/engine/collisionEngine/MeshTriangle.java

@@ -0,0 +1,20 @@
+package engine.collisionEngine;
+
+import java.awt.geom.Line2D;
+import java.awt.geom.Point2D;
+import java.util.ArrayList;
+
+public class MeshTriangle {
+    public Point2D.Float point1;
+    public Point2D.Float point2;
+    public Point2D.Float point3;
+    public ArrayList<Line2D.Float> sides;
+
+    public boolean collides(Line2D.Float line){
+        for(Line2D.Float side : sides)
+            if(line.intersectsLine(side))
+                return true;
+
+        return false;
+    }
+}

src/engine/db/handlers/dbBuildingHandler.java

@@ -13,13 +13,18 @@ import engine.Enum;
 import engine.Enum.DbObjectType;
 import engine.Enum.ProtectionState;
 import engine.Enum.TaxType;
+import engine.collisionEngine.CollisionManager;
+import engine.collisionEngine.MeshData;
+import engine.collisionEngine.MeshTriangle;
 import engine.gameManager.BuildingManager;
 import engine.gameManager.DbManager;
+import engine.math.Vector3f;
 import engine.math.Vector3fImmutable;
 import engine.objects.*;
 import org.joda.time.DateTime;
 import org.pmw.tinylog.Logger;
 
+import java.awt.geom.Point2D;
 import java.sql.Connection;
 import java.sql.PreparedStatement;
 import java.sql.ResultSet;
@@ -848,5 +853,69 @@ public class dbBuildingHandler extends dbHandlerBase {
         }
         return false;
     }
+    public boolean LOAD_STRUCTURE_MESHES() {
 
+        try (Connection connection = DbManager.getConnection();
+             PreparedStatement preparedStatement = connection.prepareStatement("SELECT * FROM `final_structure_meshes`;")) {
+
+            ResultSet rs = preparedStatement.executeQuery();
+
+            CollisionManager.structure_meshes = new HashMap<>();
+            while (rs.next()) {
+                int propId = rs.getInt("propID");
+                int meshId = rs.getInt("meshID");
+                Vector3f meshLoc = new Vector3f(rs.getFloat("locX"),rs.getFloat("locY"),rs.getFloat("locZ"));
+                Vector3f meshRef = new Vector3f(rs.getFloat("refX"),rs.getFloat("refY"),rs.getFloat("refZ"));
+                Vector3f meshEnd = new Vector3f(rs.getFloat("endX"),rs.getFloat("endY"),rs.getFloat("endZ"));
+                float maxY = rs.getFloat("maxY");
+                float minY = rs.getFloat("minY");
+                MeshData data = new MeshData(propId,meshId,meshLoc,meshRef,meshEnd,maxY,minY);
+                if(CollisionManager.structure_meshes.containsKey(propId)){
+                    CollisionManager.structure_meshes.get(propId).add(data);
+                } else{
+                    ArrayList<MeshData> addList = new ArrayList<>();
+                    addList.add(data);
+                    CollisionManager.structure_meshes.put(propId,addList);
+                }
+            }
+            return (preparedStatement.executeUpdate() > 0);
+        } catch (SQLException e) {
+            Logger.error(e);
+        }
+        return false;
+    }
+    public boolean LOAD_MESH_TRIANGLES() {
+
+        try (Connection connection = DbManager.getConnection();
+             PreparedStatement preparedStatement = connection.prepareStatement("SELECT * FROM `final_mesh_triangles`;")) {
+
+            ResultSet rs = preparedStatement.executeQuery();
+
+            CollisionManager.mesh_triangles = new HashMap<>();
+            while (rs.next()) {
+                int meshId = rs.getInt("meshID");
+                Point2D.Float point1 = new Point2D.Float(rs.getFloat("P1X"), rs.getFloat("P1Z"));
+                Point2D.Float point2 = new Point2D.Float(rs.getFloat("P2X"), rs.getFloat("P2Z"));
+                Point2D.Float point3 = new Point2D.Float(rs.getFloat("P3X"), rs.getFloat("P3Z"));
+
+                MeshTriangle newTri = new MeshTriangle();
+                newTri.point1 = point1;
+                newTri.point2 = point2;
+                newTri.point3 = point3;
+
+                if(CollisionManager.mesh_triangles.containsKey(meshId)){
+                    CollisionManager.mesh_triangles.get(meshId).add(newTri);
+                } else{
+                    ArrayList<MeshTriangle> addList = new ArrayList<>();
+                    addList.add(newTri);
+                    CollisionManager.mesh_triangles.put(meshId,addList);
+                }
+
+            }
+            return (preparedStatement.executeUpdate() > 0);
+        } catch (SQLException e) {
+            Logger.error(e);
+        }
+        return false;
+    }
 }

src/engine/db/handlers/dbWarehouseHandler.java

@@ -109,7 +109,7 @@ public class dbWarehouseHandler extends dbHandlerBase {
              PreparedStatement preparedStatement = connection.prepareStatement("UPDATE `obj_warehouse` SET `warehouse_locks`=? WHERE `UID` = ?")) {
 
             preparedStatement.setLong(1, locks);
-            preparedStatement.setInt(2, wh.UID);
+            preparedStatement.setInt(2, wh.getUID());
 
             return (preparedStatement.executeUpdate() > 0);
 
@@ -125,7 +125,7 @@ public class dbWarehouseHandler extends dbHandlerBase {
              PreparedStatement preparedStatement = connection.prepareStatement("UPDATE `obj_warehouse` SET `warehouse_gold`=? WHERE `UID` = ?")) {
 
             preparedStatement.setInt(1, amount);
-            preparedStatement.setInt(2, wh.UID);
+            preparedStatement.setInt(2, wh.getUID());
 
             return (preparedStatement.executeUpdate() > 0);
 
@@ -141,7 +141,7 @@ public class dbWarehouseHandler extends dbHandlerBase {
              PreparedStatement preparedStatement = connection.prepareStatement("UPDATE `obj_warehouse` SET `warehouse_stone`=? WHERE `UID` = ?")) {
 
             preparedStatement.setInt(1, amount);
-            preparedStatement.setInt(2, wh.UID);
+            preparedStatement.setInt(2, wh.getUID());
 
             return (preparedStatement.executeUpdate() > 0);
 
@@ -157,7 +157,7 @@ public class dbWarehouseHandler extends dbHandlerBase {
              PreparedStatement preparedStatement = connection.prepareStatement("UPDATE `obj_warehouse` SET `warehouse_truesteel`=? WHERE `UID` = ?")) {
 
             preparedStatement.setInt(1, amount);
-            preparedStatement.setInt(2, wh.UID);
+            preparedStatement.setInt(2, wh.getUID());
 
             return (preparedStatement.executeUpdate() > 0);
 
@@ -173,7 +173,7 @@ public class dbWarehouseHandler extends dbHandlerBase {
              PreparedStatement preparedStatement = connection.prepareStatement("UPDATE `obj_warehouse` SET `warehouse_iron`=? WHERE `UID` = ?")) {
 
             preparedStatement.setInt(1, amount);
-            preparedStatement.setInt(2, wh.UID);
+            preparedStatement.setInt(2, wh.getUID());
 
             return (preparedStatement.executeUpdate() > 0);
 
@@ -189,7 +189,7 @@ public class dbWarehouseHandler extends dbHandlerBase {
              PreparedStatement preparedStatement = connection.prepareStatement("UPDATE `obj_warehouse` SET `warehouse_adamant`=? WHERE `UID` = ?")) {
 
             preparedStatement.setInt(1, amount);
-            preparedStatement.setInt(2, wh.UID);
+            preparedStatement.setInt(2, wh.getUID());
 
             return (preparedStatement.executeUpdate() > 0);
 
@@ -205,7 +205,7 @@ public class dbWarehouseHandler extends dbHandlerBase {
              PreparedStatement preparedStatement = connection.prepareStatement("UPDATE `obj_warehouse` SET `warehouse_lumber`=? WHERE `UID` = ?")) {
 
             preparedStatement.setInt(1, amount);
-            preparedStatement.setInt(2, wh.UID);
+            preparedStatement.setInt(2, wh.getUID());
 
             return (preparedStatement.executeUpdate() > 0);
 
@@ -221,7 +221,7 @@ public class dbWarehouseHandler extends dbHandlerBase {
              PreparedStatement preparedStatement = connection.prepareStatement("UPDATE `obj_warehouse` SET `warehouse_oak`=? WHERE `UID` = ?")) {
 
             preparedStatement.setInt(1, amount);
-            preparedStatement.setInt(2, wh.UID);
+            preparedStatement.setInt(2, wh.getUID());
 
             return (preparedStatement.executeUpdate() > 0);
 
@@ -237,7 +237,7 @@ public class dbWarehouseHandler extends dbHandlerBase {
              PreparedStatement preparedStatement = connection.prepareStatement("UPDATE `obj_warehouse` SET `warehouse_bronzewood`=? WHERE `UID` = ?")) {
 
             preparedStatement.setInt(1, amount);
-            preparedStatement.setInt(2, wh.UID);
+            preparedStatement.setInt(2, wh.getUID());
 
             return (preparedStatement.executeUpdate() > 0);
 
@@ -253,7 +253,7 @@ public class dbWarehouseHandler extends dbHandlerBase {
              PreparedStatement preparedStatement = connection.prepareStatement("UPDATE `obj_warehouse` SET `warehouse_mandrake`=? WHERE `UID` = ?")) {
 
             preparedStatement.setInt(1, amount);
-            preparedStatement.setInt(2, wh.UID);
+            preparedStatement.setInt(2, wh.getUID());
 
             return (preparedStatement.executeUpdate() > 0);
 
@@ -269,7 +269,7 @@ public class dbWarehouseHandler extends dbHandlerBase {
              PreparedStatement preparedStatement = connection.prepareStatement("UPDATE `obj_warehouse` SET `warehouse_coal`=? WHERE `UID` = ?")) {
 
             preparedStatement.setInt(1, amount);
-            preparedStatement.setInt(2, wh.UID);
+            preparedStatement.setInt(2, wh.getUID());
 
             return (preparedStatement.executeUpdate() > 0);
 
@@ -285,7 +285,7 @@ public class dbWarehouseHandler extends dbHandlerBase {
              PreparedStatement preparedStatement = connection.prepareStatement("UPDATE `obj_warehouse` SET `warehouse_agate`=? WHERE `UID` = ?")) {
 
             preparedStatement.setInt(1, amount);
-            preparedStatement.setInt(2, wh.UID);
+            preparedStatement.setInt(2, wh.getUID());
 
             return (preparedStatement.executeUpdate() > 0);
 
@@ -301,7 +301,7 @@ public class dbWarehouseHandler extends dbHandlerBase {
              PreparedStatement preparedStatement = connection.prepareStatement("UPDATE `obj_warehouse` SET `warehouse_diamond`=? WHERE `UID` = ?")) {
 
             preparedStatement.setInt(1, amount);
-            preparedStatement.setInt(2, wh.UID);
+            preparedStatement.setInt(2, wh.getUID());
 
             return (preparedStatement.executeUpdate() > 0);
 
@@ -317,7 +317,7 @@ public class dbWarehouseHandler extends dbHandlerBase {
              PreparedStatement preparedStatement = connection.prepareStatement("UPDATE `obj_warehouse` SET `warehouse_onyx`=? WHERE `UID` = ?")) {
 
             preparedStatement.setInt(1, amount);
-            preparedStatement.setInt(2, wh.UID);
+            preparedStatement.setInt(2, wh.getUID());
 
             return (preparedStatement.executeUpdate() > 0);
 
@@ -333,7 +333,7 @@ public class dbWarehouseHandler extends dbHandlerBase {
              PreparedStatement preparedStatement = connection.prepareStatement("UPDATE `obj_warehouse` SET `warehouse_azoth`=? WHERE `UID` = ?")) {
 
             preparedStatement.setInt(1, amount);
-            preparedStatement.setInt(2, wh.UID);
+            preparedStatement.setInt(2, wh.getUID());
 
             return (preparedStatement.executeUpdate() > 0);
 
@@ -349,7 +349,7 @@ public class dbWarehouseHandler extends dbHandlerBase {
              PreparedStatement preparedStatement = connection.prepareStatement("UPDATE `obj_warehouse` SET `warehouse_orichalk`=? WHERE `UID` = ?")) {
 
             preparedStatement.setInt(1, amount);
-            preparedStatement.setInt(2, wh.UID);
+            preparedStatement.setInt(2, wh.getUID());
 
             return (preparedStatement.executeUpdate() > 0);
 
@@ -365,7 +365,7 @@ public class dbWarehouseHandler extends dbHandlerBase {
              PreparedStatement preparedStatement = connection.prepareStatement("UPDATE `obj_warehouse` SET `warehouse_antimony`=? WHERE `UID` = ?")) {
 
             preparedStatement.setInt(1, amount);
-            preparedStatement.setInt(2, wh.UID);
+            preparedStatement.setInt(2, wh.getUID());
 
             return (preparedStatement.executeUpdate() > 0);
 
@@ -381,7 +381,7 @@ public class dbWarehouseHandler extends dbHandlerBase {
              PreparedStatement preparedStatement = connection.prepareStatement("UPDATE `obj_warehouse` SET `warehouse_sulfur`=? WHERE `UID` = ?")) {
 
             preparedStatement.setInt(1, amount);
-            preparedStatement.setInt(2, wh.UID);
+            preparedStatement.setInt(2, wh.getUID());
 
             return (preparedStatement.executeUpdate() > 0);
 
@@ -397,7 +397,7 @@ public class dbWarehouseHandler extends dbHandlerBase {
              PreparedStatement preparedStatement = connection.prepareStatement("UPDATE `obj_warehouse` SET `warehouse_quicksilver`=? WHERE `UID` = ?")) {
 
             preparedStatement.setInt(1, amount);
-            preparedStatement.setInt(2, wh.UID);
+            preparedStatement.setInt(2, wh.getUID());
 
             return (preparedStatement.executeUpdate() > 0);
 
@@ -413,7 +413,7 @@ public class dbWarehouseHandler extends dbHandlerBase {
              PreparedStatement preparedStatement = connection.prepareStatement("UPDATE `obj_warehouse` SET `warehouse_galvor`=? WHERE `UID` = ?")) {
 
             preparedStatement.setInt(1, amount);
-            preparedStatement.setInt(2, wh.UID);
+            preparedStatement.setInt(2, wh.getUID());
 
             return (preparedStatement.executeUpdate() > 0);
 
@@ -429,7 +429,7 @@ public class dbWarehouseHandler extends dbHandlerBase {
              PreparedStatement preparedStatement = connection.prepareStatement("UPDATE `obj_warehouse` SET `warehouse_wormwood`=? WHERE `UID` = ?")) {
 
             preparedStatement.setInt(1, amount);
-            preparedStatement.setInt(2, wh.UID);
+            preparedStatement.setInt(2, wh.getUID());
 
             return (preparedStatement.executeUpdate() > 0);
 
@@ -445,7 +445,7 @@ public class dbWarehouseHandler extends dbHandlerBase {
              PreparedStatement preparedStatement = connection.prepareStatement("UPDATE `obj_warehouse` SET `warehouse_obsidian`=? WHERE `UID` = ?")) {
 
             preparedStatement.setInt(1, amount);
-            preparedStatement.setInt(2, wh.UID);
+            preparedStatement.setInt(2, wh.getUID());
 
             return (preparedStatement.executeUpdate() > 0);
 
@@ -461,7 +461,7 @@ public class dbWarehouseHandler extends dbHandlerBase {
              PreparedStatement preparedStatement = connection.prepareStatement("UPDATE `obj_warehouse` SET `warehouse_bloodstone`=? WHERE `UID` = ?")) {
 
             preparedStatement.setInt(1, amount);
-            preparedStatement.setInt(2, wh.UID);
+            preparedStatement.setInt(2, wh.getUID());
 
             return (preparedStatement.executeUpdate() > 0);
 
@@ -477,7 +477,7 @@ public class dbWarehouseHandler extends dbHandlerBase {
              PreparedStatement preparedStatement = connection.prepareStatement("UPDATE `obj_warehouse` SET `warehouse_mithril`=? WHERE `UID` = ?")) {
 
             preparedStatement.setInt(1, amount);
-            preparedStatement.setInt(2, wh.UID);
+            preparedStatement.setInt(2, wh.getUID());
 
             return (preparedStatement.executeUpdate() > 0);
 
@@ -545,7 +545,7 @@ public class dbWarehouseHandler extends dbHandlerBase {
             while (rs.next()) {
                 warehouse = new Warehouse(rs);
                 warehouse.runAfterLoad();
-                Warehouse.loadAllTransactions(warehouse);
+                warehouse.loadAllTransactions();
             }
 
         } catch (SQLException e) {

src/engine/devcmd/cmds/AddMobCmd.java

@@ -10,15 +10,18 @@
 package engine.devcmd.cmds;
 
 import engine.Enum;
+import engine.Enum.GameObjectType;
 import engine.devcmd.AbstractDevCmd;
 import engine.gameManager.ChatManager;
+import engine.gameManager.DbManager;
 import engine.gameManager.ZoneManager;
-import engine.objects.AbstractGameObject;
-import engine.objects.Mob;
-import engine.objects.PlayerCharacter;
-import engine.objects.Zone;
+import engine.math.Vector3fImmutable;
+import engine.objects.*;
 import org.pmw.tinylog.Logger;
 
+/**
+ * @author Eighty
+ */
 public class AddMobCmd extends AbstractDevCmd {
 
     public AddMobCmd() {
@@ -35,8 +38,27 @@ public class AddMobCmd extends AbstractDevCmd {
 
         Zone zone = ZoneManager.findSmallestZone(pc.getLoc());
 
-        int loadID;
+        if (words[0].equals("all")) {
 
+            for (AbstractGameObject mobbaseAGO : DbManager.getList(GameObjectType.MobBase)) {
+                MobBase mb = (MobBase) mobbaseAGO;
+                int loadID = mb.getObjectUUID();
+                Mob mob = Mob.createMob(loadID, Vector3fImmutable.getRandomPointInCircle(pc.getLoc(), 100),
+                        null, zone, null, null, "", 1, Enum.AIAgentType.MOBILE);
+                if (mob != null) {
+                    mob.updateDatabase();
+                    this.setResult(String.valueOf(mob.getDBID()));
+                } else {
+                    throwbackError(pc, "Failed to create mob of type " + loadID);
+                    Logger.error("Failed to create mob of type "
+                            + loadID);
+                }
+            }
+            return;
+        }
+
+
+        int loadID;
         try {
             loadID = Integer.parseInt(words[0]);
         } catch (NumberFormatException e) {
@@ -50,6 +72,7 @@ public class AddMobCmd extends AbstractDevCmd {
             return; // NaN
         }
 
+
         if (zone == null) {
             throwbackError(pc, "Failed to find zone to place mob in.");
             return;
@@ -60,9 +83,9 @@ public class AddMobCmd extends AbstractDevCmd {
             return;
         }
 
+
         Mob mob = Mob.createMob(loadID, pc.getLoc(),
                 null, zone, null, null, "", 1, Enum.AIAgentType.MOBILE);
-
         if (mob != null) {
             mob.updateDatabase();
             ChatManager.chatSayInfo(pc,

src/engine/devcmd/cmds/ColliderCmd.java

@@ -0,0 +1,79 @@
+// • ▌ ▄ ·.  ▄▄▄·  ▄▄ • ▪   ▄▄· ▄▄▄▄·  ▄▄▄·  ▐▄▄▄  ▄▄▄ .
+// ·██ ▐███▪▐█ ▀█ ▐█ ▀ ▪██ ▐█ ▌▪▐█ ▀█▪▐█ ▀█ •█▌ ▐█▐▌·
+// ▐█ ▌▐▌▐█·▄█▀▀█ ▄█ ▀█▄▐█·██ ▄▄▐█▀▀█▄▄█▀▀█ ▐█▐ ▐▌▐▀▀▀
+// ██ ██▌▐█▌▐█ ▪▐▌▐█▄▪▐█▐█▌▐███▌██▄▪▐█▐█ ▪▐▌██▐ █▌▐█▄▄▌
+// ▀▀  █▪▀▀▀ ▀  ▀ ·▀▀▀▀ ▀▀▀·▀▀▀ ·▀▀▀▀  ▀  ▀ ▀▀  █▪ ▀▀▀
+//      Magicbane Emulator Project © 2013 - 2022
+//                www.magicbane.com
+
+
+package engine.devcmd.cmds;
+
+import engine.Enum;
+import engine.Enum.BuildingGroup;
+import engine.Enum.GameObjectType;
+import engine.Enum.TargetColor;
+import engine.collisionEngine.Mesh;
+import engine.devcmd.AbstractDevCmd;
+import engine.gameManager.BuildingManager;
+import engine.gameManager.SessionManager;
+import engine.math.Vector3fImmutable;
+import engine.objects.*;
+import engine.util.StringUtils;
+
+import java.text.DecimalFormat;
+import java.util.ArrayList;
+import java.util.concurrent.ConcurrentHashMap;
+
+
+/**
+ * @author
+ */
+public class ColliderCmd extends AbstractDevCmd {
+
+    public ColliderCmd() {
+        super("collider");
+    }
+
+    @Override
+    protected void _doCmd(PlayerCharacter pc, String[] words,
+                          AbstractGameObject target) {
+        // Arg Count Check
+        if (words.length != 1) {
+            this.sendUsage(pc);
+            return;
+        }
+        if (pc == null) {
+            return;
+        }
+
+        String newline = "\r\n ";
+        String output = "----------Collider Information----------" + newline;
+        if(target.getObjectType().equals(GameObjectType.Building) == false)
+            throwbackInfo(pc, "You Must Select A Building");
+
+        Building building = (Building)target;
+        output += "Mesh Count: " + ((Building) target).buildingMeshes.size() + newline;
+        for(Mesh mesh : building.buildingMeshes){
+            output += "------------------------------" + newline;
+            output += "Mesh ID: " + mesh.meshData.meshID + newline;
+            output += "Mesh Tri Count: " + mesh.triangles.size() + newline;
+            output += "Mesh Bounds: " + mesh.bounds + newline;
+            output += "Mesh Min/Max: " + mesh.mesh_minY + " / " + mesh.mesh_maxY + newline;
+            output += "Location Inside: " + mesh.bounds.contains(pc.loc.x,pc.loc.z * -1) + newline;
+            output += "------------------------------" + newline;
+        }
+        throwbackInfo(pc, output);
+    }
+
+    @Override
+    protected String _getHelpString() {
+        return "Gets information on an Object.";
+    }
+
+    @Override
+    protected String _getUsageString() {
+        return "' /info targetID'";
+    }
+
+}

src/engine/devcmd/cmds/InfoCmd.java

@@ -220,16 +220,16 @@ public class InfoCmd extends AbstractDevCmd {
 
                 // List hirelings
 
-                if (targetBuilding.hirelings.isEmpty() == false) {
+                if (targetBuilding.getHirelings().isEmpty() == false) {
 
                     output += newline;
                     output += "Hirelings List: name / slot / floor";
 
-                    for (AbstractCharacter npc : targetBuilding.hirelings) {
+                    for (AbstractCharacter npc : targetBuilding.getHirelings().keySet()) {
 
                         if (npc.getObjectType() != GameObjectType.NPC)
                             continue;
-                        output += newline + npc.getName() + " slot " + targetBuilding.hirelings.indexOf(npc);
+                        output += newline + npc.getName() + " slot " + targetBuilding.getHirelings().get(npc);
                         output += newline + "location " + npc.getLoc();
                     }
                 }

src/engine/devcmd/cmds/PurgeObjectsCmd.java

@@ -48,7 +48,7 @@ public class PurgeObjectsCmd extends AbstractDevCmd {
         for (Building building : zone.zoneBuildingSet) {
             if (!BuildingManager.IsWallPiece(building))
                 continue;
-            for (AbstractCharacter ac : building.hirelings) {
+            for (AbstractCharacter ac : building.getHirelings().keySet()) {
                 NPC npc = null;
                 Mob mobA = null;
 
@@ -143,7 +143,7 @@ public class PurgeObjectsCmd extends AbstractDevCmd {
         if ((building.getBlueprint() != null) && (building.getBlueprint().getBuildingGroup() == BuildingGroup.SPIRE))
             building.disableSpire(false);
 
-        for (AbstractCharacter ac : building.hirelings) {
+        for (AbstractCharacter ac : building.getHirelings().keySet()) {
             NPC npc = null;
             Mob mobA = null;
 

src/engine/devcmd/cmds/RemoveObjectCmd.java

@@ -144,7 +144,7 @@ public class RemoveObjectCmd extends AbstractDevCmd {
 
         // Remove hirelings for this building
 
-        for (AbstractCharacter abstractCharacter : building.hirelings)
+        for (AbstractCharacter abstractCharacter : building.getHirelings().keySet())
             BuildingManager.removeHireling(building, abstractCharacter);
 
         Zone zone = building.getParentZone();
@@ -173,7 +173,7 @@ public class RemoveObjectCmd extends AbstractDevCmd {
         // Remove npc from hirelings list.
 
         if (npc.building != null)
-            npc.building.hirelings.remove(npc);
+            npc.building.getHirelings().remove(npc);
 
         for (Integer minionUUID : npc.minions) {
             Mob minionMob = Mob.getMob(minionUUID);
@@ -210,7 +210,7 @@ public class RemoveObjectCmd extends AbstractDevCmd {
         // Remove npc from hirelings list.
 
         if (mob.building != null)
-            mob.building.hirelings.remove(mob);
+            mob.building.getHirelings().remove(mob);
 
         mob.setLoc(Vector3fImmutable.ZERO);    //Move it off the plane..
         mob.setBindLoc(Vector3fImmutable.ZERO);    //Reset the bind loc..

src/engine/devcmd/cmds/SlotTestCmd.java

@@ -53,7 +53,7 @@ public class SlotTestCmd extends AbstractDevCmd {
 
             int slot = Integer.parseInt(args[0]);
             Vector3fImmutable slotLocation;
-            BuildingLocation buildingLocation = BuildingManager._slotLocations.get(building.meshUUID).get(slot);
+            BuildingLocation buildingLocation = BuildingManager._slotLocations.get(building.meshUUID).get(slot - 1);
             slotLocation = building.getLoc().add(buildingLocation.getLocation());
             slotLocation = Vector3fImmutable.rotateAroundPoint(building.getLoc(), slotLocation, building.getBounds().getQuaternion().angleY);
             playerCharacter.teleport(slotLocation);
@@ -61,17 +61,17 @@ public class SlotTestCmd extends AbstractDevCmd {
         }
 
         for (BuildingLocation buildingLocation : BuildingManager._slotLocations.get(building.meshUUID))
-            outString += buildingLocation.getSlot() - 1 + buildingLocation.getLocation().toString() + "\r\n";
+            outString += buildingLocation.getSlot() + buildingLocation.getLocation().toString() + "\r\n";
 
         outString += "\r\nNext Available Slot: " + BuildingManager.getAvailableSlot(building);
 
-        if (building.hirelings.isEmpty() == false) {
+        if (building.getHirelings().isEmpty() == false) {
 
             outString += "\r\n\r\n";
             outString += "Hirelings List:";
 
-            for (AbstractCharacter hireling : building.hirelings)
-                outString += "\r\n" + hireling.getName() + " slot : " + building.hirelings.indexOf(hireling);
+            for (AbstractCharacter hireling : building.getHirelings().keySet())
+                outString += "\r\n" + hireling.getName() + " slot : " + building.getHirelings().get(hireling);
 
         }
 

src/engine/gameManager/BuildingManager.java

@@ -14,12 +14,18 @@ import engine.Enum.BuildingGroup;
 import engine.Enum.GameObjectType;
 import engine.InterestManagement.InterestManager;
 import engine.InterestManagement.WorldGrid;
+import engine.collisionEngine.CollisionManager;
+import engine.collisionEngine.Mesh;
+import engine.collisionEngine.MeshData;
 import engine.job.JobContainer;
 import engine.job.JobScheduler;
 import engine.jobs.UpgradeBuildingJob;
 import engine.math.Bounds;
 import engine.math.Vector3fImmutable;
+import engine.net.client.ClientConnection;
 import engine.net.client.msg.ErrorPopupMsg;
+import engine.net.client.msg.ManageCityAssetsMsg;
+import engine.net.client.msg.PlaceAssetMsg;
 import engine.objects.*;
 import engine.server.MBServerStatics;
 import org.pmw.tinylog.Logger;
@@ -53,14 +59,12 @@ public enum BuildingManager {
 
         int numOfSlots = _slotLocations.get(building.meshUUID).size();
 
-        if (building.hirelings.isEmpty())
-            return 0;
-
-        if (building.hirelings.size() == numOfSlots)
-            return -1;
-
-        return building.hirelings.size();
+        for (int i = 1; i <= numOfSlots; i++) {
 
+            if (!building.getHirelings().containsValue(i))
+                return i;
+        }
+        return -1;
     }
 
     public static int getLastAvailableSlot(Building building) {
@@ -75,13 +79,12 @@ public enum BuildingManager {
 
         int numOfSlots = _slotLocations.get(building.meshUUID).size();
 
-        if (building.hirelings.isEmpty())
-            return numOfSlots - 1;
+        for (int i = numOfSlots; i > 0; i--) {
 
-        if (building.hirelings.size() == numOfSlots)
-            return -1;
-
-        return numOfSlots - 1;
+            if (!building.getHirelings().containsValue(i))
+                return i;
+        }
+        return -1;
     }
 
     public static BuildingLocation getSlotLocation(Building building, int slot) {
@@ -91,7 +94,7 @@ public enum BuildingManager {
         if (slot == -1)
             return buildingLocation;
 
-        buildingLocation = _slotLocations.get(building.meshUUID).get(slot); // array index
+        buildingLocation = _slotLocations.get(building.meshUUID).get(slot - 1); // array index
 
         if (buildingLocation == null) {
             Logger.error("Invalid slot for building: " + building.getObjectUUID());
@@ -100,20 +103,6 @@ public enum BuildingManager {
         return buildingLocation;
     }
 
-    public static void updateHirelingSlots(Building building) {
-
-        // Method updates all hirelings with the slot locations
-        // for the current mesh.  The position and region can change
-        // such as when upgrading a TOL to R8.
-
-        for (AbstractCharacter hireling : building.hirelings) {
-
-            hireling.setLoc(getSlotLocation(building, building.hirelings.indexOf(hireling)).getLocation());
-            InterestManager.setObjectDirty(hireling);
-        }
-
-    }
-
     public static boolean playerCanManage(PlayerCharacter player, Building building) {
 
         if (player == null)
@@ -248,15 +237,15 @@ public enum BuildingManager {
                         ChatManager.chatSystemInfo(player, "You can not carry any more of that item.");
                         return false;
                     }
-                    if (warehouse.resources.get(resourceBase) == null)
+                    if (warehouse.getResources().get(resourceBase) == null)
                         continue;
 
-                    int resourceAmount = warehouse.resources.get(resourceBase);
+                    int resourceAmount = warehouse.getResources().get(resourceBase);
 
                     if (resourceAmount <= 0)
                         continue;
 
-                    if (Warehouse.loot(warehouse, player, resourceBase, resourceAmount, true))
+                    if (warehouse.loot(player, resourceBase, resourceAmount, true))
                         ChatManager.chatInfoInfo(player, "You have looted " + resourceAmount + ' ' + resourceBase.getName());
                 }
                 break;
@@ -305,7 +294,7 @@ public enum BuildingManager {
 
     // Method transfers ownership of all hirelings in a building
 
-    public static void updateHirelingGuilds(Building building) {
+    public static void refreshHirelings(Building building) {
 
         if (building == null)
             return;
@@ -317,7 +306,7 @@ public enum BuildingManager {
         else
             newGuild = building.getOwner().getGuild();
 
-        for (AbstractCharacter hireling : building.hirelings) {
+        for (AbstractCharacter hireling : building.getHirelings().keySet()) {
             hireling.setGuild(newGuild);
             WorldGrid.updateObject(hireling);
         }
@@ -350,7 +339,7 @@ public enum BuildingManager {
 
         // Remove hireling from building
 
-        building.hirelings.remove(hireling);
+        building.getHirelings().remove(hireling);
 
         // Remove from zone mob set
 
@@ -402,7 +391,7 @@ public enum BuildingManager {
 
         if (building.getRank() < 1) {
 
-            for (AbstractCharacter slottedNPC : building.hirelings)
+            for (AbstractCharacter slottedNPC : building.getHirelings().keySet())
                 BuildingManager.removeHireling(building, slottedNPC);
 
             return;
@@ -410,17 +399,14 @@ public enum BuildingManager {
 
         // Delete hireling if building has deranked.
 
-        for (AbstractCharacter hireling : building.hirelings) {
+        for (AbstractCharacter hireling : building.getHirelings().keySet()) {
 
-            if (building.hirelings.indexOf(hireling) + 1 > building.getBlueprint().getSlotsForRank(building.getRank()))
+            if (building.getHirelings().get(hireling) > building.getBlueprint().getSlotsForRank(building.getRank()))
                 BuildingManager.removeHireling(building, hireling);
 
         }
 
-        // Update hireling guilds and locations
-
-        updateHirelingGuilds(building);
-        updateHirelingSlots(building);
+        refreshHirelings(building);
     }
 
     public static Building getBuilding(int id) {
@@ -562,7 +548,7 @@ public enum BuildingManager {
 
         NPC npc = null;
 
-        npc = NPC.createNPC(pirateName, NpcID.getObjectUUID(), NpcLoc, building.getGuild(), zone, (short) rank, building);
+        npc = NPC.createNPC(pirateName, NpcID.getObjectUUID(), NpcLoc, null, zone, (short) rank, building);
 
         if (npc == null)
             return false;
@@ -581,7 +567,7 @@ public enum BuildingManager {
         if (building.getBlueprintUUID() == 0)
             return false;
 
-        if (building.getBlueprint().getMaxSlots() == building.hirelings.size())
+        if (building.getBlueprint().getMaxSlots() == building.getHirelings().size())
             return false;
 
         String pirateName = NPCManager.getPirateName(contract.getMobbaseID());
@@ -794,6 +780,83 @@ public enum BuildingManager {
 
     }
 
+    public static void processRedeedNPC(NPC npc, Building building, ClientConnection origin) {
+
+        // Member variable declaration
+        PlayerCharacter player;
+        Contract contract;
+        CharacterItemManager itemMan;
+        ItemBase itemBase;
+        Item item;
+
+        npc.lock.writeLock().lock();
+
+        try {
+
+
+            if (building == null)
+                return;
+            player = SessionManager.getPlayerCharacter(origin);
+            itemMan = player.getCharItemManager();
+
+            contract = npc.getContract();
+
+            if (!player.getCharItemManager().hasRoomInventory((short) 1)) {
+                ErrorPopupMsg.sendErrorPopup(player, 21);
+                return;
+            }
+
+
+            if (!building.getHirelings().containsKey(npc))
+                return;
+
+            if (!npc.remove()) {
+                PlaceAssetMsg.sendPlaceAssetError(player.getClientConnection(), 1, "A Serious error has occurred. Please post details for to ensure transaction integrity");
+                return;
+            }
+
+            building.getHirelings().remove(npc);
+
+            itemBase = ItemBase.getItemBase(contract.getContractID());
+
+            if (itemBase == null) {
+                Logger.error("Could not find Contract for npc: " + npc.getObjectUUID());
+                return;
+            }
+
+            boolean itemWorked = false;
+
+            item = new Item(itemBase, player.getObjectUUID(), Enum.OwnerType.PlayerCharacter, (byte) ((byte) npc.getRank() - 1), (byte) ((byte) npc.getRank() - 1),
+                    (short) 1, (short) 1, true, false, Enum.ItemContainerType.INVENTORY, (byte) 0,
+                    new ArrayList<>(), "");
+            item.setNumOfItems(1);
+            item.containerType = Enum.ItemContainerType.INVENTORY;
+
+            try {
+                item = DbManager.ItemQueries.ADD_ITEM(item);
+                itemWorked = true;
+            } catch (Exception e) {
+                Logger.error(e);
+            }
+            if (itemWorked) {
+                itemMan.addItemToInventory(item);
+                itemMan.updateInventory();
+            }
+
+            ManageCityAssetsMsg mca = new ManageCityAssetsMsg();
+            mca.actionType = NPC.SVR_CLOSE_WINDOW;
+            mca.setTargetType(building.getObjectType().ordinal());
+            mca.setTargetID(building.getObjectUUID());
+            origin.sendMsg(mca);
+
+        } catch (Exception e) {
+            Logger.error(e);
+        } finally {
+            npc.lock.writeLock().unlock();
+        }
+
+    }
+
     public static void rebuildMine(Building mineBuilding) {
         setRank(mineBuilding, 1);
         mineBuilding.meshUUID = mineBuilding.getBlueprint().getMeshForRank(mineBuilding.rank);
@@ -811,7 +874,7 @@ public enum BuildingManager {
         boolean success;
 
 
-        // If this building has no blueprint then set rank and exit immediately.
+        // If this building has no blueprint then set rank and exit immediatly.
 
         if (building.blueprintUUID == 0 || building.getBlueprint() != null && building.getBlueprint().getBuildingGroup().equals(BuildingGroup.MINE)) {
             building.rank = rank;
@@ -829,7 +892,7 @@ public enum BuildingManager {
                 Logger.error("failed to cancel existing upgrade job.");
         }
 
-        // Attempt to write to database, or delete the building
+        // Attempt write to database, or delete the building
         // if we are destroying it.
 
         if (rank == -1)
@@ -883,6 +946,8 @@ public enum BuildingManager {
         cleanupHirelings(building);
 
         building.isDeranking.compareAndSet(true, false);
+
+        BakeColliders(building);
     }
 
     public static Building getBuildingAtLocation(Vector3fImmutable loc) {
@@ -900,4 +965,17 @@ public enum BuildingManager {
         return null;
     }
 
+    public static void BakeColliders(Building building){
+        int propID = building.meshUUID;
+
+        if(CollisionManager.structure_meshes.containsKey(propID) == false) {
+            Logger.error("Bake for PropID: " + propID + " Failed");
+            return;
+        }
+        building.buildingMeshes = new ArrayList<>();
+        for(MeshData data : CollisionManager.structure_meshes.get(propID)){
+            building.buildingMeshes.add(new Mesh(data,building.getObjectUUID()));
+        }
+    }
+
 }

src/engine/gameManager/DevCmdManager.java

@@ -58,6 +58,7 @@ public enum DevCmdManager {
         DevCmdManager.registerDevCmd(new PrintResistsCmd());
         DevCmdManager.registerDevCmd(new PrintLocationCmd());
         DevCmdManager.registerDevCmd(new InfoCmd());
+        DevCmdManager.registerDevCmd(new ColliderCmd());
         DevCmdManager.registerDevCmd(new aiInfoCmd());
         DevCmdManager.registerDevCmd(new SimulateBootyCmd());
         DevCmdManager.registerDevCmd(new GetHeightCmd());

src/engine/gameManager/MaintenanceManager.java

@@ -184,7 +184,7 @@ public enum MaintenanceManager {
         if ((overDraft > 0))
             if ((building.getBlueprint().getBuildingGroup().equals(Enum.BuildingGroup.SHRINE) == false) &&
                     (warehouse != null) && building.assetIsProtected() == true &&
-                    (warehouse.resources.get(ItemBase.GOLD_ITEM_BASE)) >= overDraft) {
+                    (warehouse.getResources().get(ItemBase.GOLD_ITEM_BASE)) >= overDraft) {
                 hasFunds = true;
             }
 
@@ -199,22 +199,22 @@ public enum MaintenanceManager {
                 hasResources = false;
             else {
 
-                resourceValue = warehouse.resources.get(ItemBase.getItemBase(1580000));
+                resourceValue = warehouse.getResources().get(Warehouse.stoneIB);
 
                 if (resourceValue < 1500)
                     hasResources = false;
 
-                resourceValue = warehouse.resources.get(ItemBase.getItemBase(1580004));
+                resourceValue = warehouse.getResources().get(Warehouse.lumberIB);
 
                 if (resourceValue < 1500)
                     hasResources = false;
 
-                resourceValue = warehouse.resources.get(ItemBase.getItemBase(1580017));
+                resourceValue = warehouse.getResources().get(Warehouse.galvorIB);
 
                 if (resourceValue < 5)
                     hasResources = false;
 
-                resourceValue = warehouse.resources.get(ItemBase.getItemBase(1580018));
+                resourceValue = warehouse.getResources().get(Warehouse.wormwoodIB);
 
                 if (resourceValue < 5)
                     hasResources = false;
@@ -247,11 +247,11 @@ public enum MaintenanceManager {
 
         if (overDraft > 0) {
 
-            resourceValue = warehouse.resources.get(ItemBase.getItemBase(7));
+            resourceValue = warehouse.getResources().get(Warehouse.goldIB);
 
             if (DbManager.WarehouseQueries.updateGold(warehouse, resourceValue - overDraft) == true) {
-                warehouse.resources.put(ItemBase.getItemBase(7), resourceValue - overDraft);
-                Warehouse.AddTransactionToWarehouse(warehouse, Enum.GameObjectType.Building, building.getObjectUUID(), Enum.TransactionType.WITHDRAWL, Resource.GOLD, overDraft);
+                warehouse.getResources().put(Warehouse.goldIB, resourceValue - overDraft);
+                warehouse.AddTransactionToWarehouse(Enum.GameObjectType.Building, building.getObjectUUID(), Enum.TransactionType.WITHDRAWL, Resource.GOLD, overDraft);
             } else {
                 Logger.error("gold update failed for warehouse of UUID:" + warehouse.getObjectUUID());
                 return true;
@@ -267,11 +267,11 @@ public enum MaintenanceManager {
 
         // Withdraw Stone
 
-        resourceValue = warehouse.resources.get(ItemBase.getItemBase(1580000));
+        resourceValue = warehouse.getResources().get(Warehouse.stoneIB);
 
         if (DbManager.WarehouseQueries.updateStone(warehouse, resourceValue - 1500) == true) {
-            warehouse.resources.put(ItemBase.getItemBase(1580000), resourceValue - 1500);
-            Warehouse.AddTransactionToWarehouse(warehouse, Enum.GameObjectType.Building, building.getObjectUUID(), Enum.TransactionType.WITHDRAWL, Resource.STONE, 1500);
+            warehouse.getResources().put(Warehouse.stoneIB, resourceValue - 1500);
+            warehouse.AddTransactionToWarehouse(Enum.GameObjectType.Building, building.getObjectUUID(), Enum.TransactionType.WITHDRAWL, Resource.STONE, 1500);
         } else {
             Logger.error("stone update failed for warehouse of UUID:" + warehouse.getObjectUUID());
             return true;
@@ -279,11 +279,11 @@ public enum MaintenanceManager {
 
         // Withdraw Lumber
 
-        resourceValue = warehouse.resources.get(ItemBase.getItemBase(1580004));
+        resourceValue = warehouse.getResources().get(Warehouse.lumberIB);
 
         if (DbManager.WarehouseQueries.updateLumber(warehouse, resourceValue - 1500) == true) {
-            warehouse.resources.put(ItemBase.getItemBase(1580004), resourceValue - 1500);
-            Warehouse.AddTransactionToWarehouse(warehouse, Enum.GameObjectType.Building, building.getObjectUUID(), Enum.TransactionType.WITHDRAWL, Resource.LUMBER, 1500);
+            warehouse.getResources().put(Warehouse.lumberIB, resourceValue - 1500);
+            warehouse.AddTransactionToWarehouse(Enum.GameObjectType.Building, building.getObjectUUID(), Enum.TransactionType.WITHDRAWL, Resource.LUMBER, 1500);
         } else {
             Logger.error("lumber update failed for warehouse of UUID:" + warehouse.getObjectUUID());
             return true;
@@ -291,21 +291,21 @@ public enum MaintenanceManager {
 
         // Withdraw Galvor
 
-        resourceValue = warehouse.resources.get(ItemBase.getItemBase(1580017));
+        resourceValue = warehouse.getResources().get(Warehouse.galvorIB);
 
         if (DbManager.WarehouseQueries.updateGalvor(warehouse, resourceValue - 5) == true) {
-            warehouse.resources.put(ItemBase.getItemBase(1580017), resourceValue - 5);
-            Warehouse.AddTransactionToWarehouse(warehouse, Enum.GameObjectType.Building, building.getObjectUUID(), Enum.TransactionType.WITHDRAWL, Resource.GALVOR, 5);
+            warehouse.getResources().put(Warehouse.galvorIB, resourceValue - 5);
+            warehouse.AddTransactionToWarehouse(Enum.GameObjectType.Building, building.getObjectUUID(), Enum.TransactionType.WITHDRAWL, Resource.GALVOR, 5);
         } else {
             Logger.error("galvor update failed for warehouse of UUID:" + warehouse.getObjectUUID());
             return true;
         }
 
-        resourceValue = warehouse.resources.get(ItemBase.getItemBase(1580018));
+        resourceValue = warehouse.getResources().get(Warehouse.wormwoodIB);
 
         if (DbManager.WarehouseQueries.updateWormwood(warehouse, resourceValue - 5) == true) {
-            warehouse.resources.put(ItemBase.getItemBase(1580018), resourceValue - 5);
-            Warehouse.AddTransactionToWarehouse(warehouse, Enum.GameObjectType.Building, building.getObjectUUID(), Enum.TransactionType.WITHDRAWL, Resource.WORMWOOD, 5);
+            warehouse.getResources().put(Warehouse.wormwoodIB, resourceValue - 5);
+            warehouse.AddTransactionToWarehouse(Enum.GameObjectType.Building, building.getObjectUUID(), Enum.TransactionType.WITHDRAWL, Resource.WORMWOOD, 5);
         } else {
             Logger.error("wyrmwood update failed for warehouse of UUID:" + warehouse.getObjectUUID());
         }

src/engine/gameManager/NPCManager.java

@@ -5,14 +5,17 @@ import engine.InterestManagement.WorldGrid;
 import engine.math.Quaternion;
 import engine.math.Vector3f;
 import engine.math.Vector3fImmutable;
+import engine.mobileAI.MobAI;
 import engine.net.Dispatch;
 import engine.net.DispatchMessage;
 import engine.net.client.msg.PetMsg;
 import engine.objects.*;
 import engine.powers.EffectsBase;
+import engine.powers.PowersBase;
 import engine.powers.RuneSkillAdjustEntry;
 import org.pmw.tinylog.Logger;
 
+import javax.smartcardio.ATR;
 import java.util.ArrayList;
 import java.util.HashMap;
 import java.util.concurrent.ThreadLocalRandom;
@@ -197,7 +200,7 @@ public enum NPCManager {
 
         // Pets are regular mobiles not hirelings (Siege engines)
         if (abstractCharacter.contract != null)
-            abstractCharacter.building.hirelings.addIfAbsent(abstractCharacter);
+            abstractCharacter.building.getHirelings().put(abstractCharacter, buildingSlot);
 
         // Override bind and location for  this npc derived
         // from BuildingManager slot location data.

src/engine/mobileAI/MobAI.java

@@ -939,16 +939,16 @@ public class MobAI {
 
                 //dont scan self.
 
-                if (mob.equals(awoMob))
+                if (mob.equals(awoMob) || (mob.agentType.equals(Enum.AIAgentType.GUARDCAPTAIN)) == true)
                     continue;
 
                 Mob aggroMob = (Mob) awoMob;
 
                 //don't attack other guards
-                if (aggroMob.isGuard() == true)
+
+                if ((aggroMob.agentType.equals(Enum.AIAgentType.GUARDCAPTAIN)))
                     continue;
 
-                //don't attack pets
                 if (aggroMob.agentType.equals(Enum.AIAgentType.PET))
                     continue;
 

src/engine/net/client/handlers/ActivateNPCMsgHandler.java

@@ -78,7 +78,7 @@ public class ActivateNPCMsgHandler extends AbstractClientMsgHandler {
                 return false;
             }
 
-            if (building.getBlueprint().getMaxSlots() == building.hirelings.size())
+            if (building.getBlueprint().getMaxSlots() == building.getHirelings().size())
                 return false;
 
             Item contractItem = Item.getFromCache(msg.getContractItem());

src/engine/net/client/handlers/ManageCityAssetMsgHandler.java

@@ -171,7 +171,7 @@ public class ManageCityAssetMsgHandler extends AbstractClientMsgHandler {
                                 if (warehouse == null)
                                     return true;
 
-                                if (Warehouse.isEmpty(warehouse)) {
+                                if (warehouse.isEmpty()) {
                                     ErrorPopupMsg.sendErrorPopup(player, 167); // no more resources.
                                     return true;
                                 }

src/engine/net/client/handlers/MerchantMsgHandler.java

@@ -437,13 +437,13 @@ public class MerchantMsgHandler extends AbstractClientMsgHandler {
                 DispatchMessage.dispatchMsgDispatch(dispatch, Enum.DispatchChannel.SECONDARY);
                 break;
             case 17:
-                Warehouse.warehouseWithdraw(msg, player, npc);
+                Warehouse.warehouseWithdraw(msg, player, npc, origin);
                 break;
             case 18:
-                Warehouse.warehouseDeposit(msg, player, npc);
+                Warehouse.warehouseDeposit(msg, player, npc, origin);
                 break;
             case 19:
-                Warehouse.warehouseLock(msg, player, npc);
+                Warehouse.warehouseLock(msg, player, npc, origin);
                 break;
         }
 

src/engine/net/client/handlers/ObjectActionMsgHandler.java

@@ -98,27 +98,27 @@ public class ObjectActionMsgHandler extends AbstractClientMsgHandler {
             return false;
         }
 
-        resourceValue = warehouse.resources.get(ItemBase.getItemBase(7));
+        resourceValue = warehouse.getResources().get(Warehouse.goldIB);
 
         if (resourceValue < 5000000)
             hasResources = false;
 
-        resourceValue = warehouse.resources.get(ItemBase.getItemBase(1580000));
+        resourceValue = warehouse.getResources().get(Warehouse.stoneIB);
 
         if (resourceValue < 8000)
             hasResources = false;
 
-        resourceValue = warehouse.resources.get(ItemBase.getItemBase(1580004));
+        resourceValue = warehouse.getResources().get(Warehouse.lumberIB);
 
         if (resourceValue < 8000)
             hasResources = false;
 
-        resourceValue = warehouse.resources.get(ItemBase.getItemBase(1580017));
+        resourceValue = warehouse.getResources().get(Warehouse.galvorIB);
 
         if (resourceValue < 15)
             hasResources = false;
 
-        resourceValue = warehouse.resources.get(ItemBase.getItemBase(1580018));
+        resourceValue = warehouse.getResources().get(Warehouse.wormwoodIB);
 
         if (resourceValue < 15)
             hasResources = false;
@@ -130,51 +130,51 @@ public class ObjectActionMsgHandler extends AbstractClientMsgHandler {
 
         // Remove resources from warehouse before claiming realm
 
-        resourceValue = warehouse.resources.get(ItemBase.getItemBase(7));
+        resourceValue = warehouse.getResources().get(Warehouse.goldIB);
 
         if (DbManager.WarehouseQueries.updateGold(warehouse, resourceValue - 5000000) == true) {
-            warehouse.resources.put(ItemBase.getItemBase(7), resourceValue - 5000000);
-            Warehouse.AddTransactionToWarehouse(warehouse, engine.Enum.GameObjectType.Building, tol.getObjectUUID(), Enum.TransactionType.WITHDRAWL, Resource.GOLD, 5000000);
+            warehouse.getResources().put(Warehouse.goldIB, resourceValue - 5000000);
+            warehouse.AddTransactionToWarehouse(engine.Enum.GameObjectType.Building, tol.getObjectUUID(), Enum.TransactionType.WITHDRAWL, Resource.GOLD, 5000000);
         } else {
             Logger.error("gold update failed for warehouse of UUID:" + warehouse.getObjectUUID());
             return false;
         }
 
-        resourceValue = warehouse.resources.get(ItemBase.getItemBase(1580000));
+        resourceValue = warehouse.getResources().get(Warehouse.stoneIB);
 
         if (DbManager.WarehouseQueries.updateStone(warehouse, resourceValue - 8000) == true) {
-            warehouse.resources.put(ItemBase.getItemBase(1580000), resourceValue - 8000);
-            Warehouse.AddTransactionToWarehouse(warehouse, engine.Enum.GameObjectType.Building, tol.getObjectUUID(), Enum.TransactionType.WITHDRAWL, Resource.STONE, 8000);
+            warehouse.getResources().put(Warehouse.stoneIB, resourceValue - 8000);
+            warehouse.AddTransactionToWarehouse(engine.Enum.GameObjectType.Building, tol.getObjectUUID(), Enum.TransactionType.WITHDRAWL, Resource.STONE, 8000);
         } else {
             Logger.error("stone update failed for warehouse of UUID:" + warehouse.getObjectUUID());
             return false;
         }
 
-        resourceValue = warehouse.resources.get(ItemBase.getItemBase(1580004));
+        resourceValue = warehouse.getResources().get(Warehouse.lumberIB);
 
         if (DbManager.WarehouseQueries.updateLumber(warehouse, resourceValue - 8000) == true) {
-            warehouse.resources.put(ItemBase.getItemBase(1580004), resourceValue - 8000);
-            Warehouse.AddTransactionToWarehouse(warehouse, engine.Enum.GameObjectType.Building, tol.getObjectUUID(), Enum.TransactionType.WITHDRAWL, Resource.LUMBER, 8000);
+            warehouse.getResources().put(Warehouse.lumberIB, resourceValue - 8000);
+            warehouse.AddTransactionToWarehouse(engine.Enum.GameObjectType.Building, tol.getObjectUUID(), Enum.TransactionType.WITHDRAWL, Resource.LUMBER, 8000);
         } else {
             Logger.error("lumber update failed for warehouse of UUID:" + warehouse.getObjectUUID());
             return false;
         }
 
-        resourceValue = warehouse.resources.get(ItemBase.getItemBase(1580017));
+        resourceValue = warehouse.getResources().get(Warehouse.galvorIB);
 
         if (DbManager.WarehouseQueries.updateGalvor(warehouse, resourceValue - 15) == true) {
-            warehouse.resources.put(ItemBase.getItemBase(1580017), resourceValue - 15);
-            Warehouse.AddTransactionToWarehouse(warehouse, engine.Enum.GameObjectType.Building, tol.getObjectUUID(), Enum.TransactionType.WITHDRAWL, Resource.GALVOR, 15);
+            warehouse.getResources().put(Warehouse.galvorIB, resourceValue - 15);
+            warehouse.AddTransactionToWarehouse(engine.Enum.GameObjectType.Building, tol.getObjectUUID(), Enum.TransactionType.WITHDRAWL, Resource.GALVOR, 15);
         } else {
             Logger.error("galvor update failed for warehouse of UUID:" + warehouse.getObjectUUID());
             return false;
         }
 
-        resourceValue = warehouse.resources.get(ItemBase.getItemBase(1580018));
+        resourceValue = warehouse.getResources().get(Warehouse.wormwoodIB);
 
         if (DbManager.WarehouseQueries.updateWormwood(warehouse, resourceValue - 15) == true) {
-            warehouse.resources.put(ItemBase.getItemBase(1580018), resourceValue - 15);
-            Warehouse.AddTransactionToWarehouse(warehouse, engine.Enum.GameObjectType.Building, tol.getObjectUUID(), Enum.TransactionType.WITHDRAWL, Resource.WORMWOOD, 15);
+            warehouse.getResources().put(Warehouse.wormwoodIB, resourceValue - 15);
+            warehouse.AddTransactionToWarehouse(engine.Enum.GameObjectType.Building, tol.getObjectUUID(), Enum.TransactionType.WITHDRAWL, Resource.WORMWOOD, 15);
         } else {
             Logger.error("wormwood update failed for warehouse of UUID:" + warehouse.getObjectUUID());
             return false;

src/engine/net/client/handlers/OrderNPCMsgHandler.java

@@ -54,7 +54,7 @@ public class OrderNPCMsgHandler extends AbstractClientMsgHandler {
             return;
         }
 
-        if (!building.hirelings.contains(hireling))
+        if (!building.getHirelings().containsKey(hireling))
             return;
 
         BuildingManager.removeHireling(building, hireling);
@@ -402,7 +402,7 @@ public class OrderNPCMsgHandler extends AbstractClientMsgHandler {
             if (building == null)
                 return true;
 
-            if (building.hirelings.contains(npc) == false)
+            if (building.getHirelings().containsKey(npc) == false)
                 return true;
 
 
@@ -421,7 +421,7 @@ public class OrderNPCMsgHandler extends AbstractClientMsgHandler {
                     if (ManageCityAssetMsgHandler.playerCanManageNotFriends(player, building) == false)
                         return true;
 
-                    if (building.hirelings.contains(npc) == false)
+                    if (building.getHirelings().containsKey(npc) == false)
                         return true;
 
                     if (npc.remove() == false) {
@@ -515,7 +515,7 @@ public class OrderNPCMsgHandler extends AbstractClientMsgHandler {
             if (building == null)
                 return true;
 
-            if (!building.hirelings.contains(mob))
+            if (!building.getHirelings().containsKey(mob))
                 return true;
 
             if (player.getCharItemManager().getTradingWith() != null) {
@@ -531,7 +531,7 @@ public class OrderNPCMsgHandler extends AbstractClientMsgHandler {
                     if (BuildingManager.playerCanManage(player, building) == false)
                         return true;
 
-                    if (building.hirelings.contains(mob) == false)
+                    if (building.getHirelings().containsKey(mob) == false)
                         return true;
 
                     BuildingManager.removeHireling(building, mob);

src/engine/net/client/handlers/TaxCityMsgHandler.java

@@ -111,7 +111,7 @@ public class TaxCityMsgHandler extends AbstractClientMsgHandler {
 
         ViewResourcesMessage vrm = new ViewResourcesMessage(player);
         vrm.setGuild(building.getGuild());
-        vrm.setWarehouseBuilding(BuildingManager.getBuildingFromCache(building.getCity().getWarehouse().buildingUID));
+        vrm.setWarehouseBuilding(BuildingManager.getBuildingFromCache(building.getCity().getWarehouse().getBuildingUID()));
         vrm.configure();
         Dispatch dispatch = Dispatch.borrow(player, msg);
         DispatchMessage.dispatchMsgDispatch(dispatch, Enum.DispatchChannel.SECONDARY);

src/engine/net/client/msg/ArcViewAssetTransactionsMsg.java

@@ -61,13 +61,13 @@ public class ArcViewAssetTransactionsMsg extends ClientNetMsg {
 
     public void configure() {
 
-        warehouseBuilding = BuildingManager.getBuilding(this.warehouse.buildingUID);
+        warehouseBuilding = BuildingManager.getBuilding(this.warehouse.getBuildingUID());
         transactions = new ArrayList<>(50);
 
-        if (this.warehouse.transactions.size() > 150) {
-            transactions.addAll(this.warehouse.transactions.subList(this.warehouse.transactions.size() - 150, this.warehouse.transactions.size()));
+        if (this.warehouse.getTransactions().size() > 150) {
+            transactions.addAll(this.warehouse.getTransactions().subList(this.warehouse.getTransactions().size() - 150, this.warehouse.getTransactions().size()));
         } else
-            transactions = this.warehouse.transactions;
+            transactions = this.warehouse.getTransactions();
 
     }
 
@@ -78,7 +78,7 @@ public class ArcViewAssetTransactionsMsg extends ClientNetMsg {
     protected void _serialize(ByteBufferWriter writer) throws SerializationException {
 
         writer.putInt(this.transactionID);
-        writer.putInt(this.warehouse.buildingUID);
+        writer.putInt(this.warehouse.getBuildingUID());
         writer.putInt(transactions.size()); //list Size
 
         for (Transaction transaction : transactions) {
@@ -124,7 +124,7 @@ public class ArcViewAssetTransactionsMsg extends ClientNetMsg {
             writer.putInt(transaction.getTargetUUID()); //ID
             writer.putString(name); //Name of depositer/withdrawler or mine name
             writer.putInt(GameObjectType.Building.ordinal()); //Type
-            writer.putInt(warehouse.buildingUID); //ID
+            writer.putInt(warehouse.getBuildingUID()); //ID
             writer.putString(warehouseBuilding.getName()); //warehouse
             writer.putInt(transaction.getTransactionType().getID()); //79,80 withdrew, 81 mine produced, 82 deposit
             writer.putInt(transaction.getAmount()); //amount

src/engine/net/client/msg/ManageCityAssetsMsg.java

@@ -24,6 +24,7 @@ import org.joda.time.Period;
 import java.time.LocalDateTime;
 import java.time.ZoneId;
 import java.util.Set;
+import java.util.concurrent.ConcurrentHashMap;
 
 /**
  * Open manage city asset window
@@ -228,8 +229,8 @@ public class ManageCityAssetsMsg extends ClientNetMsg {
             writer.putInt(asset.getObjectUUID());
             writer.putInt(0);
             writer.putInt(0);
-            writer.putInt(asset.hirelings.size());
-            for (AbstractCharacter hireling : asset.hirelings) {
+            writer.putInt(asset.getHirelings().size());
+            for (AbstractCharacter hireling : asset.getHirelings().keySet()) {
                 if (!hireling.getObjectType().equals(GameObjectType.NPC))
                     writer.putString(hireling.getName());
                 else {
@@ -261,9 +262,9 @@ public class ManageCityAssetsMsg extends ClientNetMsg {
             for (int a = 0; a < 5; a++)
                 writer.putInt(0);
 
-            writer.putInt(asset.hirelings.size());
+            writer.putInt(asset.getHirelings().size());
 
-            for (AbstractCharacter npcHire : asset.hirelings) {
+            for (AbstractCharacter npcHire : asset.getHirelings().keySet()) {
                 writer.putInt(npcHire.getObjectType().ordinal());
                 writer.putInt(npcHire.getObjectUUID());
                 if (npcHire.getObjectType() == GameObjectType.NPC)
@@ -667,11 +668,11 @@ public class ManageCityAssetsMsg extends ClientNetMsg {
                 writer.putInt(1);
             }
 
-            writer.putInt(building.hirelings.size());
 
-            if (building.hirelings.size() > 0) {
-
-                for (AbstractCharacter npcHire : building.hirelings) {
+            ConcurrentHashMap<AbstractCharacter, Integer> npcList = building.getHirelings();
+            writer.putInt(npcList.size());
+            if (npcList.size() > 0) {
+                for (AbstractCharacter npcHire : npcList.keySet()) {
                     writer.putInt(npcHire.getObjectType().ordinal());
                     if (npcHire.getObjectType() == GameObjectType.Mob)
                         writer.putInt(((Mob) npcHire).getDBID());

src/engine/net/client/msg/ViewResourcesMessage.java

@@ -89,23 +89,23 @@ public class ViewResourcesMessage extends ClientNetMsg {
     @Override
     protected void _serialize(ByteBufferWriter writer) {
 
-        writer.putInt(warehouseObject.resources.size());
+        writer.putInt(warehouseObject.getResources().size());
 
-        for (ItemBase ib : (warehouseObject.resources.keySet())) {
+        for (ItemBase ib : (warehouseObject.getResources().keySet())) {
 
             writer.putInt(ib.getHashID());
-            writer.putInt((warehouseObject.resources.get(ib)));
+            writer.putInt((warehouseObject.getResources().get(ib)));
 
 
-            if (Warehouse.isResourceLocked(warehouseObject, ib) == true)
+            if (warehouseObject.isResourceLocked(ib) == true)
                 writer.put((byte) 1);
             else
                 writer.put((byte) 0);
         }
 
-        writer.putInt(warehouseObject.resources.size());
+        writer.putInt(warehouseObject.getResources().size());
 
-        for (ItemBase ib : warehouseObject.resources.keySet()) {
+        for (ItemBase ib : warehouseObject.getResources().keySet()) {
             writer.putInt(ib.getHashID());
             writer.putInt(0); //available?
             writer.putInt(Warehouse.getMaxResources().get(ib.getUUID())); //max?

src/engine/objects/AbstractCharacter.java

@@ -45,6 +45,7 @@ import java.util.concurrent.locks.ReentrantReadWriteLock;
 
 public abstract class AbstractCharacter extends AbstractWorldObject {
 
+    public float characterHeight = 0;
     protected CharacterItemManager charItemManager;
     private final ReentrantReadWriteLock healthLock = new ReentrantReadWriteLock();
     public short level;

src/engine/objects/Building.java

@@ -14,6 +14,7 @@ import engine.Enum.*;
 import engine.InterestManagement.RealmMap;
 import engine.InterestManagement.Terrain;
 import engine.InterestManagement.WorldGrid;
+import engine.collisionEngine.Mesh;
 import engine.db.archive.CityRecord;
 import engine.db.archive.DataWarehouse;
 import engine.db.archive.MineRecord;
@@ -41,7 +42,6 @@ import java.time.ZoneId;
 import java.util.ArrayList;
 import java.util.HashMap;
 import java.util.concurrent.ConcurrentHashMap;
-import java.util.concurrent.CopyOnWriteArrayList;
 import java.util.concurrent.ThreadLocalRandom;
 import java.util.concurrent.atomic.AtomicBoolean;
 import java.util.concurrent.locks.ReentrantReadWriteLock;
@@ -52,7 +52,7 @@ public class Building extends AbstractWorldObject {
     // Used for thread safety
 
     private final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
-    public CopyOnWriteArrayList<AbstractCharacter> hirelings = new CopyOnWriteArrayList<>();
+    private final ConcurrentHashMap<AbstractCharacter, Integer> hirelings = new ConcurrentHashMap<>(MBServerStatics.CHM_INIT_CAP, MBServerStatics.CHM_LOAD, MBServerStatics.CHM_THREAD_LOW);
     private final HashMap<Integer, DoorCloseJob> doorJobs = new HashMap<>();
     public int meshUUID;
     public Zone parentZone;
@@ -100,6 +100,8 @@ public class Building extends AbstractWorldObject {
     private ConcurrentHashMap<Integer, Condemned> condemned;
     private ArrayList<Building> children = null;
 
+    public ArrayList<Mesh> buildingMeshes;
+
     /**
      * ResultSet Constructor
      */
@@ -648,11 +650,12 @@ public class Building extends AbstractWorldObject {
 
         // Add costs associated with hirelings
 
-        for (AbstractCharacter npc : this.hirelings) {
+        for (AbstractCharacter npc : this.hirelings.keySet()) {
 
             if (npc.getObjectType() != GameObjectType.NPC)
                 continue;
 
+
             maintCost += Blueprint.getNpcMaintCost(npc.getRank());
         }
 
@@ -1003,6 +1006,8 @@ public class Building extends AbstractWorldObject {
 
         if (this.upgradeDateTime != null)
             BuildingManager.submitUpgradeJob(this);
+
+        BuildingManager.BakeColliders(this);
     }
 
     public synchronized boolean setOwner(AbstractCharacter newOwner) {
@@ -1031,7 +1036,7 @@ public class Building extends AbstractWorldObject {
             // Set new guild for hirelings and refresh all clients
 
             this.refreshGuild();
-            BuildingManager.updateHirelingGuilds(this);
+            BuildingManager.refreshHirelings(this);
 
         } catch (Exception e) {
             Logger.error("Error updating owner! UUID: " + this.getObjectUUID());
@@ -1213,6 +1218,10 @@ public class Building extends AbstractWorldObject {
         this.updateEffects();
     }
 
+    public ConcurrentHashMap<AbstractCharacter, Integer> getHirelings() {
+        return hirelings;
+    }
+
     public final boolean isSpireIsActive() {
         return spireIsActive;
     }
@@ -1425,18 +1434,18 @@ public class Building extends AbstractWorldObject {
         if (this.getCity().getWarehouse() == null)
             return amount;
 
-        if (this.getCity().getWarehouse().resources.get(ItemBase.getGoldItemBase()) >= Warehouse.getMaxResources().get(ItemBase.getGoldItemBase().getUUID()))
+        if (this.getCity().getWarehouse().getResources().get(ItemBase.getGoldItemBase()) >= Warehouse.getMaxResources().get(ItemBase.getGoldItemBase().getUUID()))
             return amount;
 
         int profitAmount = (int) (amount * (taxAmount * .01f));
 
-        if (this.getCity().getWarehouse().resources.get(ItemBase.getGoldItemBase()) + profitAmount <= Warehouse.getMaxResources().get(ItemBase.getGoldItemBase().getUUID())) {
-            Warehouse.depositProfitTax(ItemBase.getGoldItemBase(), profitAmount, this,this.getCity().getWarehouse());
+        if (this.getCity().getWarehouse().getResources().get(ItemBase.getGoldItemBase()) + profitAmount <= Warehouse.getMaxResources().get(ItemBase.getGoldItemBase().getUUID())) {
+            this.getCity().getWarehouse().depositProfitTax(ItemBase.getGoldItemBase(), profitAmount, this);
             return amount - profitAmount;
         }
         //overDrafting
-        int warehouseDeposit = Warehouse.getMaxResources().get(ItemBase.getGoldItemBase().getUUID()) - this.getCity().getWarehouse().resources.get(ItemBase.getGoldItemBase());
-        Warehouse.depositProfitTax(ItemBase.getGoldItemBase(), warehouseDeposit, this,this.getCity().getWarehouse());
+        int warehouseDeposit = Warehouse.getMaxResources().get(ItemBase.getGoldItemBase().getUUID()) - this.getCity().getWarehouse().getResources().get(ItemBase.getGoldItemBase());
+        this.getCity().getWarehouse().depositProfitTax(ItemBase.getGoldItemBase(), warehouseDeposit, this);
         return amount - warehouseDeposit;
     }
 

src/engine/objects/CharacterItemManager.java

@@ -12,7 +12,10 @@ package engine.objects;
 import engine.Enum;
 import engine.Enum.GameObjectType;
 import engine.Enum.ItemType;
-import engine.gameManager.*;
+import engine.gameManager.BuildingManager;
+import engine.gameManager.ChatManager;
+import engine.gameManager.ConfigManager;
+import engine.gameManager.DbManager;
 import engine.math.Vector3fImmutable;
 import engine.net.Dispatch;
 import engine.net.DispatchMessage;
@@ -342,13 +345,13 @@ public class CharacterItemManager {
                 Warehouse warehouse = (Warehouse) object;
 
                 if (amount < 0) {
-                    if (!Warehouse.deposit((PlayerCharacter) this.absCharacter, this.getGoldInventory(), amount * -1, true, true,warehouse)) {
+                    if (!warehouse.deposit((PlayerCharacter) this.absCharacter, this.getGoldInventory(), amount * -1, true, true)) {
 
                         ErrorPopupMsg.sendErrorPopup((PlayerCharacter) this.absCharacter, 203);
                         return false;
                     }
                 } else {
-                    if (!Warehouse.withdraw(warehouse, (PlayerCharacter) this.absCharacter, this.getGoldInventory().getItemBase(), amount * -1, true, true)) {
+                    if (!warehouse.withdraw((PlayerCharacter) this.absCharacter, this.getGoldInventory().getItemBase(), amount * -1, true, true)) {
 
                         ErrorPopupMsg.sendErrorPopup((PlayerCharacter) this.absCharacter, 203);
                         return false;

src/engine/objects/City.java

@@ -656,7 +656,7 @@ public class City extends AbstractWorldObject {
         if (this.getTOL() == null)
             return outNPC;
 
-        for (AbstractCharacter npc : getTOL().hirelings) {
+        for (AbstractCharacter npc : getTOL().getHirelings().keySet()) {
             if (npc.getObjectType() == GameObjectType.NPC)
                 if (((NPC) npc).getContract().isRuneMaster() == true)
                     outNPC = (NPC) npc;
@@ -1358,7 +1358,7 @@ public class City extends AbstractWorldObject {
             ItemBase ib = ItemBase.getItemBase(itemBaseID);
             if (ib == null)
                 continue;
-            if (Warehouse.isAboveCap(ruledWarehouse, ib, (int) (city.getWarehouse().resources.get(ib) * taxPercent))) {
+            if (ruledWarehouse.isAboveCap(ib, (int) (city.getWarehouse().getResources().get(ib) * taxPercent))) {
                 ErrorPopupMsg.sendErrorMsg(player, "You're warehouse has enough " + ib.getName() + " already!");
                 return true;
             }
@@ -1371,7 +1371,7 @@ public class City extends AbstractWorldObject {
         }
 
         try {
-            Warehouse.transferResources(city.getWarehouse(), player, msg, resources, taxPercent);
+            city.getWarehouse().transferResources(player, msg, resources, taxPercent, ruledWarehouse);
         } catch (Exception e) {
             Logger.info(e.getMessage());
         }
@@ -1380,7 +1380,7 @@ public class City extends AbstractWorldObject {
 
         ViewResourcesMessage vrm = new ViewResourcesMessage(player);
         vrm.setGuild(building.getGuild());
-        vrm.setWarehouseBuilding(BuildingManager.getBuildingFromCache(building.getCity().getWarehouse().buildingUID));
+        vrm.setWarehouseBuilding(BuildingManager.getBuildingFromCache(building.getCity().getWarehouse().getBuildingUID()));
         vrm.configure();
         Dispatch dispatch = Dispatch.borrow(player, vrm);
         DispatchMessage.dispatchMsgDispatch(dispatch, Enum.DispatchChannel.SECONDARY);

src/engine/objects/ItemFactory.java

@@ -119,7 +119,7 @@ public class ItemFactory {
             // ROLL BANE SCROLL.
 
             if (ib.getUUID() > 910010 && ib.getUUID() < 910019) {
-                ConcurrentHashMap<ItemBase, Integer> resources = cityWarehouse.resources;
+                ConcurrentHashMap<ItemBase, Integer> resources = cityWarehouse.getResources();
 
 
                 int buildingWithdraw = BuildingManager.GetWithdrawAmountForRolling(forge, ib.getBaseValue());
@@ -131,7 +131,7 @@ public class ItemFactory {
                     return null;
                 }
 
-                if (overdraft > 0 && Warehouse.isResourceLocked(cityWarehouse, ItemBase.GOLD_ITEM_BASE)) {
+                if (overdraft > 0 && cityWarehouse.isResourceLocked(ItemBase.GOLD_ITEM_BASE)) {
                     if (pc != null)
                         ErrorPopupMsg.sendErrorMsg(pc, "Warehouse gold is barred! Overdraft cannot be withdrawn from warehouse." + " " + ib.getName());
                     return null;
@@ -167,9 +167,9 @@ public class ItemFactory {
                 }
 
                 if (overdraft > 0)
-                    if (!Warehouse.withdraw(cityWarehouse, npc, ItemBase.GOLD_ITEM_BASE, overdraft, true)) {
+                    if (!cityWarehouse.withdraw(npc, ItemBase.GOLD_ITEM_BASE, overdraft, false, true)) {
                         //ChatManager.chatGuildError(pc, "Failed to create Item");
-                        Logger.error("Warehouse With UID of " + cityWarehouse.UID + " Failed to Create Item." + ib.getName());
+                        Logger.error("Warehouse With UID of " + cityWarehouse.getUID() + " Failed to Create Item." + ib.getName());
                         return null;
                     }
 
@@ -263,13 +263,13 @@ public class ItemFactory {
                 }
 
             if (galvorAmount > 0) {
-                if (Warehouse.isResourceLocked(cityWarehouse, galvor)) {
+                if (cityWarehouse.isResourceLocked(galvor)) {
                     if (pc != null)
                         ErrorPopupMsg.sendErrorMsg(pc, "Galvor is locked." + ib.getName());
                     return null;
                 }
 
-                if (cityWarehouse.resources.get(galvor) < galvorAmount) {
+                if (cityWarehouse.getResources().get(galvor) < galvorAmount) {
                     if (pc != null)
                         ErrorPopupMsg.sendErrorMsg(pc, "Not enough Galvor in warehouse to roll this item." + ib.getName());
                     return null;
@@ -277,13 +277,13 @@ public class ItemFactory {
             }
 
             if (wormwoodAmount > 0) {
-                if (Warehouse.isResourceLocked(cityWarehouse, wormwood)) {
+                if (cityWarehouse.isResourceLocked(wormwood)) {
                     if (pc != null)
                         ErrorPopupMsg.sendErrorMsg(pc, "Wormwood is locked." + ib.getName());
                     return null;
                 }
 
-                if (cityWarehouse.resources.get(wormwood) < wormwoodAmount) {
+                if (cityWarehouse.getResources().get(wormwood) < wormwoodAmount) {
                     if (pc != null)
                         ErrorPopupMsg.sendErrorMsg(pc, "Not enough Wormwood in warehouse to roll this item." + ib.getName());
                     return null;
@@ -320,20 +320,20 @@ public class ItemFactory {
                     return null;
                 }
 
-                if (overdraft > 0 && Warehouse.isResourceLocked(cityWarehouse, ItemBase.GOLD_ITEM_BASE)) {
+                if (overdraft > 0 && cityWarehouse.isResourceLocked(ItemBase.GOLD_ITEM_BASE)) {
                     if (pc != null)
                         ErrorPopupMsg.sendErrorMsg(pc, "Warehouse gold is barred! Overdraft cannot be withdrawn from warehouse." + ib.getName());
                     return null;
                 }
 
-                if (overdraft > cityWarehouse.resources.get(ItemBase.GOLD_ITEM_BASE)) {
+                if (overdraft > cityWarehouse.getResources().get(ItemBase.GOLD_ITEM_BASE)) {
                     if (pc != null)
                         ErrorPopupMsg.sendErrorMsg(pc, "Not enough Gold in Warehouse for overdraft." + ib.getName());
                     return null;
                 }
                 prefixResourceCosts = prefix.getResourcesForEffect();
                 for (ItemBase ibResources : prefixResourceCosts.keySet()) {
-                    int warehouseAmount = cityWarehouse.resources.get(ibResources);
+                    int warehouseAmount = cityWarehouse.getResources().get(ibResources);
                     int creationAmount = prefixResourceCosts.get(ibResources);
                     //ChatManager.chatInfoError(pc, "Prefix : " + ibResources.getName() + " / " + creationAmount);
                     if (warehouseAmount < creationAmount) {
@@ -374,13 +374,13 @@ public class ItemFactory {
                     return null;
                 }
 
-                if (overdraft > 0 && Warehouse.isResourceLocked(cityWarehouse, ItemBase.GOLD_ITEM_BASE)) {
+                if (overdraft > 0 && cityWarehouse.isResourceLocked(ItemBase.GOLD_ITEM_BASE)) {
                     if (pc != null)
                         ErrorPopupMsg.sendErrorMsg(pc, "Warehouse gold is barred! Overdraft cannot be withdrawn from warehouse." + ib.getName());
                     return null;
                 }
 
-                if (overdraft > cityWarehouse.resources.get(ItemBase.GOLD_ITEM_BASE)) {
+                if (overdraft > cityWarehouse.getResources().get(ItemBase.GOLD_ITEM_BASE)) {
                     if (pc != null)
                         ErrorPopupMsg.sendErrorMsg(pc, "Not enough Gold in Warehouse for overdraft." + ib.getName());
                     return null;
@@ -388,7 +388,7 @@ public class ItemFactory {
 
 
                 for (ItemBase ibResources : suffixResourceCosts.keySet()) {
-                    int warehouseAmount = cityWarehouse.resources.get(ibResources);
+                    int warehouseAmount = cityWarehouse.getResources().get(ibResources);
                     int creationAmount = suffixResourceCosts.get(ibResources);
                     if (warehouseAmount < creationAmount) {
                         //					if (pc != null)
@@ -414,13 +414,13 @@ public class ItemFactory {
                 return null;
             }
 
-            if (overdraft > 0 && useWarehouse && Warehouse.isResourceLocked(cityWarehouse, ItemBase.GOLD_ITEM_BASE)) {
+            if (overdraft > 0 && useWarehouse && cityWarehouse.isResourceLocked(ItemBase.GOLD_ITEM_BASE)) {
                 if (pc != null)
                     ErrorPopupMsg.sendErrorMsg(pc, "Warehouse gold is barred! Overdraft cannot be withdrawn from warehouse." + ib.getName());
                 return null;
             }
 
-            if (useWarehouse && overdraft > cityWarehouse.resources.get(ItemBase.GOLD_ITEM_BASE)) {
+            if (useWarehouse && overdraft > cityWarehouse.getResources().get(ItemBase.GOLD_ITEM_BASE)) {
                 if (pc != null)
                     ErrorPopupMsg.sendErrorMsg(pc, "Not enough Gold in Warehouse for overdraft." + ib.getName());
                 return null;
@@ -440,7 +440,7 @@ public class ItemFactory {
                     ErrorPopupMsg.sendErrorMsg(pc, "Building does not have enough gold to produce this item." + ib.getName());
                     return null;
                 } else {
-                    if (overdraft > cityWarehouse.resources.get(ItemBase.GOLD_ITEM_BASE)) {
+                    if (overdraft > cityWarehouse.getResources().get(ItemBase.GOLD_ITEM_BASE)) {
                         ErrorPopupMsg.sendErrorMsg(pc, "Not enough Gold in Warehouse to produce this item." + ib.getName());
                         return null;
                     }
@@ -448,9 +448,9 @@ public class ItemFactory {
             }
 
             if (overdraft > 0 && useWarehouse)
-                if (!Warehouse.withdraw(cityWarehouse, npc, ItemBase.GOLD_ITEM_BASE, overdraft, true)) {
+                if (!cityWarehouse.withdraw(npc, ItemBase.GOLD_ITEM_BASE, overdraft, false, true)) {
                     //ChatManager.chatGuildError(pc, "Failed to create Item");
-                    Logger.error("Warehouse With UID of " + cityWarehouse.UID + " Failed to Create Item." + ib.getName());
+                    Logger.error("Warehouse With UID of " + cityWarehouse.getUID() + " Failed to Create Item." + ib.getName());
                     return null;
                 }
 
@@ -467,18 +467,18 @@ public class ItemFactory {
 
                     int creationAmount = prefixResourceCosts.get(ibResources);
 
-                    if (Warehouse.isResourceLocked(cityWarehouse, ibResources) == true)
+                    if (cityWarehouse.isResourceLocked(ibResources) == true)
                         return null;
 
-                    int oldAmount = cityWarehouse.resources.get(ibResources);
+                    int oldAmount = cityWarehouse.getResources().get(ibResources);
                     int amount = creationAmount;
 
                     if (oldAmount < amount)
                         amount = oldAmount;
 
-                    if (!Warehouse.withdraw(cityWarehouse, npc, ibResources, amount, true)) {
+                    if (!cityWarehouse.withdraw(npc, ibResources, amount, false, true)) {
                         //ChatManager.chatGuildError(pc, "Failed to create Item");
-                        Logger.error("Warehouse With UID of " + cityWarehouse.UID + " Failed to Create Item." + ib.getName());
+                        Logger.error("Warehouse With UID of " + cityWarehouse.getUID() + " Failed to Create Item." + ib.getName());
                         return null;
                     }
                 }
@@ -489,18 +489,18 @@ public class ItemFactory {
                 for (ItemBase ibResources : suffixResourceCosts.keySet()) {
                     int creationAmount = suffixResourceCosts.get(ibResources);
 
-                    if (Warehouse.isResourceLocked(cityWarehouse, ibResources) == true) {
+                    if (cityWarehouse.isResourceLocked(ibResources) == true) {
                         ChatManager.chatSystemError(pc, ibResources.getName() + " is locked!" + ib.getName());
                         return null;
                     }
 
-                    int oldAmount = cityWarehouse.resources.get(ibResources);
+                    int oldAmount = cityWarehouse.getResources().get(ibResources);
                     int amount = creationAmount;
                     if (oldAmount < amount)
                         amount = oldAmount;
-                    if (!Warehouse.withdraw(cityWarehouse, npc, ibResources, amount, true)) {
+                    if (!cityWarehouse.withdraw(npc, ibResources, amount, false, true)) {
                         //ChatManager.chatGuildError(pc, "Failed to create Item");
-                        Logger.error("Warehouse With UID of " + cityWarehouse.UID + " Failed to Create Item." + ib.getName());
+                        Logger.error("Warehouse With UID of " + cityWarehouse.getUID() + " Failed to Create Item." + ib.getName());
                         return null;
                     }
                 }
@@ -522,14 +522,14 @@ public class ItemFactory {
                     return null;
                 }
 
-                if (overdraft > 0 && Warehouse.isResourceLocked(cityWarehouse, ItemBase.GOLD_ITEM_BASE)) {
+                if (overdraft > 0 && cityWarehouse.isResourceLocked(ItemBase.GOLD_ITEM_BASE)) {
 
                     if (pc != null)
                         ErrorPopupMsg.sendErrorMsg(pc, "Warehouse gold is barred! Overdraft cannot be withdrawn from warehouse." + ib.getName());
                     return null;
                 }
 
-                if (useWarehouse && overdraft > cityWarehouse.resources.get(ItemBase.GOLD_ITEM_BASE)) {
+                if (useWarehouse && overdraft > cityWarehouse.getResources().get(ItemBase.GOLD_ITEM_BASE)) {
 
                     if (pc != null)
                         ErrorPopupMsg.sendErrorMsg(pc, "Not enough Gold in Warehouse for overdraft." + ib.getName());
@@ -544,7 +544,7 @@ public class ItemFactory {
                     ErrorPopupMsg.sendErrorMsg(pc, "Building does not have enough gold to produce this item." + ib.getName());
                     return null;
                 } else {
-                    if (overdraft > cityWarehouse.resources.get(ItemBase.GOLD_ITEM_BASE)) {
+                    if (overdraft > cityWarehouse.getResources().get(ItemBase.GOLD_ITEM_BASE)) {
                         ErrorPopupMsg.sendErrorMsg(pc, "Not enough Gold in Warehouse to produce this item." + ib.getName());
                         return null;
                     }
@@ -552,16 +552,16 @@ public class ItemFactory {
             }
 
             if (overdraft > 0)
-                if (!Warehouse.withdraw(cityWarehouse, npc, ItemBase.GOLD_ITEM_BASE, overdraft, true)) {
+                if (!cityWarehouse.withdraw(npc, ItemBase.GOLD_ITEM_BASE, overdraft, false, true)) {
                     //ChatManager.chatGuildError(pc, "Failed to create Item");
-                    Logger.error("Warehouse With UID of " + cityWarehouse.UID + " Failed to Create Item." + ib.getName());
+                    Logger.error("Warehouse With UID of " + cityWarehouse.getUID() + " Failed to Create Item." + ib.getName());
                     return null;
                 }
 
             //	ChatManager.chatGuildInfo(pc, "Gold Cost = " + total);
 
             if (galvorAmount > 0) {
-                if (!Warehouse.withdraw(cityWarehouse, npc, galvor, galvorAmount, true)) {
+                if (!cityWarehouse.withdraw(npc, galvor, galvorAmount, false, true)) {
                     ErrorPopupMsg.sendErrorMsg(pc, "Failed to withdraw Galvor from warehouse!" + ib.getName());
                     Logger.error("Warehouse with UID of" + cityWarehouse.getObjectUUID() + "Failed to Withdrawl ");
                     return null;
@@ -569,7 +569,7 @@ public class ItemFactory {
             }
 
             if (wormwoodAmount > 0) {
-                if (!Warehouse.withdraw(cityWarehouse, npc, wormwood, wormwoodAmount, true)) {
+                if (!cityWarehouse.withdraw(npc, wormwood, wormwoodAmount, false, true)) {
                     ErrorPopupMsg.sendErrorMsg(pc, "Failed to withdraw Wormwood from warehouse!" + ib.getName());
                     Logger.error("Warehouse with UID of" + cityWarehouse.getObjectUUID() + "Failed to Withdrawl ");
                     return null;
@@ -815,7 +815,7 @@ public class ItemFactory {
             ConcurrentHashMap<ItemBase, Integer> resources = null;
 
             if (useWarehouse)
-                resources = cityWarehouse.resources;
+                resources = cityWarehouse.getResources();
 
             int galvorAmount = 0;
             int wormwoodAmount = 0;
@@ -852,24 +852,24 @@ public class ItemFactory {
                     return null;
 
             if (galvorAmount > 0) {
-                if (Warehouse.isResourceLocked(cityWarehouse, galvor)) {
+                if (cityWarehouse.isResourceLocked(galvor)) {
                     ErrorPopupMsg.sendErrorMsg(pc, "Galvor is locked." + ib.getName());
                     return null;
                 }
 
-                if (cityWarehouse.resources.get(galvor) < galvorAmount) {
+                if (cityWarehouse.getResources().get(galvor) < galvorAmount) {
                     ErrorPopupMsg.sendErrorMsg(pc, "Not enough Galvor in warehouse to roll this item." + ib.getName());
                     return null;
                 }
             }
 
             if (wormwoodAmount > 0) {
-                if (Warehouse.isResourceLocked(cityWarehouse, wormwood)) {
+                if (cityWarehouse.isResourceLocked(wormwood)) {
                     ErrorPopupMsg.sendErrorMsg(pc, "Galvor is locked." + ib.getName());
                     return null;
                 }
 
-                if (cityWarehouse.resources.get(wormwood) < wormwoodAmount) {
+                if (cityWarehouse.getResources().get(wormwood) < wormwoodAmount) {
                     ErrorPopupMsg.sendErrorMsg(pc, "Not enough Galvor in warehouse to roll this item." + ib.getName());
                     return null;
                 }
@@ -908,7 +908,7 @@ public class ItemFactory {
                     return null;
                 }
 
-                if (useWarehouse && overdraft > 0 && Warehouse.isResourceLocked(cityWarehouse, ItemBase.GOLD_ITEM_BASE)) {
+                if (useWarehouse && overdraft > 0 && cityWarehouse.isResourceLocked(ItemBase.GOLD_ITEM_BASE)) {
                     if (pc != null)
                         ErrorPopupMsg.sendErrorMsg(pc, "Warehouse gold is barred! Overdraft cannot be withdrawn from warehouse." + ib.getName());
                     return null;
@@ -936,12 +936,12 @@ public class ItemFactory {
                 // there was an overdraft, withdraw the rest from warehouse.
                 if (overdraft > 0) {
                     if (pc != null) {
-                        if (!Warehouse.withdraw(cityWarehouse, pc, ItemBase.GOLD_ITEM_BASE, overdraft, false, true)) {
+                        if (!cityWarehouse.withdraw(pc, ItemBase.GOLD_ITEM_BASE, overdraft, false, true)) {
                             Logger.error("Warehouse with UID of" + cityWarehouse.getObjectUUID() + "Failed to Withdrawl ");
                             return null;
                         }
                     } else {
-                        if (!Warehouse.withdraw(cityWarehouse, npc, ItemBase.GOLD_ITEM_BASE, overdraft, true)) {
+                        if (!cityWarehouse.withdraw(npc, ItemBase.GOLD_ITEM_BASE, overdraft, false, true)) {
                             Logger.error("Warehouse with UID of" + cityWarehouse.getObjectUUID() + "Failed to Withdrawl ");
                             return null;
                         }
@@ -959,7 +959,7 @@ public class ItemFactory {
                     return null;
                 }
 
-                if (useWarehouse && overdraft > 0 && Warehouse.isResourceLocked(cityWarehouse, ItemBase.GOLD_ITEM_BASE)) {
+                if (useWarehouse && overdraft > 0 && cityWarehouse.isResourceLocked(ItemBase.GOLD_ITEM_BASE)) {
                     if (pc != null)
                         ErrorPopupMsg.sendErrorMsg(pc, "Warehouse gold is barred! Overdraft cannot be withdrawn from warehouse." + ib.getName());
                     return null;
@@ -987,12 +987,12 @@ public class ItemFactory {
                 if (overdraft > 0 && useWarehouse) {
 
                     if (pc != null) {
-                        if (!Warehouse.withdraw(cityWarehouse, pc, ItemBase.GOLD_ITEM_BASE, overdraft, false, true)) {
+                        if (!cityWarehouse.withdraw(pc, ItemBase.GOLD_ITEM_BASE, overdraft, false, true)) {
                             Logger.error("Warehouse with UID of" + cityWarehouse.getObjectUUID() + "Failed to Withdrawl ");
                             return null;
                         }
                     } else {
-                        if (!Warehouse.withdraw(cityWarehouse, npc, ItemBase.GOLD_ITEM_BASE, overdraft, true)) {
+                        if (!cityWarehouse.withdraw(npc, ItemBase.GOLD_ITEM_BASE, overdraft, false, true)) {
                             Logger.error("Warehouse with UID of" + cityWarehouse.getObjectUUID() + "Failed to Withdrawl ");
                             return null;
                         }
@@ -1002,7 +1002,7 @@ public class ItemFactory {
 
             if (galvorAmount > 0 && useWarehouse) {
                 //ChatManager.chatGuildInfo(pc, "Withdrawing " + galvorAmount + " galvor from warehouse");
-                if (!Warehouse.withdraw(cityWarehouse, npc, galvor, galvorAmount, true)) {
+                if (!cityWarehouse.withdraw(npc, galvor, galvorAmount, false, true)) {
                     ErrorPopupMsg.sendErrorMsg(pc, "Failed to withdraw Galvor from warehouse!" + ib.getName());
                     Logger.error("Warehouse with UID of" + cityWarehouse.getObjectUUID() + "Failed to Withdrawl ");
                     return null;
@@ -1011,7 +1011,7 @@ public class ItemFactory {
 
             if (wormwoodAmount > 0 && useWarehouse) {
                 //ChatManager.chatGuildInfo(pc, "Withdrawing " + wormwoodAmount + " wormwood from warehouse");
-                if (!Warehouse.withdraw(cityWarehouse, npc, wormwood, wormwoodAmount, true)) {
+                if (!cityWarehouse.withdraw(npc, wormwood, wormwoodAmount, false, true)) {
                     ErrorPopupMsg.sendErrorMsg(pc, "Failed to withdraw Wormwood from warehouse for " + ib.getName());
                     Logger.error("Warehouse with UID of" + cityWarehouse.getObjectUUID() + "Failed to Withdrawl ");
 

src/engine/objects/Mine.java

@@ -483,7 +483,7 @@ public class Mine extends AbstractGameObject {
             return false;
 
         ItemBase resourceIB = ItemBase.getItemBase(this.production.UUID);
-        return Warehouse.depositFromMine(this, resourceIB, this.getModifiedProductionAmount(),this.owningGuild.getOwnedCity().getWarehouse());
+        return this.owningGuild.getOwnedCity().getWarehouse().depositFromMine(this, resourceIB, this.getModifiedProductionAmount());
     }
 
     public boolean updateGuildOwner(PlayerCharacter playerCharacter) {
@@ -550,7 +550,7 @@ public class Mine extends AbstractGameObject {
         Building mineBuilding = BuildingManager.getBuilding(this.buildingID);
         if (mineBuilding == null)
             return this.production.baseProduction;
-        for (AbstractCharacter harvester : mineBuilding.hirelings) {
+        for (AbstractCharacter harvester : mineBuilding.getHirelings().keySet()) {
             totalModded += baseModValue;
             totalModded += rankModValue * harvester.getRank();
         }

src/engine/objects/Mob.java

@@ -406,6 +406,7 @@ public class Mob extends AbstractIntelligenceAgent implements Delayed {
 
         Mob mobile = new Mob();
         mobile.dbID = MBServerStatics.NO_DB_ROW_ASSIGNED_YET;
+        //mobile.agentType = AIAgentType.MOBILE; this method is only called to make guard captains and wall archers
         mobile.agentType = mobType;
         mobile.behaviourType = MobBehaviourType.None;
         mobile.loadID = loadID;
@@ -417,17 +418,12 @@ public class Mob extends AbstractIntelligenceAgent implements Delayed {
             mobile.guildUUID = guild.getObjectUUID();
 
         mobile.parentZoneUUID = parent.getObjectUUID();
-
-        if (building == null)
-            mobile.buildingUUID = 0;
-        else
-            mobile.buildingUUID = building.getObjectUUID();
+        mobile.buildingUUID = building.getObjectUUID();
 
         if (mobile.buildingUUID != 0)
             mobile.bindLoc = Vector3fImmutable.ZERO;
         else
-            mobile.bindLoc = ZoneManager.worldToLocal(spawn, parent);
-        ;
+            mobile.bindLoc = spawn;
 
         mobile.firstName = pirateName;
 
@@ -1893,19 +1889,6 @@ public class Mob extends AbstractIntelligenceAgent implements Delayed {
         }
     }
 
-    public Boolean isGuard(){
-
-        switch(this.behaviourType){
-            case GuardMinion:
-            case GuardCaptain:
-            case GuardWallArcher:
-            case HamletGuard:
-            case SimpleStandingGuard:
-                return true;
-        }
-        return false;
-    }
-
     @Override
     public long getDelay(@NotNull TimeUnit unit) {
         long timeRemaining = this.respawnTime - System.currentTimeMillis();

src/engine/objects/NPC.java

@@ -461,20 +461,16 @@ public class NPC extends AbstractCharacter {
 
         NPC newNPC = new NPC();
 
-        newNPC.parentZoneUUID = parent.getObjectUUID();
-
         newNPC.name = name;
         newNPC.contractUUID = contractID;
 
         if (building == null)
-            newNPC.bindLoc = ZoneManager.worldToLocal(spawn, parent);
+            newNPC.bindLoc = spawn;
         else
             newNPC.bindLoc = Vector3fImmutable.ZERO;
 
-        if (guild == null)
-            newNPC.guildUUID = 0;
-        else
-            newNPC.guildUUID = guild.getObjectUUID();
+        newNPC.parentZoneUUID = parent.getObjectUUID();
+        newNPC.guildUUID = guild.getObjectUUID();
 
         if (building == null)
             newNPC.buildingUUID = 0;
@@ -1119,7 +1115,7 @@ public class NPC extends AbstractCharacter {
         building = this.building;
 
         if (building != null) {
-            building.hirelings.remove(this);
+            building.getHirelings().remove(this);
             this.removeMinions();
         }
 

src/engine/objects/PlayerCharacter.java

@@ -165,7 +165,6 @@ public class PlayerCharacter extends AbstractCharacter {
     private boolean wasTripped75 = false;
     private boolean wasTripped50 = false;
     private boolean wasTripped25 = false;
-    private float characterHeight = 0;
     private boolean lastSwimming = false;
     private boolean isTeleporting = false;
     private boolean dirtyLoad = true;
@@ -5427,10 +5426,6 @@ public class PlayerCharacter extends AbstractCharacter {
         this.lastStamUpdateTime = System.currentTimeMillis();
     }
 
-    public float getCharacterHeight() {
-        return characterHeight;
-    }
-
     public boolean isEnteredWorld() {
         return enteredWorld;
     }

src/engine/server/world/WorldServer.java

@@ -315,6 +315,10 @@ public class WorldServer {
 		Logger.info("Initializing PowersManager.");
 		PowersManager.initPowersManager(true);
 
+		Logger.info("Loading Collider Data");
+		DbManager.BuildingQueries.LOAD_MESH_TRIANGLES();
+		DbManager.BuildingQueries.LOAD_STRUCTURE_MESHES();
+
 		Logger.info("Loading granted Skills for Runes");
 		DbManager.SkillsBaseQueries.LOAD_ALL_RUNE_SKILLS();