Visible Garrisoning in animated gate bug

[Stan]

It would be nice to to have animated meshes like boats with units on them, without losing the animation.

This currently works with siege towers however it doesn't work well with things like gates.

To reproduce this download the terra magna mod and play around garrisoning the Chinese gate. If it's not locked and you garrison units inside it the gate will stay open. If it's locked, it'll stay closed, and correctly switch to the right anim when opening then it won't close anymore.

Another story happens with garrison flags. If the gate is garrisoned the flags will appear. They'll have their Idle animation set. What will happen is if you try to move units near the gate the same bugs as before will occur and also the flags will freeze as they will be set to an anim they do not have, such as gate_opening, gate_closing, gate_closed, gate_opened. They ought to play Idle in this case.

[sanderd17]

In 18266:

Allow simulation to set the actor variant selection, and use it for the garrison flag. Fixes #3952 Refs #2679, #3983

[temple]

​Phab:D1418 for correct opening and closing of garrisoned gates.

[Imarok]

Move tickets to Simulation as UI & Simulation got some sub components.

[wraitii]

I think this also has a visual component: if gates receive contradictory orders on the same turn (ie. they start closed, then open, then close again on the same turn) they'll 'reset' their animation.

I don't really see a very good fix for that except if only actually open/close in "Update-End" to sorta make sure that all messages have gone through.

[wraitii]

In 23710:

Enable garrisoning on gates / fix movement of units with visible garrison points

This addresses two related issues:

• Units visibly garrisoned on gates keep the gate open.
• Units visibly garrisoned on entities keep their pathfinding blocker flags.

Remove the block-movement flag from visibly garrisoned entities.
Keep a list of entities that cannot move and thus should not count towards gate-opening logic.

Packing logic is kept separate: it is more related to entities having 'alternate forms' with different capabilities than being currently incapable of moving.

Based on work by temple

Comments by: Freagarach

Tested by: Nescio

Fixes #2679
Fixes #5151

Differential Revision: ​https://code.wildfiregames.com/D1418

[wraitii]

Changes reverted, reopened at D2775

[wraitii]

In 23731:

Enable garrisoning on gates / fix movement of units with visible garrison points (2nd commit)

This is the same commit as rP23710 / D1418, fixing noted issues.

This addresses two related issues:

• Units visibly garrisoned on gates keep the gate open.
• Units visibly garrisoned on entities keep their pathfinding blocker flags.

De-activate the obstruction of visibly-garrisoned entities, fixing the 2nd issue.
Keep a list of entities that cannot move and thus should not count towards gate-opening logic.

Packing logic is kept separate: it is more related to entities having 'alternate forms' with different capabilities than being currently incapable of moving.

Based on work by temple

Fixes #2679.
Fixes #5151.

Differential Revision: ​https://code.wildfiregames.com/D2775

[Stan]

Should we open another ticket for gates resetting each time you have a new unit in them?

[wraitii]

I think so, it's rather unrelated. It happens because the garrisonHolder sets the garrisoned flag, which sets the animation, which then reruns from scratch.