FAForever / fa

Lua code for FAF

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

External factory can be deattached by deploying

Garanas opened this issue · comments

Describe the bug

When you issue a deploy order with the Fatboy and/or the Atlantis the external factory (and any unit that is being built at that moment) is detached from the primary unit.

To Reproduce

Spawn in a Fatboy and/or an Atlantis, build a unit and issue the deploy order immediately after.

Expected behavior

For things not to detach 😞 !

Screenshots

image

Additional context

Initially the feature had an entity in between the primary unit and the external factory. I can not entirely recall why we removed that entity, but I'm sure that re-introducing the entity will fix the bug as the entity will not be 'deployed' like other units.

@clyfordv you worked on the mobile factories, can you recall the reason we removed the entity?

commented

Yes, it was to make it so the external factories could only be selected with the button (and not from the worldview). Just a smoke and mirrors thing, so definitely contradicted in this case.

EDIT: Appears to only be the case with the Fatboy, which is strange (also don't need to build a unit, just deploying detached it for me). But what's also strange is that the Fatboy is the only exFacUnit I can select the exFac from in the worldview--carriers, CZAR etc. are button only.

You can select those too but the selection box of the primary units are too big and therefore always overlap the selection box of the external factory.

Initially the external factory was always at the 'back' of the unit just to make it easier to select. This was traded in favor to the button you worked on and now the external factories are where the 'factory' is visually in the mesh.

commented

It's the "CARRIER" category--adding it to the Fatboy both prevents the exFac from being selected in the worldview, and prevents it from detaching (no idea how or why).

commented

Took a swing at it: #6093