Lesser Fire Storm - damage incorrect?

Cadiz

Area name:
Issue Location:
Quest Name:
Store Name:
NPC Name:
NPC Location:
Server Version:
Screen Shot:
Time: https://www.timeanddate.com/

Issue Description:

spell description:
Deals 1d8 fire damage/level (Reflex halves) to all targets in 10-meter radius around caster

i have used this several times at caster level 8, so should be doing 8d8 damage, i havent done more than 15 yet.
the damage seems bugged, i will test more but it seems too low

Echo

Relevant code here. Maybe someone can see why it doesnt work properly?

            if (!MyResistSpell(OBJECT_SELF, oTarget, fDelay)) {
                nDamage = d8(nCasterLevel);

                if (GetMetaMagicFeat() == METAMAGIC_MAXIMIZE) {
                    nDamage = 8 * nCasterLevel;
                } else if (GetMetaMagicFeat() == METAMAGIC_EMPOWER) {
                    nDamage = nDamage + (nDamage / 2);
                }

                nDamage = GetReflexAdjustedDamage(nDamage / 2, oTarget, GetAdjustedSpellDC(GetSpellId(), oPC, GetSpellSaveDC()), SAVING_THROW_TYPE_FIRE);

                if(nDamage > 0) {
                    effect eFire = EffectDamage(nDamage, DAMAGE_TYPE_FIRE);
                    DelayCommand(fDelay, ApplyEffectToObject(DURATION_TYPE_INSTANT, eFire, oTarget));
                    DelayCommand(fDelay, ApplyEffectToObject(DURATION_TYPE_INSTANT, eVis, oTarget));

Wulfric

@Echo said in Lesser Fire Storm - damage incorrect?:

Relevant code here. Maybe someone can see why it doesnt work properly?


                nDamage = GetReflexAdjustedDamage(nDamage / 2, oTarget, GetAdjustedSpellDC(GetSpellId(), PC, GetSpellSaveDC()), SAVING_THROW_TYPE_FIRE);

The documentation for GetReflexAdjustedDamage states that the first parameter should be the full damage without adjustment.

https://nwnlexicon.com/index.php/GetReflexAdjustedDamage

Strife and Discord

Correct. That is how Fireball works as well.
Should just be:

nDamage = GetReflexAdjustedDamage(nDamage, oTarget, GetAdjustedSpellDC(GetSpellId(), oPC, GetSpellSaveDC()), SAVING_THROW_TYPE_FIRE);

Strife and Discord

Should be corrected as of V6971.

Puffy

Anyone tested this?

toportime

@Puffy I just tested it and it seems to be working as intended. Here