VisualActor changes are not applied for building placement previews, corpses and fogged entities
— at Version 4
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.
Change History
(4)
Milestone: |
Alpha 18 → Alpha 19
|
Milestone: |
Alpha 19 → Backlog
|
Description: |
modified (diff)
|
Milestone: |
Backlog → Work In Progress
|
Patch: |
→ Phab:D576
|
Replying to leper:
To the extent that appearance indicates something important about the building being previewed, I think it is worth it.