[PATCH] Celt tower has a misleading unit in it

[fabio]

Celt tower is the only tower showing a unit inside it, this may misleads to think it is garrisoned by a unit. Sample image:

[Stan]

What do you suggest ? Removing it & adding the ability to garrison one ?

[fabio]

Probably the best choice is to use a model without any support for inner unit, there is already another celt tower that has the top covered. This way all towers are consistent in their appearing leading to no possible misleadings. This is especially worth now that walls have the ability to garrison units that appears in a similar way as in this celt tower.

[Stan]

I'll ask Enrique about it though I think we should use the engine as much as possible.

[Enrique]

I agree with Fabio here. Now with units showing when garrisoned on walls, this could lead to confusion easily. I'll keep the tower version with the unit when garrisoned and leave it empty when ungarrisoned.

[enrique]

In 15876:

Prevents gaul defense_tower_a from showing a unit when not garrisoned. Fixes #2876

[Stan]

As a fighting unit or a prop ? 

[fabio]

Thanks for the udpate, I would prefer anyway to disable the inner unit for a copule reasons:

• it may still be misleading since it's not consistent with all the other towers, including the other celt version, which doesn't have this behaviour;
• the shown soldier is always the same and doesn't reflect the unit in it, that can also be a woman.

[Jaison]

@enrique: Work you currently on this defect?

[Stan]

The first fix remove the unit.

The second fix attached add a visible garisonning for the tower like this ​http://i.imgur.com/GDFQyhP.jpg

[fabio]

I think the best approach here is either to remove this tower variant, or modify the variant closing the empty room on the top. This way it will be consistent with all other towers. A quick fix would just to leave the room always empty even when garrisoned units are inside.

[Stan]

Well 1. needs Enrique approval, and you'll need to ask him.

2. Is no unit patch.

[fabio]

Reported also on forum here: http://www.wildfiregames.com/forum/index.php?showtopic=19557

[Itms]

I agree with fabio, we need to remove entirely this unit in the tower. Also no need to have Enrique's approval IMO because it is a gameplay decision, not an artistic one.

However, the current fix_no_unit patch only removes the unit and doesn't remove the useless attachpoint from the tower model. Stan, could you take care of that and propose an updated patch? Thanks for working on it. :-)

[Stan]

Fix the garrisonning point in the dae.

[Itms]

In 16268:

Remove entirely the unit in the Celt outpost. Also clean out the dae model to remove the associated prop point.

Fix by stanislas69, fixes #2876.

[Itms]

Thanks for taking care of the issue!