Page 1 of 1

Resist Pierce Archery

PostPosted: Mon Jun 18, 2012 12:27 am
by asterick
Added effect to Archery.
I've tested this. Damage seems pretty close to correct. In any case resist pierce effect is now working for Archery.
SpellHandler.cs.patch
updated
(1.55 KiB) Downloaded 22 times

Re: Resist Pierce Archery

PostPosted: Mon Jun 18, 2012 1:07 am
by Tolakram
That does not do what you intended to do. :)

if (resiPierce > 0 || resiPierce > 0 && ad.AttackType == AttackData.eAttackType.Spell && Spell.SpellType == "Archery")

Not grouped correctly, since it mixes AND and OR, but regardless this will always be true if resiPierce > 0

What you need to do here is discover when you don't want to execute this code and then make logic for it. Perhaps it should always happen?

Re: Resist Pierce Archery

PostPosted: Mon Jun 18, 2012 4:56 am
by asterick
As far as I can recall Spell Pierce effected all magic damage in forms of spells/proc/charges..so I reverted this back and tested simply as:
Code: Select all
if (resiPierce > 0)
I got the same results as before...increased damage, so it worked as Geshi had pointed out in the other thread, not sure what had happened last time for me but it was a late night, I probably had missed something and never realized it.