Core/Mechanics: Fix prevent fleeing + fear

Closes #9754

Players should never have been affected by SPELL_AURA_PREVENTS_FLEEING in the first place.
Curse of Recklessness will no longer prevent Fear effects, only prevent NPCs from fleeing (e.g. at low health).
Judgement of Justice will no longer prevent Fear effects, but will only prevent NPCs from fleeing (e.g. at low health).
Source: http://www.wowwiki.com/Patch_3.0.2
Core/Achievements: Fixed wrong packet generation in AchievementMgr::SendCriteriaUpdate
Merge pull request #13214 from mik1893/achievement

Core/Achievements: fix old typo in AchievementMgr::SendCriteriaUpdate
Scripting/Isle of Queldanas: Moved Converted Sentry to SAI.
Closes #13121
DB/Brewfest: Fix "Ram Master Ray" gossip
Scripts/Icecrown Citadel

* Fixed Deathbringer Saurfang faction
* Fixed Coldflame faction
Core/Misc: Refactor scripts to fix static analysis warnings

Eighth batch of fixes targeting 80 issues reported by Coverity
Scripts/Misc: Make InstanceScript::Initialize() obsolete

Move all InstanceScript initialization code from Initialize() to the constructor.
InstanceScript::Initialize() is now obsolete, please don't use it anymore. The only reason it's still in the sources is for backward compatibility.
DB/Quest: Tirion's Gambit

Script for chosen zealots is not perfect as this should surround player/tirion in a circle but cant think of anyway other than every chosen zealot having its own wp and sai.

Closes #10474
Scripts/UtigardePinnacle: Prevent Ymiron from triggering a phase transition at death

...caused by the health percent multiplers used to check transitions
DB/Creature: Delete incorrectly spawned Felhunter
Scripts/ManaTombs: Nexus-Prince Shaffar rewritten to BossAI/EventMap
Core/Debugging: Extended ASSERT macro to allow passing additional formatted string
Scripts: Fixed compiler warning
DB/Quest: Fix LimitTime for Quest "Bark for Drohn's Distillery!" / "Bark for T'chali's Voodoo Brewery!" / "Bark for the Barleybrews!" / "Bark for the Thunderbrews!"
Core/Spells: Always add threat for spells from spell_threat table, not only if target was hit. Unsuccessful cast results in 0 threat added instead of the full amount and creating threat list entry for caster. This fixes various exploits with taunt spells bugging out scripts due to taunt immunity/passive state (creatures would immediately evade resetting script state if no entries were present on threat list - entry with no threat is not the same as no entry)

Closes #11883
DB/Brewfest: Fix quest start for "Bark for T'chali's Voodoo Brewery!" / "Bark for Drohn's Distillery!"
To/Do: Fix quest kill credit
Core/Misc: Refactor scripts to fix static analysis warnings

Last issue to be fixed, fixes #12960
Misc/Brewfest: Fix Quest "11407 - Bark for Drohn''s Distillery!" / "11408 -Bark for T''chali''s Voodoo Brewery!"
DB/Gameobject: Fix wrong "Brewfest Banner" position in durotar