diff options
-rw-r--r-- | src/game/ItemHandler.cpp | 4 | ||||
-rw-r--r-- | src/game/Object.cpp | 22 | ||||
-rw-r--r-- | src/game/Object.h | 4 | ||||
-rw-r--r-- | src/game/ObjectAccessor.cpp | 18 | ||||
-rw-r--r-- | src/game/ObjectAccessor.h | 1 | ||||
-rw-r--r-- | src/game/PetAI.cpp | 4 | ||||
-rw-r--r-- | src/game/PetHandler.cpp | 6 | ||||
-rw-r--r-- | src/game/Player.cpp | 38 |
8 files changed, 28 insertions, 69 deletions
diff --git a/src/game/ItemHandler.cpp b/src/game/ItemHandler.cpp index b31edcf2b42..a0e800439db 100644 --- a/src/game/ItemHandler.cpp +++ b/src/game/ItemHandler.cpp @@ -568,12 +568,12 @@ void WorldSession::HandleSellItemOpcode( WorldPacket & recv_data ) pItem->SetCount( pItem->GetCount() - count ); _player->ItemRemovedQuestCheck( pItem->GetEntry(), count ); if( _player->IsInWorld() ) - pItem->SendUpdateToPlayer( _player ); + pItem->SendCreateUpdateToPlayer( _player ); pItem->SetState(ITEM_CHANGED, _player); _player->AddItemToBuyBackSlot( pNewItem ); if( _player->IsInWorld() ) - pNewItem->SendUpdateToPlayer( _player ); + pNewItem->SendCreateUpdateToPlayer( _player ); } else { diff --git a/src/game/Object.cpp b/src/game/Object.cpp index 0c8c215d09a..1522f8db754 100644 --- a/src/game/Object.cpp +++ b/src/game/Object.cpp @@ -232,13 +232,7 @@ void Object::BuildCreateUpdateBlockForPlayer(UpdateData *data, Player *target) c data->AddUpdateBlock(buf); } -void Object::BuildUpdate(UpdateDataMapType &update_players) -{ - ObjectAccessor::_buildUpdateObject(this,update_players); - ClearUpdateMask(true); -} - -void Object::SendUpdateToPlayer(Player* player) +void Object::SendCreateUpdateToPlayer(Player* player) { // send create update to player UpdateData upd; @@ -711,20 +705,6 @@ void Object::ClearUpdateMask(bool remove) } } -// Send current value fields changes to all viewers -void Object::SendUpdateObjectToAllExcept(Player* exceptPlayer) -{ - // changes will be send in create packet - if(!IsInWorld()) - return; - - // nothing do - if(!m_objectUpdated) - return; - - ObjectAccessor::UpdateObject(this,exceptPlayer); -} - bool Object::LoadValues(const char* data) { if(!m_uint32Values) _InitValues(); diff --git a/src/game/Object.h b/src/game/Object.h index 93cf0d40e85..e3c8ee33399 100644 --- a/src/game/Object.h +++ b/src/game/Object.h @@ -160,12 +160,11 @@ class TRINITY_DLL_SPEC Object bool isType(uint16 mask) const { return (mask & m_objectType); } virtual void BuildCreateUpdateBlockForPlayer( UpdateData *data, Player *target ) const; - void SendUpdateToPlayer(Player* player); + void SendCreateUpdateToPlayer(Player* player); void BuildValuesUpdateBlockForPlayer( UpdateData *data, Player *target ) const; void BuildOutOfRangeUpdateBlock( UpdateData *data ) const; void BuildMovementUpdateBlock( UpdateData * data, uint32 flags = 0 ) const; - void BuildUpdate(UpdateDataMapType &); virtual void DestroyForPlayer( Player *target, bool anim = false ) const; @@ -307,7 +306,6 @@ class TRINITY_DLL_SPEC Object } void ClearUpdateMask(bool remove); - void SendUpdateObjectToAllExcept(Player* exceptPlayer); bool LoadValues(const char* data); diff --git a/src/game/ObjectAccessor.cpp b/src/game/ObjectAccessor.cpp index e52423eabd8..29692a7264d 100644 --- a/src/game/ObjectAccessor.cpp +++ b/src/game/ObjectAccessor.cpp @@ -185,24 +185,6 @@ ObjectAccessor::SaveAllPlayers() } void -ObjectAccessor::UpdateObject(Object* obj, Player* exceptPlayer) -{ - UpdateDataMapType update_players; - obj->BuildUpdate(update_players); - - WorldPacket packet; - for (UpdateDataMapType::iterator iter = update_players.begin(); iter != update_players.end(); ++iter) - { - if(iter->first == exceptPlayer) - continue; - - iter->second.BuildPacket(&packet); - iter->first->GetSession()->SendPacket(&packet); - packet.clear(); - } -} - -void ObjectAccessor::_buildUpdateObject(Object *obj, UpdateDataMapType &update_players) { if(obj->isType(TYPEMASK_ITEM)) diff --git a/src/game/ObjectAccessor.h b/src/game/ObjectAccessor.h index 780c108e5e4..185420d96ea 100644 --- a/src/game/ObjectAccessor.h +++ b/src/game/ObjectAccessor.h @@ -232,7 +232,6 @@ class MANGOS_DLL_DECL ObjectAccessor : public MaNGOS::Singleton<ObjectAccessor, void AddCorpsesToGrid(GridPair const& gridpair,GridType& grid,Map* map); Corpse* ConvertCorpseForPlayer(uint64 player_guid, bool insignia = false); - static void UpdateObject(Object* obj, Player* exceptPlayer); static void _buildUpdateObject(Object* obj, UpdateDataMapType &); static void UpdateObjectVisibility(WorldObject* obj); diff --git a/src/game/PetAI.cpp b/src/game/PetAI.cpp index fc305d5f122..bb69e1a0241 100644 --- a/src/game/PetAI.cpp +++ b/src/game/PetAI.cpp @@ -208,10 +208,10 @@ void PetAI::UpdateAI(const uint32 diff) { m_creature->SetInFront(target); if( target->GetTypeId() == TYPEID_PLAYER ) - m_creature->SendUpdateToPlayer( (Player*)target ); + m_creature->SendCreateUpdateToPlayer( (Player*)target ); if(owner && owner->GetTypeId() == TYPEID_PLAYER) - m_creature->SendUpdateToPlayer( (Player*)owner ); + m_creature->SendCreateUpdateToPlayer( (Player*)owner ); } m_creature->AddCreatureSpellCooldown(spell->m_spellInfo->Id); diff --git a/src/game/PetHandler.cpp b/src/game/PetHandler.cpp index ee8c02dfb43..fc11d8a8b75 100644 --- a/src/game/PetHandler.cpp +++ b/src/game/PetHandler.cpp @@ -269,17 +269,17 @@ void WorldSession::HandlePetActionHelper(Unit *pet, uint64 guid1, uint16 spellid { pet->SetInFront(unit_target); if (unit_target->GetTypeId() == TYPEID_PLAYER) - pet->SendUpdateToPlayer( (Player*)unit_target ); + pet->SendCreateUpdateToPlayer( (Player*)unit_target ); } else if(Unit *unit_target2 = spell->m_targets.getUnitTarget()) { pet->SetInFront(unit_target2); if (unit_target2->GetTypeId() == TYPEID_PLAYER) - pet->SendUpdateToPlayer( (Player*)unit_target2 ); + pet->SendCreateUpdateToPlayer( (Player*)unit_target2 ); } if (Unit* powner = pet->GetCharmerOrOwner()) if(powner->GetTypeId() == TYPEID_PLAYER) - pet->SendUpdateToPlayer((Player*)powner); + pet->SendCreateUpdateToPlayer((Player*)powner); result = SPELL_CAST_OK; } diff --git a/src/game/Player.cpp b/src/game/Player.cpp index 51fbc3413c0..e607b9a453f 100644 --- a/src/game/Player.cpp +++ b/src/game/Player.cpp @@ -11004,7 +11004,7 @@ Item* Player::_StoreItem( uint16 pos, Item *pItem, uint32 count, bool clone, boo if (IsInWorld() && update) { pItem->AddToWorld(); - pItem->SendUpdateToPlayer( this ); + pItem->SendCreateUpdateToPlayer( this ); } pItem->SetState(ITEM_CHANGED, this); @@ -11015,7 +11015,7 @@ Item* Player::_StoreItem( uint16 pos, Item *pItem, uint32 count, bool clone, boo if( IsInWorld() && update ) { pItem->AddToWorld(); - pItem->SendUpdateToPlayer( this ); + pItem->SendCreateUpdateToPlayer( this ); } pItem->SetState(ITEM_CHANGED, this); pBag->SetState(ITEM_CHANGED, this); @@ -11035,7 +11035,7 @@ Item* Player::_StoreItem( uint16 pos, Item *pItem, uint32 count, bool clone, boo pItem2->SetCount( pItem2->GetCount() + count ); if (IsInWorld() && update) - pItem2->SendUpdateToPlayer( this ); + pItem2->SendCreateUpdateToPlayer( this ); if (!clone) { @@ -11125,7 +11125,7 @@ Item* Player::EquipItem( uint16 pos, Item *pItem, bool update ) if( IsInWorld() && update ) { pItem->AddToWorld(); - pItem->SendUpdateToPlayer( this ); + pItem->SendCreateUpdateToPlayer( this ); } ApplyEquipCooldown(pItem); @@ -11152,7 +11152,7 @@ Item* Player::EquipItem( uint16 pos, Item *pItem, bool update ) { pItem2->SetCount( pItem2->GetCount() + pItem->GetCount() ); if( IsInWorld() && update ) - pItem2->SendUpdateToPlayer( this ); + pItem2->SendCreateUpdateToPlayer( this ); // delete item (it not in any slot currently) //pItem->DeleteFromDB(); @@ -11193,7 +11193,7 @@ void Player::QuickEquipItem( uint16 pos, Item *pItem) if( IsInWorld() ) { pItem->AddToWorld(); - pItem->SendUpdateToPlayer( this ); + pItem->SendCreateUpdateToPlayer( this ); } GetAchievementMgr().UpdateAchievementCriteria(ACHIEVEMENT_CRITERIA_TYPE_EQUIP_ITEM, pItem->GetEntry()); @@ -11320,7 +11320,7 @@ void Player::RemoveItem(uint8 bag, uint8 slot, bool update) // pItem->SetUInt64Value(ITEM_FIELD_OWNER, 0); not clear owner at remove (it will be set at store). This used in mail and auction code pItem->SetSlot(NULL_SLOT); if(IsInWorld() && update) - pItem->SendUpdateToPlayer(this); + pItem->SendCreateUpdateToPlayer(this); } } @@ -11472,7 +11472,7 @@ void Player::DestroyItemCount(uint32 item, uint32 count, bool update, bool unequ ItemRemovedQuestCheck(pItem->GetEntry(), count - remcount); pItem->SetCount(pItem->GetCount() - count + remcount); if (IsInWorld() & update) - pItem->SendUpdateToPlayer( this ); + pItem->SendCreateUpdateToPlayer( this ); pItem->SetState(ITEM_CHANGED, this); return; } @@ -11500,7 +11500,7 @@ void Player::DestroyItemCount(uint32 item, uint32 count, bool update, bool unequ ItemRemovedQuestCheck( pItem->GetEntry(), count - remcount ); pItem->SetCount( pItem->GetCount() - count + remcount ); if (IsInWorld() & update) - pItem->SendUpdateToPlayer( this ); + pItem->SendCreateUpdateToPlayer( this ); pItem->SetState(ITEM_CHANGED, this); return; } @@ -11533,7 +11533,7 @@ void Player::DestroyItemCount(uint32 item, uint32 count, bool update, bool unequ ItemRemovedQuestCheck( pItem->GetEntry(), count - remcount ); pItem->SetCount( pItem->GetCount() - count + remcount ); if (IsInWorld() && update) - pItem->SendUpdateToPlayer( this ); + pItem->SendCreateUpdateToPlayer( this ); pItem->SetState(ITEM_CHANGED, this); return; } @@ -11566,7 +11566,7 @@ void Player::DestroyItemCount(uint32 item, uint32 count, bool update, bool unequ ItemRemovedQuestCheck( pItem->GetEntry(), count - remcount ); pItem->SetCount( pItem->GetCount() - count + remcount ); if (IsInWorld() & update) - pItem->SendUpdateToPlayer( this ); + pItem->SendCreateUpdateToPlayer( this ); pItem->SetState(ITEM_CHANGED, this); return; } @@ -11651,7 +11651,7 @@ void Player::DestroyItemCount( Item* pItem, uint32 &count, bool update ) pItem->SetCount( pItem->GetCount() - count ); count = 0; if( IsInWorld() & update ) - pItem->SendUpdateToPlayer( this ); + pItem->SendCreateUpdateToPlayer( this ); pItem->SetState(ITEM_CHANGED, this); } } @@ -11716,7 +11716,7 @@ void Player::SplitItem( uint16 src, uint16 dst, uint32 count ) } if( IsInWorld() ) - pSrcItem->SendUpdateToPlayer( this ); + pSrcItem->SendCreateUpdateToPlayer( this ); pSrcItem->SetState(ITEM_CHANGED, this); StoreItem( dest, pNewItem, true); } @@ -11736,7 +11736,7 @@ void Player::SplitItem( uint16 src, uint16 dst, uint32 count ) } if( IsInWorld() ) - pSrcItem->SendUpdateToPlayer( this ); + pSrcItem->SendCreateUpdateToPlayer( this ); pSrcItem->SetState(ITEM_CHANGED, this); BankItem( dest, pNewItem, true); } @@ -11756,7 +11756,7 @@ void Player::SplitItem( uint16 src, uint16 dst, uint32 count ) } if( IsInWorld() ) - pSrcItem->SendUpdateToPlayer( this ); + pSrcItem->SendCreateUpdateToPlayer( this ); pSrcItem->SetState(ITEM_CHANGED, this); EquipItem( dest, pNewItem, true); AutoUnequipOffhandIfNeed(); @@ -11927,8 +11927,8 @@ void Player::SwapItem( uint16 src, uint16 dst ) pDstItem->SetState(ITEM_CHANGED, this); if( IsInWorld() ) { - pSrcItem->SendUpdateToPlayer( this ); - pDstItem->SendUpdateToPlayer( this ); + pSrcItem->SendCreateUpdateToPlayer( this ); + pDstItem->SendCreateUpdateToPlayer( this ); } } return; @@ -17920,7 +17920,7 @@ void Player::HandleStealthedUnitsDetection() { if(!hasAtClient) { - (*i)->SendUpdateToPlayer(this); + (*i)->SendCreateUpdateToPlayer(this); m_clientGUIDs.insert((*i)->GetGUID()); #ifdef MANGOS_DEBUG @@ -19343,7 +19343,7 @@ void Player::UpdateVisibilityOf(WorldObject* target) //if(target->isType(TYPEMASK_UNIT) && ((Unit*)target)->m_Vehicle) // UpdateVisibilityOf(((Unit*)target)->m_Vehicle); - target->SendUpdateToPlayer(this); + target->SendCreateUpdateToPlayer(this); m_clientGUIDs.insert(target->GetGUID()); #ifdef TRINITY_DEBUG |