Core/Player: moved player collections to separate class

* the idea is to have all collections (toybox, heirlooms, mount and later even battle pets) in one class to make it easier and don't increase size of already big Player and WorldSession class

1 files changed, 2 insertions, 6 deletions

diff --git a/src/server/game/Handlers/ToyHandler.cpp b/src/server/game/Handlers/ToyHandler.cpp
index 89ed9426c0f..a68c2ca5064 100644
--- a/src/server/game/Handlers/ToyHandler.cpp
+++ b/src/server/game/Handlers/ToyHandler.cpp
@@ -41,7 +41,7 @@ void WorldSession::HandleAddToy(WorldPackets::Toy::AddToy& packet)
         return;
     }
 
-    if (_player->AddToy(item->GetEntry(), false))
+    if (_collectionMgr->AddToy(item->GetEntry(), false))
         _player->DestroyItem(item->GetBagSlot(), item->GetSlot(), true);
 }
 
@@ -74,9 +74,5 @@ void WorldSession::HandleUseToy(WorldPackets::Toy::UseToy& packet)
 
 void WorldSession::HandleToySetFavorite(WorldPackets::Toy::ToySetFavorite& packet)
 {
-    ToyBoxContainer::iterator itr = _toys.find(packet.ItemID);
-    if (itr == _toys.end())
-        return;
-
-    itr->second = packet.Favorite;
+    _collectionMgr->ToySetFavorite(packet.ItemID, packet.Favorite);
 }