epic/OpenRA
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Properly account for disabled Armaments in various places

Browse Source 
These places didn't care if an Armament was disabled, which could lead to unexpected behavior.
This commit is contained in:
reaperrr
2017-08-19 14:01:48 +02:00
committed by Paul Chote
parent 951ee2210c
commit c75e64a952
3 changed files with 7 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
OpenRA.Mods.Common/Activities/Air/FlyAttack.cs
View File

@@ -59,7 +59,7 @@ namespace OpenRA.Mods.Common.Activities
return NextActivity;
// TODO: This should fire each weapon at its maximum range
if (attackPlane != null && target.IsInRange(self.CenterPosition, attackPlane.Armaments.Select(a => a.Weapon.MinRange).Min()))
if (attackPlane != null && target.IsInRange(self.CenterPosition, attackPlane.Armaments.Where(Exts.IsTraitEnabled).Select(a => a.Weapon.MinRange).Min()))
ChildActivity = ActivityUtils.SequenceActivities(new FlyTimed(ticksUntilTurn, self), new Fly(self, target), new FlyTimed(ticksUntilTurn, self));
else
ChildActivity = ActivityUtils.SequenceActivities(new Fly(self, target), new FlyTimed(ticksUntilTurn, self));

3
OpenRA.Mods.Common/Traits/Attack/AttackGarrisoned.cs
View File

@@ -151,6 +151,9 @@ namespace OpenRA.Mods.Common.Traits
foreach (var a in Armaments)
{
if (a.IsTraitDisabled)
continue;
var port = SelectFirePort(self, targetYaw);
if (port == null)
return;

3
OpenRA.Mods.Common/Traits/CombatDebugOverlay.cs
View File

@@ -115,6 +115,9 @@ namespace OpenRA.Mods.Common.Traits
foreach (var a in attack.Armaments)
{
if (a.IsTraitDisabled)
continue;
foreach (var b in a.Barrels)
{
var muzzle = self.CenterPosition + a.MuzzleOffset(self, b);