X-Git-Url: http://git.xonotic.org/?a=blobdiff_plain;f=qcsrc%2Fserver%2Fmutators%2Fmutator%2Fgamemode_assault.qc;h=20b3b54c04bb5f4d106064c2cb6db090024ea674;hb=3a2b76cf66fc552ebfc0253f2d8b83308faf66c1;hp=b822911c8993d81dc70a13fa4aef88a2858075a6;hpb=cd3d62397dfaa03707d6b273538e8ce0157fbde9;p=xonotic%2Fxonotic-data.pk3dir.git

diff --git a/qcsrc/server/mutators/mutator/gamemode_assault.qc b/qcsrc/server/mutators/mutator/gamemode_assault.qc
index b822911c8..20b3b54c0 100644
--- a/qcsrc/server/mutators/mutator/gamemode_assault.qc
+++ b/qcsrc/server/mutators/mutator/gamemode_assault.qc
@@ -1,7 +1,9 @@
+#include "gamemode_assault.qh"
 #ifndef GAMEMODE_ASSAULT_H
 #define GAMEMODE_ASSAULT_H
 
 void assault_ScoreRules();
+void ActivateTeamplay();
 
 REGISTER_MUTATOR(as, false)
 {
@@ -43,51 +45,49 @@ const int HAVOCBOT_AST_ROLE_OFFENSE = 4;
 .int havocbot_role_flags;
 .float havocbot_attack_time;
 
-.void() havocbot_role;
-.void() havocbot_previous_role;
+.void(entity this) havocbot_role;
+.void(entity this) havocbot_previous_role;
 
-void() havocbot_role_ast_defense;
-void() havocbot_role_ast_offense;
+void(entity this) havocbot_role_ast_defense;
+void(entity this) havocbot_role_ast_offense;
 .entity havocbot_ast_target;
 
 void(entity bot) havocbot_ast_reset_role;
 
-void(float ratingscale, vector org, float sradius) havocbot_goalrating_items;
-void(float ratingscale, vector org, float sradius) havocbot_goalrating_enemyplayers;
+void(entity this, float ratingscale, vector org, float sradius) havocbot_goalrating_items;
+void(entity this, float ratingscale, vector org, float sradius) havocbot_goalrating_enemyplayers;
 
 // scoreboard stuff
 const float ST_ASSAULT_OBJECTIVES = 1;
 const float SP_ASSAULT_OBJECTIVES = 4;
 
 // predefined spawnfuncs
-void target_objective_decrease_activate();
+void target_objective_decrease_activate(entity this);
 #endif
 
 #ifdef IMPLEMENTATION
 .entity sprite;
 
 // random functions
-void assault_objective_use()
-{SELFPARAM();
+void assault_objective_use(entity this, entity actor, entity trigger)
+{
 	// activate objective
-	self.health = 100;
-	//print("^2Activated objective ", self.targetname, "=", etos(self), "\n");
-	//print("Activator is ", activator.classname, "\n");
+	this.health = 100;
+	//print("^2Activated objective ", this.targetname, "=", etos(this), "\n");
+	//print("Activator is ", actor.classname, "\n");
 
-	for (entity e = world; (e = find(e, target, this.targetname)); )
+	for (entity e = NULL; (e = find(e, target, this.targetname)); )
 	{
 		if (e.classname == "target_objective_decrease")
 		{
-			WITH(entity, self, e, target_objective_decrease_activate());
+			target_objective_decrease_activate(e);
 		}
 	}
-
-	setself(this);
 }
 
-vector target_objective_spawn_evalfunc(entity player, entity spot, vector current)
-{SELFPARAM();
-	if(self.health < 0 || self.health >= ASSAULT_VALUE_INACTIVE)
+vector target_objective_spawn_evalfunc(entity this, entity player, entity spot, vector current)
+{
+	if(this.health < 0 || this.health >= ASSAULT_VALUE_INACTIVE)
 		return '-1 0 0';
 	return current;
 }
@@ -100,115 +100,103 @@ void assault_objective_reset(entity this)
 }
 
 // decrease the health of targeted objectives
-void assault_objective_decrease_use()
-{SELFPARAM();
-	if(activator.team != assault_attacker_team)
+void assault_objective_decrease_use(entity this, entity actor, entity trigger)
+{
+	if(actor.team != assault_attacker_team)
 	{
 		// wrong team triggered decrease
 		return;
 	}
 
-	if(other.assault_sprite)
+	if(trigger.assault_sprite)
 	{
-		WaypointSprite_Disown(other.assault_sprite, waypointsprite_deadlifetime);
-		if(other.classname == "func_assault_destructible")
-			other.sprite = world;
+		WaypointSprite_Disown(trigger.assault_sprite, waypointsprite_deadlifetime);
+		if(trigger.classname == "func_assault_destructible")
+			trigger.sprite = NULL; // TODO: just unsetting it?!
 	}
 	else
 		return; // already activated! cannot activate again!
 
-	if(self.enemy.health < ASSAULT_VALUE_INACTIVE)
+	if(this.enemy.health < ASSAULT_VALUE_INACTIVE)
 	{
-		if(self.enemy.health - self.dmg > 0.5)
+		if(this.enemy.health - this.dmg > 0.5)
 		{
-			PlayerTeamScore_Add(activator, SP_SCORE, ST_SCORE, self.dmg);
-			self.enemy.health = self.enemy.health - self.dmg;
+			PlayerTeamScore_Add(actor, SP_SCORE, ST_SCORE, this.dmg);
+			this.enemy.health = this.enemy.health - this.dmg;
 		}
 		else
 		{
-			PlayerTeamScore_Add(activator, SP_SCORE, ST_SCORE, self.enemy.health);
-			PlayerTeamScore_Add(activator, SP_ASSAULT_OBJECTIVES, ST_ASSAULT_OBJECTIVES, 1);
-			self.enemy.health = -1;
-
-			entity oldactivator;
+			PlayerTeamScore_Add(actor, SP_SCORE, ST_SCORE, this.enemy.health);
+			PlayerTeamScore_Add(actor, SP_ASSAULT_OBJECTIVES, ST_ASSAULT_OBJECTIVES, 1);
+			this.enemy.health = -1;
 
-			setself(this.enemy);
-			if(self.message)
-				FOREACH_CLIENT(IS_PLAYER(it), LAMBDA(centerprint(it, self.message)));
+			if(this.enemy.message)
+				FOREACH_CLIENT(IS_PLAYER(it), LAMBDA(centerprint(it, this.enemy.message)));
 
-			oldactivator = activator;
-			activator = this;
-			SUB_UseTargets();
-			activator = oldactivator;
-			setself(this);
+			SUB_UseTargets(this.enemy, this, trigger);
 		}
 	}
 }
 
-void assault_setenemytoobjective()
-{SELFPARAM();
-	entity objective;
-	for(objective = world; (objective = find(objective, targetname, self.target)); )
+void assault_setenemytoobjective(entity this)
+{
+	FOREACH_ENTITY_STRING(targetname, this.target,
 	{
-		if(objective.classname == "target_objective")
+		if(it.classname == "target_objective")
 		{
-			if(self.enemy == world)
-				self.enemy = objective;
+			if(this.enemy == NULL)
+				this.enemy = it;
 			else
-				objerror("more than one objective as target - fix the map!");
+				objerror(this, "more than one objective as target - fix the map!");
 			break;
 		}
-	}
+	});
 
-	if(self.enemy == world)
-		objerror("no objective as target - fix the map!");
+	if(this.enemy == NULL)
+		objerror(this, "no objective as target - fix the map!");
 }
 
-float assault_decreaser_sprite_visible(entity e)
-{SELFPARAM();
-	entity decreaser;
-
-	decreaser = self.assault_decreaser;
-
-	if(decreaser.enemy.health >= ASSAULT_VALUE_INACTIVE)
+bool assault_decreaser_sprite_visible(entity this, entity player, entity view)
+{
+	if(this.assault_decreaser.enemy.health >= ASSAULT_VALUE_INACTIVE)
 		return false;
 
 	return true;
 }
 
-void target_objective_decrease_activate()
-{SELFPARAM();
-	entity ent, spr;
-	self.owner = world;
-	for(ent = world; (ent = find(ent, target, self.targetname)); )
+void target_objective_decrease_activate(entity this)
+{
+	entity spr;
+	this.owner = NULL;
+	FOREACH_ENTITY_STRING(target, this.targetname,
 	{
-		if(ent.assault_sprite != world)
+		if(it.assault_sprite != NULL)
 		{
-			WaypointSprite_Disown(ent.assault_sprite, waypointsprite_deadlifetime);
-			if(ent.classname == "func_assault_destructible")
-				ent.sprite = world;
+			WaypointSprite_Disown(it.assault_sprite, waypointsprite_deadlifetime);
+			if(it.classname == "func_assault_destructible")
+				it.sprite = NULL; // TODO: just unsetting it?!
 		}
 
-		spr = WaypointSprite_SpawnFixed(WP_Assault, 0.5 * (ent.absmin + ent.absmax), ent, assault_sprite, RADARICON_OBJECTIVE);
-		spr.assault_decreaser = self;
+		spr = WaypointSprite_SpawnFixed(WP_Assault, 0.5 * (it.absmin + it.absmax), it, assault_sprite, RADARICON_OBJECTIVE);
+		spr.assault_decreaser = this;
 		spr.waypointsprite_visible_for_player = assault_decreaser_sprite_visible;
 		spr.classname = "sprite_waypoint";
 		WaypointSprite_UpdateRule(spr, assault_attacker_team, SPRITERULE_TEAMPLAY);
-		if(ent.classname == "func_assault_destructible")
+		if(it.classname == "func_assault_destructible")
 		{
 			WaypointSprite_UpdateSprites(spr, WP_AssaultDefend, WP_AssaultDestroy, WP_AssaultDestroy);
-			WaypointSprite_UpdateMaxHealth(spr, ent.max_health);
-			WaypointSprite_UpdateHealth(spr, ent.health);
-			ent.sprite = spr;
+			WaypointSprite_UpdateMaxHealth(spr, it.max_health);
+			WaypointSprite_UpdateHealth(spr, it.health);
+			it.sprite = spr;
 		}
 		else
 			WaypointSprite_UpdateSprites(spr, WP_AssaultDefend, WP_AssaultPush, WP_AssaultPush);
-	}
+	});
 }
 
-void target_objective_decrease_findtarget()
+void target_objective_decrease_findtarget(entity this)
 {
-	assault_setenemytoobjective();
+	assault_setenemytoobjective(this);
 }
 
 void target_assault_roundend_reset(entity this)
@@ -218,15 +206,14 @@ void target_assault_roundend_reset(entity this)
 	this.winning = false; // up round
 }
 
-void target_assault_roundend_use()
-{SELFPARAM();
-	self.winning = 1; // round has been won by attackers
+void target_assault_roundend_use(entity this, entity actor, entity trigger)
+{
+	this.winning = 1; // round has been won by attackers
 }
 
-void assault_roundstart_use()
-{SELFPARAM();
-	activator = self;
-	SUB_UseTargets();
+void assault_roundstart_use(entity this, entity actor, entity trigger)
+{
+	SUB_UseTargets(this, this, trigger);
 
 	//(Re)spawn all turrets
 	FOREACH_ENTITY_CLASS("turret_main", true, LAMBDA(
@@ -236,48 +223,49 @@ void assault_roundstart_use()
 		else
 			it.team = NUM_TEAM_1;
 
-		// Dubbles as teamchange
-		WITH(entity, self, it, turret_respawn());
+		// Doubles as teamchange
+		turret_respawn(it);
 	));
 }
+void assault_roundstart_use_this(entity this)
+{
+	assault_roundstart_use(this, NULL, NULL);
+}
 
-void assault_wall_think()
-{SELFPARAM();
-	if(self.enemy.health < 0)
+void assault_wall_think(entity this)
+{
+	if(this.enemy.health < 0)
 	{
-		self.model = "";
-		self.solid = SOLID_NOT;
+		this.model = "";
+		this.solid = SOLID_NOT;
 	}
 	else
 	{
-		self.model = self.mdl;
-		self.solid = SOLID_BSP;
+		this.model = this.mdl;
+		this.solid = SOLID_BSP;
 	}
 
-	self.nextthink = time + 0.2;
+	this.nextthink = time + 0.2;
 }
 
 // trigger new round
 // reset objectives, toggle spawnpoints, reset triggers, ...
 void vehicles_clearreturn(entity veh);
-void vehicles_spawn();
-void assault_new_round()
-{SELFPARAM();
+void vehicles_spawn(entity this);
+void assault_new_round(entity this)
+{
 	//bprint("ASSAULT: new round\n");
 
 	// Eject players from vehicles
-    FOREACH_CLIENT(IS_PLAYER(it) && it.vehicle, LAMBDA(WITH(entity, self, it, vehicles_exit(VHEF_RELEASE))));
+    FOREACH_CLIENT(IS_PLAYER(it) && it.vehicle, vehicles_exit(it.vehicle, VHEF_RELEASE));
 
     FOREACH_ENTITY_FLAGS(vehicle_flags, VHF_ISVEHICLE, LAMBDA(
-    	setself(it);
-        vehicles_clearreturn(self);
-        vehicles_spawn();
+        vehicles_clearreturn(it);
+        vehicles_spawn(it);
     ));
 
-    setself(this);
-
 	// up round counter
-	self.winning = self.winning + 1;
+	this.winning = this.winning + 1;
 
 	// swap attacker/defender roles
 	if(assault_attacker_team == NUM_TEAM_1)
@@ -299,7 +287,7 @@ void assault_new_round()
 // they win. Otherwise the defending team wins once the timelimit passes.
 int WinningCondition_Assault()
 {
-	WinningConditionHelper(); // set worldstatus
+	WinningConditionHelper(NULL); // set worldstatus
 
 	int status = WINNING_NO;
 	// as the timelimit has not yet passed just assume the defending team will win
@@ -313,7 +301,7 @@ int WinningCondition_Assault()
 	}
 
 	entity ent;
-	ent = find(world, classname, "target_assault_roundend");
+	ent = find(NULL, classname, "target_assault_roundend");
 	if(ent)
 	{
 		if(ent.winning)	// round end has been triggered by attacking team
@@ -329,7 +317,7 @@ int WinningCondition_Assault()
 			}
 			else
 			{
-				WITH(entity, self, ent, assault_new_round());
+				assault_new_round(ent);
 			}
 		}
 	}
@@ -340,17 +328,17 @@ int WinningCondition_Assault()
 // spawnfuncs
 spawnfunc(info_player_attacker)
 {
-	if (!g_assault) { remove(self); return; }
+	if (!g_assault) { remove(this); return; }
 
-	self.team = NUM_TEAM_1; // red, gets swapped every round
+	this.team = NUM_TEAM_1; // red, gets swapped every round
 	spawnfunc_info_player_deathmatch(this);
 }
 
 spawnfunc(info_player_defender)
 {
-	if (!g_assault) { remove(self); return; }
+	if (!g_assault) { remove(this); return; }
 
-	self.team = NUM_TEAM_2; // blue, gets swapped every round
+	this.team = NUM_TEAM_2; // blue, gets swapped every round
 	spawnfunc_info_player_deathmatch(this);
 }
 
@@ -367,78 +355,79 @@ spawnfunc(target_objective)
 
 spawnfunc(target_objective_decrease)
 {
-	if (!g_assault) { remove(self); return; }
+	if (!g_assault) { remove(this); return; }
 
-	self.classname = "target_objective_decrease";
+	this.classname = "target_objective_decrease";
 
-	if(!self.dmg)
-		self.dmg = 101;
+	if(!this.dmg)
+		this.dmg = 101;
 
-	self.use = assault_objective_decrease_use;
-	self.health = ASSAULT_VALUE_INACTIVE;
-	self.max_health = ASSAULT_VALUE_INACTIVE;
-	self.enemy = world;
+	this.use = assault_objective_decrease_use;
+	this.health = ASSAULT_VALUE_INACTIVE;
+	this.max_health = ASSAULT_VALUE_INACTIVE;
+	this.enemy = NULL;
 
-	InitializeEntity(self, target_objective_decrease_findtarget, INITPRIO_FINDTARGET);
+	InitializeEntity(this, target_objective_decrease_findtarget, INITPRIO_FINDTARGET);
 }
 
 // destructible walls that can be used to trigger target_objective_decrease
 spawnfunc(func_breakable);
 spawnfunc(func_assault_destructible)
 {
-	if (!g_assault) { remove(self); return; }
+	if (!g_assault) { remove(this); return; }
 
-	self.spawnflags = 3;
-	self.classname = "func_assault_destructible";
+	this.spawnflags = 3;
+	this.classname = "func_assault_destructible";
 
 	if(assault_attacker_team == NUM_TEAM_1)
-		self.team = NUM_TEAM_2;
+		this.team = NUM_TEAM_2;
 	else
-		self.team = NUM_TEAM_1;
+		this.team = NUM_TEAM_1;
 
 	spawnfunc_func_breakable(this);
 }
 
 spawnfunc(func_assault_wall)
 {
-	if (!g_assault) { remove(self); return; }
+	if (!g_assault) { remove(this); return; }
 
-	self.classname = "func_assault_wall";
-	self.mdl = self.model;
-	_setmodel(self, self.mdl);
-	self.solid = SOLID_BSP;
-	self.think = assault_wall_think;
-	self.nextthink = time;
-	InitializeEntity(self, assault_setenemytoobjective, INITPRIO_FINDTARGET);
+	this.classname = "func_assault_wall";
+	this.mdl = this.model;
+	_setmodel(this, this.mdl);
+	this.solid = SOLID_BSP;
+	setthink(this, assault_wall_think);
+	this.nextthink = time;
+	InitializeEntity(this, assault_setenemytoobjective, INITPRIO_FINDTARGET);
 }
 
 spawnfunc(target_assault_roundend)
 {
-	if (!g_assault) { remove(self); return; }
+	if (!g_assault) { remove(this); return; }
 
-	self.winning = 0; // round not yet won by attackers
-	self.classname = "target_assault_roundend";
-	self.use = target_assault_roundend_use;
-	self.cnt = 0; // first round
-	self.reset = target_assault_roundend_reset;
+	this.winning = 0; // round not yet won by attackers
+	this.classname = "target_assault_roundend";
+	this.use = target_assault_roundend_use;
+	this.cnt = 0; // first round
+	this.reset = target_assault_roundend_reset;
 }
 
 spawnfunc(target_assault_roundstart)
 {
-	if (!g_assault) { remove(self); return; }
+	if (!g_assault) { remove(this); return; }
 
 	assault_attacker_team = NUM_TEAM_1;
-	self.classname = "target_assault_roundstart";
-	self.use = assault_roundstart_use;
-	self.reset2 = assault_roundstart_use;
-	InitializeEntity(self, assault_roundstart_use, INITPRIO_FINDTARGET);
+	this.classname = "target_assault_roundstart";
+	this.use = assault_roundstart_use;
+	this.reset2 = assault_roundstart_use_this;
+	InitializeEntity(this, assault_roundstart_use_this, INITPRIO_FINDTARGET);
 }
 
 // legacy bot code
-void havocbot_goalrating_ast_targets(float ratingscale)
-{SELFPARAM();
-	entity ad, best, wp, tod;
-	float radius, found, bestvalue;
+void havocbot_goalrating_ast_targets(entity this, float ratingscale)
+{
+	entity ad, best, wp;
+	float radius, bestvalue;
+	bool found;
 	vector p;
 
 	ad = findchain(classname, "func_assault_destructible");
@@ -452,18 +441,18 @@ void havocbot_goalrating_ast_targets(float ratingscale)
 			continue;
 
 		found = false;
-		for(tod = world; (tod = find(tod, targetname, ad.target)); )
+		FOREACH_ENTITY_STRING(targetname, ad.target,
 		{
-			if(tod.classname == "target_objective_decrease")
+			if(it.classname == "target_objective_decrease")
 			{
-				if(tod.enemy.health > 0 && tod.enemy.health < ASSAULT_VALUE_INACTIVE)
+				if(it.enemy.health > 0 && it.enemy.health < ASSAULT_VALUE_INACTIVE)
 				{
 				//	dprint(etos(ad),"\n");
 					found = true;
 					break;
 				}
 			}
-		}
+		});
 
 		if(!found)
 		{
@@ -476,11 +465,11 @@ void havocbot_goalrating_ast_targets(float ratingscale)
 		p = 0.5 * (ad.absmin + ad.absmax);
 	//	dprint(vtos(ad.origin), " ", vtos(ad.absmin), " ", vtos(ad.absmax),"\n");
 	//	te_knightspike(p);
-	//	te_lightning2(world, '0 0 0', p);
+	//	te_lightning2(NULL, '0 0 0', p);
 
 		// Find and rate waypoints around it
 		found = false;
-		best = world;
+		best = NULL;
 		bestvalue = 99999999999;
 		for(radius=0; radius<1500 && !found; radius+=500)
 		{
@@ -503,154 +492,158 @@ void havocbot_goalrating_ast_targets(float ratingscale)
 		if(best)
 		{
 		///	dprint("waypoints around target were found\n");
-		//	te_lightning2(world, '0 0 0', best.origin);
+		//	te_lightning2(NULL, '0 0 0', best.origin);
 		//	te_knightspike(best.origin);
 
-			navigation_routerating(best, ratingscale, 4000);
+			navigation_routerating(this, best, ratingscale, 4000);
 			best.cnt += 1;
 
-			self.havocbot_attack_time = 0;
+			this.havocbot_attack_time = 0;
 
-			if(checkpvs(self.view_ofs,ad))
-			if(checkpvs(self.view_ofs,best))
+			if(checkpvs(this.view_ofs,ad))
+			if(checkpvs(this.view_ofs,best))
 			{
 			//	dprint("increasing attack time for this target\n");
-				self.havocbot_attack_time = time + 2;
+				this.havocbot_attack_time = time + 2;
 			}
 		}
 	}
 }
 
-void havocbot_role_ast_offense()
-{SELFPARAM();
-	if(IS_DEAD(self))
+void havocbot_role_ast_offense(entity this)
+{
+	if(IS_DEAD(this))
 	{
-		self.havocbot_attack_time = 0;
-		havocbot_ast_reset_role(self);
+		this.havocbot_attack_time = 0;
+		havocbot_ast_reset_role(this);
 		return;
 	}
 
 	// Set the role timeout if necessary
-	if (!self.havocbot_role_timeout)
-		self.havocbot_role_timeout = time + 120;
+	if (!this.havocbot_role_timeout)
+		this.havocbot_role_timeout = time + 120;
 
-	if (time > self.havocbot_role_timeout)
+	if (time > this.havocbot_role_timeout)
 	{
-		havocbot_ast_reset_role(self);
+		havocbot_ast_reset_role(this);
 		return;
 	}
 
-	if(self.havocbot_attack_time>time)
+	if(this.havocbot_attack_time>time)
 		return;
 
-	if (self.bot_strategytime < time)
+	if (this.bot_strategytime < time)
 	{
-		navigation_goalrating_start();
-		havocbot_goalrating_enemyplayers(20000, self.origin, 650);
-		havocbot_goalrating_ast_targets(20000);
-		havocbot_goalrating_items(15000, self.origin, 10000);
-		navigation_goalrating_end();
+		navigation_goalrating_start(this);
+		havocbot_goalrating_enemyplayers(this, 20000, this.origin, 650);
+		havocbot_goalrating_ast_targets(this, 20000);
+		havocbot_goalrating_items(this, 15000, this.origin, 10000);
+		navigation_goalrating_end(this);
 
-		self.bot_strategytime = time + autocvar_bot_ai_strategyinterval;
+		this.bot_strategytime = time + autocvar_bot_ai_strategyinterval;
 	}
 }
 
-void havocbot_role_ast_defense()
-{SELFPARAM();
-	if(IS_DEAD(self))
+void havocbot_role_ast_defense(entity this)
+{
+	if(IS_DEAD(this))
 	{
-		self.havocbot_attack_time = 0;
-		havocbot_ast_reset_role(self);
+		this.havocbot_attack_time = 0;
+		havocbot_ast_reset_role(this);
 		return;
 	}
 
 	// Set the role timeout if necessary
-	if (!self.havocbot_role_timeout)
-		self.havocbot_role_timeout = time + 120;
+	if (!this.havocbot_role_timeout)
+		this.havocbot_role_timeout = time + 120;
 
-	if (time > self.havocbot_role_timeout)
+	if (time > this.havocbot_role_timeout)
 	{
-		havocbot_ast_reset_role(self);
+		havocbot_ast_reset_role(this);
 		return;
 	}
 
-	if(self.havocbot_attack_time>time)
+	if(this.havocbot_attack_time>time)
 		return;
 
-	if (self.bot_strategytime < time)
+	if (this.bot_strategytime < time)
 	{
-		navigation_goalrating_start();
-		havocbot_goalrating_enemyplayers(20000, self.origin, 3000);
-		havocbot_goalrating_ast_targets(20000);
-		havocbot_goalrating_items(15000, self.origin, 10000);
-		navigation_goalrating_end();
+		navigation_goalrating_start(this);
+		havocbot_goalrating_enemyplayers(this, 20000, this.origin, 3000);
+		havocbot_goalrating_ast_targets(this, 20000);
+		havocbot_goalrating_items(this, 15000, this.origin, 10000);
+		navigation_goalrating_end(this);
 
-		self.bot_strategytime = time + autocvar_bot_ai_strategyinterval;
+		this.bot_strategytime = time + autocvar_bot_ai_strategyinterval;
 	}
 }
 
-void havocbot_role_ast_setrole(entity bot, float role)
+void havocbot_role_ast_setrole(entity this, float role)
 {
 	switch(role)
 	{
 		case HAVOCBOT_AST_ROLE_DEFENSE:
-			bot.havocbot_role = havocbot_role_ast_defense;
-			bot.havocbot_role_flags = HAVOCBOT_AST_ROLE_DEFENSE;
-			bot.havocbot_role_timeout = 0;
+			this.havocbot_role = havocbot_role_ast_defense;
+			this.havocbot_role_flags = HAVOCBOT_AST_ROLE_DEFENSE;
+			this.havocbot_role_timeout = 0;
 			break;
 		case HAVOCBOT_AST_ROLE_OFFENSE:
-			bot.havocbot_role = havocbot_role_ast_offense;
-			bot.havocbot_role_flags = HAVOCBOT_AST_ROLE_OFFENSE;
-			bot.havocbot_role_timeout = 0;
+			this.havocbot_role = havocbot_role_ast_offense;
+			this.havocbot_role_flags = HAVOCBOT_AST_ROLE_OFFENSE;
+			this.havocbot_role_timeout = 0;
 			break;
 	}
 }
 
-void havocbot_ast_reset_role(entity bot)
-{SELFPARAM();
-	if(IS_DEAD(self))
+void havocbot_ast_reset_role(entity this)
+{
+	if(IS_DEAD(this))
 		return;
 
-	if(bot.team == assault_attacker_team)
-		havocbot_role_ast_setrole(bot, HAVOCBOT_AST_ROLE_OFFENSE);
+	if(this.team == assault_attacker_team)
+		havocbot_role_ast_setrole(this, HAVOCBOT_AST_ROLE_OFFENSE);
 	else
-		havocbot_role_ast_setrole(bot, HAVOCBOT_AST_ROLE_DEFENSE);
+		havocbot_role_ast_setrole(this, HAVOCBOT_AST_ROLE_DEFENSE);
 }
 
 // mutator hooks
 MUTATOR_HOOKFUNCTION(as, PlayerSpawn)
-{SELFPARAM();
-	if(self.team == assault_attacker_team)
-		Send_Notification(NOTIF_ONE, self, MSG_CENTER, CENTER_ASSAULT_ATTACKING);
-	else
-		Send_Notification(NOTIF_ONE, self, MSG_CENTER, CENTER_ASSAULT_DEFENDING);
+{
+	entity player = M_ARGV(0, entity);
 
-	return false;
+	if(player.team == assault_attacker_team)
+		Send_Notification(NOTIF_ONE, player, MSG_CENTER, CENTER_ASSAULT_ATTACKING);
+	else
+		Send_Notification(NOTIF_ONE, player, MSG_CENTER, CENTER_ASSAULT_DEFENDING);
 }
 
 MUTATOR_HOOKFUNCTION(as, TurretSpawn)
-{SELFPARAM();
-	if(!self.team || self.team == MAX_SHOT_DISTANCE)
-		self.team = 5; // this gets reversed when match starts?
+{
+	entity turret = M_ARGV(0, entity);
 
-	return false;
+	if(!turret.team || turret.team == MAX_SHOT_DISTANCE)
+		turret.team = 5; // this gets reversed when match starts?
 }
 
 MUTATOR_HOOKFUNCTION(as, VehicleSpawn)
-{SELFPARAM();
-	self.nextthink = time + 0.5;
+{
+	entity veh = M_ARGV(0, entity);
 
-	return false;
+	veh.nextthink = time + 0.5;
 }
 
 MUTATOR_HOOKFUNCTION(as, HavocBot_ChooseRole)
-{SELFPARAM();
-	havocbot_ast_reset_role(self);
+{
+	entity bot = M_ARGV(0, entity);
+
+	havocbot_ast_reset_role(bot);
 	return true;
 }
 
 MUTATOR_HOOKFUNCTION(as, PlayHitsound)
 {
+	entity frag_victim = M_ARGV(0, entity);
+
 	return (frag_victim.classname == "func_assault_destructible");
 }
 
@@ -663,7 +656,7 @@ MUTATOR_HOOKFUNCTION(as, GetTeamCount)
 
 MUTATOR_HOOKFUNCTION(as, CheckRules_World)
 {
-	ret_float = WinningCondition_Assault();
+	M_ARGV(0, float) = WinningCondition_Assault();
 	return true;
 }
 
@@ -671,12 +664,13 @@ MUTATOR_HOOKFUNCTION(as, ReadLevelCvars)
 {
 	// no assault warmups
 	warmup_stage = 0;
-	return false;
 }
 
 MUTATOR_HOOKFUNCTION(as, OnEntityPreSpawn)
 {
-	switch(self.classname)
+    entity ent = M_ARGV(0, entity);
+
+	switch(ent.classname)
 	{
 		case "info_player_team1":
 		case "info_player_team2":
@@ -684,8 +678,6 @@ MUTATOR_HOOKFUNCTION(as, OnEntityPreSpawn)
 		case "info_player_team4":
 			return true;
 	}
-
-	return false;
 }
 
 // scoreboard setup