Fixup for last commits.
Core/Scripts: Removed useless const modifier on method parameters accepting primitive types passed by value
DB/Creatures: Update damage done by ICC bosses to more blizzlike values

Closes #8580
DB/Creature: Set Triggerflags & behaviour to triggerNPC

Closes #8950
DB/Conditions: add conditions to Lokhtos Darkbargainer Thx badmoon for initial fix

Closes #9028
DB/SAI: Add sai for quest Gather the Orbs (10859)

Closes #7636
Merge pull request #9168 from Kirkhammett/Ebonroc_Bwl

 Npc/Core: Was using Onyxia's Wing Buffet spell which can 1 shot lowbies...
Core/DBLayer: Comment out MySQL client/server version mismatch warning

MySQL version above 5.1 IS required in both client and server and there is no known issue with different versions above 5.1
Core/NPCs: Fix some equipment issues introduced with 028c72a9f2c606e8af3

Also changed equips defaults to 0 (no equipment) instead of 1.

It seems that some parts of the original commit were reverted locally, my bad.
Core/Players: Allow players to log out in combat when they are in resting area (logout in rest area is instant by default, but not when in combat)
Core/Logging: Fix mem leak. Thanks to @Sar777 for the report
Core/Misc: Remove devel log msg that shouldn't have been commited
Core/Vehicles: Also remove mount auras for passengers not in charm seat

May fix related crashes:
Ref #9256 #9255
Core/Creatures: Set hover movement flag if bytes1 have hover flag and creature cannot fly
Core/Vehicles: Fixed accessing freed memory in Vehicle::RemoveAllPassengers - _pendingJoinEvents needs to be cleaned from events that had already executed or were aborted
Core/Vehicles: Minor cleanup in header file
Core/Vehicles: Corrected logging message format specifiers
Core/Vehicles: Fix a crash in VehicleJoinEvent::Execute

Crash would happen if multiple events were scheduled for the same seat.
Fixes #9255
Core/Vehicles: Make RemovePendingEventsForSeat iterating crashsafe
Core/Vehicles: Really make RemovePendingEventsForSeat iterating crash safe