Fix done & taken procs

Corrects the logic for dot/hot procs and fixes spells such as althor's abacus, egg of mortal essence, black magic, spiritual attunement etc etc.
Reverts the remaining part of 8edf6247c087884acd8c873a57c6f414e19091ec
Merge pull request #11853 from Unholychick/patch-2

Done & taken procs & a few trinket fixes
Scripts/Ulduar: Archivum Console
- Scripts for Archivum Console, Brann Bronzebeard (33235), Prospector Doren/Loren
- Fixing quest chaining for both versions of the archivum data disc chains
- Fixes loot so correct quest starter item drops from assembly when done in hard mode rather than 10m quest item dropping in 25 and nothing dropping in 10 man
- Scripted events for turn in of Archivum Data Disk, The celestial Planetarium and watcher Analysis's and turn in of ancient history (Val'anyr, Hammer of Ancient Kings chain)
- Fix both versions of the Celestial Planetarium

Closes #11547
Closes #11556
Core/Misc: Added missing field in npc_text table for loading
Fix
Core/Spells: Convert Avenging Wrath to spellscript

Closes #11896
Closes #3464
Core/MMAPs: Improve NPC behavior with fences and high climbs

Increase the max walkable angle and max walkable climb in MMAPs generator to allow NPCs to jump on fences and climb steps.
This requires MMAPs to be extracted again with mmaps_generator, old extracted MMAPs are still compatible but will not provide the benefits of this commit.
Fixes #9965
DB/Quest: Script replacement of/or issueing of items needed for quests

By dr-j closes #11901
DB/Quest: Plundering the Plunderers

By dr-j, closes #11902
Scripts/Ruby Sanctum: Fixed corporeality scaling changes.

Thanks to Unholychick.

P.S.: #9231 if still valid will be fixed soon (tm)
Rename 2014_04_19_world_misc.sql to 2014_04_19_00_world_misc.sql
Core/Misc: Ensure GetVictim() returns a valid reference when expected

Add a new method EnsureVictim() which asserts a valid not NULL reference will be returned to the caller. Use this whenever the GetVictim() would be dereferenced .
Core/Misc: Check GetVictim() for NULL before dereferencing it

Add additional NULL checks all around the code before dereferencing GetVictim() result.
Keep in mind UpdateVictim() result has nothing to do with GetVictim() result and the caller shouldn't assume anything about GetVictim() based on UpdateVictim().
DB/Creature: Fix Birgitte Cranston

By untaught, closes #11842
DB/Creature Text: Add texts for Injured Soldiers

By untaught closes #11872
NOTE: don't posts multiple unrelated bugs on a single ticket.
Tools: Enabled struct packing to ensure that structures are read from files properly
Core/Misc: Fix change faction on dead Night Elves keeping the speed buff

Fix change faction on dead Night Elves causing the new player keeping the 175% wisp speed buff.
Players will be resurrected automatically on change faction.
Closes #255
Core/Misc: Fixed some -Weffc++ warnings
Repository: Set line endings in .patch/.diff files to LF via .gitattributes
DB/Creature: Fix Earthcaller Ryga

By untaught, closes #11827