josh/TCPlayerbotCore
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Some random cleanup here and there

Browse Source
This commit is contained in:
Spp
2012-03-09 13:42:52 +01:00
parent a1a7a2d7c0
commit f1949b20a6
32 changed files with 219 additions and 224 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/server/game/Accounts/AccountMgr.h
+17 -17
View File
@@ -36,25 +36,25 @@ enum AccountOpResult
namespace AccountMgr namespace AccountMgr
{ {
AccountOpResult CreateAccount(std::string username, std::string password); AccountOpResult CreateAccount(std::string username, std::string password);
AccountOpResult DeleteAccount(uint32 accountId); AccountOpResult DeleteAccount(uint32 accountId);
AccountOpResult ChangeUsername(uint32 accountId, std::string newUsername, std::string newPassword); AccountOpResult ChangeUsername(uint32 accountId, std::string newUsername, std::string newPassword);
AccountOpResult ChangePassword(uint32 accountId, std::string newPassword); AccountOpResult ChangePassword(uint32 accountId, std::string newPassword);
bool CheckPassword(uint32 accountId, std::string password); bool CheckPassword(uint32 accountId, std::string password);
uint32 GetId(std::string username); uint32 GetId(std::string username);
uint32 GetSecurity(uint32 accountId); uint32 GetSecurity(uint32 accountId);
uint32 GetSecurity(uint64 accountId, int32 realmId); uint32 GetSecurity(uint64 accountId, int32 realmId);
bool GetName(uint32 accountId, std::string& name); bool GetName(uint32 accountId, std::string& name);
uint32 GetCharactersCount(uint32 accountId); uint32 GetCharactersCount(uint32 accountId);
std::string CalculateShaPassHash(std::string& name, std::string& password); std::string CalculateShaPassHash(std::string& name, std::string& password);
bool normalizeString(std::string& utf8String); bool normalizeString(std::string& utf8String);
bool IsPlayerAccount(uint32 gmlevel); bool IsPlayerAccount(uint32 gmlevel);
bool IsModeratorAccount(uint32 gmlevel); bool IsModeratorAccount(uint32 gmlevel);
bool IsGMAccount(uint32 gmlevel); bool IsGMAccount(uint32 gmlevel);
bool IsAdminAccount(uint32 gmlevel); bool IsAdminAccount(uint32 gmlevel);
bool IsConsoleAccount(uint32 gmlevel); bool IsConsoleAccount(uint32 gmlevel);
}; };
#endif #endif
src/server/game/Entities/Corpse/Corpse.h
-1
View File
@@ -91,4 +91,3 @@ class Corpse : public WorldObject, public GridObject<Corpse>
GridCoord _gridCoord; // gride for corpse position for fast search GridCoord _gridCoord; // gride for corpse position for fast search
}; };
#endif #endif
src/server/game/Entities/Creature/Creature.cpp
+15 -15
View File
@@ -1663,9 +1663,9 @@ bool Creature::IsImmunedToSpellEffect(SpellInfo const* spellInfo, uint32 index)
return Unit::IsImmunedToSpellEffect(spellInfo, index); return Unit::IsImmunedToSpellEffect(spellInfo, index);
} }
SpellInfo const* Creature::reachWithSpellAttack(Unit* pVictim) SpellInfo const* Creature::reachWithSpellAttack(Unit* victim)
{ {
if (!pVictim) if (!victim)
return NULL; return NULL;
for (uint32 i=0; i < CREATURE_MAX_SPELLS; ++i) for (uint32 i=0; i < CREATURE_MAX_SPELLS; ++i)
@@ -1698,7 +1698,7 @@ SpellInfo const* Creature::reachWithSpellAttack(Unit* pVictim)
continue; continue;
float range = spellInfo->GetMaxRange(false); float range = spellInfo->GetMaxRange(false);
float minrange = spellInfo->GetMinRange(false); float minrange = spellInfo->GetMinRange(false);
float dist = GetDistance(pVictim); float dist = GetDistance(victim);
if (dist > range || dist < minrange) if (dist > range || dist < minrange)
continue; continue;
if (spellInfo->PreventionType == SPELL_PREVENTION_TYPE_SILENCE && HasFlag(UNIT_FIELD_FLAGS, UNIT_FLAG_SILENCED)) if (spellInfo->PreventionType == SPELL_PREVENTION_TYPE_SILENCE && HasFlag(UNIT_FIELD_FLAGS, UNIT_FLAG_SILENCED))
@@ -1710,9 +1710,9 @@ SpellInfo const* Creature::reachWithSpellAttack(Unit* pVictim)
return NULL; return NULL;
} }
SpellInfo const* Creature::reachWithSpellCure(Unit* pVictim) SpellInfo const* Creature::reachWithSpellCure(Unit* victim)
{ {
if (!pVictim) if (!victim)
return NULL; return NULL;
for (uint32 i=0; i < CREATURE_MAX_SPELLS; ++i) for (uint32 i=0; i < CREATURE_MAX_SPELLS; ++i)
@@ -1742,8 +1742,8 @@ SpellInfo const* Creature::reachWithSpellCure(Unit* pVictim)
float range = spellInfo->GetMaxRange(true); float range = spellInfo->GetMaxRange(true);
float minrange = spellInfo->GetMinRange(true); float minrange = spellInfo->GetMinRange(true);
float dist = GetDistance(pVictim); float dist = GetDistance(victim);
//if (!isInFront(pVictim, range) && spellInfo->AttributesEx) //if (!isInFront(victim, range) && spellInfo->AttributesEx)
// continue; // continue;
if (dist > range || dist < minrange) if (dist > range || dist < minrange)
continue; continue;
@@ -1980,30 +1980,30 @@ void Creature::SaveRespawnTime()
} }
// this should not be called by petAI or // this should not be called by petAI or
bool Creature::canCreatureAttack(Unit const* pVictim, bool /*force*/) const bool Creature::canCreatureAttack(Unit const* victim, bool /*force*/) const
{ {
if (!pVictim->IsInMap(this)) if (!victim->IsInMap(this))
return false; return false;
if (!IsValidAttackTarget(pVictim)) if (!IsValidAttackTarget(victim))
return false; return false;
if (!pVictim->isInAccessiblePlaceFor(this)) if (!victim->isInAccessiblePlaceFor(this))
return false; return false;
if (IsAIEnabled && !AI()->CanAIAttack(pVictim)) if (IsAIEnabled && !AI()->CanAIAttack(victim))
return false; return false;
if (sMapStore.LookupEntry(GetMapId())->IsDungeon()) if (sMapStore.LookupEntry(GetMapId())->IsDungeon())
return true; return true;
//Use AttackDistance in distance check if threat radius is lower. This prevents creature bounce in and out of combat every update tick. //Use AttackDistance in distance check if threat radius is lower. This prevents creature bounce in and out of combat every update tick.
float dist = std::max(GetAttackDistance(pVictim), sWorld->getFloatConfig(CONFIG_THREAT_RADIUS)) + m_CombatDistance; float dist = std::max(GetAttackDistance(victim), sWorld->getFloatConfig(CONFIG_THREAT_RADIUS)) + m_CombatDistance;
if (Unit* unit = GetCharmerOrOwner()) if (Unit* unit = GetCharmerOrOwner())
return pVictim->IsWithinDist(unit, dist); return victim->IsWithinDist(unit, dist);
else else
return pVictim->IsInDist(&m_homePosition, dist); return victim->IsInDist(&m_homePosition, dist);
} }
CreatureAddon const* Creature::GetCreatureAddon() const CreatureAddon const* Creature::GetCreatureAddon() const
src/server/game/Entities/Creature/TemporarySummon.h
-1
View File
@@ -107,4 +107,3 @@ private:
TempSummon& m_owner; TempSummon& m_owner;
}; };
#endif #endif
src/server/game/Entities/DynamicObject/DynamicObject.cpp
+2 -2
View File
@@ -116,7 +116,7 @@ bool DynamicObject::CreateDynamicObject(uint32 guidlow, Unit* caster, uint32 spe
void DynamicObject::Update(uint32 p_time) void DynamicObject::Update(uint32 p_time)
{ {
// caster has to be always avalible and in the same map // caster has to be always available and in the same map
ASSERT(_caster); ASSERT(_caster);
ASSERT(_caster->GetMap() == GetMap()); ASSERT(_caster->GetMap() == GetMap());
@@ -127,7 +127,7 @@ void DynamicObject::Update(uint32 p_time)
if (!_aura->IsRemoved()) if (!_aura->IsRemoved())
_aura->UpdateOwner(p_time, this); _aura->UpdateOwner(p_time, this);
// m_aura may be set to null in Aura::UpdateOwner call // _aura may be set to null in Aura::UpdateOwner call
if (_aura && (_aura->IsRemoved() || _aura->IsExpired())) if (_aura && (_aura->IsRemoved() || _aura->IsExpired()))
expired = true; expired = true;
} }
src/server/game/Entities/GameObject/GameObject.cpp
-2
View File
@@ -223,7 +223,6 @@ bool GameObject::Create(uint32 guidlow, uint32 name_id, Map* map, uint32 phaseMa
SetGoArtKit(0); // unknown what this is SetGoArtKit(0); // unknown what this is
SetByteValue(GAMEOBJECT_BYTES_1, 2, artKit); SetByteValue(GAMEOBJECT_BYTES_1, 2, artKit);
switch (goinfo->type) switch (goinfo->type)
{ {
case GAMEOBJECT_TYPE_DESTRUCTIBLE_BUILDING: case GAMEOBJECT_TYPE_DESTRUCTIBLE_BUILDING:
@@ -252,7 +251,6 @@ bool GameObject::Create(uint32 guidlow, uint32 name_id, Map* map, uint32 phaseMa
m_invisibility.AddFlag(INVISIBILITY_TRAP); m_invisibility.AddFlag(INVISIBILITY_TRAP);
m_invisibility.AddValue(INVISIBILITY_TRAP, 300); m_invisibility.AddValue(INVISIBILITY_TRAP, 300);
} }
break; break;
default: default:
SetGoAnimProgress(animprogress); SetGoAnimProgress(animprogress);
src/server/game/Entities/GameObject/GameObject.h
-1
View File
@@ -402,7 +402,6 @@ struct GameObjectTemplate
} raw; } raw;
}; };
std::string AIName; std::string AIName;
uint32 ScriptId; uint32 ScriptId;
src/server/game/Entities/Player/Player.cpp
+38 -38
View File
@@ -11547,7 +11547,7 @@ InventoryResult Player::CanBankItem(uint8 bag, uint8 slot, ItemPosCountVec &dest
uint32 count = pItem->GetCount(); uint32 count = pItem->GetCount();
sLog->outDebug(LOG_FILTER_PLAYER_ITEMS, "STORAGE: CanBankItem bag = %u, slot = %u, item = %u, count = %u", bag, slot, pItem->GetEntry(), pItem->GetCount()); sLog->outDebug(LOG_FILTER_PLAYER_ITEMS, "STORAGE: CanBankItem bag = %u, slot = %u, item = %u, count = %u", bag, slot, pItem->GetEntry(), pItem->GetCount());
ItemTemplate const* pProto = pItem->GetTemplate(); ItemTemplate const* pProto = pItem->GetTemplate();
if (!pProto) if (!pProto)
return swap ? EQUIP_ERR_ITEMS_CANT_BE_SWAPPED : EQUIP_ERR_ITEM_NOT_FOUND; return swap ? EQUIP_ERR_ITEMS_CANT_BE_SWAPPED : EQUIP_ERR_ITEM_NOT_FOUND;
@@ -11731,7 +11731,7 @@ InventoryResult Player::CanUseItem(Item* pItem, bool not_loading) const
{ {
if (pItem) if (pItem)
{ {
sLog->outDebug(LOG_FILTER_PLAYER_ITEMS, "STORAGE: CanUseItem item = %u", pItem->GetEntry()); sLog->outDebug(LOG_FILTER_PLAYER_ITEMS, "STORAGE: CanUseItem item = %u", pItem->GetEntry());
if (!isAlive() && not_loading) if (!isAlive() && not_loading)
return EQUIP_ERR_YOU_ARE_DEAD; return EQUIP_ERR_YOU_ARE_DEAD;
@@ -11914,7 +11914,7 @@ InventoryResult Player::CanRollForItemInLFG(ItemTemplate const* proto, WorldObje
InventoryResult Player::CanUseAmmo(uint32 item) const InventoryResult Player::CanUseAmmo(uint32 item) const
{ {
sLog->outDebug(LOG_FILTER_PLAYER_ITEMS, "STORAGE: CanUseAmmo item = %u", item); sLog->outDebug(LOG_FILTER_PLAYER_ITEMS, "STORAGE: CanUseAmmo item = %u", item);
if (!isAlive()) if (!isAlive())
return EQUIP_ERR_YOU_ARE_DEAD; return EQUIP_ERR_YOU_ARE_DEAD;
//if (isStunned()) //if (isStunned())
@@ -12060,7 +12060,7 @@ Item* Player::_StoreItem(uint16 pos, Item* pItem, uint32 count, bool clone, bool
uint8 bag = pos >> 8; uint8 bag = pos >> 8;
uint8 slot = pos & 255; uint8 slot = pos & 255;
sLog->outDebug(LOG_FILTER_PLAYER_ITEMS, "STORAGE: StoreItem bag = %u, slot = %u, item = %u, count = %u, guid = %u", bag, slot, pItem->GetEntry(), count, pItem->GetGUIDLow()); sLog->outDebug(LOG_FILTER_PLAYER_ITEMS, "STORAGE: StoreItem bag = %u, slot = %u, item = %u, count = %u, guid = %u", bag, slot, pItem->GetEntry(), count, pItem->GetGUIDLow());
Item* pItem2 = GetItemByPos(bag, slot); Item* pItem2 = GetItemByPos(bag, slot);
@@ -12316,7 +12316,7 @@ void Player::VisualizeItem(uint8 slot, Item* pItem)
if (pItem->GetTemplate()->Bonding == BIND_WHEN_EQUIPED || pItem->GetTemplate()->Bonding == BIND_WHEN_PICKED_UP || pItem->GetTemplate()->Bonding == BIND_QUEST_ITEM) if (pItem->GetTemplate()->Bonding == BIND_WHEN_EQUIPED || pItem->GetTemplate()->Bonding == BIND_WHEN_PICKED_UP || pItem->GetTemplate()->Bonding == BIND_QUEST_ITEM)
pItem->SetBinding(true); pItem->SetBinding(true);
sLog->outDebug(LOG_FILTER_PLAYER_ITEMS, "STORAGE: EquipItem slot = %u, item = %u", slot, pItem->GetEntry()); sLog->outDebug(LOG_FILTER_PLAYER_ITEMS, "STORAGE: EquipItem slot = %u, item = %u", slot, pItem->GetEntry());
m_items[slot] = pItem; m_items[slot] = pItem;
SetUInt64Value(PLAYER_FIELD_INV_SLOT_HEAD + (slot * 2), pItem->GetGUID()); SetUInt64Value(PLAYER_FIELD_INV_SLOT_HEAD + (slot * 2), pItem->GetGUID());
@@ -12341,7 +12341,7 @@ void Player::RemoveItem(uint8 bag, uint8 slot, bool update)
Item* pItem = GetItemByPos(bag, slot); Item* pItem = GetItemByPos(bag, slot);
if (pItem) if (pItem)
{ {
sLog->outDebug(LOG_FILTER_PLAYER_ITEMS, "STORAGE: RemoveItem bag = %u, slot = %u, item = %u", bag, slot, pItem->GetEntry()); sLog->outDebug(LOG_FILTER_PLAYER_ITEMS, "STORAGE: RemoveItem bag = %u, slot = %u, item = %u", bag, slot, pItem->GetEntry());
RemoveEnchantmentDurations(pItem); RemoveEnchantmentDurations(pItem);
RemoveItemDurations(pItem); RemoveItemDurations(pItem);
@@ -12460,7 +12460,7 @@ void Player::DestroyItem(uint8 bag, uint8 slot, bool update)
Item* pItem = GetItemByPos(bag, slot); Item* pItem = GetItemByPos(bag, slot);
if (pItem) if (pItem)
{ {
sLog->outDebug(LOG_FILTER_PLAYER_ITEMS, "STORAGE: DestroyItem bag = %u, slot = %u, item = %u", bag, slot, pItem->GetEntry()); sLog->outDebug(LOG_FILTER_PLAYER_ITEMS, "STORAGE: DestroyItem bag = %u, slot = %u, item = %u", bag, slot, pItem->GetEntry());
// Also remove all contained items if the item is a bag. // Also remove all contained items if the item is a bag.
// This if () prevents item saving crashes if the condition for a bag to be empty before being destroyed was bypassed somehow. // This if () prevents item saving crashes if the condition for a bag to be empty before being destroyed was bypassed somehow.
if (pItem->IsNotEmptyBag()) if (pItem->IsNotEmptyBag())
@@ -12551,7 +12551,7 @@ void Player::DestroyItem(uint8 bag, uint8 slot, bool update)
void Player::DestroyItemCount(uint32 item, uint32 count, bool update, bool unequip_check) void Player::DestroyItemCount(uint32 item, uint32 count, bool update, bool unequip_check)
{ {
sLog->outDebug(LOG_FILTER_PLAYER_ITEMS, "STORAGE: DestroyItemCount item = %u, count = %u", item, count); sLog->outDebug(LOG_FILTER_PLAYER_ITEMS, "STORAGE: DestroyItemCount item = %u, count = %u", item, count);
uint32 remcount = 0; uint32 remcount = 0;
// in inventory // in inventory
@@ -12680,7 +12680,7 @@ void Player::DestroyItemCount(uint32 item, uint32 count, bool update, bool unequ
void Player::DestroyZoneLimitedItem(bool update, uint32 new_zone) void Player::DestroyZoneLimitedItem(bool update, uint32 new_zone)
{ {
sLog->outDebug(LOG_FILTER_PLAYER_ITEMS, "STORAGE: DestroyZoneLimitedItem in map %u and area %u", GetMapId(), new_zone); sLog->outDebug(LOG_FILTER_PLAYER_ITEMS, "STORAGE: DestroyZoneLimitedItem in map %u and area %u", GetMapId(), new_zone);
// in inventory // in inventory
for (uint8 i = INVENTORY_SLOT_ITEM_START; i < INVENTORY_SLOT_ITEM_END; i++) for (uint8 i = INVENTORY_SLOT_ITEM_START; i < INVENTORY_SLOT_ITEM_END; i++)
@@ -12712,7 +12712,7 @@ void Player::DestroyConjuredItems(bool update)
{ {
// used when entering arena // used when entering arena
// destroys all conjured items // destroys all conjured items
sLog->outDebug(LOG_FILTER_PLAYER_ITEMS, "STORAGE: DestroyConjuredItems"); sLog->outDebug(LOG_FILTER_PLAYER_ITEMS, "STORAGE: DestroyConjuredItems");
// in inventory // in inventory
for (uint8 i = INVENTORY_SLOT_ITEM_START; i < INVENTORY_SLOT_ITEM_END; i++) for (uint8 i = INVENTORY_SLOT_ITEM_START; i < INVENTORY_SLOT_ITEM_END; i++)
@@ -12763,7 +12763,7 @@ void Player::DestroyItemCount(Item* pItem, uint32 &count, bool update)
if (!pItem) if (!pItem)
return; return;
sLog->outDebug(LOG_FILTER_PLAYER_ITEMS, "STORAGE: DestroyItemCount item (GUID: %u, Entry: %u) count = %u", pItem->GetGUIDLow(), pItem->GetEntry(), count); sLog->outDebug(LOG_FILTER_PLAYER_ITEMS, "STORAGE: DestroyItemCount item (GUID: %u, Entry: %u) count = %u", pItem->GetGUIDLow(), pItem->GetEntry(), count);
if (pItem->GetCount() <= count) if (pItem->GetCount() <= count)
{ {
@@ -12826,7 +12826,7 @@ void Player::SplitItem(uint16 src, uint16 dst, uint32 count)
return; return;
} }
sLog->outDebug(LOG_FILTER_PLAYER_ITEMS, "STORAGE: SplitItem bag = %u, slot = %u, item = %u, count = %u", dstbag, dstslot, pSrcItem->GetEntry(), count); sLog->outDebug(LOG_FILTER_PLAYER_ITEMS, "STORAGE: SplitItem bag = %u, slot = %u, item = %u, count = %u", dstbag, dstslot, pSrcItem->GetEntry(), count);
Item* pNewItem = pSrcItem->CloneItem(count, this); Item* pNewItem = pSrcItem->CloneItem(count, this);
if (!pNewItem) if (!pNewItem)
{ {
@@ -12911,7 +12911,7 @@ void Player::SwapItem(uint16 src, uint16 dst)
if (!pSrcItem) if (!pSrcItem)
return; return;
sLog->outDebug(LOG_FILTER_PLAYER_ITEMS, "STORAGE: SwapItem bag = %u, slot = %u, item = %u", dstbag, dstslot, pSrcItem->GetEntry()); sLog->outDebug(LOG_FILTER_PLAYER_ITEMS, "STORAGE: SwapItem bag = %u, slot = %u, item = %u", dstbag, dstslot, pSrcItem->GetEntry());
if (!isAlive()) if (!isAlive())
{ {
@@ -12941,7 +12941,7 @@ void Player::SwapItem(uint16 src, uint16 dst)
} }
// prevent put equipped/bank bag in self // prevent put equipped/bank bag in self
if (IsBagPos (src) && srcslot == dstbag) if (IsBagPos(src) && srcslot == dstbag)
{ {
SendEquipError(EQUIP_ERR_NONEMPTY_BAG_OVER_OTHER_BAG, pSrcItem, pDstItem); SendEquipError(EQUIP_ERR_NONEMPTY_BAG_OVER_OTHER_BAG, pSrcItem, pDstItem);
return; return;
@@ -12999,7 +12999,7 @@ void Player::SwapItem(uint16 src, uint16 dst)
if (IsBankPos(src)) if (IsBankPos(src))
ItemAddedQuestCheck(pSrcItem->GetEntry(), pSrcItem->GetCount()); ItemAddedQuestCheck(pSrcItem->GetEntry(), pSrcItem->GetCount());
} }
else if (IsBankPos (dst)) else if (IsBankPos(dst))
{ {
ItemPosCountVec dest; ItemPosCountVec dest;
InventoryResult msg = CanBankItem(dstbag, dstslot, dest, pSrcItem, false); InventoryResult msg = CanBankItem(dstbag, dstslot, dest, pSrcItem, false);
@@ -13013,7 +13013,7 @@ void Player::SwapItem(uint16 src, uint16 dst)
BankItem(dest, pSrcItem, true); BankItem(dest, pSrcItem, true);
ItemRemovedQuestCheck(pSrcItem->GetEntry(), pSrcItem->GetCount()); ItemRemovedQuestCheck(pSrcItem->GetEntry(), pSrcItem->GetCount());
} }
else if (IsEquipmentPos (dst)) else if (IsEquipmentPos(dst))
{ {
uint16 dest; uint16 dest;
InventoryResult msg = CanEquipItem(dstslot, dest, pSrcItem, false); InventoryResult msg = CanEquipItem(dstslot, dest, pSrcItem, false);
@@ -13039,9 +13039,9 @@ void Player::SwapItem(uint16 src, uint16 dst)
uint16 eDest = 0; uint16 eDest = 0;
if (IsInventoryPos(dst)) if (IsInventoryPos(dst))
msg = CanStoreItem(dstbag, dstslot, sDest, pSrcItem, false); msg = CanStoreItem(dstbag, dstslot, sDest, pSrcItem, false);
else if (IsBankPos (dst)) else if (IsBankPos(dst))
msg = CanBankItem(dstbag, dstslot, sDest, pSrcItem, false); msg = CanBankItem(dstbag, dstslot, sDest, pSrcItem, false);
else if (IsEquipmentPos (dst)) else if (IsEquipmentPos(dst))
msg = CanEquipItem(dstslot, eDest, pSrcItem, false); msg = CanEquipItem(dstslot, eDest, pSrcItem, false);
else else
return; return;
@@ -13055,9 +13055,9 @@ void Player::SwapItem(uint16 src, uint16 dst)
if (IsInventoryPos(dst)) if (IsInventoryPos(dst))
StoreItem(sDest, pSrcItem, true); StoreItem(sDest, pSrcItem, true);
else if (IsBankPos (dst)) else if (IsBankPos(dst))
BankItem(sDest, pSrcItem, true); BankItem(sDest, pSrcItem, true);
else if (IsEquipmentPos (dst)) else if (IsEquipmentPos(dst))
{ {
EquipItem(eDest, pSrcItem, true); EquipItem(eDest, pSrcItem, true);
AutoUnequipOffhandIfNeed(); AutoUnequipOffhandIfNeed();
@@ -13287,7 +13287,7 @@ void Player::AddItemToBuyBackSlot(Item* pItem)
} }
RemoveItemFromBuyBackSlot(slot, true); RemoveItemFromBuyBackSlot(slot, true);
sLog->outDebug(LOG_FILTER_PLAYER_ITEMS, "STORAGE: AddItemToBuyBackSlot item = %u, slot = %u", pItem->GetEntry(), slot); sLog->outDebug(LOG_FILTER_PLAYER_ITEMS, "STORAGE: AddItemToBuyBackSlot item = %u, slot = %u", pItem->GetEntry(), slot);
m_items[slot] = pItem; m_items[slot] = pItem;
time_t base = time(NULL); time_t base = time(NULL);
@@ -13295,8 +13295,8 @@ void Player::AddItemToBuyBackSlot(Item* pItem)
uint32 eslot = slot - BUYBACK_SLOT_START; uint32 eslot = slot - BUYBACK_SLOT_START;
SetUInt64Value(PLAYER_FIELD_VENDORBUYBACK_SLOT_1 + (eslot * 2), pItem->GetGUID()); SetUInt64Value(PLAYER_FIELD_VENDORBUYBACK_SLOT_1 + (eslot * 2), pItem->GetGUID());
if (ItemTemplate const* pProto = pItem->GetTemplate()) if (ItemTemplate const* proto = pItem->GetTemplate())
SetUInt32Value(PLAYER_FIELD_BUYBACK_PRICE_1 + eslot, pProto->SellPrice * pItem->GetCount()); SetUInt32Value(PLAYER_FIELD_BUYBACK_PRICE_1 + eslot, proto->SellPrice * pItem->GetCount());
else else
SetUInt32Value(PLAYER_FIELD_BUYBACK_PRICE_1 + eslot, 0); SetUInt32Value(PLAYER_FIELD_BUYBACK_PRICE_1 + eslot, 0);
SetUInt32Value(PLAYER_FIELD_BUYBACK_TIMESTAMP_1 + eslot, (uint32)etime); SetUInt32Value(PLAYER_FIELD_BUYBACK_TIMESTAMP_1 + eslot, (uint32)etime);
@@ -13309,7 +13309,7 @@ void Player::AddItemToBuyBackSlot(Item* pItem)
Item* Player::GetItemFromBuyBackSlot(uint32 slot) Item* Player::GetItemFromBuyBackSlot(uint32 slot)
{ {
sLog->outDebug(LOG_FILTER_PLAYER_ITEMS, "STORAGE: GetItemFromBuyBackSlot slot = %u", slot); sLog->outDebug(LOG_FILTER_PLAYER_ITEMS, "STORAGE: GetItemFromBuyBackSlot slot = %u", slot);
if (slot >= BUYBACK_SLOT_START && slot < BUYBACK_SLOT_END) if (slot >= BUYBACK_SLOT_START && slot < BUYBACK_SLOT_END)
return m_items[slot]; return m_items[slot];
return NULL; return NULL;
@@ -13317,7 +13317,7 @@ Item* Player::GetItemFromBuyBackSlot(uint32 slot)
void Player::RemoveItemFromBuyBackSlot(uint32 slot, bool del) void Player::RemoveItemFromBuyBackSlot(uint32 slot, bool del)
{ {
sLog->outDebug(LOG_FILTER_PLAYER_ITEMS, "STORAGE: RemoveItemFromBuyBackSlot slot = %u", slot); sLog->outDebug(LOG_FILTER_PLAYER_ITEMS, "STORAGE: RemoveItemFromBuyBackSlot slot = %u", slot);
if (slot >= BUYBACK_SLOT_START && slot < BUYBACK_SLOT_END) if (slot >= BUYBACK_SLOT_START && slot < BUYBACK_SLOT_END)
{ {
Item* pItem = m_items[slot]; Item* pItem = m_items[slot];
@@ -14374,7 +14374,7 @@ void Player::OnGossipSelect(WorldObject* source, uint32 gossipListId, uint32 men
return; return;
} }
GetSession()->SendBattlegGroundList(guid, bgTypeId); GetSession()->SendBattleGroundList(guid, bgTypeId);
break; break;
} }
} }
@@ -14429,15 +14429,15 @@ uint32 Player::GetDefaultGossipMenuForSource(WorldObject* source)
void Player::PrepareQuestMenu(uint64 guid) void Player::PrepareQuestMenu(uint64 guid)
{ {
QuestRelationBounds pObjectQR; QuestRelationBounds objectQR;
QuestRelationBounds pObjectQIR; QuestRelationBounds objectQIR;
// pets also can have quests // pets also can have quests
Creature* creature = ObjectAccessor::GetCreatureOrPetOrVehicle(*this, guid); Creature* creature = ObjectAccessor::GetCreatureOrPetOrVehicle(*this, guid);
if (creature) if (creature)
{ {
pObjectQR = sObjectMgr->GetCreatureQuestRelationBounds(creature->GetEntry()); objectQR = sObjectMgr->GetCreatureQuestRelationBounds(creature->GetEntry());
pObjectQIR = sObjectMgr->GetCreatureQuestInvolvedRelationBounds(creature->GetEntry()); objectQIR = sObjectMgr->GetCreatureQuestInvolvedRelationBounds(creature->GetEntry());
} }
else else
{ {
@@ -14448,8 +14448,8 @@ void Player::PrepareQuestMenu(uint64 guid)
GameObject* pGameObject = _map->GetGameObject(guid); GameObject* pGameObject = _map->GetGameObject(guid);
if (pGameObject) if (pGameObject)
{ {
pObjectQR = sObjectMgr->GetGOQuestRelationBounds(pGameObject->GetEntry()); objectQR = sObjectMgr->GetGOQuestRelationBounds(pGameObject->GetEntry());
pObjectQIR = sObjectMgr->GetGOQuestInvolvedRelationBounds(pGameObject->GetEntry()); objectQIR = sObjectMgr->GetGOQuestInvolvedRelationBounds(pGameObject->GetEntry());
} }
else else
return; return;
@@ -14458,7 +14458,7 @@ void Player::PrepareQuestMenu(uint64 guid)
QuestMenu &qm = PlayerTalkClass->GetQuestMenu(); QuestMenu &qm = PlayerTalkClass->GetQuestMenu();
qm.ClearMenu(); qm.ClearMenu();
for (QuestRelations::const_iterator i = pObjectQIR.first; i != pObjectQIR.second; ++i) for (QuestRelations::const_iterator i = objectQIR.first; i != objectQIR.second; ++i)
{ {
uint32 quest_id = i->second; uint32 quest_id = i->second;
QuestStatus status = GetQuestStatus(quest_id); QuestStatus status = GetQuestStatus(quest_id);
@@ -14470,7 +14470,7 @@ void Player::PrepareQuestMenu(uint64 guid)
// qm.AddMenuItem(quest_id, 2); // qm.AddMenuItem(quest_id, 2);
} }
for (QuestRelations::const_iterator i = pObjectQR.first; i != pObjectQR.second; ++i) for (QuestRelations::const_iterator i = objectQR.first; i != objectQR.second; ++i)
{ {
uint32 quest_id = i->second; uint32 quest_id = i->second;
Quest const* quest = sObjectMgr->GetQuestTemplate(quest_id); Quest const* quest = sObjectMgr->GetQuestTemplate(quest_id);
@@ -14590,11 +14590,11 @@ bool Player::IsActiveQuest(uint32 quest_id) const
Quest const* Player::GetNextQuest(uint64 guid, Quest const* quest) Quest const* Player::GetNextQuest(uint64 guid, Quest const* quest)
{ {
QuestRelationBounds pObjectQR; QuestRelationBounds objectQR;
Creature* creature = ObjectAccessor::GetCreatureOrPetOrVehicle(*this, guid); Creature* creature = ObjectAccessor::GetCreatureOrPetOrVehicle(*this, guid);
if (creature) if (creature)
pObjectQR = sObjectMgr->GetCreatureQuestRelationBounds(creature->GetEntry()); objectQR = sObjectMgr->GetCreatureQuestRelationBounds(creature->GetEntry());
else else
{ {
//we should obtain map pointer from GetMap() in 99% of cases. Special case //we should obtain map pointer from GetMap() in 99% of cases. Special case
@@ -14603,13 +14603,13 @@ Quest const* Player::GetNextQuest(uint64 guid, Quest const* quest)
ASSERT(_map); ASSERT(_map);
GameObject* pGameObject = _map->GetGameObject(guid); GameObject* pGameObject = _map->GetGameObject(guid);
if (pGameObject) if (pGameObject)
pObjectQR = sObjectMgr->GetGOQuestRelationBounds(pGameObject->GetEntry()); objectQR = sObjectMgr->GetGOQuestRelationBounds(pGameObject->GetEntry());
else else
return NULL; return NULL;
} }
uint32 nextQuestID = quest->GetNextQuestInChain(); uint32 nextQuestID = quest->GetNextQuestInChain();
for (QuestRelations::const_iterator itr = pObjectQR.first; itr != pObjectQR.second; ++itr) for (QuestRelations::const_iterator itr = objectQR.first; itr != objectQR.second; ++itr)
{ {
if (itr->second == nextQuestID) if (itr->second == nextQuestID)
return sObjectMgr->GetQuestTemplate(nextQuestID); return sObjectMgr->GetQuestTemplate(nextQuestID);
src/server/game/Entities/Player/Player.h
+4 -4
View File
@@ -1543,7 +1543,7 @@ class Player : public Unit, public GridObject<Player>
void setRegenTimerCount(uint32 time) {m_regenTimerCount = time;} void setRegenTimerCount(uint32 time) {m_regenTimerCount = time;}
void setWeaponChangeTimer(uint32 time) {m_weaponChangeTimer = time;} void setWeaponChangeTimer(uint32 time) {m_weaponChangeTimer = time;}
uint32 GetMoney() const { return GetUInt32Value (PLAYER_FIELD_COINAGE); } uint32 GetMoney() const { return GetUInt32Value(PLAYER_FIELD_COINAGE); }
void ModifyMoney(int32 d); void ModifyMoney(int32 d);
bool HasEnoughMoney(uint32 amount) const { return (GetMoney() >= amount); } bool HasEnoughMoney(uint32 amount) const { return (GetMoney() >= amount); }
bool HasEnoughMoney(int32 amount) const bool HasEnoughMoney(int32 amount) const
@@ -1555,13 +1555,13 @@ class Player : public Unit, public GridObject<Player>
void SetMoney(uint32 value) void SetMoney(uint32 value)
{ {
SetUInt32Value (PLAYER_FIELD_COINAGE, value); SetUInt32Value(PLAYER_FIELD_COINAGE, value);
MoneyChanged(value); MoneyChanged(value);
UpdateAchievementCriteria(ACHIEVEMENT_CRITERIA_TYPE_HIGHEST_GOLD_VALUE_OWNED); UpdateAchievementCriteria(ACHIEVEMENT_CRITERIA_TYPE_HIGHEST_GOLD_VALUE_OWNED);
} }
RewardedQuestSet const& getRewardedQuests() const { return m_RewardedQuests; } RewardedQuestSet const& getRewardedQuests() const { return m_RewardedQuests; }
QuestStatusMap& getQuestStatusMap() { return m_QuestStatus; }; QuestStatusMap& getQuestStatusMap() { return m_QuestStatus; }
size_t GetRewardedQuestCount() const { return m_RewardedQuests.size(); } size_t GetRewardedQuestCount() const { return m_RewardedQuests.size(); }
bool IsQuestRewarded(uint32 quest_id) const bool IsQuestRewarded(uint32 quest_id) const
@@ -1574,7 +1574,7 @@ class Player : public Unit, public GridObject<Player>
Player* GetSelectedPlayer() const; Player* GetSelectedPlayer() const;
void SetSelection(uint64 guid) { m_curSelection = guid; SetUInt64Value(UNIT_FIELD_TARGET, guid); } void SetSelection(uint64 guid) { m_curSelection = guid; SetUInt64Value(UNIT_FIELD_TARGET, guid); }
uint8 GetComboPoints() { return m_comboPoints; } uint8 GetComboPoints() const { return m_comboPoints; }
uint64 GetComboTarget() const { return m_comboTarget; } uint64 GetComboTarget() const { return m_comboTarget; }
void AddComboPoints(Unit* target, int8 count, Spell* spell = NULL); void AddComboPoints(Unit* target, int8 count, Spell* spell = NULL);
src/server/game/Entities/Totem/Totem.h
-1
View File
@@ -60,4 +60,3 @@ class Totem : public Minion
uint32 m_duration; uint32 m_duration;
}; };
#endif #endif
src/server/game/Handlers/BattleGroundHandler.cpp
+2 -2
View File
@@ -60,10 +60,10 @@ void WorldSession::HandleBattlemasterHelloOpcode(WorldPacket & recv_data)
return; return;
} }
SendBattlegGroundList(guid, bgTypeId); SendBattleGroundList(guid, bgTypeId);
} }
void WorldSession::SendBattlegGroundList(uint64 guid, BattlegroundTypeId bgTypeId) void WorldSession::SendBattleGroundList(uint64 guid, BattlegroundTypeId bgTypeId)
{ {
WorldPacket data; WorldPacket data;
sBattlegroundMgr->BuildBattlegroundListPacket(&data, guid, _player, bgTypeId, 0); sBattlegroundMgr->BuildBattlegroundListPacket(&data, guid, _player, bgTypeId, 0);
src/server/game/Handlers/QuestHandler.cpp
+96 -101
View File
@@ -53,7 +53,7 @@ void WorldSession::HandleQuestgiverStatusQueryOpcode(WorldPacket & recv_data)
{ {
sLog->outDebug(LOG_FILTER_NETWORKIO, "WORLD: Received CMSG_QUESTGIVER_STATUS_QUERY for npc, guid = %u", uint32(GUID_LOPART(guid))); sLog->outDebug(LOG_FILTER_NETWORKIO, "WORLD: Received CMSG_QUESTGIVER_STATUS_QUERY for npc, guid = %u", uint32(GUID_LOPART(guid)));
Creature* cr_questgiver=questgiver->ToCreature(); Creature* cr_questgiver=questgiver->ToCreature();
if (!cr_questgiver->IsHostileTo(_player)) // not show quest status to enemies if (!cr_questgiver->IsHostileTo(_player)) // do not show quest status to enemies
{ {
questStatus = sScriptMgr->GetDialogStatus(_player, cr_questgiver); questStatus = sScriptMgr->GetDialogStatus(_player, cr_questgiver);
if (questStatus > 6) if (questStatus > 6)
@@ -112,17 +112,17 @@ void WorldSession::HandleQuestgiverHelloOpcode(WorldPacket & recv_data)
void WorldSession::HandleQuestgiverAcceptQuestOpcode(WorldPacket & recv_data) void WorldSession::HandleQuestgiverAcceptQuestOpcode(WorldPacket & recv_data)
{ {
uint64 guid; uint64 guid;
uint32 quest; uint32 questId;
uint32 unk1; uint32 unk1;
recv_data >> guid >> quest >> unk1; recv_data >> guid >> questId >> unk1;
sLog->outDebug(LOG_FILTER_NETWORKIO, "WORLD: Received CMSG_QUESTGIVER_ACCEPT_QUEST npc = %u, quest = %u, unk1 = %u", uint32(GUID_LOPART(guid)), quest, unk1); sLog->outDebug(LOG_FILTER_NETWORKIO, "WORLD: Received CMSG_QUESTGIVER_ACCEPT_QUEST npc = %u, quest = %u, unk1 = %u", uint32(GUID_LOPART(guid)), questId, unk1);
Object* pObject = ObjectAccessor::GetObjectByTypeMask(*_player, guid, TYPEMASK_UNIT|TYPEMASK_GAMEOBJECT|TYPEMASK_ITEM|TYPEMASK_PLAYER); Object* object = ObjectAccessor::GetObjectByTypeMask(*_player, guid, TYPEMASK_UNIT|TYPEMASK_GAMEOBJECT|TYPEMASK_ITEM|TYPEMASK_PLAYER);
// no or incorrect quest giver // no or incorrect quest giver
if (!pObject || (pObject->GetTypeId() != TYPEID_PLAYER && !pObject->hasQuest(quest)) || if (!object || (object->GetTypeId() != TYPEID_PLAYER && !object->hasQuest(questId)) ||
(pObject->GetTypeId() == TYPEID_PLAYER && !pObject->ToPlayer()->CanShareQuest(quest))) (object->GetTypeId() == TYPEID_PLAYER && !object->ToPlayer()->CanShareQuest(questId)))
{ {
_player->PlayerTalkClass->SendCloseGossip(); _player->PlayerTalkClass->SendCloseGossip();
_player->SetDivider(0); _player->SetDivider(0);
@@ -130,14 +130,13 @@ void WorldSession::HandleQuestgiverAcceptQuestOpcode(WorldPacket & recv_data)
} }
// some kind of WPE protection // some kind of WPE protection
if (!_player->CanInteractWithQuestGiver(pObject)) if (!_player->CanInteractWithQuestGiver(object))
return; return;
Quest const* qInfo = sObjectMgr->GetQuestTemplate(quest); if (Quest const* quest = sObjectMgr->GetQuestTemplate(questId))
if (qInfo)
{ {
// prevent cheating // prevent cheating
if (!GetPlayer()->CanTakeQuest(qInfo, true)) if (!GetPlayer()->CanTakeQuest(quest, true))
{ {
_player->PlayerTalkClass->SendCloseGossip(); _player->PlayerTalkClass->SendCloseGossip();
_player->SetDivider(0); _player->SetDivider(0);
@@ -154,11 +153,11 @@ void WorldSession::HandleQuestgiverAcceptQuestOpcode(WorldPacket & recv_data)
} }
} }
if (_player->CanAddQuest(qInfo, true)) if (_player->CanAddQuest(quest, true))
{ {
_player->AddQuest(qInfo, pObject); _player->AddQuest(quest, object);
if (qInfo->HasFlag(QUEST_FLAGS_PARTY_ACCEPT)) if (quest->HasFlag(QUEST_FLAGS_PARTY_ACCEPT))
{ {
if (Group* group = _player->GetGroup()) if (Group* group = _player->GetGroup())
{ {
@@ -169,38 +168,38 @@ void WorldSession::HandleQuestgiverAcceptQuestOpcode(WorldPacket & recv_data)
if (!player || player == _player) // not self if (!player || player == _player) // not self
continue; continue;
if (player->CanTakeQuest(qInfo, true)) if (player->CanTakeQuest(quest, true))
{ {
player->SetDivider(_player->GetGUID()); player->SetDivider(_player->GetGUID());
//need confirmation that any gossip window will close //need confirmation that any gossip window will close
player->PlayerTalkClass->SendCloseGossip(); player->PlayerTalkClass->SendCloseGossip();
_player->SendQuestConfirmAccept(qInfo, player); _player->SendQuestConfirmAccept(quest, player);
} }
} }
} }
} }
if (_player->CanCompleteQuest(quest)) if (_player->CanCompleteQuest(questId))
_player->CompleteQuest(quest); _player->CompleteQuest(questId);
switch (pObject->GetTypeId()) switch (object->GetTypeId())
{ {
case TYPEID_UNIT: case TYPEID_UNIT:
sScriptMgr->OnQuestAccept(_player, (pObject->ToCreature()), qInfo); sScriptMgr->OnQuestAccept(_player, (object->ToCreature()), quest);
(pObject->ToCreature())->AI()->sQuestAccept(_player, qInfo); (object->ToCreature())->AI()->sQuestAccept(_player, quest);
break; break;
case TYPEID_ITEM: case TYPEID_ITEM:
case TYPEID_CONTAINER: case TYPEID_CONTAINER:
{ {
sScriptMgr->OnQuestAccept(_player, ((Item*)pObject), qInfo); sScriptMgr->OnQuestAccept(_player, ((Item*)object), quest);
// destroy not required for quest finish quest starting item // destroy not required for quest finish quest starting item
bool destroyItem = true; bool destroyItem = true;
for (int i = 0; i < QUEST_ITEM_OBJECTIVES_COUNT; ++i) for (int i = 0; i < QUEST_ITEM_OBJECTIVES_COUNT; ++i)
{ {
if ((qInfo->RequiredItemId[i] == ((Item*)pObject)->GetEntry()) && (((Item*)pObject)->GetTemplate()->MaxCount > 0)) if ((quest->RequiredItemId[i] == ((Item*)object)->GetEntry()) && (((Item*)object)->GetTemplate()->MaxCount > 0))
{ {
destroyItem = false; destroyItem = false;
break; break;
@@ -208,21 +207,21 @@ void WorldSession::HandleQuestgiverAcceptQuestOpcode(WorldPacket & recv_data)
} }
if (destroyItem) if (destroyItem)
_player->DestroyItem(((Item*)pObject)->GetBagSlot(), ((Item*)pObject)->GetSlot(), true); _player->DestroyItem(((Item*)object)->GetBagSlot(), ((Item*)object)->GetSlot(), true);
break; break;
} }
case TYPEID_GAMEOBJECT: case TYPEID_GAMEOBJECT:
sScriptMgr->OnQuestAccept(_player, ((GameObject*)pObject), qInfo); sScriptMgr->OnQuestAccept(_player, ((GameObject*)object), quest);
(pObject->ToGameObject())->AI()->QuestAccept(_player, qInfo); (object->ToGameObject())->AI()->QuestAccept(_player, quest);
break; break;
default: default:
break; break;
} }
_player->PlayerTalkClass->SendCloseGossip(); _player->PlayerTalkClass->SendCloseGossip();
if (qInfo->GetSrcSpell() > 0) if (quest->GetSrcSpell() > 0)
_player->CastSpell(_player, qInfo->GetSrcSpell(), true); _player->CastSpell(_player, quest->GetSrcSpell(), true);
return; return;
} }
@@ -247,8 +246,7 @@ void WorldSession::HandleQuestgiverQueryQuestOpcode(WorldPacket & recv_data)
return; return;
} }
Quest const* quest = sObjectMgr->GetQuestTemplate(questId); if (Quest const* quest = sObjectMgr->GetQuestTemplate(questId))
if (quest)
{ {
// not sure here what should happen to quests with QUEST_FLAGS_AUTOCOMPLETE // not sure here what should happen to quests with QUEST_FLAGS_AUTOCOMPLETE
// if this breaks them, add && object->GetTypeId() == TYPEID_ITEM to this check // if this breaks them, add && object->GetTypeId() == TYPEID_ITEM to this check
@@ -275,15 +273,12 @@ void WorldSession::HandleQuestQueryOpcode(WorldPacket & recv_data)
if (!_player) if (!_player)
return; return;
uint32 quest; uint32 questId;
recv_data >> quest; recv_data >> questId;
sLog->outDebug(LOG_FILTER_NETWORKIO, "WORLD: Received CMSG_QUEST_QUERY quest = %u", quest); sLog->outDebug(LOG_FILTER_NETWORKIO, "WORLD: Received CMSG_QUEST_QUERY quest = %u", questId);
Quest const* pQuest = sObjectMgr->GetQuestTemplate(quest); if (Quest const* quest = sObjectMgr->GetQuestTemplate(questId))
if (pQuest) _player->PlayerTalkClass->SendQuestQueryResponse(quest);
{
_player->PlayerTalkClass->SendQuestQueryResponse(pQuest);
}
} }
void WorldSession::HandleQuestgiverChooseRewardOpcode(WorldPacket & recv_data) void WorldSession::HandleQuestgiverChooseRewardOpcode(WorldPacket & recv_data)
@@ -372,28 +367,28 @@ void WorldSession::HandleQuestgiverChooseRewardOpcode(WorldPacket & recv_data)
void WorldSession::HandleQuestgiverRequestRewardOpcode(WorldPacket & recv_data) void WorldSession::HandleQuestgiverRequestRewardOpcode(WorldPacket & recv_data)
{ {
uint32 quest; uint32 questId;
uint64 guid; uint64 guid;
recv_data >> guid >> quest; recv_data >> guid >> questId;
sLog->outDebug(LOG_FILTER_NETWORKIO, "WORLD: Received CMSG_QUESTGIVER_REQUEST_REWARD npc = %u, quest = %u", uint32(GUID_LOPART(guid)), quest); sLog->outDebug(LOG_FILTER_NETWORKIO, "WORLD: Received CMSG_QUESTGIVER_REQUEST_REWARD npc = %u, quest = %u", uint32(GUID_LOPART(guid)), questId);
Object* pObject = ObjectAccessor::GetObjectByTypeMask(*_player, guid, TYPEMASK_UNIT|TYPEMASK_GAMEOBJECT); Object* object = ObjectAccessor::GetObjectByTypeMask(*_player, guid, TYPEMASK_UNIT|TYPEMASK_GAMEOBJECT);
if (!pObject || !pObject->hasInvolvedQuest(quest)) if (!object || !object->hasInvolvedQuest(questId))
return; return;
// some kind of WPE protection // some kind of WPE protection
if (!_player->CanInteractWithQuestGiver(pObject)) if (!_player->CanInteractWithQuestGiver(object))
return; return;
if (_player->CanCompleteQuest(quest)) if (_player->CanCompleteQuest(questId))
_player->CompleteQuest(quest); _player->CompleteQuest(questId);
if (_player->GetQuestStatus(quest) != QUEST_STATUS_COMPLETE) if (_player->GetQuestStatus(questId) != QUEST_STATUS_COMPLETE)
return; return;
if (Quest const* pQuest = sObjectMgr->GetQuestTemplate(quest)) if (Quest const* quest = sObjectMgr->GetQuestTemplate(questId))
_player->PlayerTalkClass->SendQuestGiverOfferReward(pQuest, guid, true); _player->PlayerTalkClass->SendQuestGiverOfferReward(quest, guid, true);
} }
void WorldSession::HandleQuestgiverCancel(WorldPacket& /*recv_data*/) void WorldSession::HandleQuestgiverCancel(WorldPacket& /*recv_data*/)
@@ -425,22 +420,22 @@ void WorldSession::HandleQuestLogRemoveQuest(WorldPacket& recv_data)
if (slot < MAX_QUEST_LOG_SIZE) if (slot < MAX_QUEST_LOG_SIZE)
{ {
if (uint32 quest = _player->GetQuestSlotQuestId(slot)) if (uint32 questId = _player->GetQuestSlotQuestId(slot))
{ {
if (!_player->TakeQuestSourceItem(quest, true)) if (!_player->TakeQuestSourceItem(questId, true))
return; // can't un-equip some items, reject quest cancel return; // can't un-equip some items, reject quest cancel
if (const Quest *pQuest = sObjectMgr->GetQuestTemplate(quest)) if (const Quest *quest = sObjectMgr->GetQuestTemplate(questId))
{ {
if (pQuest->HasFlag(QUEST_TRINITY_FLAGS_TIMED)) if (quest->HasFlag(QUEST_TRINITY_FLAGS_TIMED))
_player->RemoveTimedQuest(quest); _player->RemoveTimedQuest(questId);
} }
_player->TakeQuestSourceItem(quest, true); // remove quest src item from player _player->TakeQuestSourceItem(questId, true); // remove quest src item from player
_player->RemoveActiveQuest(quest); _player->RemoveActiveQuest(questId);
_player->GetAchievementMgr().RemoveTimedAchievement(ACHIEVEMENT_TIMED_TYPE_QUEST, quest); _player->GetAchievementMgr().RemoveTimedAchievement(ACHIEVEMENT_TIMED_TYPE_QUEST, questId);
sLog->outDetail("Player %u abandoned quest %u", _player->GetGUIDLow(), quest); sLog->outDetail("Player %u abandoned quest %u", _player->GetGUIDLow(), questId);
} }
_player->SetQuestSlot(slot, 0); _player->SetQuestSlot(slot, 0);
@@ -451,14 +446,14 @@ void WorldSession::HandleQuestLogRemoveQuest(WorldPacket& recv_data)
void WorldSession::HandleQuestConfirmAccept(WorldPacket& recv_data) void WorldSession::HandleQuestConfirmAccept(WorldPacket& recv_data)
{ {
uint32 quest; uint32 questId;
recv_data >> quest; recv_data >> questId;
sLog->outDebug(LOG_FILTER_NETWORKIO, "WORLD: Received CMSG_QUEST_CONFIRM_ACCEPT quest = %u", quest); sLog->outDebug(LOG_FILTER_NETWORKIO, "WORLD: Received CMSG_QUEST_CONFIRM_ACCEPT quest = %u", questId);
if (const Quest* pQuest = sObjectMgr->GetQuestTemplate(quest)) if (const Quest* quest = sObjectMgr->GetQuestTemplate(questId))
{ {
if (!pQuest->HasFlag(QUEST_FLAGS_PARTY_ACCEPT)) if (!quest->HasFlag(QUEST_FLAGS_PARTY_ACCEPT))
return; return;
Player* pOriginalPlayer = ObjectAccessor::FindPlayer(_player->GetDivider()); Player* pOriginalPlayer = ObjectAccessor::FindPlayer(_player->GetDivider());
@@ -466,7 +461,7 @@ void WorldSession::HandleQuestConfirmAccept(WorldPacket& recv_data)
if (!pOriginalPlayer) if (!pOriginalPlayer)
return; return;
if (pQuest->IsRaidQuest()) if (quest->IsRaidQuest())
{ {
if (!_player->IsInSameRaidWith(pOriginalPlayer)) if (!_player->IsInSameRaidWith(pOriginalPlayer))
return; return;
@@ -477,8 +472,8 @@ void WorldSession::HandleQuestConfirmAccept(WorldPacket& recv_data)
return; return;
} }
if (_player->CanAddQuest(pQuest, true)) if (_player->CanAddQuest(quest, true))
_player->AddQuest(pQuest, NULL); // NULL, this prevent DB script from duplicate running _player->AddQuest(quest, NULL); // NULL, this prevent DB script from duplicate running
_player->SetDivider(0); _player->SetDivider(0);
} }
@@ -486,48 +481,48 @@ void WorldSession::HandleQuestConfirmAccept(WorldPacket& recv_data)
void WorldSession::HandleQuestgiverCompleteQuest(WorldPacket& recv_data) void WorldSession::HandleQuestgiverCompleteQuest(WorldPacket& recv_data)
{ {
uint32 quest; uint32 questId;
uint64 guid; uint64 guid;
recv_data >> guid >> quest;
sLog->outDebug(LOG_FILTER_NETWORKIO, "WORLD: Received CMSG_QUESTGIVER_COMPLETE_QUEST npc = %u, quest = %u", uint32(GUID_LOPART(guid)), quest); recv_data >> guid >> questId;
Object* pObject = ObjectAccessor::GetObjectByTypeMask(*_player, guid, TYPEMASK_UNIT|TYPEMASK_GAMEOBJECT); sLog->outDebug(LOG_FILTER_NETWORKIO, "WORLD: Received CMSG_QUESTGIVER_COMPLETE_QUEST npc = %u, quest = %u", uint32(GUID_LOPART(guid)), questId);
if (!pObject || !pObject->hasInvolvedQuest(quest))
Object* object = ObjectAccessor::GetObjectByTypeMask(*_player, guid, TYPEMASK_UNIT|TYPEMASK_GAMEOBJECT);
if (!object || !object->hasInvolvedQuest(questId))
return; return;
// some kind of WPE protection // some kind of WPE protection
if (!_player->CanInteractWithQuestGiver(pObject)) if (!_player->CanInteractWithQuestGiver(object))
return; return;
Quest const* pQuest = sObjectMgr->GetQuestTemplate(quest); if (Quest const* quest = sObjectMgr->GetQuestTemplate(questId))
if (pQuest)
{ {
if (!_player->CanSeeStartQuest(pQuest) && _player->GetQuestStatus(quest)==QUEST_STATUS_NONE) if (!_player->CanSeeStartQuest(quest) && _player->GetQuestStatus(questId) == QUEST_STATUS_NONE)
{ {
sLog->outError("Possible hacking attempt: Player %s [guid: %u] tried to complete quest [entry: %u] without being in possession of the quest!", sLog->outError("Possible hacking attempt: Player %s [guid: %u] tried to complete quest [entry: %u] without being in possession of the quest!",
_player->GetName(), _player->GetGUIDLow(), quest); _player->GetName(), _player->GetGUIDLow(), questId);
return; return;
} }
// TODO: need a virtual function // TODO: need a virtual function
if (_player->InBattleground()) if (_player->InBattleground())
if (Battleground* bg = _player->GetBattleground()) if (Battleground* bg = _player->GetBattleground())
if (bg->GetTypeID() == BATTLEGROUND_AV) if (bg->GetTypeID() == BATTLEGROUND_AV)
((BattlegroundAV*)bg)->HandleQuestComplete(quest, _player); ((BattlegroundAV*)bg)->HandleQuestComplete(questId, _player);
if (_player->GetQuestStatus(quest) != QUEST_STATUS_COMPLETE) if (_player->GetQuestStatus(questId) != QUEST_STATUS_COMPLETE)
{ {
if (pQuest->IsRepeatable()) if (quest->IsRepeatable())
_player->PlayerTalkClass->SendQuestGiverRequestItems(pQuest, guid, _player->CanCompleteRepeatableQuest(pQuest), false); _player->PlayerTalkClass->SendQuestGiverRequestItems(quest, guid, _player->CanCompleteRepeatableQuest(quest), false);
else else
_player->PlayerTalkClass->SendQuestGiverRequestItems(pQuest, guid, _player->CanRewardQuest(pQuest, false), false); _player->PlayerTalkClass->SendQuestGiverRequestItems(quest, guid, _player->CanRewardQuest(quest, false), false);
} }
else else
{ {
if (pQuest->GetReqItemsCount()) // some items required if (quest->GetReqItemsCount()) // some items required
_player->PlayerTalkClass->SendQuestGiverRequestItems(pQuest, guid, _player->CanRewardQuest(pQuest, false), false); _player->PlayerTalkClass->SendQuestGiverRequestItems(quest, guid, _player->CanRewardQuest(quest, false), false);
else // no items required else // no items required
_player->PlayerTalkClass->SendQuestGiverOfferReward(pQuest, guid, true); _player->PlayerTalkClass->SendQuestGiverOfferReward(quest, guid, true);
} }
} }
} }
@@ -544,7 +539,7 @@ void WorldSession::HandlePushQuestToParty(WorldPacket& recvPacket)
sLog->outDebug(LOG_FILTER_NETWORKIO, "WORLD: Received CMSG_PUSHQUESTTOPARTY quest = %u", questId); sLog->outDebug(LOG_FILTER_NETWORKIO, "WORLD: Received CMSG_PUSHQUESTTOPARTY quest = %u", questId);
if (Quest const* pQuest = sObjectMgr->GetQuestTemplate(questId)) if (Quest const* quest = sObjectMgr->GetQuestTemplate(questId))
{ {
if (Group* group = _player->GetGroup()) if (Group* group = _player->GetGroup())
{ {
@@ -557,7 +552,7 @@ void WorldSession::HandlePushQuestToParty(WorldPacket& recvPacket)
_player->SendPushToPartyResponse(player, QUEST_PARTY_MSG_SHARING_QUEST); _player->SendPushToPartyResponse(player, QUEST_PARTY_MSG_SHARING_QUEST);
if (!player->SatisfyQuestStatus(pQuest, false)) if (!player->SatisfyQuestStatus(quest, false))
{ {
_player->SendPushToPartyResponse(player, QUEST_PARTY_MSG_HAVE_QUEST); _player->SendPushToPartyResponse(player, QUEST_PARTY_MSG_HAVE_QUEST);
continue; continue;
@@ -569,7 +564,7 @@ void WorldSession::HandlePushQuestToParty(WorldPacket& recvPacket)
continue; continue;
} }
if (!player->CanTakeQuest(pQuest, false)) if (!player->CanTakeQuest(quest, false))
{ {
_player->SendPushToPartyResponse(player, QUEST_PARTY_MSG_CANT_TAKE_QUEST); _player->SendPushToPartyResponse(player, QUEST_PARTY_MSG_CANT_TAKE_QUEST);
continue; continue;
@@ -587,7 +582,7 @@ void WorldSession::HandlePushQuestToParty(WorldPacket& recvPacket)
continue; continue;
} }
player->PlayerTalkClass->SendQuestGiverQuestDetails(pQuest, _player->GetGUID(), true); player->PlayerTalkClass->SendQuestGiverQuestDetails(quest, _player->GetGUID(), true);
player->SetDivider(_player->GetGUID()); player->SetDivider(_player->GetGUID());
} }
} }
@@ -647,18 +642,18 @@ uint32 WorldSession::getDialogStatus(Player* player, Object* questgiver, uint32
{ {
uint32 result2 = 0; uint32 result2 = 0;
uint32 quest_id = i->second; uint32 quest_id = i->second;
Quest const* pQuest = sObjectMgr->GetQuestTemplate(quest_id); Quest const* quest = sObjectMgr->GetQuestTemplate(quest_id);
if (!pQuest) continue; if (!quest) continue;
ConditionList conditions = sConditionMgr->GetConditionsForNotGroupedEntry(CONDITION_SOURCE_TYPE_QUEST_SHOW_MARK, pQuest->GetQuestId()); ConditionList conditions = sConditionMgr->GetConditionsForNotGroupedEntry(CONDITION_SOURCE_TYPE_QUEST_SHOW_MARK, quest->GetQuestId());
if (!sConditionMgr->IsObjectMeetToConditions(player, conditions)) if (!sConditionMgr->IsObjectMeetToConditions(player, conditions))
continue; continue;
QuestStatus status = player->GetQuestStatus(quest_id); QuestStatus status = player->GetQuestStatus(quest_id);
if ((status == QUEST_STATUS_COMPLETE && !player->GetQuestRewardStatus(quest_id)) || if ((status == QUEST_STATUS_COMPLETE && !player->GetQuestRewardStatus(quest_id)) ||
(pQuest->IsAutoComplete() && player->CanTakeQuest(pQuest, false))) (quest->IsAutoComplete() && player->CanTakeQuest(quest, false)))
{ {
if (pQuest->IsAutoComplete() && pQuest->IsRepeatable()) if (quest->IsAutoComplete() && quest->IsRepeatable())
result2 = DIALOG_STATUS_REWARD_REP; result2 = DIALOG_STATUS_REWARD_REP;
else else
result2 = DIALOG_STATUS_REWARD; result2 = DIALOG_STATUS_REWARD;
@@ -674,26 +669,26 @@ uint32 WorldSession::getDialogStatus(Player* player, Object* questgiver, uint32
{ {
uint32 result2 = 0; uint32 result2 = 0;
uint32 quest_id = i->second; uint32 quest_id = i->second;
Quest const* pQuest = sObjectMgr->GetQuestTemplate(quest_id); Quest const* quest = sObjectMgr->GetQuestTemplate(quest_id);
if (!pQuest) if (!quest)
continue; continue;
ConditionList conditions = sConditionMgr->GetConditionsForNotGroupedEntry(CONDITION_SOURCE_TYPE_QUEST_SHOW_MARK, pQuest->GetQuestId()); ConditionList conditions = sConditionMgr->GetConditionsForNotGroupedEntry(CONDITION_SOURCE_TYPE_QUEST_SHOW_MARK, quest->GetQuestId());
if (!sConditionMgr->IsObjectMeetToConditions(player, conditions)) if (!sConditionMgr->IsObjectMeetToConditions(player, conditions))
continue; continue;
QuestStatus status = player->GetQuestStatus(quest_id); QuestStatus status = player->GetQuestStatus(quest_id);
if (status == QUEST_STATUS_NONE) if (status == QUEST_STATUS_NONE)
{ {
if (player->CanSeeStartQuest(pQuest)) if (player->CanSeeStartQuest(quest))
{ {
if (player->SatisfyQuestLevel(pQuest, false)) if (player->SatisfyQuestLevel(quest, false))
{ {
if (pQuest->IsAutoComplete() || (pQuest->IsRepeatable() && player->IsQuestRewarded(quest_id))) if (quest->IsAutoComplete() || (quest->IsRepeatable() && player->IsQuestRewarded(quest_id)))
result2 = DIALOG_STATUS_REWARD_REP; result2 = DIALOG_STATUS_REWARD_REP;
else if (player->getLevel() <= ((player->GetQuestLevel(pQuest) == -1) ? player->getLevel() : player->GetQuestLevel(pQuest) + sWorld->getIntConfig(CONFIG_QUEST_LOW_LEVEL_HIDE_DIFF))) else if (player->getLevel() <= ((player->GetQuestLevel(quest) == -1) ? player->getLevel() : player->GetQuestLevel(quest) + sWorld->getIntConfig(CONFIG_QUEST_LOW_LEVEL_HIDE_DIFF)))
{ {
if (pQuest->HasFlag(QUEST_FLAGS_DAILY) || pQuest->HasFlag(QUEST_FLAGS_WEEKLY)) if (quest->HasFlag(QUEST_FLAGS_DAILY) || quest->HasFlag(QUEST_FLAGS_WEEKLY))
result2 = DIALOG_STATUS_AVAILABLE_REP; result2 = DIALOG_STATUS_AVAILABLE_REP;
else else
result2 = DIALOG_STATUS_AVAILABLE; result2 = DIALOG_STATUS_AVAILABLE;
src/server/game/Handlers/SkillHandler.cpp
-1
View File
@@ -90,4 +90,3 @@ void WorldSession::HandleUnlearnSkillOpcode(WorldPacket & recv_data)
recv_data >> skill_id; recv_data >> skill_id;
GetPlayer()->SetSkill(skill_id, 0, 0, 0); GetPlayer()->SetSkill(skill_id, 0, 0, 0);
} }
src/server/game/Handlers/TaxiHandler.cpp
+1 -1
View File
@@ -103,7 +103,7 @@ void WorldSession::SendTaxiMenu(Creature* unit)
sLog->outDebug(LOG_FILTER_NETWORKIO, "WORLD: CMSG_TAXINODE_STATUS_QUERY %u ", curloc); sLog->outDebug(LOG_FILTER_NETWORKIO, "WORLD: CMSG_TAXINODE_STATUS_QUERY %u ", curloc);
WorldPacket data(SMSG_SHOWTAXINODES, (4+8+4+8*4)); WorldPacket data(SMSG_SHOWTAXINODES, (4 + 8 + 4 + 8 * 4));
data << uint32(1); data << uint32(1);
data << uint64(unit->GetGUID()); data << uint64(unit->GetGUID());
data << uint32(curloc); data << uint32(curloc);
src/server/game/Handlers/TradeHandler.cpp
+3 -7
View File
@@ -546,15 +546,12 @@ void WorldSession::HandleCancelTradeOpcode(WorldPacket& /*recvPacket*/)
void WorldSession::HandleInitiateTradeOpcode(WorldPacket& recvPacket) void WorldSession::HandleInitiateTradeOpcode(WorldPacket& recvPacket)
{ {
if (GetPlayer()->m_trade)
{
recvPacket.rfinish();
return;
}
uint64 ID; uint64 ID;
recvPacket >> ID; recvPacket >> ID;
if (GetPlayer()->m_trade)
return;
if (!GetPlayer()->isAlive()) if (!GetPlayer()->isAlive())
{ {
SendTradeStatus(TRADE_STATUS_YOU_DEAD); SendTradeStatus(TRADE_STATUS_YOU_DEAD);
@@ -728,4 +725,3 @@ void WorldSession::HandleClearTradeItemOpcode(WorldPacket& recvPacket)
my_trade->SetItem(TradeSlots(tradeSlot), NULL); my_trade->SetItem(TradeSlots(tradeSlot), NULL);
} }
src/server/game/Instances/InstanceSaveMgr.cpp
+1 -1
View File
@@ -639,4 +639,4 @@ uint32 InstanceSaveManager::GetNumBoundGroupsTotal()
ret += itr->second->GetGroupCount(); ret += itr->second->GetGroupCount();
return ret; return ret;
} }
src/server/game/Mails/Mail.h
+1
View File
@@ -15,6 +15,7 @@
* You should have received a copy of the GNU General Public License along * You should have received a copy of the GNU General Public License along
* with this program. If not, see <http://www.gnu.org/licenses/>. * with this program. If not, see <http://www.gnu.org/licenses/>.
*/ */
#ifndef TRINITY_MAIL_H #ifndef TRINITY_MAIL_H
#define TRINITY_MAIL_H #define TRINITY_MAIL_H
src/server/game/OutdoorPvP/OutdoorPvP.cpp
+5 -5
View File
@@ -436,20 +436,20 @@ void OutdoorPvP::HandleKill(Player* killer, Unit* killed)
{ {
for (GroupReference* itr = group->GetFirstMember(); itr != NULL; itr = itr->next()) for (GroupReference* itr = group->GetFirstMember(); itr != NULL; itr = itr->next())
{ {
Player* pGroupGuy = itr->getSource(); Player* groupGuy = itr->getSource();
if (!pGroupGuy) if (!groupGuy)
continue; continue;
// skip if too far away // skip if too far away
if (!pGroupGuy->IsAtGroupRewardDistance(killed)) if (!groupGuy->IsAtGroupRewardDistance(killed))
continue; continue;
// creature kills must be notified, even if not inside objective / not outdoor pvp active // creature kills must be notified, even if not inside objective / not outdoor pvp active
// player kills only count if active and inside objective // player kills only count if active and inside objective
if ((pGroupGuy->IsOutdoorPvPActive() && IsInsideObjective(pGroupGuy)) || killed->GetTypeId() == TYPEID_UNIT) if ((groupGuy->IsOutdoorPvPActive() && IsInsideObjective(groupGuy)) || killed->GetTypeId() == TYPEID_UNIT)
{ {
HandleKillImpl(pGroupGuy, killed); HandleKillImpl(groupGuy, killed);
} }
} }
} }
src/server/game/Scripting/ScriptLoader.h
+16 -3
View File
@@ -1,6 +1,19 @@
/* Copyright (C) 2006 - 2009 ScriptDev2 <https://scriptdev2.svn.sourceforge.net/> /*
* This program is free software licensed under GPL version 2 * Copyright (C) 2008-2012 TrinityCore <http://www.trinitycore.org/>
* Please see the included DOCS/LICENSE.TXT for more information */ *
* This program is free software; you can redistribute it and/or modify it
* under the terms of the GNU General Public License as published by the
* Free Software Foundation; either version 2 of the License, or (at your
* option) any later version.
*
* This program is distributed in the hope that it will be useful, but WITHOUT
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for
* more details.
*
* You should have received a copy of the GNU General Public License along
* with this program. If not, see <http://www.gnu.org/licenses/>.
*/
#ifndef SC_SCRIPTLOADER_H #ifndef SC_SCRIPTLOADER_H
#define SC_SCRIPTLOADER_H #define SC_SCRIPTLOADER_H
src/server/game/Scripting/ScriptSystem.cpp
+6 -3
View File
@@ -79,7 +79,8 @@ void SystemMgr::LoadScriptTexts()
m_mTextDataMap[iId] = temp; m_mTextDataMap[iId] = temp;
++uiCount; ++uiCount;
} while (result->NextRow()); }
while (result->NextRow());
sLog->outString(">> Loaded %u additional Script Texts data in %u ms", uiCount, GetMSTimeDiffToNow(oldMSTime)); sLog->outString(">> Loaded %u additional Script Texts data in %u ms", uiCount, GetMSTimeDiffToNow(oldMSTime));
sLog->outString(); sLog->outString();
@@ -140,7 +141,8 @@ void SystemMgr::LoadScriptTextsCustom()
m_mTextDataMap[iId] = temp; m_mTextDataMap[iId] = temp;
++uiCount; ++uiCount;
} while (result->NextRow()); }
while (result->NextRow());
sLog->outString(">> Loaded %u additional Custom Texts data.", uiCount); sLog->outString(">> Loaded %u additional Custom Texts data.", uiCount);
sLog->outString(); sLog->outString();
@@ -199,7 +201,8 @@ void SystemMgr::LoadScriptWaypoints()
m_mPointMoveMap[uiEntry].push_back(temp); m_mPointMoveMap[uiEntry].push_back(temp);
++count; ++count;
} while (result->NextRow()); }
while (result->NextRow());
sLog->outString(">> Loaded %u Script Waypoint nodes in %u ms", count, GetMSTimeDiffToNow(oldMSTime)); sLog->outString(">> Loaded %u Script Waypoint nodes in %u ms", count, GetMSTimeDiffToNow(oldMSTime));
sLog->outString(); sLog->outString();
src/server/game/Scripting/ScriptSystem.h
+1
View File
@@ -4,6 +4,7 @@
#ifndef SC_SYSTEM_H #ifndef SC_SYSTEM_H
#define SC_SYSTEM_H #define SC_SYSTEM_H
#include <ace/Singleton.h> #include <ace/Singleton.h>
#define TEXT_SOURCE_RANGE -1000000 //the amount of entries each text source has available #define TEXT_SOURCE_RANGE -1000000 //the amount of entries each text source has available
src/server/game/Server/Protocol/Opcodes.h
+1 -1
View File
@@ -26,7 +26,7 @@
#include "Common.h" #include "Common.h"
// Note: this include need for be sure have full definition of class WorldSession // Note: this include need for be sure have full definition of class WorldSession
// if this class definition not complite then VS for x64 release use different size for // if this class definition not complete then VS for x64 release use different size for
// struct OpcodeHandler in this header and Opcode.cpp and get totally wrong data from // struct OpcodeHandler in this header and Opcode.cpp and get totally wrong data from
// table opcodeTable in source when Opcode.h included but WorldSession.h not included // table opcodeTable in source when Opcode.h included but WorldSession.h not included
#include "WorldSession.h" #include "WorldSession.h"
src/server/game/Server/WorldSession.cpp
+7 -7
View File
@@ -47,7 +47,7 @@
bool MapSessionFilter::Process(WorldPacket* packet) bool MapSessionFilter::Process(WorldPacket* packet)
{ {
OpcodeHandler const &opHandle = opcodeTable[packet->GetOpcode()]; OpcodeHandler const& opHandle = opcodeTable[packet->GetOpcode()];
//let's check if our opcode can be really processed in Map::Update() //let's check if our opcode can be really processed in Map::Update()
if (opHandle.packetProcessing == PROCESS_INPLACE) if (opHandle.packetProcessing == PROCESS_INPLACE)
@@ -69,7 +69,7 @@ bool MapSessionFilter::Process(WorldPacket* packet)
//OR packet handler is not thread-safe! //OR packet handler is not thread-safe!
bool WorldSessionFilter::Process(WorldPacket* packet) bool WorldSessionFilter::Process(WorldPacket* packet)
{ {
OpcodeHandler const &opHandle = opcodeTable[packet->GetOpcode()]; OpcodeHandler const& opHandle = opcodeTable[packet->GetOpcode()];
//check if packet handler is supposed to be safe //check if packet handler is supposed to be safe
if (opHandle.packetProcessing == PROCESS_INPLACE) if (opHandle.packetProcessing == PROCESS_INPLACE)
return true; return true;
@@ -105,7 +105,7 @@ isRecruiter(isARecruiter), timeLastWhoCommand(0)
m_Address = sock->GetRemoteAddress(); m_Address = sock->GetRemoteAddress();
sock->AddReference(); sock->AddReference();
ResetTimeOutTime(); ResetTimeOutTime();
LoginDatabase.PExecute("UPDATE account SET online = 1 WHERE id = %u;", GetAccountId()); // One-time query LoginDatabase.PExecute("UPDATE account SET online = 1 WHERE id = %u;", GetAccountId());
} }
InitializeQueryCallbackParameters(); InitializeQueryCallbackParameters();
@@ -134,10 +134,10 @@ WorldSession::~WorldSession()
while (_recvQueue.next(packet)) while (_recvQueue.next(packet))
delete packet; delete packet;
LoginDatabase.PExecute("UPDATE account SET online = 0 WHERE id = %u;", GetAccountId()); // One-time query LoginDatabase.PExecute("UPDATE account SET online = 0 WHERE id = %u;", GetAccountId());
} }
void WorldSession::SizeError(WorldPacket const &packet, uint32 size) const void WorldSession::SizeError(WorldPacket const& packet, uint32 size) const
{ {
sLog->outError("Client (account %u) send packet %s (%u) with size " SIZEFMTD " but expected %u (attempt to crash server?), skipped", sLog->outError("Client (account %u) send packet %s (%u) with size " SIZEFMTD " but expected %u (attempt to crash server?), skipped",
GetAccountId(), LookupOpcodeName(packet.GetOpcode()), packet.GetOpcode(), packet.size(), size); GetAccountId(), LookupOpcodeName(packet.GetOpcode()), packet.GetOpcode(), packet.size(), size);
@@ -345,7 +345,7 @@ bool WorldSession::Update(uint32 diff, PacketFilter& updater)
break; break;
} }
} }
catch (ByteBufferException &) catch(ByteBufferException &)
{ {
sLog->outError("WorldSession::Update ByteBufferException occured while parsing a packet (opcode: %u) from client %s, accountid=%i. Skipped packet.", sLog->outError("WorldSession::Update ByteBufferException occured while parsing a packet (opcode: %u) from client %s, accountid=%i. Skipped packet.",
packet->GetOpcode(), GetRemoteAddress().c_str(), GetAccountId()); packet->GetOpcode(), GetRemoteAddress().c_str(), GetAccountId());
@@ -896,7 +896,7 @@ void WorldSession::ReadAddonsInfo(WorldPacket &data)
ByteBuffer addonInfo; ByteBuffer addonInfo;
addonInfo.resize(size); addonInfo.resize(size);
if (uncompress(const_cast<uint8 *>(addonInfo.contents()), &uSize, const_cast<uint8 *>(data.contents() + pos), data.size() - pos) == Z_OK) if (uncompress(const_cast<uint8*>(addonInfo.contents()), &uSize, const_cast<uint8*>(data.contents() + pos), data.size() - pos) == Z_OK)
{ {
uint32 addonsCount; uint32 addonsCount;
addonInfo >> addonsCount; // addons count addonInfo >> addonsCount; // addons count
src/server/game/Server/WorldSession.h
+1 -1
View File
@@ -294,7 +294,7 @@ class WorldSession
void SendAttackStop(Unit const* enemy); void SendAttackStop(Unit const* enemy);
void SendBattlegGroundList(uint64 guid, BattlegroundTypeId bgTypeId); void SendBattleGroundList(uint64 guid, BattlegroundTypeId bgTypeId);
void SendTradeStatus(TradeStatus status); void SendTradeStatus(TradeStatus status);
void SendUpdateTrade(bool trader_data = true); void SendUpdateTrade(bool trader_data = true);
src/server/game/Skills/SkillDiscovery.cpp
+2 -2
View File
@@ -132,7 +132,8 @@ void LoadSkillDiscoveryTable()
} }
++count; ++count;
} while (result->NextRow()); }
while (result->NextRow());
if (!ssNonDiscoverableEntries.str().empty()) if (!ssNonDiscoverableEntries.str().empty())
sLog->outErrorDb("Some items can't be successfully discovered: have in chance field value < 0.000001 in `skill_discovery_template` DB table . List:\n%s", ssNonDiscoverableEntries.str().c_str()); sLog->outErrorDb("Some items can't be successfully discovered: have in chance field value < 0.000001 in `skill_discovery_template` DB table . List:\n%s", ssNonDiscoverableEntries.str().c_str());
@@ -246,4 +247,3 @@ uint32 GetSkillDiscoverySpell(uint32 skillId, uint32 spellId, Player* player)
return 0; return 0;
} }
src/server/game/Skills/SkillDiscovery.h
-1
View File
@@ -28,4 +28,3 @@ uint32 GetSkillDiscoverySpell(uint32 skillId, uint32 spellId, Player* player);
bool HasDiscoveredAllSpells(uint32 spellId, Player* player); bool HasDiscoveredAllSpells(uint32 spellId, Player* player);
uint32 GetExplicitDiscoverySpell(uint32 spellId, Player* player); uint32 GetExplicitDiscoverySpell(uint32 spellId, Player* player);
#endif #endif
src/server/game/Skills/SkillExtraItems.cpp
-1
View File
@@ -138,4 +138,3 @@ bool canCreateExtraItems(Player* player, uint32 spellId, float &additionalChance
// enable extra item creation // enable extra item creation
return true; return true;
} }
src/server/game/Skills/SkillExtraItems.h
-1
View File
@@ -28,4 +28,3 @@ bool canCreateExtraItems(Player* player, uint32 spellId, float &additionalChance
// function to load the extra item creation info from DB // function to load the extra item creation info from DB
void LoadSkillExtraItemTable(); void LoadSkillExtraItemTable();
#endif #endif
src/server/game/Weather/Weather.h
-1
View File
@@ -89,4 +89,3 @@ class Weather
WeatherData const* m_weatherChances; WeatherData const* m_weatherChances;
}; };
#endif #endif
src/server/scripts/OutdoorPvP/OutdoorPvPEP.cpp
-1
View File
@@ -244,7 +244,6 @@ void OPvPCapturePointEP_NPT::ChangeState()
// complete quest objective // complete quest objective
if (m_TowerState == EP_TS_A || m_TowerState == EP_TS_H) if (m_TowerState == EP_TS_A || m_TowerState == EP_TS_H)
SendObjectiveComplete(EP_NPT_CM, 0); SendObjectiveComplete(EP_NPT_CM, 0);
} }
void OPvPCapturePointEP_NPT::SendChangePhase() void OPvPCapturePointEP_NPT::SendChangePhase()
src/server/scripts/OutdoorPvP/OutdoorPvPEP.h
-1
View File
@@ -329,4 +329,3 @@ class OutdoorPvPEP : public OutdoorPvP
}; };
#endif #endif
src/server/scripts/OutdoorPvP/OutdoorPvPZM.cpp
-1
View File
@@ -176,7 +176,6 @@ OutdoorPvPZM::OutdoorPvPZM()
m_GraveYard = NULL; m_GraveYard = NULL;
m_AllianceTowersControlled = 0; m_AllianceTowersControlled = 0;
m_HordeTowersControlled = 0; m_HordeTowersControlled = 0;
} }
bool OutdoorPvPZM::SetupOutdoorPvP() bool OutdoorPvPZM::SetupOutdoorPvP()