[7252] [BUG] Lava Lash

Auntie Mangos · January 1, 1970

yes, it is very important thing!!!

there is the problem too:

http://getmangos.eu/community/viewtopic.php?id=6561

:mellow::mellow:

xzbbzx · February 14, 2009

http://www.wowhead.com/?spell=60103

It always says that you need a weapon to be equipped, even though you have equipped one.

DiSlord · February 14, 2009

It always says that you need a weapon to be equipped, even though you have equipped one.

in offhand?

kozelo · February 14, 2009

Works if you have fist weapon equipped in offhand.

xzbbzx · February 14, 2009

in offhand?

Yep, I've got an axe in off hand.

Serker · February 24, 2009

Rev 7324(clean), a shaman with mace in offhand and the same problem...

goodbadguy · February 24, 2009

Somehow only fist weapons are working. Anyone know why?

Rolandmaffia · February 27, 2009

rev: 7294

no changes, and with off hand axe, or with one hand axe in offhand is not works

hikikomori18 · February 27, 2009

I believe its because offhand and ranged auras haven't been implemented yet.

Xlayer · February 27, 2009

Lava Lash is working if you have an explicit Off-hand weapon equipped, but it does not work with One-hand weapons.

hikikomori18 · February 27, 2009

Lava Lash is working if you have an explicit Off-hand weapon equipped, but it does not work with One-hand weapons.

thats very useful data

NIKE · March 16, 2009

Lava Lash is working if you have an explicit Off-hand weapon equipped, but it does not work with One-hand weapons.

This is the way it should be? or One-hand should do the same work?

hikikomori18 · March 16, 2009

This is the way it should be? or One-hand should do the same work?

he's refering to off hand weapons and one hand weapons, if you have an off hand in the off hand slot it works, if you have a one hand weapon in the off hand slot it doesn't work. that's the bug. Its not working on non-explicit offhand weapons, like one handers.

NIKE · March 19, 2009

// Lava Lash
           if (m_spellInfo->SpellFamilyFlags2 & 0x00000004)
           {
               if (m_caster->GetTypeId()!=TYPEID_PLAYER)
                   return;
               Item *item = ((Player*)m_caster)->GetItemByPos(INVENTORY_SLOT_BAG_0, EQUIPMENT_SLOT_OFFHAND);
               if (item)
               {
                   // Damage is increased if your off-hand weapon is enchanted with Flametongue.
                   Unit::AuraList const& auraDummy = m_caster->GetAurasByType(SPELL_AURA_DUMMY);
                   for(Unit::AuraList::const_iterator itr = auraDummy.begin(); itr != auraDummy.end(); ++itr)
                   {
                       if( (*itr)->GetSpellProto()->SpellFamilyName==SPELLFAMILY_SHAMAN &&
                           (*itr)->GetSpellProto()->SpellFamilyFlags & 0x0000000000200000LL &&
                           (*itr)->GetCastItemGUID() == item->GetGUID())
                       {
                          m_damage += m_damage * damage / 100;
                          return;
                       }
                   }
               }
               return;
           }
           break;

maybe use

  
 SpellEntry const *spellInfo = sSpellStore.LookupEntry( triggered_spell_id );
if(spellInfo->AttributesEx3 & SPELL_ATTR_EX3_REQ_OFFHAND)
       {
           Item* item = ((Player*)m_caster)->GetWeaponForAttack(OFF_ATTACK);

           // skip spell if no weapon in slot or broken
           if(!item || item->IsBroken() )
               return;

           // skip spell if weapon not fit to triggered spell
           if(!item->IsFitToSpellRequirements(spellInfo))
               return;
       }

cletusjoe · March 29, 2009

Can anyone confirm whether or not that works?

KJChernov · June 2, 2009

Temporary hack patch:

--- Item.cpp.1    2009-06-02 20:56:46.000000000 +0700
+++ Item.cpp    2009-06-02 20:54:27.000000000 +0700
@@ -745,6 +745,15 @@
      (spellInfo->EquippedItemClass == ITEM_CLASS_WEAPON && IsWeaponVellum())))
        return true;

+    //Lava Lash
+    if (spellInfo->Id==60103) {
+    if (spellInfo->EquippedItemClass==ITEM_CLASS_WEAPON) {
+        return true;
+    } else {
+        return false;
+    }
+    }
+
    if (spellInfo->EquippedItemClass != -1)                 // -1 == any item class
    {
        if(spellInfo->EquippedItemClass != int32(proto->Class))

xzbbzx · June 5, 2009

Temporary hack patch:

--- Item.cpp.1    2009-06-02 20:56:46.000000000 +0700
+++ Item.cpp    2009-06-02 20:54:27.000000000 +0700
@@ -745,6 +745,15 @@
      (spellInfo->EquippedItemClass == ITEM_CLASS_WEAPON && IsWeaponVellum())))
        return true;

+    //Lava Lash
+    if (spellInfo->Id==60103) {
+    if (spellInfo->EquippedItemClass==ITEM_CLASS_WEAPON) {
+        return true;
+    } else {
+        return false;
+    }
+    }
+
    if (spellInfo->EquippedItemClass != -1)                 // -1 == any item class
    {
        if(spellInfo->EquippedItemClass != int32(proto->Class))

Tested and works

nos4r2zod · July 6, 2009

Maybe a better fix would be

@@ -763,11 +763,12 @@ bool Item::IsFitToSpellRequirements(SpellEntry const* spellInfo) const
        }
    }

    if(spellInfo->EquippedItemInventoryTypeMask != 0)       // 0 == any inventory type
    {
-        if((spellInfo->EquippedItemInventoryTypeMask  & (1 << proto->InventoryType)) == 0)
+        if(!((spellInfo->EquippedItemInventoryTypeMask & (1 << proto->InventoryType)) ||
+           (spellInfo->EquippedItemInventoryTypeMask & (1 << 22) && proto->InventoryType == 13)))
            return false;                                   // inventory type not present in mask
    }

    return true;
}

(check one-hand weapons (type = 13) when looking for offhand weapons (type = 22)). But it's still a nuisance because Lava Lash is the only used spell that requires such a check.

megosh · July 26, 2009

Maybe a better fix would be

@@ -763,11 +763,12 @@ bool Item::IsFitToSpellRequirements(SpellEntry const* spellInfo) const
        }
    }

    if(spellInfo->EquippedItemInventoryTypeMask != 0)       // 0 == any inventory type
    {
-        if((spellInfo->EquippedItemInventoryTypeMask  & (1 << proto->InventoryType)) == 0)
+        if(!((spellInfo->EquippedItemInventoryTypeMask & (1 << proto->InventoryType)) ||
+           (spellInfo->EquippedItemInventoryTypeMask & (1 << 22) && proto->InventoryType == 13)))
            return false;                                   // inventory type not present in mask
    }

    return true;
}

(check one-hand weapons (type = 13) when looking for offhand weapons (type = 22)). But it's still a nuisance because Lava Lash is the only used spell that requires such a check.

Anyone know if this fix works?

Xlayer · July 26, 2009

(check one-hand weapons (type = 13) when looking for offhand weapons (type = 22)). But it's still a nuisance because Lava Lash is the only used spell that requires such a check.

Shiv (http://www.wowhead.com/?spell=5938) works the same way; although I don't know how it's currently handled in the core. Maybe it can provide an hint on how to fix Lava Lash.

nos4r2zod · July 26, 2009

Anyone know if this fix works?

Wouldn't have posted it if it didn't do what was advertised

Shiv (http://www.wowhead.com/?spell=5938) works the same way; although I don't know how it's currently handled in the core. Maybe it can provide an hint on how to fix Lava Lash.

Shiv has EquippedItemInventoryTypeMask = 0, so it'll pass over this block completely:

   if(spellInfo->EquippedItemInventoryTypeMask != 0)       // 0 == any inventory type
   {
       if((spellInfo->EquippedItemInventoryTypeMask  & (1 << proto->InventoryType)) == 0)
           return false;                                   // inventory type not present in mask
   }

VladimirMangos · August 17, 2009

Why offhand interpreted as _any_ weapon in offhand? Base at spell data this must be _real_ offhand weapon only. And i not see any references that this not like way.

nos4r2zod · August 17, 2009

Why offhand interpreted as _any_ weapon in offhand? Base at spell data this must be _real_ offhand weapon only. And i not see any references that this not like way.

This is the way it worked prior to patch to 3.0.8. And in 3.0.8 patch notes,

Lava Lash - Fixed a bug which allowed you to use the ability even if you had a shield in offhand. Now requires a 1hand axe, fist or dagger to be able to be used.

no mention that the weapon had to be specifically an off-hand weapon only item.

apendix · August 18, 2009

try test something like this

diff --git a/src/game/Item.cpp b/src/game/Item.cpp
index 52ed24b..e9475e1 100644
--- a/src/game/Item.cpp
+++ b/src/game/Item.cpp
@@ -741,6 +741,10 @@ bool Item::IsFitToSpellRequirements(SpellEntry const* spellInfo) const
{
    ItemPrototype const* proto = GetProto();

+    //Lava Lash
+    if (spellInfo->Id==60103 && spellInfo->EquippedItemClass==ITEM_CLASS_WEAPON)
+         return true;
+
    if (spellInfo->EquippedItemClass != -1)                 // -1 == any item class
    {
        if(spellInfo->EquippedItemClass != int32(proto->Class))

NoFantasy · August 18, 2009

no mention that the weapon had to be specifically an off-hand weapon only item.

Agree, i think we must see the difference between the offhand meele attack and the offhand item inventory slot. Many item slot types (one hand, offhand, etc) can be used for the offhand meele attack.

Sign In

[7252] [BUG] Lava Lash

Recommended Posts

Auntie Mangos

xzbbzx

DiSlord

kozelo

xzbbzx

Serker

goodbadguy

Rolandmaffia

hikikomori18

Xlayer

hikikomori18

NIKE

hikikomori18

NIKE

cletusjoe

KJChernov

xzbbzx

nos4r2zod

megosh

Xlayer

nos4r2zod

VladimirMangos

nos4r2zod

apendix

NoFantasy

Recently Browsing 0 members

Contact Us

Repositories

Support

Wiki

Browse

Activity

My Activity Streams

Important Information