Notice and respect conf changes to min/max bots (#218)

Author: bozimmerman

src/modules/Bots/playerbot/RandomPlayerbotMgr.cpp

@@ -54,6 +54,23 @@ void RandomPlayerbotMgr::UpdateAIInternal(uint32 elapsed)
         return;
     }
 
+    sLog.outBasic("Processing random bots...");
+
+    uint32 cachedMin = GetEventValue(0, "config_min");
+    uint32 cachedMax = GetEventValue(0, "config_max");
+
+    if (cachedMin != sPlayerbotAIConfig.minRandomBots ||
+        cachedMax != sPlayerbotAIConfig.maxRandomBots)
+    {
+        sLog.outString("Bot count range changed from %d-%d to %d-%d, regenerating target...",
+            cachedMin, cachedMax,
+            sPlayerbotAIConfig.minRandomBots, sPlayerbotAIConfig.maxRandomBots);
+
+        SetEventValue(0, "bot_count", 0, 0);  // Invalidate
+        SetEventValue(0, "config_min", sPlayerbotAIConfig.minRandomBots, 999999);
+        SetEventValue(0, "config_max", sPlayerbotAIConfig.maxRandomBots, 999999);
+    }
+
     int maxAllowedBotCount = GetEventValue(0, "bot_count");
     if (!maxAllowedBotCount)
     {
@@ -286,6 +303,21 @@ bool RandomPlayerbotMgr::ProcessBot(Player* player)
         return true;
     }
 
+    // Check if bot level is outside configured min/max range
+    uint32 botLevel = player->getLevel();
+    uint32 maxLevel = sPlayerbotAIConfig.randomBotMaxLevel;
+    if (maxLevel > sWorld.getConfig(CONFIG_UINT32_MAX_PLAYER_LEVEL))
+    {
+        maxLevel = sWorld.getConfig(CONFIG_UINT32_MAX_PLAYER_LEVEL);
+    }
+    if (botLevel < sPlayerbotAIConfig.randomBotMinLevel || botLevel > maxLevel)
+    {
+        sLog.outDetail("Bot %d level %d is outside valid range (%d-%d), scheduling immediate re-randomization",
+                       bot, botLevel, sPlayerbotAIConfig.randomBotMinLevel, maxLevel);
+        ScheduleRandomize(bot, 0);
+        return true;
+    }
+
     uint32 teleport = GetEventValue(bot, "teleport");
     if (!teleport)
     {