Merge pull request #11077 from Elron103/pullrequests2

DB/Gossip: Update some data for trainer NPCs from sniffs
Core/Misc: Get rid of some useless code
Merge pull request #11050 from Ascathor/master

Core/Command: Improve .group list to feature several notes and features ...
DB/SAI: Misc Fixes from IT

By @dr-j and @untaught
Fixes #11081 fixes #11080 fixes #11082
Core/Quest: Remove assertion

Remove assertion about Quest ExclusiveGroup since they are triggered even when adding valid quests to npcs. Fixes #8336 .
How to reproduce steps:
- .gm off
- add quest 11335 in creature_queststarter
- .reload creature_queststarter
- get in range of that creature
- .gm on
- assertion triggered
Core/SAI: Rename SMART_EVENT_TARGET_CASTING to SMART_EVENT_VICTIM_CASTING as that's what it actually does. It was often confused to use the target_type field instead of the victim.
Core/Transports: Fixed players taking fall damage when teleporting from transports

Closes #11076
Core/Auras: Added serverside checking for UNIT_FLAG_PACIFIED when attempting to start melee attack

Closes #8623
Core: add missing comment
Merge pull request #11092 from gerripeach/pull_request

Core: add missing comment
Scripts/Misc: Coding Style unification? (by Aokromes)
Core/Spell: Shaman Bloodlust/Heroism ignore LOS

closes: #6307
Rename sql file and apply some code style. *sigh*
Rename SQL file (thx Aokromes)
Core/IoC: Fix uninitialized timer

Fix uninitialized Docks timer in Isle Of Conquest .

Valgrind log:
 Conditional jump or move depends on uninitialised value(s)
  at : BattlegroundIC::HandleCapturedNodes(ICNodePoint*, bool) (BattlegroundIC.cpp:665)
  by : BattlegroundIC::PostUpdateImpl(unsigned int) (BattlegroundIC.cpp:228)
  by : Battleground::Update(unsigned int) (Battleground.cpp:300)
  by : BattlegroundMgr::Update(unsigned int) (BattlegroundMgr.cpp:93)
  by : World::Update(unsigned int) (World.cpp:2062)
DB/SAI: This just in: Fire still hot!

By @untaught fixes #10752
DB/SAI: Add Gymer text and faction friendly to horde also.
DB/SAI: Fix scripts using SMART_TARGET_NONE that were 'broken' by 9653576cb4cc9449898d6af691c045d68e9361c5. 'Broken' is quoted because scripts using SMART_TARGET_NONE are most of the time broken by default anyway.
DB/Errors: Db error clean up

By @untaught fixes #11116
DB/SAI: Swift Discipline

By @untaught fixes #11118