Opened 2 years ago

Closed 2 years ago

#6399 closed enhancement (fixed)

Move possible formations from Identity to UnitAI.

Reported by: Freagarach Owned by: Freagarach
Priority: Nice to Have Milestone: Alpha 26
Component: Simulation Keywords:
Cc: Patch: ​Phab:D4443

Description (last modified by Freagarach)

A formation member needs UnitAI, so it would be best to have the possible formation stored there, instead of in cmpIdentity.

Reason: Keep data close to where it is used.

Change History (4)

comment:1 by Freagarach, 2 years ago

Summary: Move possible formation from Identity to UnitAI. → Move possible formations from Identity to UnitAI.

comment:2 by Freagarach, 2 years ago

Description: modified (diff)

comment:3 by Freagarach, 2 years ago

Keywords: simple removed
Owner: set to Freagarach
Patch: → Phab:D4443

comment:4 by Freagarach, 2 years ago

Resolution: → fixed
Status: new → closed

In [26235]:

Move the allowed formations from cmpIdentity to cmpUnitAI.

One needs UnitAI to be able to use a formation, so include the spec therein (to keep information close to where it is used). Modders can change their templates with the following script: ​​https://code.wildfiregames.com/P261.

Differential revision: ​​https://code.wildfiregames.com/D4443
Comments by: @Stan
Idea accepted by: @wraitii
Fixes: #6399

(In the future, one might want to consider having a separate formationmember component, especially when BuildingAI and UnitAI are combined #5889.)

Note: See TracTickets for help on using tickets.