Fix an issue added in 024b57bb74 that didn't properly skip ReputationSpillover rows with invalid data.
Fix also a static analysis issue reported by Coverity.
(cherry picked from commit 95b1204798d3ab2a141f81b294b064255c335b8a)
Conflicts:
src/server/game/Globals/ObjectMgr.cpp
Fix debug assert thrown by G3D caused by models with empty bounding box being exported by vmap4assembler and imported by worldserver. This change will skip those models, proper investigation on the issue of why the bounding box is empty is still required.
(cherry picked from commit a7ca78b2fe12c68e561782416bf5726410a3c01b)
Script http://www.wowhead.com/npc=18230/trayanise from sniff
Only the Tiny Arcane Construct appears to have fixed WP ie same waypoints repeat in sniff over and over while trayanise waypoints are different each time,
Also adds the texts to this npc
mesh_extractor is not ready for production and it will be developed
in a different branch
(cherry picked from commit a87442be68da4e2ea51a10e3bd8a07ef7b9765b2)
Conflicts:
src/tools/mesh_extractor/ContinentBuilder.h
src/tools/mesh_extractor/DBC.cpp
Fix http://www.wowhead.com/quest=12481/adding-injury-to-insult so it works as on retail
* Bjorn Halgurdsson unattackable until the Vrykul Insult is used on him, by setting passive + flags 2 unattackable since unlike using immune to pc this still allows the insult to be cast without getting invalid target
* It is now possible to get Bjorn Halgurdsson to Lydias Ambush without him evading as previously you could only get him as far as the water
* ensuring this resets correctly in event of evade.
* Prevent event from restarting if theres already an instance of either the Lich King, any of the 2 versions of vegard or the first 2 light's vengence objects already spawned
* Prevent further ghouls from being summoned by same npc if there is already ghoul summoned by that npc
Closes#13856
Script working, there are some issues with spell targeting for aod spells used by Lich King and Vegard to summon ghouls but this is fixed by applying the dbc hack by Rushor from original issue to limit spell targets.
Fix AuraScript::Remove() triggering an assert with default parameter and log an error when Aura is removed with invalid mode AURA_REMOVE_NONE instead of triggering an assert deep down in the call tree.
Fix https://github.com/TrinityCore/TrinityCore/issues/714 , triggered by spell 50240 "Evasive Maneuvers" script
Rushor