Wow nice work bot. Does this override the regular Booster combo then? 

Also, you might consider using a timer instead of a tick as to avoid the rounding errors. 

There's got to be some way to suck the package name off a class object or something so as to prevent the naming problem. 

Would it be possible to set it so that when your shield is full, the combo exits - rather than waiting until your adrenaline runs out? 

Szlat wrote:

Would it be possible to set it so that when your shield is full, the combo exits - rather than waiting until your adrenaline runs out? 

Certainly.

The next question is: Dru, do you want it that way? 

I suppose it ought to actually check for health and shield being full, just in case someone starts off the booster, then picks up 150 shield. It would be sad if it exited with them only having 50 health. 

 function Timer()
 
 {
 	local Pawn P;
 	local bool ShieldNotFull;
 
 // We need to default this to True, so that we don't stop unless it's false.
 	ShieldNotFull = True;
 	P = Pawn(Owner);
 	if (P == None || P.Controller == None)
 	{
 		Destroy();
 		return;
 	}
 
 	if (P.Role == ROLE_Authority)
 
 	{
 
 		P.GiveHealth(5, P.SuperHealthMax);
 
 		if ( P.Health >= Pawn(Owner).SuperHealthMax )
 // AddShieldStrength returns True if ShieldStrength is changed
 // We won't react to the change in ShieldNotFull until after adren is eaten.
 
 			ShieldNotFull = P.AddShieldStrength(5);
 // Flag Carriers loose adren twice as fast!
 		if (P.Controller.PlayerReplicationInfo != None && P.Controller.PlayerReplicationInfo.HasFlag != None)
 		{
 			P.Controller.Adrenaline -= 6;
 		}else
 		{
 			P.Controller.Adrenaline -= 3;
 		}
 		if (P.Controller.Adrenaline <= 0)
 		{
 			P.Controller.Adrenaline = 0;
 			Destroy();
 			return;
 		}
 // If Shield is full, then stop.
 		if (!ShieldNotFull)
 		{
 			Destroy();
 			return;
 		}
 	}
 }
 
 // Overwrite Combo's Tick - adren use now in Timer
 simulated function Tick(float DeltaTime)
 {
 	// Do nothing.
 }

As an aside, the two artifacts HealingBlast and SphereHealing currently check for players running a 'ComboDefensive', and will not heal those players. If we add BotBooster instead of ComboDefensive, that code is still probably ok. In this case it will heal the player - which is ok (since the player will be running a 'BotBooster' not a 'ComboDefensive'). 

There's got to be some way to suck the package name off a class object or something so as to prevent the naming problem. 

It looks like everytime you use "BotBooster.BotComboDefensive" it is as a string. Can you just have a UT2004RPG.ini config variable that contains that string? Then the package can be changed in the ini file without recompiling?
Not perfect, but better than hard coded. 

as for disabling tick, in PostNetBeginPlay()

just Engine.disable('tick');