Made husks targetable with force fire only #4941
Conversation
| @@ -199,6 +199,9 @@ bool CanTargetActor(Actor self, Target target, TargetModifiers modifiers, ref st | |||
| if (modifiers.HasModifier(TargetModifiers.ForceMove)) | |||
| return false; | |||
|
|
|||
| if (target.Actor.HasTrait<Husk>()) | |||
There was a problem hiding this comment.
👎 for hardcoding a gameplay decision that other mods may not want into our low-level shared traits.
This probably wants to live as a flag on the ITargetable interface / implementations.
There was a problem hiding this comment.
True, it also fits good at bridges. Updated.
|
Renamed the internal functions and variables as @pchote suggested. |
| @@ -85,6 +85,11 @@ public bool IsValidFor(Actor targeter) | |||
| return true; | |||
| } | |||
|
|
|||
| public bool RequiresForceFire() | |||
There was a problem hiding this comment.
This should be a property to be consistent with the rest of Target.
There was a problem hiding this comment.
You mean get { return targetable != null && targetable.RequiresForceFire; }?
|
I can't test this at the moment but if it all works and is nice internally: 👍 |
|
👍 after a couple of final consistency fixes:
|
|
Updated. |
|
Good job. It will be nice to finally put the husk-abuse exploits to rest. One (unconsidered?) side effect is that nukes will now vaporize the units in their blast, instead of leaving husks. |
Made husks targetable with force fire only
|
I also noticed that shooting with lot's of rockets on a tank can also kill the husk afterwards. We can combat this with more HP (and higher Burns: damage) for the wrecks. |
|
I don't think we should change HP due to a visual polish/husk-abuse-fix. |
This is a remake of #3955 which allows force fire to kill husks without making the AI go crazy or annoy players in the height of the battle.