Opened 10 years ago
Last modified 3 years ago
#2907 closed defect
VisualActor changes are not applied for building placement previews, corpses and fogged entities — at Initial Version
| Reported by: | leper | Owned by: | |
|---|---|---|---|
| Priority: | Nice to Have | Milestone: | Alpha 24 |
| Component: | Core engine | Keywords: | simple |
| Cc: | Patch: |
Description
As reported on the forums the actor change technology does not apply to the Fogging, Health (corpse), and placementPreview cases.
corpse| and preview| templates both have the Identity component, but we don't apply technologies to them (see SendGlobalMessage() (note that the comment should read local instead of non-local). The reasoning why we don't do that makes sense, and we need a different way for Fogging anyways, so using the same way for all of those cases makes sense.
We should add a function to set the actor to cmpVisualActor and use that to set the correct one in all three cases. The preview case should use cmpTechnologyManager's ApplyModificationsTemplate() to obtain the correct actor. I'm not sure if it is worth extra logic to handle the case where an actor changing technology is researched while the user is placing a building.
