Issue information

Issue ID
#6054
Status
Fixed
Severity
Fair
Started
Hercules Elf Bot
Jun 18, 2012 15:41
Last Post
Hercules Elf Bot
Jun 27, 2012 6:49
Confirmation
N/A

Hercules Elf Bot - Jun 18, 2012 15:41

Originally posted by [b]Vali[/b]
The turn undead damage formula is not working correctly. From lvl 1 to lvl 10 is doing always the same damage, and this damage is HUGE ( +128k ). But sometimes it does a extremely low damage (350).

Vali~

Hercules Elf Bot - Jun 18, 2012 16:31

Originally posted by [b]malufett[/b]
the 128k is the failure damage where it is not suppose to be...

can you help me look for the renewal formula for this one...thanks...so I can fix it asap....

Hercules Elf Bot - Jun 18, 2012 17:56

Originally posted by [b]Vali[/b]
I'm going to research for it :)

Hercules Elf Bot - Jun 18, 2012 19:32

Originally posted by [b]Vali[/b]
The skill changed recently in this bug report: http://rathena.org/board/tracker/issue-5302-renewal-skill-changes-are-not-applied-for-some-skills/

The only info there is that the next formula is the correct one:
[quote][color=#008000]Turn Undead - Confirmed[/color][list]
[*][color=#008000]Success rate formula has changed. [rate = 10*skill_lv + sstatus->luk + sstatus->int_ + s_level + 300 - 300*tstatus->hp/tstatus->max_hp;][/color]
[*][color=#008000]On failure, the magic damage dealt is based on your magic attack power. [ad.damage = skill_lv * (sstatus->matk_min+sstatus->matk_max);][/color]
[/list]
[/quote]

Maybe Xantara has more info about this.

Vali~

Hercules Elf Bot - Jun 18, 2012 19:41

Originally posted by [b]MarkZD[/b]
It's because this "*" should be "+":

[quote]
[left]ad.damage = status_get_lv(src)[b][size=5] * [/size][/b](sstatus->matk_min + sstatus->matk_max);[/left]
[/quote]

or the formula should be:

[quote]
[left]ad.damage = [size=5]skill_lv[/size] [size=4]* [/size](sstatus->matk_min + sstatus->matk_max);[/left]
[/quote]
As quoted in the report from skill changes.

But, it seems the second formula is the right, Xantara just wrote wrongly [left]status_get_lv(src) where it should be skill_lv.[/left]

This post has been edited by MarkZD on Jun 18, 2012 20:20

Hercules Elf Bot - Jun 18, 2012 21:26

Originally posted by [b]Xantara[/b]
[b]Fixed [/b][rev=16317]

Hercules Elf Bot - Jun 19, 2012 6:35

Originally posted by [b]malufett[/b]
is that formula official? anyways I'll try to confirm in official servers...