Issue Information
-
#000475
-
3 - Medium
-
Fixed
Issue Confirmations
-
Yes (0)No (0)
0
Originally posted by theultramage
http://www.eathena.w...ker&showbug=475
After getting inspired by the Lif's Mental Charge topic, I checked the code/db and did an aegis comparison, and found the following:
1. Lif's Healing Hands should have 2s normal delay instead of a parallel 1s delay + 1s skill block
2. Lif's Mental Charge only has normal delay (that blocks all skills) instead of a homskill block on itself
3. Lif's Urgent Escape has its time1/time2 columns misinterpreted! Time1 is used for the skill duration and time2 for the skill block, but the values in the db are in the reverse order!
Thus I'm declaring the homunculus skill_cast_db values as invalid and requesting a db/code audit.
One possible outcome could be that merc_hom_block_skill() usage gets identified as completely custom and that all skills only have a simple delay.
PS: this should be done as soon as possible, as currently Lif's Urgent Escape has 0ms status duration (side-effect of [3]).
This post has been edited by theultramage: Nov 22 2007, 12:46 PM
http://www.eathena.w...ker&showbug=475
After getting inspired by the Lif's Mental Charge topic, I checked the code/db and did an aegis comparison, and found the following:
1. Lif's Healing Hands should have 2s normal delay instead of a parallel 1s delay + 1s skill block
2. Lif's Mental Charge only has normal delay (that blocks all skills) instead of a homskill block on itself
3. Lif's Urgent Escape has its time1/time2 columns misinterpreted! Time1 is used for the skill duration and time2 for the skill block, but the values in the db are in the reverse order!
Thus I'm declaring the homunculus skill_cast_db values as invalid and requesting a db/code audit.
One possible outcome could be that merc_hom_block_skill() usage gets identified as completely custom and that all skills only have a simple delay.
PS: this should be done as soon as possible, as currently Lif's Urgent Escape has 0ms status duration (side-effect of [3]).
This post has been edited by theultramage: Nov 22 2007, 12:46 PM
