Resist Pierce Archery

A place to submit .patch fixes for the DOL SVN

Moderator: Developer Team

Resist Pierce Archery

Postby asterick » Mon Jun 18, 2012 12:27 am

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
Last edited by asterick on Mon Jun 18, 2012 4:56 am, edited 2 times in total.
asterick
DOL Visitor
 
Posts: 13
Joined: Fri May 25, 2012 3:40 pm

Re: Resist Pierce Archery

Postby Tolakram » Mon Jun 18, 2012 1:07 am

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?
- Mark
User avatar
Tolakram
Storm / Storm-D2 Admin
 
Posts: 9189
Joined: Tue Jun 13, 2006 1:49 am
Location: Kentucky, USA

Re: Resist Pierce Archery

Postby asterick » Mon Jun 18, 2012 4:56 am

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.
asterick
DOL Visitor
 
Posts: 13
Joined: Fri May 25, 2012 3:40 pm


Return to “%s” DOL Code Contributions

Who is online

Users browsing this forum: No registered users and 0 guests