Comment 21 for bug 894884

Revision history for this message
Graion Dilach (graiondilach) wrote :

Thanks!

Temporals, Armory, Slave Miner Crushing and Threshold should been retested with Bounty.Pillage=no. AFAIK, Hospital and Armory kills their units before respawning and SM always kills the Slave when it gets in. I've only added checks against these three. Crushes and Threshold should work without Pillage set.

DeathWeapons: OK, so it does Bounty. I can't decide if it should or shouldn't. That's my problem.

Drainweapon and Temporal doesn't cause a damage, so that's not surprising if that doesn't work with pillagers. But I see, Temporal should give bounty at deleting even if the unit is a pillager.

Animation damage doesn't have an owner tbh, so I don't think I can fix that. It goes to the other animation warhead problems.

OK, then todo list:
Add Marshall's proposal
Fix Spawned units' bounty strings Only MissileSpawn is checked, spawn planes will show
Add a check to Sonic
Add a check to ROF
Crushing should give bounty to pillagers
Negative damage shouldn't generate Pillage. (this one is arguable, we can speak about non-free unit repairs with this)