Bot Zerg control...

[ArkR]

It's partially working right now. Here is the behavior I'm seeing,
#bot botgroup Load (group designation) => is completely unrestricted
#bot spawn (name) => works until mob is physically attacked, can cast unlimited spells damaging mob and not get flagged as engaged

I observed behavior as a Shadow Knight using the precompiled 1180 and now the latest svn I just compiled to.

Thanks for the great work on bots btw!

[c0ncrete]

I've been able to consistently camp and spawn bots while I have aggro, effectively CHealing them in an instant. It makes things such as dealing with huge trains in Fear incredibly easy, as long as there are no casters.

To stop zerging with the #bot spawn command, try replacing this block

          /*if(c->IsGrouped()) {
             Group *g = entity_list.GetGroupByClient(c);
             for (int i=0; i<MAX_GROUP_MEMBERS; i++) {
                if(g && g->members[i] && !g->members[i]->qglobal && (g->members[i]->GetAppearance() != eaDead) && g->members[i]->IsEngaged()) {
                   c->Message(0, "You can't summon bots while you are engaged.");
                   return;
                }
                if(g && g->members[i] && g->members[i]->qglobal) {
                   return;
                }
             }
          }*/

with this block

          // Blocks spawn if any group member is engaged.
          Group *g = c->GetGroup();
          if(g){ // Client is grouped.
             for(int i = 0; i < MAX_GROUP_MEMBERS; i++) { // Check each group member.
                // Group member is in zone & ((is a client & has mob aggro) or (is a bot & has a populated hate list)).
                if(g->members[i] != NULL && ((g->members[i]->IsClient() && g->members[i]->CastToClient()->GetAggroCount() > 0) || (g->members[i]->IsBot() && g->members[i]->CastToBot()->IsEngaged()))) {
                   c->Message(0, "You can't spawn bots while your group is engaged.");
                   return;
                }
             }
          } else { // Client is not grouped.
             if(c->GetAggroCount() > 0) { // Client has mob aggro.
                c->Message(0, "You can't spawn bots while you are engaged.");
                return;
             }
          }

in zone/bot.cpp

You should also be able to insert the same block directly after this line

if(!strcasecmp(sep->arg[1], "botgroup") && !strcasecmp(sep->arg[2], "load")) {

in the same file to kill zerging using the #bot botgroup load command.

NOTES:

I'm still learning C++ and I've only spent a couple of days looking at the code, so this might not be the most efficient way of dealing with the situation. I'm not entirely sure if the issue was with checking IsEngaged() on a client instead of looking at GetAggroCount(), but it seems to work correctly now.

I'm sure there are ways I could have cleaned it up a bit (Do I need CastToMoB() and CastToClient? Do I need to use > 0 after GetAggroCount()? Do I need to use != NULL after g->members[i]?), but I haven't had any sleep...

[c0ncrete]

raising this thread from the dead as you are still able to spawn bots while you are in combat (even though they may not be at full health/mana). the current code does not check for aggro on an ungrouped client, nor does it stop client-only groups from spawning bots, as the IsEngaged() check will always return false on a client. patch included.

Index: bot.cpp
===================================================================
--- bot.cpp	(revision 2103)
+++ bot.cpp	(working copy)
@@ -12369,17 +12369,27 @@
 			}
 		}*/
 
-		if(c->IsGrouped()) {
-			Group *g = entity_list.GetGroupByClient(c);
-			for (int i=0; i<MAX_GROUP_MEMBERS; i++) {
-				if(g && g->members[i] && !g->members[i]->qglobal && (g->members[i]->GetAppearance() != eaDead) && g->members[i]->IsEngaged()) {
-					c->Message(0, "You can't summon bots while you are engaged.");
-					return;
+		// blocks spawn if any group member is engaged
+		Group *g = c->GetGroup();
+		if(g) {
+			for(int i = 0; i < MAX_GROUP_MEMBERS; i++) {
+				// if current group member is invalid or dead
+				if(!g->members[i] || g->members[i]->qglobal || g->members[i]->GetAppearance() == eaDead) {
+					continue;
 				}
-				if(g && g->members[i] && g->members[i]->qglobal) {
+				// if group member is a client and has aggro OR group member is a bot and has populated hate list
+				else if ( ( g->members[i]->IsClient() && g->members[i]->CastToClient()->GetAggroCount() > 0 )
+					 ||   ( g->members[i]->IsBot() && g->members[i]->IsEngaged() ) ) {
+					c->Message(0, "You can't spawn bots while your group is engaged.");
 					return;
 				}
 			}
+		// blocks spawn if ungrouped client has aggro
+		} else {
+			if(c->GetAggroCount() > 0) {
+				c->Message(0, "You can't spawn bots while you are engaged.");
+				return;
+			}
 		}
 
 		Mob* TempBotMob = entity_list.GetMobByBotID(botId);