Fix transmog_legion_3.3.5 build after upstream 3.3.5 merge

[Rochet2]

Summary

The scheduled merge workflow merges TrinityCore/TrinityCore 3.3.5 into transmog_legion_3.3.5 and builds with -Werror. The branch failed to compile after upstream renamed locals in Player::StoreNewItem.

This PR merges the latest upstream 3.3.5 into transmog_legion_3.3.5 and adapts the transmog collection hook to the new names.

What was wrong

Post-merge compile errors

1. src/server/game/Entities/Player/Player.cpp:11665 — Upstream renamed StoreNewItem locals/params (pItem/item/dest → item/itemId/pos). The transmog-only AddToCollection(this, pItem) call kept the old name after auto-merge → use of undeclared identifier 'pItem'.

Verification

• Reconfigured CMake against source/ from BUILD/.
• Built src/server/game/game.vcxproj (Debug) — clean.
• Built src/server/scripts/scripts.vcxproj (Debug) — clean.

Test plan

• Re-run the merge workflow on the resulting branch and confirm the _3-3-5 job is green.
• Smoke-test transmog collection when looting/receiving items in-game.