diff --git a/OpenRA.Mods.Common/Traits/RejectsOrders.cs b/OpenRA.Mods.Common/Traits/RejectsOrders.cs index 4d17fb6de3..cb9bf71f88 100644 --- a/OpenRA.Mods.Common/Traits/RejectsOrders.cs +++ b/OpenRA.Mods.Common/Traits/RejectsOrders.cs @@ -10,6 +10,7 @@ #endregion using System.Collections.Generic; +using System.Linq; using OpenRA.Traits; namespace OpenRA.Mods.Common.Traits @@ -17,7 +18,11 @@ namespace OpenRA.Mods.Common.Traits [Desc("Can be used to make a unit partly uncontrollable by the player.")] public class RejectsOrdersInfo : ConditionalTraitInfo { - [Desc("Possible values include Attack, AttackMove, Guard, Move.")] + [Desc("Explicit list of rejected orders. Leave empty to reject all minus those listed under Except.")] + public readonly HashSet Reject = new HashSet(); + + [Desc("List of orders that should *not* be rejected.", + "Also overrides other instances of this trait's Reject fields.")] public readonly HashSet Except = new HashSet(); public override object Create(ActorInitializer init) { return new RejectsOrders(this); } @@ -25,6 +30,7 @@ namespace OpenRA.Mods.Common.Traits public class RejectsOrders : ConditionalTrait { + public HashSet Reject { get { return Info.Reject; } } public HashSet Except { get { return Info.Except; } } public RejectsOrders(RejectsOrdersInfo info) @@ -35,8 +41,8 @@ namespace OpenRA.Mods.Common.Traits { public static bool AcceptsOrder(this Actor self, string orderString) { - var r = self.TraitOrDefault(); - return r == null || r.IsTraitDisabled || r.Except.Contains(orderString); + var r = self.TraitsImplementing().Where(Exts.IsTraitEnabled).ToList(); + return !r.Any() || r.Any(t => t.Reject.Any() && !t.Reject.Contains(orderString)) || r.Any(t => t.Except.Contains(orderString)); } } }