Fixes for /mob, /npc, and /equipset commands.

src/engine/db/handlers/dbNPCHandler.java

@@ -243,8 +243,12 @@ public class dbNPCHandler extends dbHandlerBase {
 
     public boolean UPDATE_EQUIPSET(NPC npc, int equipSetID) {
 
+        // Column name must match what NPC/Mob loaders read ("equipmentSet").
+        // Using the wrong column here causes the update to fail silently and
+        // dev command to report "Unable to find Equipset" despite a valid ID.
+
         try (Connection connection = DbManager.getConnection();
-             PreparedStatement preparedStatement = connection.prepareStatement("UPDATE `obj_npc` SET `equipsetID`=? WHERE `UID`=?")) {
+             PreparedStatement preparedStatement = connection.prepareStatement("UPDATE `obj_npc` SET `equipmentSet`=? WHERE `UID`=?")) {
 
             preparedStatement.setInt(1, equipSetID);
             preparedStatement.setLong(2, npc.getObjectUUID());
@@ -467,4 +471,4 @@ public class dbNPCHandler extends dbHandlerBase {
             return false;
         }
     }
-}
+}

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

@@ -15,7 +15,6 @@ import engine.Enum.GuildState;
 import engine.exception.MsgSendException;
 import engine.gameManager.ChatManager;
 import engine.gameManager.DbManager;
-import engine.gameManager.GuildManager;
 import engine.gameManager.SessionManager;
 import engine.net.Dispatch;
 import engine.net.DispatchMessage;
@@ -27,6 +26,8 @@ import engine.objects.Guild;
 import engine.objects.GuildStatusController;
 import engine.objects.PlayerCharacter;
 
+import java.util.ArrayList;
+
 public class AcceptSubInviteHandler extends AbstractClientMsgHandler {
 
     public AcceptSubInviteHandler() {
@@ -37,74 +38,69 @@ public class AcceptSubInviteHandler extends AbstractClientMsgHandler {
     protected boolean _handleNetMsg(ClientNetMsg baseMsg, ClientConnection origin) throws MsgSendException {
 
         AcceptSubInviteMsg msg = (AcceptSubInviteMsg) baseMsg;
-        PlayerCharacter playerCharacter;
-        Guild protectorate;
-        Guild nation;
+        PlayerCharacter sourcePlayer;
+        Guild sourceGuild;
+        Guild targetGuild;
         Dispatch dispatch;
 
         // get PlayerCharacter of person sending sub invite
 
-        playerCharacter = SessionManager.getPlayerCharacter(origin);
+        sourcePlayer = SessionManager.getPlayerCharacter(origin);
 
-        if (playerCharacter == null)
+        if (sourcePlayer == null)
             return true;
 
-        protectorate = playerCharacter.getGuild();
-        nation = (Guild) DbManager.getObject(GameObjectType.Guild, msg.guildUUID());
+        sourceGuild = sourcePlayer.getGuild();
+        targetGuild = (Guild) DbManager.getObject(GameObjectType.Guild, msg.guildUUID());
 
         //must be source guild to sub to
 
-        if (nation == null) {
-            ErrorPopupMsg.sendErrorPopup(playerCharacter, 45); // Failure to swear guild
+        if (targetGuild == null) {
+            ErrorPopupMsg.sendErrorPopup(sourcePlayer, 45); // Failure to swear guild
             return true;
         }
-        if (protectorate == null) {
-            ErrorPopupMsg.sendErrorPopup(playerCharacter, 45); // Failure to swear guild
+        if (sourceGuild == null) {
+            ErrorPopupMsg.sendErrorPopup(sourcePlayer, 45); // Failure to swear guild
             return true;
         }
 
-        if (protectorate.equals(nation))
+        if (sourceGuild.equals(targetGuild))
             return true;
 
-        if (GuildStatusController.isGuildLeader(playerCharacter.getGuildStatus()) == false) {
-            ErrorPopupMsg.sendErrorMsg(playerCharacter, "Only a guild leader can accept fealty!");
+        if (GuildStatusController.isGuildLeader(sourcePlayer.getGuildStatus()) == false) {
+            ErrorPopupMsg.sendErrorMsg(sourcePlayer, "Only a guild leader can accept fealty!");
             return true;
         }
 
         //source guild is limited to 7 subs
         //TODO this should be based on TOL rank
 
-        if (!nation.canSubAGuild(protectorate)) {
-            ErrorPopupMsg.sendErrorPopup(playerCharacter, 45); // Failure to swear guild
+        if (!targetGuild.canSubAGuild(sourceGuild)) {
+            ErrorPopupMsg.sendErrorPopup(sourcePlayer, 45); // Failure to swear guild
             return true;
         }
 
         //all tests passed, let's Handle code
+        //Update Target Guild State.
 
-        // Update database
+        sourceGuild.upgradeGuildState(false);
 
-        if (!DbManager.GuildQueries.UPDATE_PARENT(protectorate.getObjectUUID(), nation.getObjectUUID())) {
-            ErrorPopupMsg.sendErrorMsg(playerCharacter, "A Serious error has occured. Please post details for to ensure transaction integrity");
-            return true;
-        }
+        //Add sub so GuildMaster can Swear in.
 
-        //update Guild states.
+        ArrayList<Guild> subs = targetGuild.getSubGuildList();
+        subs.add(sourceGuild);
 
-        protectorate.setNation(nation);
-        GuildManager.updateAllGuildTags(protectorate);
-        protectorate.upgradeGuildState(false);
+        targetGuild.setGuildState(GuildState.Nation);
 
-        if (nation.getGuildState() == GuildState.Sovereign)
-            nation.upgradeGuildState(true);
 
         //Let's send the message back.
 
         msg.setUnknown02(1);
-        msg.setResponse("Your guild is now a " + protectorate.getGuildState().name() + '.');
-        dispatch = Dispatch.borrow(playerCharacter, msg);
+        msg.setResponse("Your guild is now a " + sourceGuild.getGuildState().name() + '.');
+        dispatch = Dispatch.borrow(sourcePlayer, msg);
         DispatchMessage.dispatchMsgDispatch(dispatch, Enum.DispatchChannel.SECONDARY);
 
-        ChatManager.chatSystemInfo(playerCharacter, "Your guild is now a " + protectorate.getGuildState().name() + '.');
+        ChatManager.chatSystemInfo(sourcePlayer, "Your guild is now a " + sourceGuild.getGuildState().name() + '.');
         return true;
     }
 }

src/engine/objects/Mob.java

@@ -541,8 +541,18 @@ public class Mob extends AbstractIntelligenceAgent {
         mobWithoutID.parentZone = parent;
         mobWithoutID.parentZoneID = parent.getObjectUUID();
 
-        // NPC in a Building derives position from slot
+        // Ensure spawn coordinates are persisted for DB insert
+        // statLat/statAlt/statLon represent local (zone-relative) spawn
+        // coordinates used by the DB handler when creating the mob row.
+        if (spawn != null && parent != null) {
+            // Convert world coordinates to zone-local spawn coordinates
+            Vector3fImmutable localSpawn = spawn.subtract(parent.getLoc());
+            mobWithoutID.statLat = localSpawn.x;
+            mobWithoutID.statAlt = localSpawn.y;
+            mobWithoutID.statLon = localSpawn.z;
+        }
 
+        // NPC in a Building derives position from slot
         if (mobWithoutID.building != null)
             mobWithoutID.bindLoc = Vector3fImmutable.ZERO;
 

src/engine/objects/NPC.java

@@ -470,7 +470,8 @@ public class NPC extends AbstractCharacter {
             newNPC.bindLoc = Vector3fImmutable.ZERO;
 
         newNPC.parentZoneUUID = parent.getObjectUUID();
-        newNPC.guildUUID = guild.getObjectUUID();
+        // guild may be null when spawning via ./npc; default to 0
+        newNPC.guildUUID = (guild != null) ? guild.getObjectUUID() : 0;
 
         if (building == null)
             newNPC.buildingUUID = 0;
@@ -1347,4 +1348,4 @@ public class NPC extends AbstractCharacter {
         }
     }
 
-}
+}