Opened 10 years ago

Closed 10 years ago

Last modified 3 years ago

#1600 closed defect (fixed)

Handle 'remain' corpses as discrete entities

Reported by: Deiz Owned by: Deiz
Priority: Should Have Milestone: Alpha 11
Component: UI & Simulation Keywords:
Cc: Patch:

Description (last modified by elexis)

Per IRC discussion, the current way 'remain' corpses are handled presents several issues. Currently this behaviour is only used for huntable animals.

When units with the 'remain' property in cmpHealth die, they simply play the death animation but continue to exist. UnitAI makes this work by overriding a number of movement-related functions, so that corpses don't start moving around.

Absent an ownership change to Gaia, players that own these animals will have LOS from the corpses until the resources are harvested, and enemy buildings will fire on the corpses until they're destroyed.

Ideally, the promotion logic should be genericized and implemented inside each component, so that unit promotion, gate conversion, and units dying and becoming resources all share common code.

Change History (4)

comment:1 by Deiz, 10 years ago

In 12465:

Set ownership to Gaia when units that remain on death are killed. Refs #1600.

comment:2 by Deiz, 10 years ago

Owner: set to Deiz
Resolution: fixed
Status: newclosed

In 12486:

Destroy dying 'remain' entities and replace them with visually-identical resources. Fixes #1600.

comment:3 by historic_bruno, 10 years ago

Milestone: Alpha 12Alpha 11

comment:4 by elexis, 3 years ago

Description: modified (diff)

irc discussion on 2012-08-14, 2012-08-18, 2012-08-19, 2012-08-20, perhaps also 2012-08-27, 2012-09-05 (search term Deiz and corpse) on http://irclogs.wildfiregames.com/

refs Phab:rP23601 / Phab:D2656

Note: See TracTickets for help on using tickets.