[Antithasys]

I added some error checking although I still never run GetClientName(). I think you got that error from my modified ammopacks and medipacks, which I corrected in their threads.

I went ahead and had it return if the client is not connected, which doesn't make sense because why would the event run on a client not connected, but ok its in there. I am guessing that sometimes the server runs this event and that is why the client index is 0, and I guess it is possible that during the split second that the event is run the client may disconnect. Either way, I added:

if (IsClientConnected(client)) {

Let me know if you are having any additional problems.

[bl4nk]

I don't see how this plugin will even work. The way the whole system works is that it picks a player to switch, switches them, and then fires the event. The event doesn't actually control whether or not the player was switched, it just controls whether or not the message is relayed to the players.

Taken from CTeamplayRoundBasedRules::BalanceTeams(bool bRequireSwitcheesToBeDead):

    for ( int i=0; i<vecPlayers.Count() && iNumSwitchesRequired > 0 && i < iNumEligibleSwitchees; i++ )
    {
        pPlayer = vecPlayers.Element(i);

        Assert( pPlayer );

        if ( !pPlayer )
            continue;

        if ( bRequireSwitcheesToBeDead == false || !pPlayer->IsAlive() )
        {
            pPlayer->ChangeTeam( iLightestTeam );
            pPlayer->SetLastForcedChangeTeamTimeToNow();

            IGameEvent *event = gameeventmanager->CreateEvent( "teamplay_teambalanced_player" );
            if ( event )
            {
                event->SetInt( "player", pPlayer->entindex() );
                event->SetInt( "team", iLightestTeam );
                gameeventmanager->FireEvent( event );
            }

            // tell people that we've switched this player
            UTIL_ClientPrintAll( HUD_PRINTTALK, "#game_player_was_team_balanced", pPlayer->GetPlayerName() );

            iNumSwitchesRequired--;
        }
    }

[Antithasys]

Well like I said it has been hard to test. After 3 days none of my admins were switched so I figured it worked.

Since I was getting the event in _Pre mode I figured switching the team back would make the event essentially "try again".

But from looking at what you posted, it appears that this will actually do nothing and I have just been lucky with the autobalances.

Would be nice if the event actually controlled the switching.

Oh well. Delete this thread or move to unapproved.

[ThatGuy]

I test by adding bots, then kicking them. Forcing the teams to balance, and kill yourself a few times to see if it works.

[bl4nk]

I did basically the same thing, but I modified the plugin to print a text saying that user x is going to be switched, and the block the event. It would end up with the user still being switched over. I then decided to dig into the SDK and found what I showed up above.

[DontWannaName]

All I know is it gives a whole lot of errors. Why cant you just take the code out of Pinkfaries donator plugin and make it with the a flag.

[ThatGuy]

I tried doing that but i couldn't get it to work. Now i'm looking at editing Crimson's team balancer to make it block admins. Though, I of course won't release it, because it's not my plugin to showoff.

[Antithasys]

Originally Posted by DontWannaName All I know is it gives a whole lot of errors. Why cant you just take the code out of Pinkfaries donator plugin and make it with the a flag.

Pinkfaire hooks the same event and it will have the same results.

Basically we(as in someone) just needs to make a autobalancer that does it very simply. I have already started to work on one, but it will take a week or so to test and get it on the forums.