Refactor: Replace bNeedsHazardEscape var with NeedsHazardEscape() function

Cannot add variables to native classes, so converted the hazard escape check from a class variable to a helper function. The function performs the same check: empty tile cache + unit in hazard = needs escape.

Author: Whakomatic

X2WOTCCommunityHighlander/Src/XComGame/Classes/XGAIBehavior.uc

@@ -431,8 +431,6 @@ struct native DebugAoEResult
 var array<DebugAoEResult> DebugAoEList;
 var int CurrAoEIndex;
 
-// HL-Docs: ref:AIHazardEscape; issue:1559; tags:tactical
-var bool bNeedsHazardEscape;
 
 // Generic ability selector for units that cannot be given a custom behavior tree.  
 // This should be used when the AI has no idea what abilities this unit has.  (Shadow/Clone units, etc)
@@ -1531,13 +1529,6 @@ simulated function BT_StartGetDestinations(bool bFiltered=false, bool bSkipBuild
 
 	m_kUnit.m_kReachableTilesCache.UpdateTileCacheIfNeeded();
 
-	// HL-Docs: feature:AIHazardEscape; issue:1559
-	// Allows AI units to escape when surrounded by hazardous world effects (poison, fire, acid). 
-	// When the normal pathfinding cache is empty due to hazards blocking all tiles, 
-	// the AI gathers nearby non-hazardous tiles (3-5 tiles away), scores them using the standard tactical scoring system, 
-	// and paths through hazards using BuildNonUnitPath to reach the best-scoring safe destination.
-	bNeedsHazardEscape = (m_kUnit.m_kReachableTilesCache.CoverDestinations.Length == 0 && class'XComPath'.static.TileContainsHazard(UnitState, UnitState.TileLocation));
-
 	// Special case for grapple movement.
 	if ( m_kCurrMoveRestriction.bIsGrappleMove )
 	{
@@ -1568,10 +1559,10 @@ simulated function BT_StartGetDestinations(bool bFiltered=false, bool bSkipBuild
 			AddTileToProcess(kTileScore);
 		}
 	}
-	else if (bNeedsHazardEscape)
+	// HL-Docs: ref:AIHazardEscape; issue:1559
+	// placed before CoverDestinations check because m_kReachableTilesCache will be empty
+	else if (NeedsHazardEscape())
 	{
-		// HL-Docs: ref:AIHazardEscape; issue:1559
-		// placed before CoverDestinations check because m_kReachableTilesCache will be empty
 		AllTiles = GatherHazardEscapeTiles();
 		foreach AllTiles(kTile)
 		{
@@ -1818,6 +1809,16 @@ function BT_IgnoreHazards( bool bIgnore=true )
 	bIgnoreHazards = bIgnore;
 }
 
+// HL-Docs: feature:AIHazardEscape; issue:1559; tags:tactical
+// Allows AI units to escape when surrounded by hazardous world effects (poison, fire, acid).
+// When the normal pathfinding cache is empty due to hazards blocking all tiles,
+// the AI gathers nearby non-hazardous tiles (3-5 tiles away), scores them using the standard tactical scoring system,
+// and paths through hazards using BuildNonUnitPath to reach the best-scoring safe destination.
+function bool NeedsHazardEscape()
+{
+	return (m_kUnit.m_kReachableTilesCache.CoverDestinations.Length == 0 && class'XComPath'.static.TileContainsHazard(UnitState, UnitState.TileLocation));
+}
+
 // HL-Docs: ref:AIHazardEscape; issue:1559
 // Gather nearby non-hazardous tiles for escape pathing then let BT_StepProcessDestinations score them
 function array<TTile> GatherHazardEscapeTiles()
@@ -1992,7 +1993,7 @@ simulated function BT_StepProcessDestinations()
 			// HL-Docs: ref:AIHazardEscape; issue:1559
 			// m_kReachableTilesCache will be empty if we are surrounded by a hazard
 			//See if this tile is reachable
-			if ( bValid && !bNeedsHazardEscape && !m_kCurrMoveRestriction.bIsGrappleMove && !m_kUnit.m_kReachableTilesCache.IsTileReachable(kTileData.kTile) )
+			if ( bValid && !NeedsHazardEscape() && !m_kCurrMoveRestriction.bIsGrappleMove && !m_kUnit.m_kReachableTilesCache.IsTileReachable(kTileData.kTile) )
 			{
 				if (bLogTacticalDestinationIteration)
 				{
@@ -8651,7 +8652,7 @@ simulated function bool MoveToPoint(vector vDestination, optional out string Fai
 
 	// HL-Docs: ref:AIHazardEscape; issue:1559
 	// IsTileReachable will always return false if surrounded by hazards
-	if (!bPathFailed && !bNeedsHazardEscape)
+	if (!bPathFailed && !NeedsHazardEscape())
 	{
 		bPathFailed = !m_kUnit.m_kReachableTilesCache.IsTileReachable(kTileDest);
 		if (bPathFailed)
@@ -8660,7 +8661,7 @@ simulated function bool MoveToPoint(vector vDestination, optional out string Fai
 		}
 	}
 
-	if( !bPathFailed || bForcePathIfUnreachable || bNeedsHazardEscape )
+	if( !bPathFailed || bForcePathIfUnreachable || NeedsHazardEscape() )
 	{
 		if (XGAIBehavior_Civilian(self) != none)
 		{
@@ -8669,7 +8670,7 @@ simulated function bool MoveToPoint(vector vDestination, optional out string Fai
 
 		// HL-Docs: ref:AIHazardEscape; issue:1559
 		// Use BuildNonUnitPath for hazard escape to ensure the unit can path out of hazards
-		if (bNeedsHazardEscape)
+		if (NeedsHazardEscape())
 		{
 			// Build allowed traversals from unit's character template
 			if (UnitState.GetMyTemplate().bCanUse_eTraversal_Normal) AllowedTraversals.AddItem(eTraversal_Normal);