id,summary,reporter,owner,description,type,status,priority,milestone,component,resolution,keywords,cc,phab_field 2907,"VisualActor changes are not applied for building placement previews, corpses and fogged entities",leper,wraitii,"[http://www.wildfiregames.com/forum/index.php?showtopic=18229&p=300144 As reported on the forums] the actor change technology does not apply to the [source:ps/trunk/binaries/data/mods/public/simulation/components/Fogging.js#L24 Fogging], [source:ps/trunk/binaries/data/mods/public/simulation/components/Health.js#L235 Health (corpse)], and [source:ps/trunk/binaries/data/mods/public/simulation/components/GuiInterface.js#L1141 placementPreview] cases. `corpse|` and `preview|` templates both have the Identity component, but we don't apply technologies to them (see [source:ps/trunk/source/simulation2/system/ComponentManager.cpp#L1050 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.",defect,closed,Nice to Have,Alpha 24,Core engine,fixed,simple,,Phab:D576