mirror of
https://github.com/TrinityCore/TrinityCore.git
synced 2026-01-22 10:05:32 +01:00
Core/Group: Code style corrections and logic fixes
This commit is contained in:
@@ -931,50 +931,50 @@ void WorldSession::BuildPartyMemberStatsChangedPacket(Player* player, WorldPacke
|
||||
if (mask & GROUP_UPDATE_FLAG_PET_POWER_TYPE) // same for pets
|
||||
mask |= (GROUP_UPDATE_FLAG_PET_CUR_POWER | GROUP_UPDATE_FLAG_PET_MAX_POWER);
|
||||
|
||||
data->Initialize(SMSG_PARTY_MEMBER_STATS, 80); // average value
|
||||
data->Initialize(SMSG_PARTY_MEMBER_STATS, 80); // average value
|
||||
data->append(player->GetPackGUID());
|
||||
*data << (uint32) mask;
|
||||
*data << uint32(mask);
|
||||
|
||||
if (mask & GROUP_UPDATE_FLAG_STATUS)
|
||||
{
|
||||
if (player)
|
||||
{
|
||||
if (player->IsPvP())
|
||||
*data << (uint16) (MEMBER_STATUS_ONLINE | MEMBER_STATUS_PVP);
|
||||
*data << uint16(MEMBER_STATUS_ONLINE | MEMBER_STATUS_PVP);
|
||||
else
|
||||
*data << (uint16) MEMBER_STATUS_ONLINE;
|
||||
*data << uint16(MEMBER_STATUS_ONLINE);
|
||||
}
|
||||
else
|
||||
*data << (uint16) MEMBER_STATUS_OFFLINE;
|
||||
*data << uint16(MEMBER_STATUS_OFFLINE);
|
||||
}
|
||||
|
||||
if (mask & GROUP_UPDATE_FLAG_CUR_HP)
|
||||
*data << (uint32) player->GetHealth();
|
||||
*data << uint32(player->GetHealth());
|
||||
|
||||
if (mask & GROUP_UPDATE_FLAG_MAX_HP)
|
||||
*data << (uint32) player->GetMaxHealth();
|
||||
*data << uint32(player->GetMaxHealth());
|
||||
|
||||
Powers powerType = player->getPowerType();
|
||||
if (mask & GROUP_UPDATE_FLAG_POWER_TYPE)
|
||||
*data << (uint8) powerType;
|
||||
*data << uint8(powerType);
|
||||
|
||||
if (mask & GROUP_UPDATE_FLAG_CUR_POWER)
|
||||
*data << (uint16) player->GetPower(powerType);
|
||||
*data << uint16(player->GetPower(powerType));
|
||||
|
||||
if (mask & GROUP_UPDATE_FLAG_MAX_POWER)
|
||||
*data << (uint16) player->GetMaxPower(powerType);
|
||||
*data << uint16(player->GetMaxPower(powerType));
|
||||
|
||||
if (mask & GROUP_UPDATE_FLAG_LEVEL)
|
||||
*data << (uint16) player->getLevel();
|
||||
*data << uint16(player->getLevel());
|
||||
|
||||
if (mask & GROUP_UPDATE_FLAG_ZONE)
|
||||
*data << (uint16) player->GetZoneId();
|
||||
*data << uint16(player->GetZoneId());
|
||||
|
||||
if (mask & GROUP_UPDATE_FLAG_UNK100)
|
||||
*data << uint16(0);
|
||||
|
||||
if (mask & GROUP_UPDATE_FLAG_POSITION)
|
||||
*data << (uint16) player->GetPositionX() << (uint16) player->GetPositionY() << (uint16) player->GetPositionZ();
|
||||
*data << uint16(player->GetPositionX()) << uint16(player->GetPositionY()) << uint16(player->GetPositionZ());
|
||||
|
||||
if (mask & GROUP_UPDATE_FLAG_AURAS)
|
||||
{
|
||||
@@ -1015,9 +1015,9 @@ void WorldSession::BuildPartyMemberStatsChangedPacket(Player* player, WorldPacke
|
||||
if (mask & GROUP_UPDATE_FLAG_PET_GUID)
|
||||
{
|
||||
if (pet)
|
||||
*data << (uint64) pet->GetGUID();
|
||||
*data << uint64(pet->GetGUID());
|
||||
else
|
||||
*data << (uint64) 0;
|
||||
*data << uint64(0);
|
||||
}
|
||||
|
||||
if (mask & GROUP_UPDATE_FLAG_PET_NAME)
|
||||
@@ -1025,61 +1025,63 @@ void WorldSession::BuildPartyMemberStatsChangedPacket(Player* player, WorldPacke
|
||||
if (pet)
|
||||
*data << pet->GetName();
|
||||
else
|
||||
*data << (uint8) 0;
|
||||
*data << uint8(0);
|
||||
}
|
||||
|
||||
if (mask & GROUP_UPDATE_FLAG_PET_MODEL_ID)
|
||||
{
|
||||
if (pet)
|
||||
*data << (uint16) pet->GetDisplayId();
|
||||
*data << uint16(pet->GetDisplayId());
|
||||
else
|
||||
*data << (uint16) 0;
|
||||
*data << uint16(0);
|
||||
}
|
||||
|
||||
if (mask & GROUP_UPDATE_FLAG_PET_CUR_HP)
|
||||
{
|
||||
if (pet)
|
||||
*data << (uint32) pet->GetHealth();
|
||||
*data << uint32(pet->GetHealth());
|
||||
else
|
||||
*data << (uint32) 0;
|
||||
*data << uint32(0);
|
||||
}
|
||||
|
||||
if (mask & GROUP_UPDATE_FLAG_PET_MAX_HP)
|
||||
{
|
||||
if (pet)
|
||||
*data << (uint32) pet->GetMaxHealth();
|
||||
*data << uint32(pet->GetMaxHealth());
|
||||
else
|
||||
*data << (uint32) 0;
|
||||
*data << uint32(0);
|
||||
}
|
||||
|
||||
if (mask & GROUP_UPDATE_FLAG_PET_POWER_TYPE)
|
||||
{
|
||||
if (pet)
|
||||
*data << (uint8) pet->getPowerType();
|
||||
*data << uint8(pet->getPowerType());
|
||||
else
|
||||
*data << (uint8) 0;
|
||||
*data << uint8(0);
|
||||
}
|
||||
|
||||
if (mask & GROUP_UPDATE_FLAG_PET_CUR_POWER)
|
||||
{
|
||||
if (pet)
|
||||
*data << (uint16) pet->GetPower(pet->getPowerType());
|
||||
*data << uint16(pet->GetPower(pet->getPowerType()));
|
||||
else
|
||||
*data << (uint16) 0;
|
||||
*data << uint16(0);
|
||||
}
|
||||
|
||||
if (mask & GROUP_UPDATE_FLAG_PET_MAX_POWER)
|
||||
{
|
||||
if (pet)
|
||||
*data << (uint16) pet->GetMaxPower(pet->getPowerType());
|
||||
*data << uint16(pet->GetMaxPower(pet->getPowerType()));
|
||||
else
|
||||
*data << (uint16) 0;
|
||||
*data << uint16(0);
|
||||
}
|
||||
|
||||
if (mask & GROUP_UPDATE_FLAG_VEHICLE_SEAT)
|
||||
{
|
||||
if (Vehicle* veh = player->GetVehicle())
|
||||
*data << (uint32) veh->GetVehicleInfo()->m_seatID[player->m_movementInfo.t_seat];
|
||||
*data << uint32(veh->GetVehicleInfo()->m_seatID[player->m_movementInfo.t_seat]);
|
||||
else
|
||||
*data << uint32(0);
|
||||
}
|
||||
|
||||
if (mask & GROUP_UPDATE_FLAG_PET_AURAS)
|
||||
@@ -1119,7 +1121,10 @@ void WorldSession::BuildPartyMemberStatsChangedPacket(Player* player, WorldPacke
|
||||
}
|
||||
}
|
||||
else
|
||||
*data << (uint64) 0;
|
||||
{
|
||||
*data << uint8(0);
|
||||
*data << uint64(0);
|
||||
}
|
||||
}
|
||||
|
||||
if (mask & GROUP_UPDATE_FLAG_PHASE)
|
||||
@@ -1134,15 +1139,15 @@ void WorldSession::BuildPartyMemberStatsChangedPacket(Player* player, WorldPacke
|
||||
void WorldSession::HandleRequestPartyMemberStatsOpcode(WorldPacket& recvData)
|
||||
{
|
||||
sLog->outDebug(LOG_FILTER_NETWORKIO, "WORLD: Received CMSG_REQUEST_PARTY_MEMBER_STATS");
|
||||
uint64 Guid;
|
||||
recvData >> Guid;
|
||||
uint64 guid;
|
||||
recvData >> guid;
|
||||
|
||||
Player* player = HashMapHolder<Player>::Find(Guid);
|
||||
Player* player = HashMapHolder<Player>::Find(guid);
|
||||
if (!player)
|
||||
{
|
||||
WorldPacket data(SMSG_PARTY_MEMBER_STATS_FULL, 3+4+2);
|
||||
data << uint8(0); // only for SMSG_PARTY_MEMBER_STATS_FULL, probably arena/bg related
|
||||
data.appendPackGUID(Guid);
|
||||
data.appendPackGUID(guid);
|
||||
data << (uint32) GROUP_UPDATE_FLAG_STATUS;
|
||||
data << (uint16) MEMBER_STATUS_OFFLINE;
|
||||
SendPacket(&data);
|
||||
@@ -1161,25 +1166,25 @@ void WorldSession::HandleRequestPartyMemberStatsOpcode(WorldPacket& recvData)
|
||||
mask1 &= ~GROUP_UPDATE_PET;
|
||||
|
||||
Powers powerType = player->getPowerType();
|
||||
data << (uint32) mask1; // group update mask
|
||||
data << (uint16) MEMBER_STATUS_ONLINE; // member's online status, GROUP_UPDATE_FLAG_STATUS
|
||||
data << (uint32) player->GetHealth(); // GROUP_UPDATE_FLAG_CUR_HP
|
||||
data << (uint32) player->GetMaxHealth(); // GROUP_UPDATE_FLAG_MAX_HP
|
||||
data << (uint8) powerType; // GROUP_UPDATE_FLAG_POWER_TYPE
|
||||
data << (uint16) player->GetPower(powerType); // GROUP_UPDATE_FLAG_CUR_POWER
|
||||
data << (uint16) player->GetMaxPower(powerType); // GROUP_UPDATE_FLAG_MAX_POWER
|
||||
data << (uint16) player->getLevel(); // GROUP_UPDATE_FLAG_LEVEL
|
||||
data << (uint16) player->GetZoneId(); // GROUP_UPDATE_FLAG_ZONE
|
||||
data << (uint16) player->GetPositionX(); // GROUP_UPDATE_FLAG_POSITION
|
||||
data << (uint16) player->GetPositionY(); // GROUP_UPDATE_FLAG_POSITION
|
||||
data << (uint16) player->GetPositionZ(); // GROUP_UPDATE_FLAG_POSITION
|
||||
data << uint32(mask1); // group update mask
|
||||
data << uint16(MEMBER_STATUS_ONLINE); // member's online status, GROUP_UPDATE_FLAG_STATUS
|
||||
data << uint32(player->GetHealth()); // GROUP_UPDATE_FLAG_CUR_HP
|
||||
data << uint32(player->GetMaxHealth()); // GROUP_UPDATE_FLAG_MAX_HP
|
||||
data << uint8 (powerType); // GROUP_UPDATE_FLAG_POWER_TYPE
|
||||
data << uint16(player->GetPower(powerType)); // GROUP_UPDATE_FLAG_CUR_POWER
|
||||
data << uint16(player->GetMaxPower(powerType)); // GROUP_UPDATE_FLAG_MAX_POWER
|
||||
data << uint16(player->getLevel()); // GROUP_UPDATE_FLAG_LEVEL
|
||||
data << uint16(player->GetZoneId()); // GROUP_UPDATE_FLAG_ZONE
|
||||
data << uint16(player->GetPositionX()); // GROUP_UPDATE_FLAG_POSITION
|
||||
data << uint16(player->GetPositionY()); // GROUP_UPDATE_FLAG_POSITION
|
||||
data << uint16(player->GetPositionZ()); // GROUP_UPDATE_FLAG_POSITION
|
||||
|
||||
// GROUP_UPDATE_FLAG_AURAS
|
||||
data << (uint8) 1;
|
||||
data << uint8(1);
|
||||
uint64 auramask = 0;
|
||||
data << (uint64) auramask; // placeholder
|
||||
data << uint64(auramask); // placeholder
|
||||
size_t maskPos = data.wpos();
|
||||
data << (uint32) MAX_AURAS; // count
|
||||
data << uint32(MAX_AURAS); // count
|
||||
for (uint8 i = 0; i < MAX_AURAS; ++i)
|
||||
{
|
||||
if (AuraApplication const* aurApp = player->GetVisibleAura(i))
|
||||
@@ -1201,26 +1206,26 @@ void WorldSession::HandleRequestPartyMemberStatsOpcode(WorldPacket& recvData)
|
||||
}
|
||||
}
|
||||
}
|
||||
data.put<uint64>(maskPos, auramask); // GROUP_UPDATE_FLAG_AURAS
|
||||
data.put<uint64>(maskPos, auramask); // GROUP_UPDATE_FLAG_AURAS
|
||||
|
||||
if (pet)
|
||||
{
|
||||
Powers petpowertype = pet->getPowerType();
|
||||
data << (uint64) pet->GetGUID(); // GROUP_UPDATE_FLAG_PET_GUID
|
||||
data << uint64(pet->GetGUID()); // GROUP_UPDATE_FLAG_PET_GUID
|
||||
data << pet->GetName(); // GROUP_UPDATE_FLAG_PET_NAME
|
||||
data << (uint16) pet->GetDisplayId(); // GROUP_UPDATE_FLAG_PET_MODEL_ID
|
||||
data << (uint32) pet->GetHealth(); // GROUP_UPDATE_FLAG_PET_CUR_HP
|
||||
data << (uint32) pet->GetMaxHealth(); // GROUP_UPDATE_FLAG_PET_MAX_HP
|
||||
data << (uint8) petpowertype; // GROUP_UPDATE_FLAG_PET_POWER_TYPE
|
||||
data << (uint16) pet->GetPower(petpowertype); // GROUP_UPDATE_FLAG_PET_CUR_POWER
|
||||
data << (uint16) pet->GetMaxPower(petpowertype); // GROUP_UPDATE_FLAG_PET_MAX_POWER
|
||||
data << uint16(pet->GetDisplayId()); // GROUP_UPDATE_FLAG_PET_MODEL_ID
|
||||
data << uint32(pet->GetHealth()); // GROUP_UPDATE_FLAG_PET_CUR_HP
|
||||
data << uint32(pet->GetMaxHealth()); // GROUP_UPDATE_FLAG_PET_MAX_HP
|
||||
data << uint8 (petpowertype); // GROUP_UPDATE_FLAG_PET_POWER_TYPE
|
||||
data << uint16(pet->GetPower(petpowertype)); // GROUP_UPDATE_FLAG_PET_CUR_POWER
|
||||
data << uint16(pet->GetMaxPower(petpowertype)); // GROUP_UPDATE_FLAG_PET_MAX_POWER
|
||||
|
||||
// GROUP_UPDATE_FLAG_PET_AURAS
|
||||
data << (uint8) 1;
|
||||
data << uint8(1);
|
||||
uint64 petauramask = 0;
|
||||
data << (uint64) petauramask; // placeholder
|
||||
data << uint64(petauramask); // placeholder
|
||||
size_t petMaskPos = data.wpos();
|
||||
data << (uint32) MAX_AURAS; // count
|
||||
data << uint32(MAX_AURAS); // count
|
||||
for (uint8 i = 0; i < MAX_AURAS; ++i)
|
||||
{
|
||||
if (AuraApplication const* aurApp = pet->GetVisibleAura(i))
|
||||
@@ -1242,15 +1247,10 @@ void WorldSession::HandleRequestPartyMemberStatsOpcode(WorldPacket& recvData)
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
data.put<uint64>(petMaskPos, petauramask); // GROUP_UPDATE_FLAG_PET_AURAS
|
||||
}
|
||||
else
|
||||
{
|
||||
data << (uint8) 0; // GROUP_UPDATE_FLAG_PET_NAME
|
||||
data << (uint8) 0; // GROUP_UPDATE_FLAG_PET_AURAS
|
||||
data << (uint64) 0; // GROUP_UPDATE_FLAG_PET_AURAS
|
||||
data << (uint32) 0; // GROUP_UPDATE_FLAG_PET_AURAS
|
||||
}
|
||||
// else not needed, flags do not include any PET_ update
|
||||
|
||||
// GROUP_UPDATE_FLAG_PHASE
|
||||
data << uint32(8); // either 0 or 8, same unk found in SMSG_PHASESHIFT
|
||||
@@ -1260,18 +1260,13 @@ void WorldSession::HandleRequestPartyMemberStatsOpcode(WorldPacket& recvData)
|
||||
SendPacket(&data);
|
||||
}
|
||||
|
||||
/*!*/void WorldSession::HandleRequestRaidInfoOpcode(WorldPacket & /*recvData*/)
|
||||
void WorldSession::HandleRequestRaidInfoOpcode(WorldPacket& /*recvData*/)
|
||||
{
|
||||
// every time the player checks the character screen
|
||||
_player->SendRaidInfo();
|
||||
}
|
||||
|
||||
/*void WorldSession::HandleGroupCancelOpcode(WorldPacket & recvData)
|
||||
{
|
||||
sLog->outDebug(LOG_FILTER_NETWORKIO, "WORLD: got CMSG_GROUP_CANCEL.");
|
||||
}*/
|
||||
|
||||
void WorldSession::HandleOptOutOfLootOpcode(WorldPacket & recvData)
|
||||
void WorldSession::HandleOptOutOfLootOpcode(WorldPacket& recvData)
|
||||
{
|
||||
sLog->outDebug(LOG_FILTER_NETWORKIO, "WORLD: Received CMSG_OPT_OUT_OF_LOOT");
|
||||
|
||||
|
||||
Reference in New Issue
Block a user