Core/SAI: Implement a new action type to make a creature (or targets in list) start the closest waypoint available to them
Merge pull request #12990 from Discover-/master

Core/SAI: Implement a new action type to make a creature (or targets in ...
Core/Misc: Refactor scripts to fix static analysis warnings
Core/Tickets: Send global GM system messages on ticket complete
DB/Loot: Increase drop chance for patterns and plans from Trial of Crusader

Ref #13042
Scripts/Blood Furnace: Update Broggok's script to EventMaps
Scripts/Black Temple: Supremus to BossAI and slight changes
Scripts/Black Temple: Updates Najentus to BossAI and slight changes
Scripts/Onyxia Lair: small rework
- fix crash on boss death
- avoid abusing during phase switch
- added better facing through trigger
- corrected some timers
- corrected inhabit type
- cleanup and inheriting some BossAI functionalities instead of custom stuff
Core/Misc:
- fixed an oracle/wolvar reputation change issue
- moved texts of read pronouncement to db

Closes #9105
Scripts/BF: The Maker rewritten to EventMap
DB/Misc: Shadowmoon Valley

Few scripts for shadowmoon valley, 

this is only tip of iceberg as have sniffs of gossip on a lot of tbc npcs which on tc are missing gossip

Closes #12979
Closes #12973
Remove CPP script for Oronok Torn-heart
DB/Creature: Delete spawns of Apexis Guardian

Delete spawns of Apexis Guardian http://www.wowhead.com/npc=22275/apexis-guardian#comments, TC had 12 permament spawns of this NPC when there should be none spawned these are summoned using the large simon clusters at a cost of 35 apexis shards and this is scripted as I have checked and tested this, https://github.com/TrinityCore/TrinityCore/blob/master/src/server/scripts/Outland/zone_blades_edge_mountains.cpp#L998
Scripts/Scholomance: Jandice Barov

- Updated to EventMaps
- Removed hack to spawn illusions
- Moved Illusion's script to SAI
- Corrected Illusion's unit_flags
Core/Spells: Stormstrike should stack for different casters
Solve issue when 2 enhancement shamans are attacking same target
Scripts/Commands: Move cs_misc command output hardcoded strings to DB so they can be localized too
DB/SAI: To Rule the Skies

Proper script for Obsidia / Obsidia's Egg for quest http://www.wowhead.com/quest=11078/to-rule-the-skies

Now reagents are taken and Obsidia spawned using correct spells

Ie 

http://www.wowhead.com/spell=41035 (Four Dragons: Force Cast - Obsidia) which is cast on player as invoker cast when gossip option selected which triggers
http://www.wowhead.com/spell=41036 (Four Dragons: Reagents - Obsidia) which requires 35 apexis shards as reagents this deducting these and then triggers
http://www.wowhead.com/spell=40682 (http://www.wowhead.com/spell=40682) which triggers event to spawn

Previous script was not using spells and was using sai to deduct shards and spawn obsidia.
Scripts/Auchindoun: Updates Pandemonius to EventMaps/BossAI
Allow players to track both herbs and minerals at the same time

Note: The following are client limitations and cannot be coded for:
      * The minimap tracking icon will display whichever skill is activated second
      * The minimap tracking list will only show a check mark next to the last skill activated (sometimes this
        bugs out and doesn't switch the check mark. It has no effect on the actual tracking though).
      * The minimap dots are yellow for both resources