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 Version 4

Reported by: leper Owned by:
Priority: Nice to Have Milestone: Alpha 24
Component: Core engine Keywords: simple
Cc: Patch: Phab:D576

Description (last modified by Sandarac)

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)

in reply to:  description comment:1 by historic_bruno, 9 years ago

Replying to leper:

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.

To the extent that appearance indicates something important about the building being previewed, I think it is worth it.

comment:2 by leper, 9 years ago

Milestone: Alpha 18Alpha 19

comment:3 by Itms, 9 years ago

Milestone: Alpha 19Backlog

comment:4 by Sandarac, 7 years ago

Description: modified (diff)
Milestone: BacklogWork In Progress
Patch: Phab:D576
Note: See TracTickets for help on using tickets.