[Fix][9018] Faerie Fire (Feral)

[Auntie Mangos]

no problem, nothing personally, btw this guy http://getmangos.eu/community/member.php?u=3717 has those patches on his signature

[Auntie Mangos]

no problem, nothing personally, btw this guy http://getmangos.eu/community/member.php?u=3717 has those patches on his signature

Suggested in thread patch is more complete. Also do you think that devs must look by signatures links for patches... at least while unreviwed section non empty.

Ok, i find thread: http://getmangos.eu/community/viewtopic.php?id=9909&highlight=Faerie+Fire

Base at time and aditional data i will make commit author Sarjuuk with special comment about author of prepered sql update.

I sure that both patch versions created independently

[MrLama]

What bug does the patch fix? What features does the patch add?

Adds Faerie Fire (Feral) damage, when in bear form.

For which repository revision was the patch created?

8994

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

---

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

Me

DB part

DELETE FROM spell_bonus_data WHERE entry = 60089;
INSERT INTO spell_bonus_data VALUES
(60089,0,0,0.05,"Druid - Faerie Fire (Feral)");

Core part

diff --git a/src/game/Spell.cpp b/src/game/Spell.cpp
index f7ce356..1b0aadf 100644
--- a/src/game/Spell.cpp
+++ b/src/game/Spell.cpp
@@ -2522,6 +2522,14 @@ void Spell::cast(bool skipCheck)
            }
            break;
        }
+		case SPELLFAMILY_DRUID:
+		{
+			// Faerie Fire (Feral)
+			if( m_spellInfo->Id == 16857 && m_caster->m_form != FORM_CAT )
+				AddTriggeredSpell(60089);
+
+			break;
+		}
        case SPELLFAMILY_ROGUE:
            // Fan of Knives (main hand)
            if (m_spellInfo->Id == 51723 && m_caster->GetTypeId() == TYPEID_PLAYER &&

[pelle]

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

Me

are you sure? i saw the same patch here http://spezies0815.datearea.de/ and it's not your....!

[MrLama]

are you sure? i saw the same patch here http://spezies0815.datearea.de/ and it's not your....!

I was writing this path myself. If there if some similiarity to other patch it's purely coincidental, because I wasn't aware that other patch exists up to this moment

[MrLama]

I got a message from a player with a request to fix this so I did it without looking for patch. Often it is the fastest way Personaly, I don't care which name patch would carry. I just shared my idea hoping somebody finds it useful

[VladimirMangos]

In [9018] Thank you