Core/Player: Fix a query that was wrongly deleting items

CHAR_DEL_ITEM_INSTANCE deletes by item guid but in Player::DeleteFromDB the guid being used was player's guid.
That means that if player with guid 100 was deleted, the item with guid 100 would also be deleted. This item could or could not belong to the player.

Closes #8883
Closes #9136

Thanks to @vlad852 for finding the issue
DB/Misc: Fixup quest Hot On The Trail (A:24849 H:24851)
Fix up previous commit.
Core: Fixed some compile warnings
Core: Clean up whitespace, and remove a silly warning in boss_lurker_below.cpp
Core: More clean up whitespace
Core/Accounts: sessionkey field in account table is only a temporary storage to pass data from authserver to worldserver and should only be used as such. Clearing sessionkey from database after a successful login to prevent possible exploits.
Core/Auth: Reverted part of fb43a92cc2aaffab42efebf025b6a12c01af8fde - session key is still needed after logging in to be able to switch realms
Merge branch 'master' of https://github.com/TrinityCore/TrinityCore