keeperfx icon indicating copy to clipboard operation
keeperfx copied to clipboard

New HitTypes to hit everything but the caster creature

Open ErminioLucente opened this issue 10 months ago • 3 comments

; Controls which things will be affected by the AreaDamage of the spell (1..8). ;1 - Affect all creatures and all objects. ;2 - Affect only creatures. ;3 - Affect not own creatures and objects. ;4 - Affect not own creatures. ;5 - Affect not own creatures which are not protected by Armour spell. ;6 - Affect all things. ;7 - Affect only dungeon hearts. ;8 - Affect only not own dungeon hearts. 9 - THit_CrtrsNObjctsNShot, // Affect all creatures and all objects, also allow colliding with other shots 10- THit_TrapsAll, // Affect all traps, not just the ones that are destructable

  • [ ] ;11 - Affect all things, except caster.
  • [ ] 12 ; Affect all creatures, except caster & objects
  • [ ] ;13 - Affect all creatures which are not protected by Armour spell., except caster & objects

HitType = 0

I currently have a slam attack from a big creature, that deals damage to everything, but sadly the caster receives damage too. So im request 3 additional hit types with altered properties. Previous spoke about here : https://discord.com/channels/480505152806191114/1083906845502091326/1230205774748450828

ErminioLucente avatar Apr 18 '24 13:04 ErminioLucente

relevant: https://github.com/dkfans/keeperfx/blob/master/src/thing_effects.h#L33

currently 9 and 10 are already used, but are not documented in the config

hit type 9 is:

THit_CrtrsNObjctsNShot, // Affect all creatures and all objects, also allow colliding with other shots

which is for Boulder, then there is hit type 10:

THit_TrapsAll, // Affect all traps, not just the ones that are destructable

walt253 avatar Apr 18 '24 13:04 walt253

https://github.com/dkfans/keeperfx/blob/master/src/thing_list.c#L3303

walt253 avatar Apr 18 '24 13:04 walt253

//TODO replace HitType with these ... enum HitTargetFlagsList {

I see mefisto intended to directly use the HitTargetFlagsList instead of HitType, could do it like that, or make the HitType to HitTargetFlagsList list configurable, idk wich of the 2 would make more sense

I suppose the later would be best for backwards compatibility

the specific ones requested by ermin don't have related HitTargetFlagsList, so would also need extra of those to work

PieterVdc avatar Apr 19 '24 07:04 PieterVdc