Issue information

Issue ID
#5745
Status
Fixed
Severity
Low
Started
Hercules Elf Bot
May 14, 2012 16:04
Last Post
malufett
Jan 21, 2013 12:10
Confirmation
Yes (5)
No (0)

Hercules Elf Bot - May 14, 2012 16:04

Originally posted by [b]Judas[/b]
Is Recognized Spell implemented in rA yet? It doesn't seem to change damage

Hercules Elf Bot - May 14, 2012 16:34

Originally posted by [b]malufett[/b]
oh..yeah its not yet fully implemented..only the sp consumption part...

Hercules Elf Bot - Nov 7, 2012 15:15

Originally posted by [b]Aleos[/b]
Is there any information on how the skill is supposed to work? The description seems kinda vague.

Hercules Elf Bot - Nov 7, 2012 16:53

Originally posted by [b]unknown267[/b]
The skill description says it makes all spells do max damage, so it's like a magical Maximize Power. Don't see how that's vague O.o

Hercules Elf Bot - Nov 7, 2012 19:00

Originally posted by [b]Aleos[/b]
Hm. Ok so it's setting the min = max. Ok now I understand it, I totally forgot Maximize Power acted the same way. I was thinking of it in a different method which is what was confusing me.

Hercules Elf Bot - Nov 12, 2012 0:01

Originally posted by [b]esu1214[/b]
find
[code]
#ifdef RENEWAL //Renewal MATK Appliance according to doddler (?title=Renewal_Changes#Upgrade_MATK)
/**
* min: (weaponMATK+upgradeMATK) * 2 + 1.5 * statusMATK
* max: [weaponMATK+upgradeMATK+(wMatk*wLvl)/10] * 2 + 1.5 * statusMATK
* yes this formula MATCHES their site: matk_max already holds weaponmatk+upgradematk, and
* -> statusMATK holds the %Matk modifier stuff from earlier and lastly:
* -> the mdef part is not applied at this point, but later.
**/ //1:bugreport:5101 //1:bugreport:5101
MATK_ADD((1+sstatus->matk_max) * 2 + 15/10 * sstatus->matk_min + rnd()% ( sstatus->matk_max + (1 + (sstatus->matk_max*sstatus->wlv) / 10 * 2 + 10/15 * sstatus->matk_min ) ));
#else //Ancient MATK Appliance
if (sstatus->matk_max > sstatus->matk_min) {
MATK_ADD(sstatus->matk_min+rnd()%(1+sstatus->matk_max-sstatus->matk_min));
} else {
MATK_ADD(sstatus->matk_min);
}
#endif
[/code]
replace
[code]
MATK_ADD((1+sstatus->matk_max) * 2 + 15/10 * sstatus->matk_min + rnd()% ( sstatus->matk_max + (1 + (sstatus->matk_max*sstatus->wlv) / 10 * 2 + 10/15 * sstatus->matk_min ) ));
[/code]
to
[code]
if(sc && sc->data[SC_RECOGNIZEDSPELL])
MATK_ADD(sstatus->matk_max);
else
MATK_ADD((1+sstatus->matk_max) * 2 + 15/10 * sstatus->matk_min + rnd()% ( sstatus->matk_max + (1 + (sstatus->matk_max*sstatus->wlv) / 10 * 2 + 10/15 * sstatus->matk_min ) ));
[/code]

Hercules Elf Bot - Nov 12, 2012 1:47

Originally posted by [b]MarkZD[/b]
esu1214, your formula is VERY wrong, as any value you use in your formula with RECOGNIZED SPELL will be lower than without it.

Recognized Spell should add the MAX damage.
You just have to remove rnd()%, so it'll be the max damage.

[CODE]
if(sc && sc->data[SC_RECOGNIZEDSPELL])
MATK_ADD((1+sstatus->matk_max) * 2 + 15/10 * sstatus->matk_min + ( sstatus->matk_max + (1 + (sstatus->matk_max*sstatus->wlv) / 10 * 2 + 10/15 * sstatus->matk_min ) ));
else
MATK_ADD((1+sstatus->matk_max) * 2 + 15/10 * sstatus->matk_min + rnd()% ( sstatus->matk_max + (1 + (sstatus->matk_max*sstatus->wlv) / 10 * 2 + 10/15 * sstatus->matk_min ) ));
[/CODE]

This post has been edited by MarkZD on Nov 12, 2012 1:49

Hercules Elf Bot - Nov 12, 2012 5:22

Originally posted by [b]malufett[/b]
ok..I'm with it..

:meow:

Hercules Elf Bot - Nov 29, 2012 15:58

Originally posted by [b]malufett[/b]
Fixed @ [rev='16981']

:meow: