Move RollOfUnit() to after the "finished building" animation

[lL1l1]

Description of the proposed changes

Fixes #5625. UEF and Cybran air factories have an animation that locks the unit's orientation, interfering with the rolloff rotation. The changes move the rolloff rotation to happen after the animation finishes.

Testing done on the proposed changes

Spawn factories, give a rally point on one side, order an engineer, and then change the rally point to the other side before the engineer finishes building and see that it now gets rotated correctly in the UEF and Cybran factories.

   CreateUnitAtMouse('xab1401', 0,    0.14,   -8.65, -0.00000)
   CreateUnitAtMouse('xsb0302', 0,   -4.04,    2.16,  0.00000)
   CreateUnitAtMouse('ueb0302', 0,   11.96,    2.16,  0.00000)
   CreateUnitAtMouse('urb0302', 0,    3.96,    2.16,  0.00000)
   CreateUnitAtMouse('uab0302', 0,  -12.04,    2.16,  0.00000)

Checklist

• Changes are annotated, including comments where useful
• Changes are documented in the changelog for the next game version

[clyfordv]

This introduces a bug where the a land unit built from a relevant air fac wants to nav back to the roll off point after the rally point. See something similar here where an order is slipped in right at the moment of construction with odd behavior: https://discord.com/channels/197033481883222026/1144605729911738439/1238254471662927943

Stock:

With change:

My read on why this occurs is that the rally point order chain issuing is extra special somewhere in the engine (apropos nothing it would be nice to expose that to lua a little more, to enable more sophisticated roll off behavior). I think if you want to implement this as-is you'll have to manually re-check the unit's orientation and update if necessary.

[lL1l1]

This fixes it but the code style may be sloppy, although I think it's the most efficient solution.