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

Let CreateGroup use ExtraActors field.

Browse Source
This commit is contained in:
tovl
2019-08-31 13:01:56 +02:00
committed by abcdefg30
parent 00ce1d7ee6
commit 6fb3dc050b
2 changed files with 3 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
OpenRA.Game/Network/Order.cs
View File

@@ -11,7 +11,6 @@
using System;
using System.IO;
using System.Linq;
using OpenRA.Network;
using OpenRA.Traits;
@@ -271,7 +270,7 @@ namespace OpenRA
minLength += 4 + 1 + 13 + (TargetString != null ? TargetString.Length + 1 : 0) + 4 + 4 + 4;
if (ExtraActors != null)
minLength += ExtraActors.Count() * 4;
minLength += ExtraActors.Length * 4;
// ProtocolVersion.Orders and the associated documentation MUST be updated if the serialized format changes
var ret = new MemoryStream(minLength);
@@ -352,7 +351,7 @@ namespace OpenRA
if (fields.HasField(OrderFields.ExtraActors))
{
w.Write(ExtraActors.Count());
w.Write(ExtraActors.Length);
foreach (var a in ExtraActors)
w.Write(UIntFromActor(a));
}

5
OpenRA.Game/Orders/UnitOrderGenerator.cs
View File

@@ -52,10 +52,7 @@ namespace OpenRA.Orders
// HACK: This is required by the hacky player actions-per-minute calculation
// TODO: Reimplement APM properly and then remove this
yield return new Order("CreateGroup", actorsInvolved.First().Owner.PlayerActor, false)
{
TargetString = actorsInvolved.Select(a => a.ActorID).JoinWith(",")
};
yield return new Order("CreateGroup", actorsInvolved.First().Owner.PlayerActor, false, actorsInvolved.ToArray());
foreach (var o in orders)
yield return CheckSameOrder(o.Order, o.Trait.IssueOrder(o.Actor, o.Order, o.Target, mi.Modifiers.HasModifier(Modifiers.Shift)));