Core/Conditions: implement CONDITION_SOURCE_TYPE_SPELL_PROC and use it in old and new proc system

Usage:
SourceTypeOrReferenceId: 24
- SourceGroup: always 0
- SourceEntry: spell id of aura which triggers the proc
- ConditionTarget:
 * 0 - Actor
 * 1 - ActionTarget
Core/VMaps: drop "vmap.ignoreSpellIds" in config, this is not longer necessary since we use SPELL_ATTR2_CAN_TARGET_NOT_IN_LOS
Core/DisableMgr: readd possibility to disable los check for spells
Core/Scripting: Created a hook for when a player is about to be saved.
Grammar/Spelling fix.
DB: Re-add creature text for SAI conversion
Core/Scripts: Some cleanup for boss_kirtonos_the_herald script
Update src/server/game/Achievements/AchievementMgr.cpp

Prevent achievement completion log spam.
db/commit: fix sql error
DB/SAI: Fix crash with quest "Thirst Unending"
Closes #8856
Core/Misc: Use ASSERT (custom, better macro) instead of the C macro assert()
Core/Player: Prevent a crash in CastedCreatureOrGO and add an assertion to KilledMonster
Core/Scripting: Update Kirtonos script.
Core/Misc: Fix a couple of warnings, some of them performance related

Errors were found using Cppcheck, open-source static analysis tool
Core/Scripting: Fix Typo.
Core/Spline: Fix typo, I blame not enough Ctrl+S
DB/Gossip/Waypoints/Creature_text: Db data for Harrison Jones Gong Event.
DB/SAI: Fixed some runtime errors.
Merge pull request #8848 from NightQuest/save_hook

Core/Scripts: Created a hook for when a player is about to be saved.
Core/Vehicles: Add some internal checks in vehicle code to prevent crashable behaviour caused by scripts or other areas of the core.