Core/Misc:  Fix some static analysis issues

Fix some static analysis issues, mostly false positive about fields not initialized in the constructor. It's good practice anyway to always initialize them.

Conflicts:
	src/server/collision/Models/WorldModel.h
	src/server/scripts/Kalimdor/CavernsOfTime/BattleForMountHyjal/hyjal_trash.cpp
Core/Player: GetRewSpellCast & GetRewSpell should not be cast by questgiver if effect type SPELL_EFFECT_LEARN_SPELL
Core/Misc: Fix some static analysis issues

Fix some static analysis issues about missing NULL checks.
Pass safe Creature pointer instead of summoning a Creature, passing the GUID and retrieve the same Creature from GUID.

Conflicts:
	src/server/scripts/EasternKingdoms/Karazhan/karazhan.cpp
Core/Chat: Fix ChatFlood.MessageCount not being handled correctly

Fix Anti-flood chat system to always allow 1 more message than the count specified in the ChatFlood.MessageCount config parameter.
Core/SmartAI: Fix SMART_ACTION_ALLOW_COMBAT_MOVEMENT
Core/SmartScript: Allow flag to be added to spell to determine combat movement.
Greatly simplifies SAI combat scripting.
Core/SmartAI: Fixed waypoint pathing not resuming after evade.
Core/SmartAI: Remove unneeded code added in error
Core/Commands: Fix NULL dereference crash in .npc set link

Fix NULL dereference crash in .npc set link happening when passing a not-existent Creature guid as parameter, es. ".npc set link -1" .
Issue spotted by static analysis, added in c05ed659cbc561b6e7c0016cc7b1548886d5337a
Core/Vehicle: Add an extra check for required energy before setting power.
Scripting/Grizzly Hills: Add missing include "SpellScript.h"
Mailbox Command
Mailbox fix
Core/Spells: Fixed rare issues where spell modifiers would not be correctly restored.

Happens in case of a spell having more than one modifier granted by an aura.
Core/Unit: Add switch to prevent COMMAND_ATTACK being added to action bar for supplied creature entries.
Core/Misc: Fix some static analysis issues

Fix some static analysis issues about uninitialized values in scripts.
Add spell id validation in Unit::SpellNonMeleeDamageLog() .

Conflicts:
	src/server/scripts/EasternKingdoms/BlackrockMountain/BlackwingLair/instance_blackwing_lair.cpp
	src/server/scripts/Northrend/CrusadersColiseum/TrialOfTheCrusader/boss_northrend_beasts.cpp
	src/server/scripts/Northrend/FrozenHalls/HallsOfReflection/halls_of_reflection.cpp
	src/server/scripts/Northrend/Naxxramas/boss_gothik.cpp
	src/server/scripts/Outland/Auchindoun/AuchenaiCrypts/boss_exarch_maladaar.cpp
Core/Misc: Fix some static analysis issues

Fix some static analysis issues about uninitialized values.

Conflicts:
	src/server/scripts/Kalimdor/CavernsOfTime/BattleForMountHyjal/boss_anetheron.cpp
	src/server/scripts/Kalimdor/CavernsOfTime/BattleForMountHyjal/boss_azgalor.cpp
	src/server/scripts/Kalimdor/CavernsOfTime/BattleForMountHyjal/hyjal_trash.cpp
	src/server/scripts/Kalimdor/TempleOfAhnQiraj/boss_cthun.cpp
Core/Enums: Few inns/zones use the zone itself for the resting state instead of an areatrigger.
Scripts: replace some SetOrientation() calls with spline packets and some random changes

Conflicts:
	src/server/game/Battlefield/Battlefield.cpp
	src/server/scripts/EasternKingdoms/Scholomance/boss_ras_frostwhisper.cpp
	src/server/scripts/EasternKingdoms/SunwellPlateau/boss_felmyst.cpp
	src/server/scripts/EasternKingdoms/ZulGurub/boss_jindo.cpp
	src/server/scripts/Kalimdor/TempleOfAhnQiraj/boss_bug_trio.cpp
	src/server/scripts/Northrend/zone_storm_peaks.cpp
	src/server/scripts/Outland/zone_netherstorm.cpp
+ deleted unused zone_scripts
Core/SmartScripts: speedup loading/validating sai scripts from db

Before:
>> Loaded 6953 SmartAI scripts in 3543 ms

After:
>> Loaded SmartAIMgr Helpers in 4 ms
>> Loaded 6953 SmartAI scripts in 95 ms