Core/Collision: Corrected logic for closing source file in WorldModel_Raw::Read, fixes resource leak if method succeeds and prevents double free if it fails.

Closes #9823
Reverted part of 60a88a08f5e85c1986aee49f9bfdbf8fa254f370

All 8 bytes must be read from file to match output from vmap4extractor
Scripts/Misc: Fixed many scripts not executing one of their actions because of an event with id = 0
Core/Spells: Fixed channeling spells with infinite duration
Core/Misc: Another batch of fixes for issues found by static analysis
Core/Misc: Another batch of fixes for issues found by static analysis
Merge pull request #1 from TrinityCore/master