[Patch][6860] Bloodberry - spell_elixir data

[Backbone]

What bug does the patch fix? What features does the patch add? Provide correct stacking data for spell 45373

For which SubVersion revision was the patch created? 6817

Is there a thread in the bug report section? If yes, please add a link to the thread. no thread

Who has been writing this patch? Please include either forum user names or email addresses. Malah

INSERT INTO `spell_elixir` SET `mask`=1, `entry`=45373;

[Sephiroth1983]

Only works on the Sunwell Plateau.

When we take care of this important thing instead?

[Backbone]

When we take care of this important thing instead?

--- Spell.cpp
+++ Spell.cpp
@@ -3327,7 +3327,11 @@
    // zone check
    if(!IsSpellAllowedInLocation(m_spellInfo,m_caster->GetMapId(),m_caster->GetZoneId(),m_caster->GetAreaId()))
        return SPELL_FAILED_REQUIRES_AREA;
+
+   // bloodberry elixir
+   if( (m_spellInfo->Id==45373) && zone_id != 4075  )
+       return SPELL_FAILED_REQUIRES_AREA;
+    
    // not let players cast spells at mount (and let do it to creatures)
    if( m_caster->IsMounted() && m_caster->GetTypeId()==TYPEID_PLAYER && !m_IsTriggeredSpell &&
        !IsPassiveSpell(m_spellInfo->Id) && !(m_spellInfo->Attributes & SPELL_ATTR_CASTABLE_WHILE_MOUNTED) )

[DasBlub]

why you don't do your check in IsSpellAllowedInLocation ?

this would be better, i think, because then all functions which need such a check breaks at this function.

so long

[Patman128]

Yes, he is right. Add the check in IsSpellAllowedInLocation().

[Backbone]

it is now correct?

--- SpellMgr.cpp
+++ SpellMgr.cpp
@@ -2059,9 +2059,8 @@
            }
            if(mask & ELIXIR_SHATTRATH_MASK)
            {
-                // in Tempest Keep, Serpentshrine Cavern, Caverns of Time: Mount Hyjal, Black Temple
-                // TODO: and the Sunwell Plateau
-                if(zone_id ==3607 || map_id==534 || map_id==564)
+               // in Tempest Keep, Serpentshrine Cavern, Caverns of Time: Mount Hyjal, Black Temple, Sunwell Plateau
+               if(zone_id ==3607 || map_id==534 || map_id==564 || zone_id==4075)
                    return true;

                MapEntry const* mapEntry = sMapStore.LookupEntry(map_id);
@@ -2098,6 +2097,15 @@

            return mapEntry->multimap_id==207;
        }
+       // Bloodberry Elixir
+        case 45373:
+        {
+            MapEntry const* mapEntry = sMapStore.LookupEntry(zone_id);
+            if(!mapEntry)
+                return false;
+
+            return mapEntry->zone_id==4075;
+        }
        // Dragonmaw Illusion
        case 40216:
        case 42016:
@@ -2106,6 +2114,7 @@
                return false;
            break;
        }
    }

    return true;

[Backbone]

someone from dev`s can take a look?

[VladimirMangos]

+ return mapEntry->zone_id==4075;

? This is will not build.

Code part of chnages will not work. Spell as many elexires have

22[/color][/size]22[/color][/size]22[/color][/size]22[/color][/size]2

And then added code just will not called.

I rewrite C++ part.

Patch added in [6860]. Thank you [/color][/size][/color][/size]2