state updated properly

src/engine/devcmd/cmds/SetCampLevelCmd.java

@@ -1,61 +0,0 @@
-package engine.devcmd.cmds;
-
-import engine.devcmd.AbstractDevCmd;
-import engine.gameManager.PowersManager;
-import engine.gameManager.ZoneManager;
-import engine.objects.*;
-
-public class SetCampLevelCmd extends AbstractDevCmd {
-    public SetCampLevelCmd() { super("setcamplevel"); }
-
-    @Override
-    protected void _doCmd(PlayerCharacter pcSender, String[] args, AbstractGameObject target) {
-
-        if (args.length > 1)
-        {
-             this.sendUsage(pcSender);
-        }
-
-        int targetLevel = 0;
-        if (args.length == 1) {
-            try {
-                targetLevel = Integer.parseInt(args[0]);
-            } catch (NumberFormatException nfe) {
-                throwbackError(pcSender, "Argument MUST be integer.  Received: " + args[0]);
-            } catch (Exception e) {
-                throwbackError(pcSender, "Unknown command parsing provided camp level: " + args[0]);
-            }
-        }
-
-        if ((target instanceof Mob))
-        {
-            // Get the camp that owns the targeted Mob
-            Zone campZone = ((Mob) target).parentZone;
-
-            // Make sure that the zone we're targeting is valid for action
-            if (campZone == null ||
-                    campZone.zoneMobSet.isEmpty() ||
-                    campZone.isPlayerCity()) {
-                throwbackError(pcSender, "Current zone must own mobs, and NOT be a city.");
-                return;
-            }
-
-            campZone.setCampLvl(targetLevel);
-        }
-        else if (target instanceof PlayerCharacter)
-        {
-            PlayerCharacter pc = (PlayerCharacter)target;
-            PowersManager.applyPower(pc, pc, pc.getLoc(), "CMP-001", targetLevel, false);
-        }
-    }
-
-    @Override
-    protected String _getUsageString() {
-        return "Sets the level of the currently occupied camp to the desired level";
-    }
-
-    @Override
-    protected String _getHelpString() {
-        return "'./setcamplevel levelNum'";
-    }
-}

src/engine/gameManager/DevCmdManager.java

@@ -143,7 +143,6 @@ public enum DevCmdManager {
         DevCmdManager.registerDevCmd(new ApplyBonusCmd());
         DevCmdManager.registerDevCmd(new AuditFailedItemsCmd());
         DevCmdManager.registerDevCmd(new SlotTestCmd());
-        DevCmdManager.registerDevCmd(new SetCampLevelCmd());
 
     }
 

src/engine/gameManager/LootManager.java

@@ -14,7 +14,6 @@ import engine.net.DispatchMessage;
 import engine.net.client.msg.ErrorPopupMsg;
 import engine.net.client.msg.chat.ChatSystemMsg;
 import engine.objects.*;
-import engine.util.ZoneLevel;
 import org.pmw.tinylog.Logger;
 
 import java.util.ArrayList;
@@ -101,16 +100,6 @@ public enum LootManager {
         boolean hotzoneWasRan = false;
         float dropRate = 1.0f;
 
-        if (mob.getSafeZone() == false)
-            dropRate = LootManager.NORMAL_DROP_RATE;
-
-        if (inHotzone == true)
-            dropRate = LootManager.HOTZONE_DROP_RATE;
-
-        // Adjust for camp scaling
-        Zone camp = mob.getParentZone();
-        dropRate = dropRate * ZoneLevel.getLootDropModifier(camp);
-
         // Iterate all entries in this bootySet and process accordingly
 
         for (BootySetEntry bse : entries) {
@@ -120,6 +109,12 @@ public enum LootManager {
                     break;
                 case "LOOT":
 
+                    if (mob.getSafeZone() == false)
+                        dropRate = LootManager.NORMAL_DROP_RATE;
+
+                    if (inHotzone == true)
+                        dropRate = LootManager.HOTZONE_DROP_RATE;
+
                     if (ThreadLocalRandom.current().nextInt(1, 100 + 1) < (bse.dropChance * dropRate))
                         GenerateLootDrop(mob, bse.genTable, false);  //generate normal loot drop
 
@@ -201,9 +196,6 @@ public enum LootManager {
                 Logger.error("Failed to GenerateSuffix for item: " + outItem.getName());
             }
         }
-
-        // We don't want to bother with identifying gear
-        outItem.setIsID(true);
         return outItem;
     }
 
@@ -314,9 +306,6 @@ public enum LootManager {
         else
             gold = (int) (gold * NORMAL_GOLD_RATE);
 
-        Zone camp = mob.getParentZone();
-        gold = (int) (gold * ZoneLevel.getGoldDropModifier(camp));
-
         if (gold > 0) {
             MobLoot goldAmount = new MobLoot(mob, gold);
             mob.getCharItemManager().addItemToInventory(goldAmount);

src/engine/mobileAI/Threads/MobRespawnThread.java

@@ -11,7 +11,6 @@ package engine.mobileAI.Threads;
 import engine.gameManager.ZoneManager;
 import engine.objects.Mob;
 import engine.objects.Zone;
-import engine.util.ZoneLevel;
 import org.pmw.tinylog.Logger;
 
 /**
@@ -26,6 +25,7 @@ import org.pmw.tinylog.Logger;
 
 public class MobRespawnThread implements Runnable {
 
+
     public MobRespawnThread() {
         Logger.info(" MobRespawnThread thread has started!");
 
@@ -34,85 +34,23 @@ public class MobRespawnThread implements Runnable {
     @Override
     public void run() {
 
-        long startTime = System.currentTimeMillis();
-        long rollingKeepFraction = (Zone.rollingAvgMobsAliveDepth - 1) / Zone.rollingAvgMobsAliveDepth;
-        long rollingAddFraction = 1 / Zone.rollingAvgMobsAliveDepth;
-
         while (true) {
 
             try {
-
                 for (Zone zone : ZoneManager.getAllZones()) {
 
-                    /*
-                    if (zone.respawnQue.size() > ZoneLevel.queueLengthToLevelUp) {
-                        zone.setCampLvl(zone.getCamplvl() + 1);
-                    }
-                    else if (zone.respawnQue.isEmpty() &&
-                            (zone.lastRespawn + ZoneLevel.msToLevelDown < System.currentTimeMillis()) &&
-                            zone.getCamplvl() > 0) {
-                        zone.setCampLvl(zone.getCamplvl() - 1);
-                    }
-                     */
+                    if (zone.respawnQue.isEmpty() == false && zone.lastRespawn + 100 < System.currentTimeMillis()) {
 
-                    int aliveCount = 0;
-                    int deadCount = 0;
-                    for (Mob mob : zone.zoneMobSet) {
-                        if (mob.isAlive()) {
-                            aliveCount = aliveCount + 1;
-                        }
-                        else {
-                            deadCount = deadCount + 1;
-                        }
-                    }
-
-                    zone.rollingAvgMobsAlive =
-                            ((zone.rollingAvgMobsAlive * (Zone.rollingAvgMobsAliveDepth - 1) + aliveCount) / Zone.rollingAvgMobsAliveDepth);
-
-                    /*
-                    if (startTime + ZoneLevel.msDelayToCampLevel < System.currentTimeMillis()) {
-                        if (aliveCount > Math.floor(zone.zoneMobSet.size() / 2.0)) {
-                            if (zone.levelUpTimer == 0) {
-                                zone.levelUpTimer = System.currentTimeMillis();
-                            } else if (zone.levelUpTimer + ZoneLevel.msTolevelUp < System.currentTimeMillis()) {
-                                zone.setCampLvl(zone.getCampLvl() + 1);
-
-                                zone.levelUpTimer = 0;
-                            }
-                        } else if (aliveCount == 0) {
-
-                            if (zone.levelDownTimer == 0) {
-                                zone.levelDownTimer = System.currentTimeMillis();
-                            } else if (zone.levelDownTimer + ZoneLevel.msToLevelDown < System.currentTimeMillis()) {
-                                if (zone.getCampLvl() > 0) {
-                                    zone.setCampLvl(zone.getCampLvl() + 1);
-
-                                    zone.levelDownTimer = 0;
-                                }
-                            }
-                        } else {
-                            zone.levelUpTimer = 0;
-                            zone.levelDownTimer = 0;
-                        }
-                    }
-                    */
-                }
-
-                // --------------------------------------------------------------------------------------------------------------------
-                // Manage mob respawn
-                // --------------------------------------------------------------------------------------------------------------------
-                if (!Zone.respawnQue.isEmpty() && Zone.lastRespawn + 100 < System.currentTimeMillis()) {
-
-                    Mob respawner = Zone.respawnQue.iterator().next();
+                        Mob respawner = zone.respawnQue.iterator().next();
 
                         if (respawner == null)
                             continue;
 
                         respawner.respawn();
-                    Zone.respawnQue.remove(respawner);
-                    Zone.lastRespawn = System.currentTimeMillis();
+                        zone.respawnQue.remove(respawner);
+                        zone.lastRespawn = System.currentTimeMillis();
+                    }
                 }
-
             } catch (Exception e) {
                 Logger.error(e);
             }

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

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

src/engine/objects/Mob.java

@@ -31,7 +31,6 @@ import engine.net.client.msg.PlaceAssetMsg;
 import engine.powers.EffectsBase;
 import engine.powers.MobPowerEntry;
 import engine.server.MBServerStatics;
-import engine.util.ZoneLevel;
 import org.joda.time.DateTime;
 import org.pmw.tinylog.Logger;
 
@@ -102,8 +101,6 @@ public class Mob extends AbstractIntelligenceAgent {
     private DateTime upgradeDateTime = null;
     private boolean lootSync = false;
 
-    private String originalFirstName;
-    private String originalLastName;
 
     /**
      * No Id Constructor
@@ -132,9 +129,6 @@ public class Mob extends AbstractIntelligenceAgent {
             this.lastName = "the " + contract.getName();
         }
         clearStatic();
-
-        originalFirstName = this.firstName;
-        originalLastName = this.lastName;
     }
 
     /**
@@ -156,9 +150,6 @@ public class Mob extends AbstractIntelligenceAgent {
         this.building = building;
         initializeMob(false, false, false);
         clearStatic();
-
-        originalFirstName = this.firstName;
-        originalLastName = this.lastName;
     }
 
     /**
@@ -175,9 +166,6 @@ public class Mob extends AbstractIntelligenceAgent {
         this.BehaviourType = Enum.MobBehaviourType.Pet1;
         initializeMob(true, false, false);
         clearStatic();
-
-        originalFirstName = this.firstName;
-        originalLastName = this.lastName;
     }
 
     //SIEGE CONSTRUCTOR
@@ -192,9 +180,6 @@ public class Mob extends AbstractIntelligenceAgent {
         this.equip = new HashMap<>();
         initializeMob(false, true, isPlayerGuard);
         clearStatic();
-
-        originalFirstName = this.firstName;
-        originalLastName = this.lastName;
     }
 
     /**
@@ -303,8 +288,6 @@ public class Mob extends AbstractIntelligenceAgent {
             Logger.error("Mobile:" + this.dbID + ": " + e);
         }
 
-        originalFirstName = this.firstName;
-        originalLastName = this.lastName;
     }
 
     public static void serializeMobForClientMsgOtherPlayer(Mob mob, ByteBufferWriter writer) throws SerializationException {
@@ -1399,12 +1382,6 @@ public class Mob extends AbstractIntelligenceAgent {
 
         NPCManager.applyRuneSetEffects(this);
 
-        // Set Name based on parent zone level
-        Zone camp = this.getParentZone();
-        this.lastName = this.originalLastName + ZoneLevel.getNameSuffix(camp);
-
-        //PowersManager.applyPower(this, this, this.getLoc(), "CMP-001", camp.getCamplvl(), false);
-
         this.recalculateStats();
         this.setHealth(this.healthMax);
 
@@ -1518,10 +1495,6 @@ public class Mob extends AbstractIntelligenceAgent {
             s *= (1 + this.bonuses.getFloatPercentAll(ModType.StaminaFull, SourceType.None));
         }
 
-        // Modify max health based on camp level - bad, we need to use effects for this
-        Zone camp = this.getParentZone();
-        h = h * ZoneLevel.getMaxHealthPctModifier(camp);
-
         // Set max health, mana and stamina
 
         if (h > 0)
@@ -1622,11 +1595,6 @@ public class Mob extends AbstractIntelligenceAgent {
                 Logger.error("Error: missing bonuses");
 
             defense = (defense < 1) ? 1 : defense;
-
-            // Modify defense for camp level - bad, we need to use effects for this
-            Zone camp = this.getParentZone();
-            defense = defense * ZoneLevel.getDefPctModifier(camp);
-
             this.defenseRating = (short) (defense + 0.5f);
         } catch (Exception e) {
             Logger.info("Mobbase ID " + this.getMobBaseID() + " returned an error. Setting to Default Defense." + e.getMessage());
@@ -1828,10 +1796,6 @@ public class Mob extends AbstractIntelligenceAgent {
             atr *= (1 + neg_Bonus);
         }
 
-        // Modify atr for camp level - bad, we need to use effects for this
-        Zone camp = this.getParentZone();
-        atr = atr * ZoneLevel.getAtrPctModifier(camp);
-
         atr = (atr < 1) ? 1 : atr;
 
         // set atr

src/engine/objects/Zone.java

@@ -18,10 +18,7 @@ import engine.math.Bounds;
 import engine.math.Vector2f;
 import engine.math.Vector3fImmutable;
 import engine.net.ByteBufferWriter;
-import engine.net.DispatchMessage;
-import engine.net.client.msg.chat.ChatSystemMsg;
 import engine.server.MBServerStatics;
-import engine.util.ZoneLevel;
 import org.pmw.tinylog.Logger;
 
 import java.sql.ResultSet;
@@ -64,13 +61,6 @@ public class Zone extends AbstractGameObject {
     //public static ArrayList<Mob> respawnQue = new ArrayList<>();
     public static final Set<Mob> respawnQue = Collections.newSetFromMap(new ConcurrentHashMap<>());
     public static long lastRespawn = 0;
-
-    private int campLvl = 0;
-    public long levelUpTimer = 0;
-    public long levelDownTimer = 0;
-    public int rollingAvgMobsAlive = 0;
-    public static final int rollingAvgMobsAliveDepth = 100;
-
     /**
      * ResultSet Constructor
      */
@@ -110,33 +100,8 @@ public class Zone extends AbstractGameObject {
 
         if (hash == null)
             setHash();
-    }
 
-    public void setCampLvl(int level)
-    {
-        this.campLvl = level;
 
-        if (this.campLvl > ZoneLevel.campMaxLvl)
-        {
-            this.campLvl = ZoneLevel.campMaxLvl;
-        }
-        else if (this.campLvl < 0)
-        {
-            this.campLvl = 0;
-        }
-
-        //if (this.campLvl > ZoneLevel.campLvlAnnounceThreshold)
-        {
-            ChatSystemMsg chatMsg = new ChatSystemMsg(null, this.getName() + " has reached camp level " + this.campLvl + "! Will anyone contest?!");
-            chatMsg.setMessageType(2);
-            chatMsg.setChannel(Enum.ChatChannelType.SYSTEM.getChannelID());
-            DispatchMessage.dispatchMsgToAll(chatMsg);
-        }
-    }
-
-    public int getCampLvl()
-    {
-        return this.campLvl;
     }
 
     public static void serializeForClientMsg(Zone zone, ByteBufferWriter writer) {

src/engine/server/world/WorldServer.java

@@ -288,7 +288,6 @@ public class WorldServer {
 
 	private boolean init() {
 
-		Logger.info("Server Code: [NovaTest] Branch");
 		Logger.info("MAGICBANE SERVER GREETING:");
 		Logger.info(ConfigManager.MB_WORLD_GREETING.getValue());
 

src/engine/util/ZoneLevel.java

@@ -1,84 +0,0 @@
-package engine.util;
-
-import engine.objects.Zone;
-
-public class ZoneLevel {
-
-    private static final float healthPctPerLevel = (float)0.2;
-    private static final float atrPctPerLevel = (float)0.2;
-    private static final float defPctPerLevel = (float)0.2;
-    private static final float lootPctPerLevel = (float)0.1;
-    private static final float goldPctPerLevel = (float)0.2;
-
-    public static final int campLvlAnnounceThreshold = 5;
-    public static final int campMaxLvl = 10;
-
-    public static final int queueLengthToLevelUp = 5;
-    public static final int msToLevelDown = 60 * 1000;
-    public static final int msTolevelUp = 60 * 1000;
-    public static final long msDelayToCampLevel = 60 * 1000;
-
-    private static final String[] nameMap =
-            {
-              "",
-              " I",
-              " II",
-              " III",
-              " IV",
-              " V",
-              " VI",
-              " VII",
-              " VIII",
-              " IX",
-              " X"
-            };
-
-    public static String getNameSuffix(Zone zone)
-    {
-        try {
-            return nameMap[zone.getCampLvl()];
-        }
-        catch (Exception ignored)
-        {
-        }
-
-        return "";
-    }
-
-    public static float getMaxHealthPctModifier(Zone zone)
-    {
-        return getGenericModifier(zone, healthPctPerLevel);
-    }
-
-    public static float getAtrPctModifier(Zone zone)
-    {
-        return getGenericModifier(zone, atrPctPerLevel);
-    }
-
-    public static float getDefPctModifier(Zone zone)
-    {
-        return getGenericModifier(zone, defPctPerLevel);
-    }
-
-    public static float getLootDropModifier(Zone zone)
-    {
-        return getGenericModifier(zone, lootPctPerLevel);
-    }
-
-    public static float getGoldDropModifier(Zone zone)
-    {
-        return getGenericModifier(zone, goldPctPerLevel);
-    }
-
-    private static float getGenericModifier(Zone zone, float modifierPerLevel)
-    {
-        float modifier = (float)1.0;
-
-        if (zone != null)
-        {
-            modifier += zone.getCampLvl() * modifierPerLevel;
-        }
-
-        return modifier;
-    }
-}