From 6fd41ae50e5f988473f86ad8370b60a1e29eac5f Mon Sep 17 00:00:00 2001 From: Spp- Date: Thu, 30 Jun 2011 14:24:56 +0200 Subject: Scripts: Reorder file names in Commands/CMakeList file and cosmetic changes in all scripts --- src/server/scripts/Commands/CMakeLists.txt | 20 ++++++++++---------- src/server/scripts/Commands/cs_debug.cpp | 26 +++++++++++++------------- src/server/scripts/Commands/cs_gm.cpp | 2 +- src/server/scripts/Commands/cs_gobject.cpp | 8 ++++---- src/server/scripts/Commands/cs_honor.cpp | 4 ++-- src/server/scripts/Commands/cs_modify.cpp | 12 ++++++------ src/server/scripts/Commands/cs_npc.cpp | 22 +++++++++++----------- src/server/scripts/Commands/cs_tele.cpp | 2 +- src/server/scripts/Commands/cs_titles.cpp | 2 +- src/server/scripts/Commands/cs_wp.cpp | 8 ++++---- 10 files changed, 53 insertions(+), 53 deletions(-) (limited to 'src/server/scripts/Commands') diff --git a/src/server/scripts/Commands/CMakeLists.txt b/src/server/scripts/Commands/CMakeLists.txt index eec02c0e106..19c42ba92ad 100644 --- a/src/server/scripts/Commands/CMakeLists.txt +++ b/src/server/scripts/Commands/CMakeLists.txt @@ -12,21 +12,22 @@ set(scripts_STAT_SRCS ${scripts_STAT_SRCS} Commands/cs_account.cpp Commands/cs_achievement.cpp - Commands/cs_gm.cpp - Commands/cs_npc.cpp - Commands/cs_go.cpp - Commands/cs_learn.cpp - Commands/cs_modify.cpp Commands/cs_debug.cpp - Commands/cs_tele.cpp -# Commands/cs_character.cpp Commands/cs_event.cpp + Commands/cs_gm.cpp + Commands/cs_go.cpp Commands/cs_gobject.cpp + Commands/cs_gps.cpp Commands/cs_honor.cpp - Commands/cs_wp.cpp - Commands/cs_titles.cpp + Commands/cs_learn.cpp + Commands/cs_modify.cpp + Commands/cs_npc.cpp Commands/cs_quest.cpp Commands/cs_reload.cpp + Commands/cs_tele.cpp + Commands/cs_titles.cpp + Commands/cs_wp.cpp +# Commands/cs_character.cpp # Commands/cs_list.cpp # Commands/cs_lookup.cpp # Commands/cs_pdump.cpp @@ -54,7 +55,6 @@ set(scripts_STAT_SRCS # Commands/cs_die.cpp # Commands/cs_revive.cpp # Commands/cs_dismount.cpp - Commands/cs_gps.cpp # Commands/cs_guid.cpp # Commands/cs_help.cpp # Commands/cs_itemmove.cpp diff --git a/src/server/scripts/Commands/cs_debug.cpp b/src/server/scripts/Commands/cs_debug.cpp index 4c192ec1d90..41e456289e0 100644 --- a/src/server/scripts/Commands/cs_debug.cpp +++ b/src/server/scripts/Commands/cs_debug.cpp @@ -249,7 +249,7 @@ public: static bool HandleDebugSendOpcodeCommand(ChatHandler* handler, const char* /*args*/) { - Unit *unit = handler->getSelectedUnit(); + Unit* unit = handler->getSelectedUnit(); Player* player = NULL; if (!unit || (unit->GetTypeId() != TYPEID_PLAYER)) player = handler->GetSession()->GetPlayer(); @@ -320,7 +320,7 @@ public: } else if (type == "appgoguid") { - GameObject *obj = handler->GetNearbyGameObject(); + GameObject* obj = handler->GetNearbyGameObject(); if (!obj) { handler->PSendSysMessage(LANG_COMMAND_OBJNOTFOUND, 0); @@ -332,7 +332,7 @@ public: } else if (type == "goguid") { - GameObject *obj = handler->GetNearbyGameObject(); + GameObject* obj = handler->GetNearbyGameObject(); if (!obj) { handler->PSendSysMessage(LANG_COMMAND_OBJNOTFOUND, 0); @@ -489,7 +489,7 @@ public: if (i >= BUYBACK_SLOT_START && i < BUYBACK_SLOT_END) continue; - if (Item *item = player->GetItemByPos(INVENTORY_SLOT_BAG_0, i)) + if (Item* item = player->GetItemByPos(INVENTORY_SLOT_BAG_0, i)) { if (Bag* bag = item->ToBag()) { @@ -509,7 +509,7 @@ public: std::vector &updateQueue = player->GetItemUpdateQueue(); for (size_t i = 0; i < updateQueue.size(); ++i) { - Item *item = updateQueue[i]; + Item* item = updateQueue[i]; if (!item) continue; Bag *container = item->GetContainer(); @@ -539,7 +539,7 @@ public: if (i >= BUYBACK_SLOT_START && i < BUYBACK_SLOT_END) continue; - Item *item = player->GetItemByPos(INVENTORY_SLOT_BAG_0, i); + Item* item = player->GetItemByPos(INVENTORY_SLOT_BAG_0, i); if (!item) continue; if (item->GetSlot() != i) @@ -651,7 +651,7 @@ public: for (size_t i = 0; i < updateQueue.size(); ++i) { - Item *item = updateQueue[i]; + Item* item = updateQueue[i]; if (!item) continue; if (item->GetOwnerGUID() != player->GetGUID()) @@ -667,7 +667,7 @@ public: } if (item->GetState() == ITEM_REMOVED) continue; - Item *test = player->GetItemByPos(item->GetBagSlot(), item->GetSlot()); + Item* test = player->GetItemByPos(item->GetBagSlot(), item->GetSlot()); if (test == NULL) { @@ -784,7 +784,7 @@ public: handler->GetSession()->GetPlayer()->EnterVehicle(target, seatId); else { - Creature *passenger = NULL; + Creature* passenger = NULL; Trinity::AllCreaturesOfEntryInRange check(handler->GetSession()->GetPlayer(), entry, 20.0f); Trinity::CreatureSearcher searcher(handler->GetSession()->GetPlayer(), passenger, check); handler->GetSession()->GetPlayer()->VisitNearbyObject(30.0f, searcher); @@ -828,7 +828,7 @@ public: if (!ve) return false; - Creature *v = new Creature; + Creature* v = new Creature; Map *map = handler->GetSession()->GetPlayer()->GetMap(); @@ -877,7 +877,7 @@ public: uint32 guid = (uint32)atoi(e); uint32 index = (uint32)atoi(f); - Item *i = handler->GetSession()->GetPlayer()->GetItemByGuid(MAKE_NEW_GUID(guid, 0, HIGHGUID_ITEM)); + Item* i = handler->GetSession()->GetPlayer()->GetItemByGuid(MAKE_NEW_GUID(guid, 0, HIGHGUID_ITEM)); if (!i) return false; @@ -908,7 +908,7 @@ public: uint32 index = (uint32)atoi(f); uint32 value = (uint32)atoi(g); - Item *i = handler->GetSession()->GetPlayer()->GetItemByGuid(MAKE_NEW_GUID(guid, 0, HIGHGUID_ITEM)); + Item* i = handler->GetSession()->GetPlayer()->GetItemByGuid(MAKE_NEW_GUID(guid, 0, HIGHGUID_ITEM)); if (!i) return false; @@ -932,7 +932,7 @@ public: uint32 guid = (uint32)atoi(e); - Item *i = handler->GetSession()->GetPlayer()->GetItemByGuid(MAKE_NEW_GUID(guid, 0, HIGHGUID_ITEM)); + Item* i = handler->GetSession()->GetPlayer()->GetItemByGuid(MAKE_NEW_GUID(guid, 0, HIGHGUID_ITEM)); if (!i) return false; diff --git a/src/server/scripts/Commands/cs_gm.cpp b/src/server/scripts/Commands/cs_gm.cpp index 90fbe1d1549..a520eac1ee7 100644 --- a/src/server/scripts/Commands/cs_gm.cpp +++ b/src/server/scripts/Commands/cs_gm.cpp @@ -89,7 +89,7 @@ public: if (!*args) return false; - Player *target = handler->getSelectedPlayer(); + Player* target = handler->getSelectedPlayer(); if (!target) target = handler->GetSession()->GetPlayer(); diff --git a/src/server/scripts/Commands/cs_gobject.cpp b/src/server/scripts/Commands/cs_gobject.cpp index 44d87729719..3197595812f 100644 --- a/src/server/scripts/Commands/cs_gobject.cpp +++ b/src/server/scripts/Commands/cs_gobject.cpp @@ -139,7 +139,7 @@ public: return false; } - Player *chr = handler->GetSession()->GetPlayer(); + Player* chr = handler->GetSession()->GetPlayer(); float x = float(chr->GetPositionX()); float y = float(chr->GetPositionY()); float z = float(chr->GetPositionZ()); @@ -190,7 +190,7 @@ public: if (!charID) return false; - Player *chr = handler->GetSession()->GetPlayer(); + Player* chr = handler->GetSession()->GetPlayer(); char* spawntime = strtok(NULL, " "); uint32 spawntm = 300; @@ -411,7 +411,7 @@ public: } else { - Player *chr = handler->GetSession()->GetPlayer(); + Player* chr = handler->GetSession()->GetPlayer(); o = chr->GetOrientation(); } @@ -459,7 +459,7 @@ public: if (!px) { - Player *chr = handler->GetSession()->GetPlayer(); + Player* chr = handler->GetSession()->GetPlayer(); obj->Relocate(chr->GetPositionX(), chr->GetPositionY(), chr->GetPositionZ(), obj->GetOrientation()); obj->DestroyForNearbyPlayers(); obj->UpdateObjectVisibility(); diff --git a/src/server/scripts/Commands/cs_honor.cpp b/src/server/scripts/Commands/cs_honor.cpp index b2ba23c0186..6f44000017d 100644 --- a/src/server/scripts/Commands/cs_honor.cpp +++ b/src/server/scripts/Commands/cs_honor.cpp @@ -57,7 +57,7 @@ public: if (!*args) return false; - Player *target = handler->getSelectedPlayer(); + Player* target = handler->getSelectedPlayer(); if (!target) { handler->SendSysMessage(LANG_PLAYER_NOT_FOUND); @@ -92,7 +92,7 @@ public: } static bool HandleHonorUpdateCommand(ChatHandler* handler, const char* /*args*/) { - Player *target = handler->getSelectedPlayer(); + Player* target = handler->getSelectedPlayer(); if (!target) { handler->SendSysMessage(LANG_PLAYER_NOT_FOUND); diff --git a/src/server/scripts/Commands/cs_modify.cpp b/src/server/scripts/Commands/cs_modify.cpp index b865574cd43..b54e9519da5 100644 --- a/src/server/scripts/Commands/cs_modify.cpp +++ b/src/server/scripts/Commands/cs_modify.cpp @@ -440,7 +440,7 @@ public: } else if (target->ToCreature()->isPet()) { - Unit *owner = target->GetOwner(); + Unit* owner = target->GetOwner(); if (owner && owner->GetTypeId() == TYPEID_PLAYER && ((Pet *)target)->IsPermanentPetFor(owner->ToPlayer())) { // check online security @@ -698,15 +698,15 @@ public: return false; } - if (target->GetTypeId()==TYPEID_PLAYER) + if (Player* plr = target->ToPlayer()) { // check online security - if (handler->HasLowerSecurity((Player*)target, 0)) + if (handler->HasLowerSecurity(plr, 0)) return false; - handler->PSendSysMessage(LANG_YOU_CHANGE_SIZE, Scale, handler->GetNameLink((Player*)target).c_str()); - if (handler->needReportToTarget((Player*)target)) - (ChatHandler((Player*)target)).PSendSysMessage(LANG_YOURS_SIZE_CHANGED, handler->GetNameLink().c_str(), Scale); + handler->PSendSysMessage(LANG_YOU_CHANGE_SIZE, Scale, handler->GetNameLink(plr).c_str()); + if (handler->needReportToTarget(plr)) + (ChatHandler(plr)).PSendSysMessage(LANG_YOURS_SIZE_CHANGED, handler->GetNameLink().c_str(), Scale); } target->SetFloatValue(OBJECT_FIELD_SCALE_X, Scale); diff --git a/src/server/scripts/Commands/cs_npc.cpp b/src/server/scripts/Commands/cs_npc.cpp index dd526162e66..48f91d9ec30 100644 --- a/src/server/scripts/Commands/cs_npc.cpp +++ b/src/server/scripts/Commands/cs_npc.cpp @@ -119,7 +119,7 @@ public: uint32 id = atoi(charID); - Player *chr = handler->GetSession()->GetPlayer(); + Player* chr = handler->GetSession()->GetPlayer(); float x = chr->GetPositionX(); float y = chr->GetPositionY(); float z = chr->GetPositionZ(); @@ -495,7 +495,7 @@ public: static bool HandleNpcFollowCommand(ChatHandler* handler, const char* /*args*/) { Player* player = handler->GetSession()->GetPlayer(); - Creature *creature = handler->getSelectedCreature(); + Creature* creature = handler->getSelectedCreature(); if (!creature) { @@ -667,7 +667,7 @@ public: uint32 displayId = (uint32) atoi((char*)args); - Creature *pCreature = handler->getSelectedCreature(); + Creature* pCreature = handler->getSelectedCreature(); if (!pCreature || pCreature->isPet()) { @@ -878,7 +878,7 @@ public: if (option >0.0f) mtype = RANDOM_MOTION_TYPE; - Creature *pCreature = handler->getSelectedCreature(); + Creature* pCreature = handler->getSelectedCreature(); uint32 u_guidlow = 0; if (pCreature) @@ -920,7 +920,7 @@ public: return false; } - Creature *pCreature = handler->getSelectedCreature(); + Creature* pCreature = handler->getSelectedCreature(); uint32 u_guidlow = 0; if (pCreature) @@ -986,7 +986,7 @@ public: static bool HandleNpcUnFollowCommand(ChatHandler* handler, const char* /*args*/) { Player* player = handler->GetSession()->GetPlayer(); - Creature *creature = handler->getSelectedCreature(); + Creature* creature = handler->getSelectedCreature(); if (!creature) { @@ -1078,7 +1078,7 @@ public: if (!charID) return false; - Player *chr = handler->GetSession()->GetPlayer(); + Player* chr = handler->GetSession()->GetPlayer(); uint32 id = atoi(charID); if (!id) @@ -1092,7 +1092,7 @@ public: //npc tame handling static bool HandleNpcTameCommand(ChatHandler* handler, const char* /*args*/) { - Creature *creatureTarget = handler->getSelectedCreature (); + Creature* creatureTarget = handler->getSelectedCreature (); if (!creatureTarget || creatureTarget->isPet ()) { handler->PSendSysMessage (LANG_SELECT_CREATURE); @@ -1162,7 +1162,7 @@ public: return false; uint32 leaderGUID = (uint32) atoi((char*)args); - Creature *pCreature = handler->getSelectedCreature(); + Creature* pCreature = handler->getSelectedCreature(); if (!pCreature || !pCreature->GetDBTableGUIDLow()) { @@ -1181,7 +1181,7 @@ public: if (!lowguid) return false; - Player *chr = handler->GetSession()->GetPlayer(); + Player* chr = handler->GetSession()->GetPlayer(); FormationInfo *group_member; group_member = new FormationInfo; @@ -1248,7 +1248,7 @@ public: return true; } - Creature *pCreature = ObjectAccessor::GetCreature(*handler->GetSession()->GetPlayer(), guid); + Creature* pCreature = ObjectAccessor::GetCreature(*handler->GetSession()->GetPlayer(), guid); if (!pCreature) { diff --git a/src/server/scripts/Commands/cs_tele.cpp b/src/server/scripts/Commands/cs_tele.cpp index 501cfcdb6c3..2bc777afe85 100644 --- a/src/server/scripts/Commands/cs_tele.cpp +++ b/src/server/scripts/Commands/cs_tele.cpp @@ -228,7 +228,7 @@ public: for (GroupReference *itr = grp->GetFirstMember(); itr != NULL; itr = itr->next()) { - Player *pl = itr->getSource(); + Player* pl = itr->getSource(); if (!pl || !pl->GetSession()) continue; diff --git a/src/server/scripts/Commands/cs_titles.cpp b/src/server/scripts/Commands/cs_titles.cpp index fc268afe75a..2fb22e0090d 100644 --- a/src/server/scripts/Commands/cs_titles.cpp +++ b/src/server/scripts/Commands/cs_titles.cpp @@ -210,7 +210,7 @@ public: sscanf((char*)args, UI64FMTD, &titles); - Player *target = handler->getSelectedPlayer(); + Player* target = handler->getSelectedPlayer(); if (!target) { handler->SendSysMessage(LANG_NO_CHAR_SELECTED); diff --git a/src/server/scripts/Commands/cs_wp.cpp b/src/server/scripts/Commands/cs_wp.cpp index 4701591bbcf..4addf788183 100644 --- a/src/server/scripts/Commands/cs_wp.cpp +++ b/src/server/scripts/Commands/cs_wp.cpp @@ -569,7 +569,7 @@ public: { handler->PSendSysMessage("|cff00ff00DEBUG: wp move, PathID: |r|cff00ffff%u|r", pathid); - Player *chr = handler->GetSession()->GetPlayer(); + Player* chr = handler->GetSession()->GetPlayer(); Map *map = chr->GetMap(); { // wpCreature @@ -776,7 +776,7 @@ public: uint32 id = VISUAL_WAYPOINT; - Player *chr = handler->GetSession()->GetPlayer(); + Player* chr = handler->GetSession()->GetPlayer(); Map *map = chr->GetMap(); float o = chr->GetOrientation(); @@ -827,7 +827,7 @@ public: float z = fields[2].GetFloat(); uint32 id = VISUAL_WAYPOINT; - Player *chr = handler->GetSession()->GetPlayer(); + Player* chr = handler->GetSession()->GetPlayer(); float o = chr->GetOrientation(); Map *map = chr->GetMap(); @@ -875,7 +875,7 @@ public: float z = fields[2].GetFloat(); uint32 id = VISUAL_WAYPOINT; - Player *chr = handler->GetSession()->GetPlayer(); + Player* chr = handler->GetSession()->GetPlayer(); float o = chr->GetOrientation(); Map *map = chr->GetMap(); -- cgit v1.2.3 From e9aa72cc7b09999a1cbb6df994396ae8823657f6 Mon Sep 17 00:00:00 2001 From: Spp- Date: Thu, 30 Jun 2011 14:52:44 +0200 Subject: Cosmetic: rename function escape_string to EscapeString --- src/server/game/Accounts/AccountMgr.cpp | 4 ++-- src/server/game/Addons/AddonMgr.cpp | 2 +- src/server/game/Chat/Commands/Level2.cpp | 6 +++--- src/server/game/Chat/Commands/Level3.cpp | 8 ++++---- src/server/game/Entities/Pet/Pet.cpp | 2 +- src/server/game/Entities/Player/Player.cpp | 2 +- src/server/game/Entities/Player/SocialMgr.cpp | 2 +- src/server/game/Globals/ObjectMgr.cpp | 2 +- src/server/game/Server/Protocol/Handlers/CharacterHandler.cpp | 8 ++++---- src/server/game/Server/Protocol/Handlers/GroupHandler.cpp | 2 +- src/server/game/Server/Protocol/Handlers/MiscHandler.cpp | 8 ++++---- src/server/game/Server/Protocol/Handlers/PetHandler.cpp | 4 ++-- src/server/game/Server/Protocol/Handlers/PetitionsHandler.cpp | 4 ++-- src/server/game/Server/WorldSocket.cpp | 4 ++-- src/server/game/Tools/PlayerDump.cpp | 6 +++--- src/server/scripts/Commands/cs_go.cpp | 2 +- src/server/scripts/Commands/cs_gobject.cpp | 2 +- src/server/scripts/Commands/cs_wp.cpp | 4 ++-- src/server/shared/Database/DatabaseWorkerPool.h | 6 +++--- src/server/shared/Logging/Log.cpp | 2 +- src/server/worldserver/RemoteAccess/RASocket.cpp | 6 +++--- 21 files changed, 43 insertions(+), 43 deletions(-) (limited to 'src/server/scripts/Commands') diff --git a/src/server/game/Accounts/AccountMgr.cpp b/src/server/game/Accounts/AccountMgr.cpp index 614400af21f..47320b1b542 100755 --- a/src/server/game/Accounts/AccountMgr.cpp +++ b/src/server/game/Accounts/AccountMgr.cpp @@ -108,7 +108,7 @@ AccountOpResult AccountMgr::ChangeUsername(uint32 accid, std::string new_uname, normalizeString(new_passwd); std::string safe_new_uname = new_uname; - LoginDatabase.escape_string(safe_new_uname); + LoginDatabase.EscapeString(safe_new_uname); LoginDatabase.PExecute("UPDATE account SET v='0', s='0', username='%s', sha_pass_hash='%s' WHERE id='%d'", safe_new_uname.c_str(), CalculateShaPassHash(new_uname, new_passwd).c_str(), accid); @@ -138,7 +138,7 @@ AccountOpResult AccountMgr::ChangePassword(uint32 accid, std::string new_passwd) uint32 AccountMgr::GetId(std::string username) { - LoginDatabase.escape_string(username); + LoginDatabase.EscapeString(username); QueryResult result = LoginDatabase.PQuery("SELECT id FROM account WHERE username = '%s'", username.c_str()); if (!result) return 0; diff --git a/src/server/game/Addons/AddonMgr.cpp b/src/server/game/Addons/AddonMgr.cpp index f2fc4d9d1ad..d892e64b316 100755 --- a/src/server/game/Addons/AddonMgr.cpp +++ b/src/server/game/Addons/AddonMgr.cpp @@ -67,7 +67,7 @@ void AddonMgr::LoadFromDB() void AddonMgr::SaveAddon(AddonInfo const& addon) { std::string name = addon.Name; - CharacterDatabase.escape_string(name); + CharacterDatabase.EscapeString(name); CharacterDatabase.PExecute("INSERT INTO addons (name, crc) VALUES ('%s', %u)", name.c_str(), addon.CRC); SavedAddon newAddon(addon.Name, addon.CRC); diff --git a/src/server/game/Chat/Commands/Level2.cpp b/src/server/game/Chat/Commands/Level2.cpp index e1b9b61e920..587bba4d66f 100755 --- a/src/server/game/Chat/Commands/Level2.cpp +++ b/src/server/game/Chat/Commands/Level2.cpp @@ -611,7 +611,7 @@ bool ChatHandler::HandleLookupPlayerIpCommand(const char* args) char* limit_str = strtok (NULL, " "); int32 limit = limit_str ? atoi (limit_str) : -1; - LoginDatabase.escape_string (ip); + LoginDatabase.EscapeString (ip); QueryResult result = LoginDatabase.PQuery ("SELECT id, username FROM account WHERE last_ip = '%s'", ip.c_str ()); @@ -630,7 +630,7 @@ bool ChatHandler::HandleLookupPlayerAccountCommand(const char* args) if (!AccountMgr::normalizeString (account)) return false; - LoginDatabase.escape_string (account); + LoginDatabase.EscapeString (account); QueryResult result = LoginDatabase.PQuery ("SELECT id, username FROM account WHERE username = '%s'", account.c_str ()); @@ -647,7 +647,7 @@ bool ChatHandler::HandleLookupPlayerEmailCommand(const char* args) char* limit_str = strtok (NULL, " "); int32 limit = limit_str ? atoi (limit_str) : -1; - LoginDatabase.escape_string (email); + LoginDatabase.EscapeString (email); QueryResult result = LoginDatabase.PQuery ("SELECT id, username FROM account WHERE email = '%s'", email.c_str ()); diff --git a/src/server/game/Chat/Commands/Level3.cpp b/src/server/game/Chat/Commands/Level3.cpp index 94a9c592793..83469f17100 100755 --- a/src/server/game/Chat/Commands/Level3.cpp +++ b/src/server/game/Chat/Commands/Level3.cpp @@ -225,7 +225,7 @@ bool ChatHandler::HandleAddItemCommand(const char *args) if (citemName && citemName[0]) { std::string itemName = citemName+1; - WorldDatabase.escape_string(itemName); + WorldDatabase.EscapeString(itemName); QueryResult result = WorldDatabase.PQuery("SELECT entry FROM item_template WHERE name = '%s'", itemName.c_str()); if (!result) { @@ -3184,7 +3184,7 @@ bool ChatHandler::HandleBanInfoIPCommand(const char *args) std::string IP = cIP; - LoginDatabase.escape_string(IP); + LoginDatabase.EscapeString(IP); QueryResult result = LoginDatabase.PQuery("SELECT ip, FROM_UNIXTIME(bandate), FROM_UNIXTIME(unbandate), unbandate-UNIX_TIMESTAMP(), banreason, bannedby, unbandate-bandate FROM ip_banned WHERE ip = '%s'", IP.c_str()); if (!result) { @@ -3293,7 +3293,7 @@ bool ChatHandler::HandleBanListAccountCommand(const char *args) char* cFilter = strtok((char*)args, " "); std::string filter = cFilter ? cFilter : ""; - LoginDatabase.escape_string(filter); + LoginDatabase.EscapeString(filter); QueryResult result; @@ -3398,7 +3398,7 @@ bool ChatHandler::HandleBanListIPCommand(const char *args) char* cFilter = strtok((char*)args, " "); std::string filter = cFilter ? cFilter : ""; - LoginDatabase.escape_string(filter); + LoginDatabase.EscapeString(filter); QueryResult result; diff --git a/src/server/game/Entities/Pet/Pet.cpp b/src/server/game/Entities/Pet/Pet.cpp index 861d8d1e348..baffe4cd0f4 100755 --- a/src/server/game/Entities/Pet/Pet.cpp +++ b/src/server/game/Entities/Pet/Pet.cpp @@ -384,7 +384,7 @@ void Pet::SavePetToDB(PetSaveMode mode) { uint32 owner = GUID_LOPART(GetOwnerGUID()); std::string name = m_name; - CharacterDatabase.escape_string(name); + CharacterDatabase.EscapeString(name); SQLTransaction trans = CharacterDatabase.BeginTransaction(); // remove current data trans->PAppend("DELETE FROM character_pet WHERE owner = '%u' AND id = '%u'", owner, m_charmInfo->GetPetNumber()); diff --git a/src/server/game/Entities/Player/Player.cpp b/src/server/game/Entities/Player/Player.cpp index 1ba7707e30a..7d206bcdbea 100755 --- a/src/server/game/Entities/Player/Player.cpp +++ b/src/server/game/Entities/Player/Player.cpp @@ -18186,7 +18186,7 @@ void Player::SaveToDB() outDebugValues(); std::string sql_name = m_name; - CharacterDatabase.escape_string(sql_name); + CharacterDatabase.EscapeString(sql_name); std::ostringstream ss; ss << "REPLACE INTO characters (guid, account, name, race, class, gender, level, xp, money, playerBytes, playerBytes2, playerFlags, " diff --git a/src/server/game/Entities/Player/SocialMgr.cpp b/src/server/game/Entities/Player/SocialMgr.cpp index 2b44d2a192d..9733d5e80bb 100755 --- a/src/server/game/Entities/Player/SocialMgr.cpp +++ b/src/server/game/Entities/Player/SocialMgr.cpp @@ -108,7 +108,7 @@ void PlayerSocial::SetFriendNote(uint32 friend_guid, std::string note) utf8truncate(note, 48); // DB and client size limitation - CharacterDatabase.escape_string(note); + CharacterDatabase.EscapeString(note); CharacterDatabase.PExecute("UPDATE character_social SET note = '%s' WHERE guid = '%u' AND friend = '%u'", note.c_str(), GetPlayerGUID(), friend_guid); m_playerSocialMap[friend_guid].Note = note; } diff --git a/src/server/game/Globals/ObjectMgr.cpp b/src/server/game/Globals/ObjectMgr.cpp index 2c69d83f311..356f26e880c 100755 --- a/src/server/game/Globals/ObjectMgr.cpp +++ b/src/server/game/Globals/ObjectMgr.cpp @@ -2002,7 +2002,7 @@ uint64 ObjectMgr::GetPlayerGUIDByName(std::string name) const { uint64 guid = 0; - CharacterDatabase.escape_string(name); + CharacterDatabase.EscapeString(name); // Player name safe to sending to DB (checked at login) and this function using QueryResult result = CharacterDatabase.PQuery("SELECT guid FROM characters WHERE name = '%s'", name.c_str()); diff --git a/src/server/game/Server/Protocol/Handlers/CharacterHandler.cpp b/src/server/game/Server/Protocol/Handlers/CharacterHandler.cpp index 0424bb3aab8..97a1ea0551c 100755 --- a/src/server/game/Server/Protocol/Handlers/CharacterHandler.cpp +++ b/src/server/game/Server/Protocol/Handlers/CharacterHandler.cpp @@ -1158,7 +1158,7 @@ void WorldSession::HandleCharRenameOpcode(WorldPacket& recv_data) } std::string escaped_newname = newname; - CharacterDatabase.escape_string(escaped_newname); + CharacterDatabase.EscapeString(escaped_newname); // make sure that the character belongs to the current account, that rename at login is enabled // and that there is no character with the desired new name @@ -1270,7 +1270,7 @@ void WorldSession::HandleSetPlayerDeclinedNames(WorldPacket& recv_data) } for (int i = 0; i < MAX_DECLINED_NAME_CASES; ++i) - CharacterDatabase.escape_string(declinedname.name[i]); + CharacterDatabase.EscapeString(declinedname.name[i]); SQLTransaction trans = CharacterDatabase.BeginTransaction(); trans->PAppend("DELETE FROM character_declinedname WHERE guid = '%u'", GUID_LOPART(guid)); @@ -1431,7 +1431,7 @@ void WorldSession::HandleCharCustomize(WorldPacket& recv_data) } } - CharacterDatabase.escape_string(newname); + CharacterDatabase.EscapeString(newname); if (QueryResult result = CharacterDatabase.PQuery("SELECT name FROM characters WHERE guid ='%u'", GUID_LOPART(guid))) { std::string oldname = result->Fetch()[0].GetString(); @@ -1654,7 +1654,7 @@ void WorldSession::HandleCharFactionOrRaceChange(WorldPacket& recv_data) } } - CharacterDatabase.escape_string(newname); + CharacterDatabase.EscapeString(newname); Player::Customize(guid, gender, skin, face, hairStyle, hairColor, facialHair); SQLTransaction trans = CharacterDatabase.BeginTransaction(); trans->PAppend("UPDATE `characters` SET name='%s', race='%u', at_login=at_login & ~ %u WHERE guid='%u'", newname.c_str(), race, used_loginFlag, lowGuid); diff --git a/src/server/game/Server/Protocol/Handlers/GroupHandler.cpp b/src/server/game/Server/Protocol/Handlers/GroupHandler.cpp index b906aab9218..098bd03f629 100755 --- a/src/server/game/Server/Protocol/Handlers/GroupHandler.cpp +++ b/src/server/game/Server/Protocol/Handlers/GroupHandler.cpp @@ -599,7 +599,7 @@ void WorldSession::HandleGroupChangeSubGroupOpcode(WorldPacket & recv_data) } else { - CharacterDatabase.escape_string(name); + CharacterDatabase.EscapeString(name); guid = sObjectMgr->GetPlayerGUIDByName(name.c_str()); } diff --git a/src/server/game/Server/Protocol/Handlers/MiscHandler.cpp b/src/server/game/Server/Protocol/Handlers/MiscHandler.cpp index 81f771fdada..25b64ce1e7a 100755 --- a/src/server/game/Server/Protocol/Handlers/MiscHandler.cpp +++ b/src/server/game/Server/Protocol/Handlers/MiscHandler.cpp @@ -553,7 +553,7 @@ void WorldSession::HandleAddFriendOpcode(WorldPacket & recv_data) if (!normalizePlayerName(friendName)) return; - CharacterDatabase.escape_string(friendName); // prevent SQL injection - normal name don't must changed by this call + CharacterDatabase.EscapeString(friendName); // prevent SQL injection - normal name don't must changed by this call sLog->outDebug(LOG_FILTER_NETWORKIO, "WORLD: %s asked to add friend : '%s'", GetPlayer()->GetName(), friendName.c_str()); @@ -642,7 +642,7 @@ void WorldSession::HandleAddIgnoreOpcode(WorldPacket & recv_data) if (!normalizePlayerName(IgnoreName)) return; - CharacterDatabase.escape_string(IgnoreName); // prevent SQL injection - normal name don't must changed by this call + CharacterDatabase.EscapeString(IgnoreName); // prevent SQL injection - normal name don't must changed by this call sLog->outDebug(LOG_FILTER_NETWORKIO, "WORLD: %s asked to Ignore: '%s'", GetPlayer()->GetName(), IgnoreName.c_str()); @@ -728,8 +728,8 @@ void WorldSession::HandleBugOpcode(WorldPacket & recv_data) sLog->outDebug(LOG_FILTER_NETWORKIO, "%s", type.c_str()); sLog->outDebug(LOG_FILTER_NETWORKIO, "%s", content.c_str()); - CharacterDatabase.escape_string(type); - CharacterDatabase.escape_string(content); + CharacterDatabase.EscapeString(type); + CharacterDatabase.EscapeString(content); CharacterDatabase.PExecute ("INSERT INTO bugreport (type, content) VALUES('%s', '%s')", type.c_str(), content.c_str()); } diff --git a/src/server/game/Server/Protocol/Handlers/PetHandler.cpp b/src/server/game/Server/Protocol/Handlers/PetHandler.cpp index 625ce518e28..8a40d8345fa 100755 --- a/src/server/game/Server/Protocol/Handlers/PetHandler.cpp +++ b/src/server/game/Server/Protocol/Handlers/PetHandler.cpp @@ -643,13 +643,13 @@ void WorldSession::HandlePetRename(WorldPacket & recv_data) if (isdeclined) { for (uint8 i = 0; i < MAX_DECLINED_NAME_CASES; ++i) - CharacterDatabase.escape_string(declinedname.name[i]); + CharacterDatabase.EscapeString(declinedname.name[i]); trans->PAppend("DELETE FROM character_pet_declinedname WHERE owner = '%u' AND id = '%u'", _player->GetGUIDLow(), pet->GetCharmInfo()->GetPetNumber()); trans->PAppend("INSERT INTO character_pet_declinedname (id, owner, genitive, dative, accusative, instrumental, prepositional) VALUES ('%u', '%u', '%s', '%s', '%s', '%s', '%s')", pet->GetCharmInfo()->GetPetNumber(), _player->GetGUIDLow(), declinedname.name[0].c_str(), declinedname.name[1].c_str(), declinedname.name[2].c_str(), declinedname.name[3].c_str(), declinedname.name[4].c_str()); } - CharacterDatabase.escape_string(name); + CharacterDatabase.EscapeString(name); trans->PAppend("UPDATE character_pet SET name = '%s', renamed = '1' WHERE owner = '%u' AND id = '%u'", name.c_str(), _player->GetGUIDLow(), pet->GetCharmInfo()->GetPetNumber()); CharacterDatabase.CommitTransaction(trans); diff --git a/src/server/game/Server/Protocol/Handlers/PetitionsHandler.cpp b/src/server/game/Server/Protocol/Handlers/PetitionsHandler.cpp index b47ccfaba8d..0b748594ee0 100755 --- a/src/server/game/Server/Protocol/Handlers/PetitionsHandler.cpp +++ b/src/server/game/Server/Protocol/Handlers/PetitionsHandler.cpp @@ -233,7 +233,7 @@ void WorldSession::HandlePetitionBuyOpcode(WorldPacket & recv_data) ssInvalidPetitionGUIDs << "'" << charter->GetGUIDLow() << "'"; sLog->outDebug(LOG_FILTER_NETWORKIO, "Invalid petition GUIDs: %s", ssInvalidPetitionGUIDs.str().c_str()); - CharacterDatabase.escape_string(name); + CharacterDatabase.EscapeString(name); SQLTransaction trans = CharacterDatabase.BeginTransaction(); trans->PAppend("DELETE FROM petition WHERE petitionguid IN (%s)", ssInvalidPetitionGUIDs.str().c_str()); trans->PAppend("DELETE FROM petition_sign WHERE petitionguid IN (%s)", ssInvalidPetitionGUIDs.str().c_str()); @@ -432,7 +432,7 @@ void WorldSession::HandlePetitionRenameOpcode(WorldPacket & recv_data) } std::string db_newname = newname; - CharacterDatabase.escape_string(db_newname); + CharacterDatabase.EscapeString(db_newname); CharacterDatabase.PExecute("UPDATE petition SET name = '%s' WHERE petitionguid = '%u'", db_newname.c_str(), GUID_LOPART(petitionguid)); diff --git a/src/server/game/Server/WorldSocket.cpp b/src/server/game/Server/WorldSocket.cpp index b2c8164d6f5..fb152cfd331 100755 --- a/src/server/game/Server/WorldSocket.cpp +++ b/src/server/game/Server/WorldSocket.cpp @@ -812,7 +812,7 @@ int WorldSocket::HandleAuthSession (WorldPacket& recvPacket) // Get the account information from the realmd database std::string safe_account = account; // Duplicate, else will screw the SHA hash verification below - LoginDatabase.escape_string (safe_account); + LoginDatabase.EscapeString (safe_account); // No SQL injection, username escaped. QueryResult result = @@ -978,7 +978,7 @@ int WorldSocket::HandleAuthSession (WorldPacket& recvPacket) // Update the last_ip in the database // No SQL injection, username escaped. - LoginDatabase.escape_string (address); + LoginDatabase.EscapeString (address); LoginDatabase.PExecute ("UPDATE account " "SET last_ip = '%s' " diff --git a/src/server/game/Tools/PlayerDump.cpp b/src/server/game/Tools/PlayerDump.cpp index 7f4dc15c5c3..4a1c525167b 100755 --- a/src/server/game/Tools/PlayerDump.cpp +++ b/src/server/game/Tools/PlayerDump.cpp @@ -206,7 +206,7 @@ std::string CreateDumpString(char const* tableName, QueryResult result) else ss << ", '"; std::string s = fields[i].GetString(); - CharacterDatabase.escape_string(s); + CharacterDatabase.EscapeString(s); ss << s; ss << "'"; @@ -411,7 +411,7 @@ DumpReturn PlayerDumpReader::LoadDump(const std::string& file, uint32 account, s if (ObjectMgr::CheckPlayerName(name, true) == CHAR_NAME_SUCCESS) { - CharacterDatabase.escape_string(name); // for safe, we use name only for sql quearies anyway + CharacterDatabase.EscapeString(name); // for safe, we use name only for sql quearies anyway result = CharacterDatabase.PQuery("SELECT 1 FROM characters WHERE name = '%s'", name.c_str()); if (result) name = ""; // use the one from the dump @@ -507,7 +507,7 @@ DumpReturn PlayerDumpReader::LoadDump(const std::string& file, uint32 account, s { // check if the original name already exists name = getnth(line, 3); - CharacterDatabase.escape_string(name); + CharacterDatabase.EscapeString(name); result = CharacterDatabase.PQuery("SELECT 1 FROM characters WHERE name = '%s'", name.c_str()); if (result) diff --git a/src/server/scripts/Commands/cs_go.cpp b/src/server/scripts/Commands/cs_go.cpp index 7129aa67c99..8393d338e93 100644 --- a/src/server/scripts/Commands/cs_go.cpp +++ b/src/server/scripts/Commands/cs_go.cpp @@ -113,7 +113,7 @@ public: if (!guid) { std::string name = pParam1; - WorldDatabase.escape_string(name); + WorldDatabase.EscapeString(name); whereClause << ", creature_template WHERE creature.id = creature_template.entry AND creature_template.name "_LIKE_" '" << name << "'"; } else diff --git a/src/server/scripts/Commands/cs_gobject.cpp b/src/server/scripts/Commands/cs_gobject.cpp index 3197595812f..204012224d5 100644 --- a/src/server/scripts/Commands/cs_gobject.cpp +++ b/src/server/scripts/Commands/cs_gobject.cpp @@ -233,7 +233,7 @@ public: else { std::string name = cId; - WorldDatabase.escape_string(name); + WorldDatabase.EscapeString(name); result = WorldDatabase.PQuery( "SELECT guid, id, position_x, position_y, position_z, orientation, map, phaseMask, (POW(position_x - %f, 2) + POW(position_y - %f, 2) + POW(position_z - %f, 2)) AS order_ " "FROM gameobject, gameobject_template WHERE gameobject_template.entry = gameobject.id AND map = %i AND name "_LIKE_" "_CONCAT3_("'%%'", "'%s'", "'%%'")" ORDER BY order_ ASC LIMIT 1", diff --git a/src/server/scripts/Commands/cs_wp.cpp b/src/server/scripts/Commands/cs_wp.cpp index 4addf788183..1da11795742 100644 --- a/src/server/scripts/Commands/cs_wp.cpp +++ b/src/server/scripts/Commands/cs_wp.cpp @@ -434,7 +434,7 @@ public: else { std::string arg_str_3 = arg_3; - WorldDatabase.escape_string(arg_str_3); + WorldDatabase.EscapeString(arg_str_3); WorldDatabase.PExecute("UPDATE waypoint_scripts SET %s='%s' WHERE guid='%u'", arg_2, arg_str_3.c_str(), id); } @@ -619,7 +619,7 @@ public: { // show_str check for present in list of correct values, no sql injection possible std::string text2 = text; - WorldDatabase.escape_string(text2); + WorldDatabase.EscapeString(text2); WorldDatabase.PExecute("UPDATE waypoint_data SET %s='%s' WHERE id='%u' AND point='%u'", show_str, text2.c_str(), pathid, point); } diff --git a/src/server/shared/Database/DatabaseWorkerPool.h b/src/server/shared/Database/DatabaseWorkerPool.h index 2be778a1028..5e24608715a 100755 --- a/src/server/shared/Database/DatabaseWorkerPool.h +++ b/src/server/shared/Database/DatabaseWorkerPool.h @@ -408,13 +408,13 @@ class DatabaseWorkerPool } //! Apply escape string'ing for current collation. (utf8) - void escape_string(std::string& str) + void EscapeString(std::string& str) { if (str.empty()) return; char* buf = new char[str.size()*2+1]; - escape_string(buf, str.c_str(), str.size()); + EscapeString(buf, str.c_str(), str.size()); str = buf; delete[] buf; } @@ -441,7 +441,7 @@ class DatabaseWorkerPool } private: - unsigned long escape_string(char *to, const char *from, unsigned long length) + unsigned long EscapeString(char *to, const char *from, unsigned long length) { if (!to || !from || !length) return 0; diff --git a/src/server/shared/Logging/Log.cpp b/src/server/shared/Logging/Log.cpp index e755e7208c3..ac74f893d58 100755 --- a/src/server/shared/Logging/Log.cpp +++ b/src/server/shared/Logging/Log.cpp @@ -364,7 +364,7 @@ void Log::outDB(LogTypes type, const char * str) std::string new_str(str); if (new_str.empty()) return; - LoginDatabase.escape_string(new_str); + LoginDatabase.EscapeString(new_str); LoginDatabase.PExecute("INSERT INTO logs (time, realm, type, string) " "VALUES (" UI64FMTD ", %u, %u, '%s');", uint64(time(0)), realm, type, new_str.c_str()); diff --git a/src/server/worldserver/RemoteAccess/RASocket.cpp b/src/server/worldserver/RemoteAccess/RASocket.cpp index cfdf7248611..4344bfea048 100755 --- a/src/server/worldserver/RemoteAccess/RASocket.cpp +++ b/src/server/worldserver/RemoteAccess/RASocket.cpp @@ -177,7 +177,7 @@ int RASocket::check_access_level(const std::string& user) std::string safe_user = user; AccountMgr::normalizeString(safe_user); - LoginDatabase.escape_string(safe_user); + LoginDatabase.EscapeString(safe_user); QueryResult result = LoginDatabase.PQuery("SELECT a.id, aa.gmlevel, aa.RealmID FROM account a LEFT JOIN account_access aa ON (a.id = aa.id) WHERE a.username = '%s'", safe_user.c_str()); @@ -207,11 +207,11 @@ int RASocket::check_password(const std::string& user, const std::string& pass) { std::string safe_user = user; AccountMgr::normalizeString(safe_user); - LoginDatabase.escape_string(safe_user); + LoginDatabase.EscapeString(safe_user); std::string safe_pass = pass; AccountMgr::normalizeString(safe_pass); - LoginDatabase.escape_string(safe_pass); + LoginDatabase.EscapeString(safe_pass); std::string hash = sAccountMgr->CalculateShaPassHash(safe_user, safe_pass); -- cgit v1.2.3 From a9c9ab0efa7e41c31f52e71749048300d809d69a Mon Sep 17 00:00:00 2001 From: click Date: Fri, 1 Jul 2011 03:00:39 +0200 Subject: Core: Clean up script variables to make variables consistant/standardized, and some additional warning-removals: + Fix compilewarning in Unit.cpp + Remove/comment out unused parameters for some commandscripts/scripts (warnings) + Replace "plr" with "player" all over the scriptbase for consistency --- src/server/game/Entities/Unit/Unit.cpp | 2 +- src/server/scripts/Commands/cs_debug.cpp | 8 +- src/server/scripts/Commands/cs_gps.cpp | 2 +- src/server/scripts/Commands/cs_modify.cpp | 10 +- .../ScarletEnclave/the_scarlet_enclave.cpp | 16 +-- .../ScarletMonastery/boss_headless_horseman.cpp | 32 ++--- .../SunwellPlateau/instance_sunwell_plateau.cpp | 6 +- .../scripts/EasternKingdoms/isle_of_queldanas.cpp | 6 +- .../instance_culling_of_stratholme.cpp | 4 +- .../instance_old_hillsbrad.cpp | 4 +- .../RazorfenKraul/instance_razorfen_kraul.cpp | 4 +- src/server/scripts/Kalimdor/silithus.cpp | 40 +++--- .../AzjolNerub/ahnkahet/boss_herald_volazj.cpp | 6 +- .../ObsidianSanctum/boss_sartharion.cpp | 6 +- .../IcecrownCitadel/boss_blood_queen_lana_thel.cpp | 4 +- .../IcecrownCitadel/boss_professor_putricide.cpp | 4 +- .../Northrend/Nexus/Nexus/boss_anomalus.cpp | 2 +- .../Northrend/Ulduar/ulduar/boss_kologarn.cpp | 6 +- .../scripts/Northrend/Ulduar/ulduar/boss_xt002.cpp | 8 +- .../UtgardeKeep/instance_utgarde_keep.cpp | 4 +- src/server/scripts/OutdoorPvP/OutdoorPvPEP.cpp | 148 ++++++++++----------- src/server/scripts/OutdoorPvP/OutdoorPvPEP.h | 22 +-- src/server/scripts/OutdoorPvP/OutdoorPvPHP.cpp | 68 +++++----- src/server/scripts/OutdoorPvP/OutdoorPvPHP.h | 12 +- src/server/scripts/OutdoorPvP/OutdoorPvPNA.cpp | 136 +++++++++---------- src/server/scripts/OutdoorPvP/OutdoorPvPNA.h | 16 +-- src/server/scripts/OutdoorPvP/OutdoorPvPSI.cpp | 68 +++++----- src/server/scripts/OutdoorPvP/OutdoorPvPSI.h | 12 +- src/server/scripts/OutdoorPvP/OutdoorPvPTF.cpp | 64 ++++----- src/server/scripts/OutdoorPvP/OutdoorPvPTF.h | 10 +- src/server/scripts/OutdoorPvP/OutdoorPvPZM.cpp | 132 +++++++++--------- src/server/scripts/OutdoorPvP/OutdoorPvPZM.h | 20 +-- .../Outland/BlackTemple/instance_black_temple.cpp | 4 +- src/server/scripts/Outland/shadowmoon_valley.cpp | 62 ++++----- src/server/scripts/Spells/spell_generic.cpp | 22 +-- 35 files changed, 485 insertions(+), 485 deletions(-) (limited to 'src/server/scripts/Commands') diff --git a/src/server/game/Entities/Unit/Unit.cpp b/src/server/game/Entities/Unit/Unit.cpp index 41ef0ac83fa..4f8d39d5b03 100755 --- a/src/server/game/Entities/Unit/Unit.cpp +++ b/src/server/game/Entities/Unit/Unit.cpp @@ -3797,7 +3797,7 @@ void Unit::RemoveAurasDueToSpellBySteal(uint32 spellId, uint64 casterGUID, Unit if (aura->IsSingleTarget()) aura->UnregisterSingleTarget(); - if (newAura = Aura::TryRefreshStackOrCreate(aura->GetSpellProto(), effMask, stealer, NULL, &baseDamage[0], NULL, aura->GetCasterGUID())) + if (Aura* newAura = Aura::TryRefreshStackOrCreate(aura->GetSpellProto(), effMask, stealer, NULL, &baseDamage[0], NULL, aura->GetCasterGUID())) { // created aura must not be single target aura,, so stealer won't loose it on recast if (newAura->IsSingleTarget()) diff --git a/src/server/scripts/Commands/cs_debug.cpp b/src/server/scripts/Commands/cs_debug.cpp index 41e456289e0..185cf05864d 100644 --- a/src/server/scripts/Commands/cs_debug.cpp +++ b/src/server/scripts/Commands/cs_debug.cpp @@ -392,14 +392,14 @@ public: static bool HandleDebugAreaTriggersCommand(ChatHandler* handler, const char* /*args*/) { - Player* plr = handler->GetSession()->GetPlayer(); - if (!plr->isDebugAreaTriggers) + Player* player = handler->GetSession()->GetPlayer(); + if (!player->isDebugAreaTriggers) { handler->PSendSysMessage(LANG_DEBUG_AREATRIGGER_ON); - plr->isDebugAreaTriggers = true; + player->isDebugAreaTriggers = true; } else { handler->PSendSysMessage(LANG_DEBUG_AREATRIGGER_OFF); - plr->isDebugAreaTriggers = false; + player->isDebugAreaTriggers = false; } return true; } diff --git a/src/server/scripts/Commands/cs_gps.cpp b/src/server/scripts/Commands/cs_gps.cpp index 93e8674143a..a8086f16043 100644 --- a/src/server/scripts/Commands/cs_gps.cpp +++ b/src/server/scripts/Commands/cs_gps.cpp @@ -124,7 +124,7 @@ public: return true; } - static bool HandleWPGPSCommand(ChatHandler* handler, const char *args) + static bool HandleWPGPSCommand(ChatHandler* handler, const char* /*args*/) { Player* player = handler->GetSession()->GetPlayer(); diff --git a/src/server/scripts/Commands/cs_modify.cpp b/src/server/scripts/Commands/cs_modify.cpp index b54e9519da5..c2371243150 100644 --- a/src/server/scripts/Commands/cs_modify.cpp +++ b/src/server/scripts/Commands/cs_modify.cpp @@ -698,15 +698,15 @@ public: return false; } - if (Player* plr = target->ToPlayer()) + if (Player* player = target->ToPlayer()) { // check online security - if (handler->HasLowerSecurity(plr, 0)) + if (handler->HasLowerSecurity(player, 0)) return false; - handler->PSendSysMessage(LANG_YOU_CHANGE_SIZE, Scale, handler->GetNameLink(plr).c_str()); - if (handler->needReportToTarget(plr)) - (ChatHandler(plr)).PSendSysMessage(LANG_YOURS_SIZE_CHANGED, handler->GetNameLink().c_str(), Scale); + handler->PSendSysMessage(LANG_YOU_CHANGE_SIZE, Scale, handler->GetNameLink(player).c_str()); + if (handler->needReportToTarget(player)) + (ChatHandler(player)).PSendSysMessage(LANG_YOURS_SIZE_CHANGED, handler->GetNameLink().c_str(), Scale); } target->SetFloatValue(OBJECT_FIELD_SCALE_X, Scale); diff --git a/src/server/scripts/EasternKingdoms/ScarletEnclave/the_scarlet_enclave.cpp b/src/server/scripts/EasternKingdoms/ScarletEnclave/the_scarlet_enclave.cpp index 487ec83e682..2bea5547a4d 100644 --- a/src/server/scripts/EasternKingdoms/ScarletEnclave/the_scarlet_enclave.cpp +++ b/src/server/scripts/EasternKingdoms/ScarletEnclave/the_scarlet_enclave.cpp @@ -60,13 +60,13 @@ public: { if (FlyBackTimer <= diff) { - Player* plr = NULL; + Player* player = NULL; if (me->isSummon()) if (Unit* summoner = me->ToTempSummon()->GetSummoner()) if (summoner->GetTypeId() == TYPEID_PLAYER) - plr = CAST_PLR(summoner); + player = CAST_PLR(summoner); - if (!plr) + if (!player) phase = 3; switch(phase) @@ -77,19 +77,19 @@ public: FlyBackTimer = 500; break; case 1: - plr->GetClosePoint(x, y, z, me->GetObjectSize()); + player->GetClosePoint(x, y, z, me->GetObjectSize()); z += 2.5; x -= 2; y -= 1.5; me->GetMotionMaster()->MovePoint(0, x, y, z); - me->SetUInt64Value(UNIT_FIELD_TARGET, plr->GetGUID()); + me->SetUInt64Value(UNIT_FIELD_TARGET, player->GetGUID()); me->SetVisible(true); FlyBackTimer = 4500; break; case 2: - if (!plr->isRessurectRequested()) + if (!player->isRessurectRequested()) { me->HandleEmoteCommand(EMOTE_ONESHOT_CUSTOMSPELL01); - DoCast(plr, SPELL_REVIVE, true); - me->MonsterWhisper(VALK_WHISPER, plr->GetGUID()); + DoCast(player, SPELL_REVIVE, true); + me->MonsterWhisper(VALK_WHISPER, player->GetGUID()); } FlyBackTimer = 5000; break; diff --git a/src/server/scripts/EasternKingdoms/ScarletMonastery/boss_headless_horseman.cpp b/src/server/scripts/EasternKingdoms/ScarletMonastery/boss_headless_horseman.cpp index f897f6fc90c..1fdd23fe51d 100644 --- a/src/server/scripts/EasternKingdoms/ScarletMonastery/boss_headless_horseman.cpp +++ b/src/server/scripts/EasternKingdoms/ScarletMonastery/boss_headless_horseman.cpp @@ -470,8 +470,8 @@ public: wp_reached = false; me->RemoveFlag(UNIT_FIELD_FLAGS, UNIT_FLAG_NON_ATTACKABLE); SaySound(SAY_ENTRANCE); - if (Unit* plr = Unit::GetUnit((*me), PlayerGUID)) - DoStartMovement(plr); + if (Unit* player = Unit::GetUnit((*me), PlayerGUID)) + DoStartMovement(player); break; } } @@ -490,9 +490,9 @@ public: if (withhead && Phase != 0) ScriptedAI::MoveInLineOfSight(who); } - void KilledUnit(Unit* plr) + void KilledUnit(Unit* player) { - if (plr->GetTypeId() == TYPEID_PLAYER) + if (player->GetTypeId() == TYPEID_PLAYER) { if (withhead) SaySound(SAY_PLAYER_DEATH); @@ -625,19 +625,19 @@ public: if (say_timer <= diff) { say_timer = 3000; - Player* plr = SelectRandomPlayer(100.0f, false); + Player* player = SelectRandomPlayer(100.0f, false); if (count < 3) { - if (plr) - plr->Say(Text[count], 0); + if (player) + player->Say(Text[count], 0); } else { DoCast(me, SPELL_RHYME_BIG); - if (plr) + if (player) { - plr->Say(Text[count], 0); - plr->HandleEmoteCommand(ANIM_EMOTE_SHOUT); + player->Say(Text[count], 0); + player->HandleEmoteCommand(ANIM_EMOTE_SHOUT); } wp_reached = true; IsFlying = true; @@ -671,8 +671,8 @@ public: case 2: if (conflagrate <= diff) { - if (Unit* plr = SelectRandomPlayer(30.0f)) - DoCast(plr, SPELL_CONFLAGRATION, false); + if (Unit* player = SelectRandomPlayer(30.0f)) + DoCast(player, SPELL_CONFLAGRATION, false); conflagrate = urand(10000, 16000); } else conflagrate -= diff; break; @@ -839,12 +839,12 @@ public: return true; pInstance->SetData(DATA_HORSEMAN_EVENT, IN_PROGRESS); } - /* if (soil->GetGoType() == GAMEOBJECT_TYPE_QUESTGIVER && plr->getLevel() > 64) + /* if (soil->GetGoType() == GAMEOBJECT_TYPE_QUESTGIVER && player->getLevel() > 64) { - plr->PrepareQuestMenu(soil->GetGUID()); - plr->SendPreparedQuest(soil->GetGUID()); + player->PrepareQuestMenu(soil->GetGUID()); + player->SendPreparedQuest(soil->GetGUID()); } - if (plr->GetQuestStatus(11405) == QUEST_STATUS_INCOMPLETE && plr->getLevel() > 64) + if (player->GetQuestStatus(11405) == QUEST_STATUS_INCOMPLETE && player->getLevel() > 64) { */ pPlayer->AreaExploredOrEventHappens(11405); if (Creature* horseman = soil->SummonCreature(HH_MOUNTED, FlightPoint[20].x, FlightPoint[20].y, FlightPoint[20].z, 0, TEMPSUMMON_MANUAL_DESPAWN, 0)) diff --git a/src/server/scripts/EasternKingdoms/SunwellPlateau/instance_sunwell_plateau.cpp b/src/server/scripts/EasternKingdoms/SunwellPlateau/instance_sunwell_plateau.cpp index 9cf1344beba..4ad483d7fb7 100644 --- a/src/server/scripts/EasternKingdoms/SunwellPlateau/instance_sunwell_plateau.cpp +++ b/src/server/scripts/EasternKingdoms/SunwellPlateau/instance_sunwell_plateau.cpp @@ -128,9 +128,9 @@ public: { for (Map::PlayerList::const_iterator itr = players.begin(); itr != players.end(); ++itr) { - Player* plr = itr->getSource(); - if (plr && !plr->HasAura(45839, 0)) - return plr; + Player* player = itr->getSource(); + if (player && !player->HasAura(45839, 0)) + return player; } } diff --git a/src/server/scripts/EasternKingdoms/isle_of_queldanas.cpp b/src/server/scripts/EasternKingdoms/isle_of_queldanas.cpp index 188959cbd23..31512dc6a37 100644 --- a/src/server/scripts/EasternKingdoms/isle_of_queldanas.cpp +++ b/src/server/scripts/EasternKingdoms/isle_of_queldanas.cpp @@ -130,9 +130,9 @@ public: PlayerGUID = caster->GetGUID(); if (PlayerGUID) { - Unit* plr = Unit::GetUnit((*me), PlayerGUID); - if (plr && CAST_PLR(plr)->GetQuestStatus(QUESTG) == QUEST_STATUS_INCOMPLETE) - DoCast(plr, 45110, true); + Unit* player = Unit::GetUnit((*me), PlayerGUID); + if (player && CAST_PLR(player)->GetQuestStatus(QUESTG) == QUEST_STATUS_INCOMPLETE) + DoCast(player, 45110, true); } DoCast(me, ENRAGE); Unit* Myrmidon = me->FindNearestCreature(DM, 70); diff --git a/src/server/scripts/Kalimdor/CavernsOfTime/CullingOfStratholme/instance_culling_of_stratholme.cpp b/src/server/scripts/Kalimdor/CavernsOfTime/CullingOfStratholme/instance_culling_of_stratholme.cpp index e80b0f7d734..f0b871e0cc0 100644 --- a/src/server/scripts/Kalimdor/CavernsOfTime/CullingOfStratholme/instance_culling_of_stratholme.cpp +++ b/src/server/scripts/Kalimdor/CavernsOfTime/CullingOfStratholme/instance_culling_of_stratholme.cpp @@ -182,8 +182,8 @@ class instance_culling_of_stratholme : public InstanceMapScript // Summon Chromie and global whisper if (Creature* chromie = instance->SummonCreature(NPC_CHROMIE_2, ChromieSummonPos)) if (!instance->GetPlayers().isEmpty()) - if (Player* plr = instance->GetPlayers().getFirst()->getSource()) - sCreatureTextMgr->SendChat(chromie, SAY_CRATES_COMPLETED, plr->GetGUID(), CHAT_MSG_ADDON, LANG_ADDON, TEXT_RANGE_MAP); + if (Player* player = instance->GetPlayers().getFirst()->getSource()) + sCreatureTextMgr->SendChat(chromie, SAY_CRATES_COMPLETED, player->GetGUID(), CHAT_MSG_ADDON, LANG_ADDON, TEXT_RANGE_MAP); } DoUpdateWorldState(WORLDSTATE_CRATES_REVEALED, _crateCount); break; diff --git a/src/server/scripts/Kalimdor/CavernsOfTime/EscapeFromDurnholdeKeep/instance_old_hillsbrad.cpp b/src/server/scripts/Kalimdor/CavernsOfTime/EscapeFromDurnholdeKeep/instance_old_hillsbrad.cpp index 54382449303..bf675c7d63b 100644 --- a/src/server/scripts/Kalimdor/CavernsOfTime/EscapeFromDurnholdeKeep/instance_old_hillsbrad.cpp +++ b/src/server/scripts/Kalimdor/CavernsOfTime/EscapeFromDurnholdeKeep/instance_old_hillsbrad.cpp @@ -78,8 +78,8 @@ public: { for (Map::PlayerList::const_iterator itr = players.begin(); itr != players.end(); ++itr) { - if (Player* plr = itr->getSource()) - return plr; + if (Player* player = itr->getSource()) + return player; } } diff --git a/src/server/scripts/Kalimdor/RazorfenKraul/instance_razorfen_kraul.cpp b/src/server/scripts/Kalimdor/RazorfenKraul/instance_razorfen_kraul.cpp index 5943b223857..a251e6dfdf3 100644 --- a/src/server/scripts/Kalimdor/RazorfenKraul/instance_razorfen_kraul.cpp +++ b/src/server/scripts/Kalimdor/RazorfenKraul/instance_razorfen_kraul.cpp @@ -59,8 +59,8 @@ public: { for (Map::PlayerList::const_iterator itr = players.begin(); itr != players.end(); ++itr) { - if (Player* plr = itr->getSource()) - return plr; + if (Player* player = itr->getSource()) + return player; } } sLog->outDebug(LOG_FILTER_TSCR, "TSCR: Instance Razorfen Kraul: GetPlayerInMap, but PlayerList is empty!"); diff --git a/src/server/scripts/Kalimdor/silithus.cpp b/src/server/scripts/Kalimdor/silithus.cpp index 76ee96d5ad7..c53e60f9d7d 100644 --- a/src/server/scripts/Kalimdor/silithus.cpp +++ b/src/server/scripts/Kalimdor/silithus.cpp @@ -522,14 +522,14 @@ public: void HandleAnimation() { - Player* plr = Unit::GetPlayer(*me, PlayerGUID); - if (!plr) + Player* player = Unit::GetPlayer(*me, PlayerGUID); + if (!player) return; - Unit* Fandral = plr->FindNearestCreature(C_FANDRAL_STAGHELM, 100.0f, me); - Unit* Arygos = plr->FindNearestCreature(C_ARYGOS, 100.0f, me); - Unit* Caelestrasz = plr->FindNearestCreature(C_CAELESTRASZ, 100.0f, me); - Unit* Merithra = plr->FindNearestCreature(C_MERITHRA, 100.0f, me); + Unit* Fandral = player->FindNearestCreature(C_FANDRAL_STAGHELM, 100.0f, me); + Unit* Arygos = player->FindNearestCreature(C_ARYGOS, 100.0f, me); + Unit* Caelestrasz = player->FindNearestCreature(C_CAELESTRASZ, 100.0f, me); + Unit* Merithra = player->FindNearestCreature(C_MERITHRA, 100.0f, me); if (!Fandral || !Arygos || !Caelestrasz || !Merithra) return; @@ -659,19 +659,19 @@ public: Caelestrasz->GetMotionMaster()->MoveCharge(-8050, 1473, 65, 15); break; //Text: sands will stop case 34: - DoCast(plr, 23017, true);//Arcane Channeling + DoCast(player, 23017, true);//Arcane Channeling break; case 35: me->CastSpell(-8088, 1520.43f, 2.67f, 25158, true); break; case 36: - DoCast(plr, 25159, true); + DoCast(player, 25159, true); break; case 37: me->SummonGameObject(GO_GATE_OF_AHN_QIRAJ, -8130, 1525, 17.5f, 0, 0, 0, 0, 0, 0); break; case 38: - DoCast(plr, 25166, true); + DoCast(player, 25166, true); me->SummonGameObject(GO_GLYPH_OF_AHN_QIRAJ, -8130, 1525, 17.5f, 0, 0, 0, 0, 0, 0); break; case 39: @@ -719,11 +719,11 @@ public: uint32 entries[4] = { 15423, 15424, 15414, 15422 }; for (uint8 i = 0; i < 4; ++i) { - mob = plr->FindNearestCreature(entries[i], 50, me); + mob = player->FindNearestCreature(entries[i], 50, me); while (mob) { mob->RemoveFromWorld(); - mob = plr->FindNearestCreature(15423, 50, me); + mob = player->FindNearestCreature(15423, 50, me); } } break; @@ -755,14 +755,14 @@ public: me->GetMotionMaster()->MoveCharge(-8117.99f, 1532.24f, 3.94f, 4); break; case 60: - if (plr) - DoScriptText(ANACHRONOS_SAY_10, me, plr); + if (player) + DoScriptText(ANACHRONOS_SAY_10, me, player); me->GetMotionMaster()->MoveCharge(-8113.46f, 1524.16f, 2.89f, 4); break; case 61: me->GetMotionMaster()->MoveCharge(-8057.1f, 1470.32f, 2.61f, 6); - if (plr->IsInRange(me, 0, 15)) - plr->GroupEventHappens(QUEST_A_PAWN_ON_THE_ETERNAL_BOARD , me); + if (player->IsInRange(me, 0, 15)) + player->GroupEventHappens(QUEST_A_PAWN_ON_THE_ETERNAL_BOARD , me); break; case 62: me->SetDisplayId(15500); @@ -842,7 +842,7 @@ public: void UpdateAI(const uint32 diff) { Unit* pTarget = NULL; - //Player* plr = me->GetPlayer(PlayerGUID); + //Player* player = me->GetPlayer(PlayerGUID); if (!Timers) { @@ -1083,12 +1083,12 @@ class go_crystalline_tear : public GameObjectScript public: go_crystalline_tear() : GameObjectScript("go_crystalline_tear") { } - bool OnQuestAccept(Player* plr, GameObject* go, Quest const* quest) + bool OnQuestAccept(Player* player, GameObject* go, Quest const* quest) { if (quest->GetQuestId() == QUEST_A_PAWN_ON_THE_ETERNAL_BOARD) { - if (Unit* Anachronos_Quest_Trigger = go->FindNearestCreature(15454, 100, plr)) + if (Unit* Anachronos_Quest_Trigger = go->FindNearestCreature(15454, 100, player)) { Unit* Merithra = Anachronos_Quest_Trigger->SummonCreature(15378, -8034.535f, 1535.14f, 2.61f, 0, TEMPSUMMON_TIMED_OR_DEAD_DESPAWN, 220000); @@ -1123,9 +1123,9 @@ public: if (Anachronos) { - CAST_AI(npc_anachronos_the_ancient::npc_anachronos_the_ancientAI, Anachronos->AI())->PlayerGUID = plr->GetGUID(); + CAST_AI(npc_anachronos_the_ancient::npc_anachronos_the_ancientAI, Anachronos->AI())->PlayerGUID = player->GetGUID(); CAST_AI(npc_anachronos_quest_trigger::npc_anachronos_quest_triggerAI, CAST_CRE(Anachronos_Quest_Trigger)->AI())->Failed=false; - CAST_AI(npc_anachronos_quest_trigger::npc_anachronos_quest_triggerAI, CAST_CRE(Anachronos_Quest_Trigger)->AI())->PlayerGUID = plr->GetGUID(); + CAST_AI(npc_anachronos_quest_trigger::npc_anachronos_quest_triggerAI, CAST_CRE(Anachronos_Quest_Trigger)->AI())->PlayerGUID = player->GetGUID(); CAST_AI(npc_anachronos_quest_trigger::npc_anachronos_quest_triggerAI, CAST_CRE(Anachronos_Quest_Trigger)->AI())->EventStarted=true; CAST_AI(npc_anachronos_quest_trigger::npc_anachronos_quest_triggerAI, CAST_CRE(Anachronos_Quest_Trigger)->AI())->Announced=true; } diff --git a/src/server/scripts/Northrend/AzjolNerub/ahnkahet/boss_herald_volazj.cpp b/src/server/scripts/Northrend/AzjolNerub/ahnkahet/boss_herald_volazj.cpp index 712bec836bc..1800c204e38 100644 --- a/src/server/scripts/Northrend/AzjolNerub/ahnkahet/boss_herald_volazj.cpp +++ b/src/server/scripts/Northrend/AzjolNerub/ahnkahet/boss_herald_volazj.cpp @@ -123,14 +123,14 @@ public: Map::PlayerList const &players = me->GetMap()->GetPlayers(); for (Map::PlayerList::const_iterator i = players.begin(); i != players.end(); ++i) { - Player* plr = i->getSource(); - if (!plr || !plr->isAlive()) + Player* player = i->getSource(); + if (!player || !player->isAlive()) continue; // Summon clone if (Unit* summon = me->SummonCreature(MOB_TWISTED_VISAGE, me->GetPositionX(), me->GetPositionY(), me->GetPositionZ(), me->GetOrientation(), TEMPSUMMON_CORPSE_DESPAWN, 0)) { // clone - plr->CastSpell(summon, SPELL_CLONE_PLAYER, true); + player->CastSpell(summon, SPELL_CLONE_PLAYER, true); // set phase summon->SetPhaseMask((1<<(4+insanityHandled)), true); } diff --git a/src/server/scripts/Northrend/ChamberOfAspects/ObsidianSanctum/boss_sartharion.cpp b/src/server/scripts/Northrend/ChamberOfAspects/ObsidianSanctum/boss_sartharion.cpp index 7ce54792e9b..cdd2b2e0cde 100644 --- a/src/server/scripts/Northrend/ChamberOfAspects/ObsidianSanctum/boss_sartharion.cpp +++ b/src/server/scripts/Northrend/ChamberOfAspects/ObsidianSanctum/boss_sartharion.cpp @@ -1705,7 +1705,7 @@ class achievement_twilight_assist : public AchievementCriteriaScript { } - bool OnCheck(Player* player, Unit* target) + bool OnCheck(Player* /*player*/, Unit* target) { if (!target) return false; @@ -1725,7 +1725,7 @@ class achievement_twilight_duo : public AchievementCriteriaScript { } - bool OnCheck(Player* player, Unit* target) + bool OnCheck(Player* /*player*/, Unit* target) { if (!target) return false; @@ -1745,7 +1745,7 @@ class achievement_twilight_zone : public AchievementCriteriaScript { } - bool OnCheck(Player* player, Unit* target) + bool OnCheck(Player* /*player*/, Unit* target) { if (!target) return false; diff --git a/src/server/scripts/Northrend/IcecrownCitadel/boss_blood_queen_lana_thel.cpp b/src/server/scripts/Northrend/IcecrownCitadel/boss_blood_queen_lana_thel.cpp index 355321e4d14..5cde7ada233 100644 --- a/src/server/scripts/Northrend/IcecrownCitadel/boss_blood_queen_lana_thel.cpp +++ b/src/server/scripts/Northrend/IcecrownCitadel/boss_blood_queen_lana_thel.cpp @@ -186,8 +186,8 @@ class boss_blood_queen_lana_thel : public CreatureScript if (_creditBloodQuickening) { instance->SetData(DATA_BLOOD_QUICKENING_STATE, DONE); - if (Player* plr = killer->ToPlayer()) - plr->RewardPlayerAndGroupAtEvent(NPC_INFILTRATOR_MINCHAR_BQ, plr); + if (Player* player = killer->ToPlayer()) + player->RewardPlayerAndGroupAtEvent(NPC_INFILTRATOR_MINCHAR_BQ, player); if (Creature* minchar = me->FindNearestCreature(NPC_INFILTRATOR_MINCHAR_BQ, 200.0f)) { minchar->SetUInt32Value(UNIT_NPC_EMOTESTATE, 0); diff --git a/src/server/scripts/Northrend/IcecrownCitadel/boss_professor_putricide.cpp b/src/server/scripts/Northrend/IcecrownCitadel/boss_professor_putricide.cpp index bb57325d8c0..450875662e4 100755 --- a/src/server/scripts/Northrend/IcecrownCitadel/boss_professor_putricide.cpp +++ b/src/server/scripts/Northrend/IcecrownCitadel/boss_professor_putricide.cpp @@ -1393,8 +1393,8 @@ class spell_putricide_mutated_transformation : public SpellScriptLoader if (putricide->AI()->GetData(DATA_ABOMINATION)) { - if (Player* plrCaster = caster->ToPlayer()) - Spell::SendCastResult(plrCaster, GetSpellInfo(), 0, SPELL_FAILED_CUSTOM_ERROR, SPELL_CUSTOM_ERROR_TOO_MANY_ABOMINATIONS); + if (Player* player = caster->ToPlayer()) + Spell::SendCastResult(player, GetSpellInfo(), 0, SPELL_FAILED_CUSTOM_ERROR, SPELL_CUSTOM_ERROR_TOO_MANY_ABOMINATIONS); return; } diff --git a/src/server/scripts/Northrend/Nexus/Nexus/boss_anomalus.cpp b/src/server/scripts/Northrend/Nexus/Nexus/boss_anomalus.cpp index abe4d4ce7be..4f8e15b6111 100644 --- a/src/server/scripts/Northrend/Nexus/Nexus/boss_anomalus.cpp +++ b/src/server/scripts/Northrend/Nexus/Nexus/boss_anomalus.cpp @@ -117,7 +117,7 @@ class boss_anomalus : public CreatureScript return 0; } - void SummonedCreatureDies(Creature* summoned, Unit* who) + void SummonedCreatureDies(Creature* summoned, Unit* /*who*/) { if (summoned->GetEntry() == MOB_CHAOTIC_RIFT) chaosTheory = false; diff --git a/src/server/scripts/Northrend/Ulduar/ulduar/boss_kologarn.cpp b/src/server/scripts/Northrend/Ulduar/ulduar/boss_kologarn.cpp index d19a44bc2a1..4669b953241 100644 --- a/src/server/scripts/Northrend/Ulduar/ulduar/boss_kologarn.cpp +++ b/src/server/scripts/Northrend/Ulduar/ulduar/boss_kologarn.cpp @@ -428,11 +428,11 @@ class spell_ulduar_stone_grip_cast_target : public SpellScriptLoader void HandleForceCast(SpellEffIndex i) { - Player* plr = GetHitPlayer(); - if (!plr) + Player* player = GetHitPlayer(); + if (!player) return; - plr->CastSpell(GetTargetUnit(), GetSpellInfo()->EffectTriggerSpell[i], true); // Don't send m_originalCasterGUID param here or underlying + player->CastSpell(GetTargetUnit(), GetSpellInfo()->EffectTriggerSpell[i], true); // Don't send m_originalCasterGUID param here or underlying PreventHitEffect(i); // AureEffect::HandleAuraControlVehicle will fail on caster == target } diff --git a/src/server/scripts/Northrend/Ulduar/ulduar/boss_xt002.cpp b/src/server/scripts/Northrend/Ulduar/ulduar/boss_xt002.cpp index 26f3abbd00b..082365565c4 100644 --- a/src/server/scripts/Northrend/Ulduar/ulduar/boss_xt002.cpp +++ b/src/server/scripts/Northrend/Ulduar/ulduar/boss_xt002.cpp @@ -767,10 +767,10 @@ class spell_xt002_searing_light_spawn_life_spark : public SpellScriptLoader void OnRemove(AuraEffect const* aurEff, AuraEffectHandleModes /*mode*/) { - if (Player* plr = GetOwner()->ToPlayer()) + if (Player* player = GetOwner()->ToPlayer()) if (Unit* xt002 = GetCaster()) if (xt002->HasAura(aurEff->GetAmount())) // Heartbreak aura indicating hard mode - plr->CastSpell(plr, SPELL_SUMMON_LIFE_SPARK, true); + player->CastSpell(player, SPELL_SUMMON_LIFE_SPARK, true); } void Register() @@ -803,10 +803,10 @@ class spell_xt002_gravity_bomb_aura : public SpellScriptLoader void OnRemove(AuraEffect const* aurEff, AuraEffectHandleModes /*mode*/) { - if (Player* plr = GetOwner()->ToPlayer()) + if (Player* player = GetOwner()->ToPlayer()) if (Unit* xt002 = GetCaster()) if (xt002->HasAura(aurEff->GetAmount())) // Heartbreak aura indicating hard mode - plr->CastSpell(plr, SPELL_SUMMON_VOID_ZONE, true); + player->CastSpell(player, SPELL_SUMMON_VOID_ZONE, true); } void OnPeriodic(AuraEffect const* aurEff) diff --git a/src/server/scripts/Northrend/UtgardeKeep/UtgardeKeep/instance_utgarde_keep.cpp b/src/server/scripts/Northrend/UtgardeKeep/UtgardeKeep/instance_utgarde_keep.cpp index 443512d5986..94f17de43f0 100644 --- a/src/server/scripts/Northrend/UtgardeKeep/UtgardeKeep/instance_utgarde_keep.cpp +++ b/src/server/scripts/Northrend/UtgardeKeep/UtgardeKeep/instance_utgarde_keep.cpp @@ -113,8 +113,8 @@ public: { for (Map::PlayerList::const_iterator itr = players.begin(); itr != players.end(); ++itr) { - if (Player* plr = itr->getSource()) - return plr; + if (Player* player = itr->getSource()) + return player; } } diff --git a/src/server/scripts/OutdoorPvP/OutdoorPvPEP.cpp b/src/server/scripts/OutdoorPvP/OutdoorPvPEP.cpp index 5eaa19b4f82..7fc24a75d61 100755 --- a/src/server/scripts/OutdoorPvP/OutdoorPvPEP.cpp +++ b/src/server/scripts/OutdoorPvP/OutdoorPvPEP.cpp @@ -127,23 +127,23 @@ void OPvPCapturePointEP_EWT::UpdateTowerState() m_PvP->SendUpdateWorldState(EP_EWT_N , bool(m_TowerState & EP_TS_N)); } -bool OPvPCapturePointEP_EWT::HandlePlayerEnter(Player* plr) +bool OPvPCapturePointEP_EWT::HandlePlayerEnter(Player* player) { - if (OPvPCapturePoint::HandlePlayerEnter(plr)) + if (OPvPCapturePoint::HandlePlayerEnter(player)) { - plr->SendUpdateWorldState(EP_UI_TOWER_SLIDER_DISPLAY, 1); + player->SendUpdateWorldState(EP_UI_TOWER_SLIDER_DISPLAY, 1); uint32 phase = (uint32)ceil((m_value + m_maxValue) / (2 * m_maxValue) * 100.0f); - plr->SendUpdateWorldState(EP_UI_TOWER_SLIDER_POS, phase); - plr->SendUpdateWorldState(EP_UI_TOWER_SLIDER_N, m_neutralValuePct); + player->SendUpdateWorldState(EP_UI_TOWER_SLIDER_POS, phase); + player->SendUpdateWorldState(EP_UI_TOWER_SLIDER_N, m_neutralValuePct); return true; } return false; } -void OPvPCapturePointEP_EWT::HandlePlayerLeave(Player* plr) +void OPvPCapturePointEP_EWT::HandlePlayerLeave(Player* player) { - plr->SendUpdateWorldState(EP_UI_TOWER_SLIDER_DISPLAY, 0); - OPvPCapturePoint::HandlePlayerLeave(plr); + player->SendUpdateWorldState(EP_UI_TOWER_SLIDER_DISPLAY, 0); + OPvPCapturePoint::HandlePlayerLeave(player); } void OPvPCapturePointEP_EWT::SummonSupportUnitAtNorthpassTower(uint32 team) @@ -276,23 +276,23 @@ void OPvPCapturePointEP_NPT::UpdateTowerState() m_PvP->SendUpdateWorldState(EP_NPT_N , bool(m_TowerState & EP_TS_N)); } -bool OPvPCapturePointEP_NPT::HandlePlayerEnter(Player* plr) +bool OPvPCapturePointEP_NPT::HandlePlayerEnter(Player* player) { - if (OPvPCapturePoint::HandlePlayerEnter(plr)) + if (OPvPCapturePoint::HandlePlayerEnter(player)) { - plr->SendUpdateWorldState(EP_UI_TOWER_SLIDER_DISPLAY, 1); + player->SendUpdateWorldState(EP_UI_TOWER_SLIDER_DISPLAY, 1); uint32 phase = (uint32)ceil((m_value + m_maxValue) / (2 * m_maxValue) * 100.0f); - plr->SendUpdateWorldState(EP_UI_TOWER_SLIDER_POS, phase); - plr->SendUpdateWorldState(EP_UI_TOWER_SLIDER_N, m_neutralValuePct); + player->SendUpdateWorldState(EP_UI_TOWER_SLIDER_POS, phase); + player->SendUpdateWorldState(EP_UI_TOWER_SLIDER_N, m_neutralValuePct); return true; } return false; } -void OPvPCapturePointEP_NPT::HandlePlayerLeave(Player* plr) +void OPvPCapturePointEP_NPT::HandlePlayerLeave(Player* player) { - plr->SendUpdateWorldState(EP_UI_TOWER_SLIDER_DISPLAY, 0); - OPvPCapturePoint::HandlePlayerLeave(plr); + player->SendUpdateWorldState(EP_UI_TOWER_SLIDER_DISPLAY, 0); + OPvPCapturePoint::HandlePlayerLeave(player); } void OPvPCapturePointEP_NPT::SummonGO(uint32 team) @@ -408,23 +408,23 @@ void OPvPCapturePointEP_CGT::UpdateTowerState() m_PvP->SendUpdateWorldState(EP_CGT_N , bool(m_TowerState & EP_TS_N)); } -bool OPvPCapturePointEP_CGT::HandlePlayerEnter(Player* plr) +bool OPvPCapturePointEP_CGT::HandlePlayerEnter(Player* player) { - if (OPvPCapturePoint::HandlePlayerEnter(plr)) + if (OPvPCapturePoint::HandlePlayerEnter(player)) { - plr->SendUpdateWorldState(EP_UI_TOWER_SLIDER_DISPLAY, 1); + player->SendUpdateWorldState(EP_UI_TOWER_SLIDER_DISPLAY, 1); uint32 phase = (uint32)ceil((m_value + m_maxValue) / (2 * m_maxValue) * 100.0f); - plr->SendUpdateWorldState(EP_UI_TOWER_SLIDER_POS, phase); - plr->SendUpdateWorldState(EP_UI_TOWER_SLIDER_N, m_neutralValuePct); + player->SendUpdateWorldState(EP_UI_TOWER_SLIDER_POS, phase); + player->SendUpdateWorldState(EP_UI_TOWER_SLIDER_N, m_neutralValuePct); return true; } return false; } -void OPvPCapturePointEP_CGT::HandlePlayerLeave(Player* plr) +void OPvPCapturePointEP_CGT::HandlePlayerLeave(Player* player) { - plr->SendUpdateWorldState(EP_UI_TOWER_SLIDER_DISPLAY, 0); - OPvPCapturePoint::HandlePlayerLeave(plr); + player->SendUpdateWorldState(EP_UI_TOWER_SLIDER_DISPLAY, 0); + OPvPCapturePoint::HandlePlayerLeave(player); } void OPvPCapturePointEP_CGT::LinkGraveYard(uint32 team) @@ -547,23 +547,23 @@ void OPvPCapturePointEP_PWT::UpdateTowerState() m_PvP->SendUpdateWorldState(EP_PWT_N , bool(m_TowerState & EP_TS_N)); } -bool OPvPCapturePointEP_PWT::HandlePlayerEnter(Player* plr) +bool OPvPCapturePointEP_PWT::HandlePlayerEnter(Player* player) { - if (OPvPCapturePoint::HandlePlayerEnter(plr)) + if (OPvPCapturePoint::HandlePlayerEnter(player)) { - plr->SendUpdateWorldState(EP_UI_TOWER_SLIDER_DISPLAY, 1); + player->SendUpdateWorldState(EP_UI_TOWER_SLIDER_DISPLAY, 1); uint32 phase = (uint32)ceil((m_value + m_maxValue) / (2 * m_maxValue) * 100.0f); - plr->SendUpdateWorldState(EP_UI_TOWER_SLIDER_POS, phase); - plr->SendUpdateWorldState(EP_UI_TOWER_SLIDER_N, m_neutralValuePct); + player->SendUpdateWorldState(EP_UI_TOWER_SLIDER_POS, phase); + player->SendUpdateWorldState(EP_UI_TOWER_SLIDER_N, m_neutralValuePct); return true; } return false; } -void OPvPCapturePointEP_PWT::HandlePlayerLeave(Player* plr) +void OPvPCapturePointEP_PWT::HandlePlayerLeave(Player* player) { - plr->SendUpdateWorldState(EP_UI_TOWER_SLIDER_DISPLAY, 0); - OPvPCapturePoint::HandlePlayerLeave(plr); + player->SendUpdateWorldState(EP_UI_TOWER_SLIDER_DISPLAY, 0); + OPvPCapturePoint::HandlePlayerLeave(player); } void OPvPCapturePointEP_PWT::SummonFlightMaster(uint32 team) @@ -658,58 +658,58 @@ bool OutdoorPvPEP::Update(uint32 diff) return false; } -void OutdoorPvPEP::HandlePlayerEnterZone(Player* plr, uint32 zone) +void OutdoorPvPEP::HandlePlayerEnterZone(Player* player, uint32 zone) { // add buffs - if (plr->GetTeam() == ALLIANCE) + if (player->GetTeam() == ALLIANCE) { if (m_AllianceTowersControlled && m_AllianceTowersControlled < 5) - plr->CastSpell(plr, EP_AllianceBuffs[m_AllianceTowersControlled-1], true); + player->CastSpell(player, EP_AllianceBuffs[m_AllianceTowersControlled-1], true); } else { if (m_HordeTowersControlled && m_HordeTowersControlled < 5) - plr->CastSpell(plr, EP_HordeBuffs[m_HordeTowersControlled-1], true); + player->CastSpell(player, EP_HordeBuffs[m_HordeTowersControlled-1], true); } - OutdoorPvP::HandlePlayerEnterZone(plr, zone); + OutdoorPvP::HandlePlayerEnterZone(player, zone); } -void OutdoorPvPEP::HandlePlayerLeaveZone(Player* plr, uint32 zone) +void OutdoorPvPEP::HandlePlayerLeaveZone(Player* player, uint32 zone) { // remove buffs - if (plr->GetTeam() == ALLIANCE) + if (player->GetTeam() == ALLIANCE) { for (int i = 0; i < 4; ++i) - plr->RemoveAurasDueToSpell(EP_AllianceBuffs[i]); + player->RemoveAurasDueToSpell(EP_AllianceBuffs[i]); } else { for (int i = 0; i < 4; ++i) - plr->RemoveAurasDueToSpell(EP_HordeBuffs[i]); + player->RemoveAurasDueToSpell(EP_HordeBuffs[i]); } - OutdoorPvP::HandlePlayerLeaveZone(plr, zone); + OutdoorPvP::HandlePlayerLeaveZone(player, zone); } void OutdoorPvPEP::BuffTeams() { for (PlayerSet::iterator itr = m_players[0].begin(); itr != m_players[0].end(); ++itr) { - Player* plr = *itr; + Player* player = *itr; { for (int i = 0; i < 4; ++i) - plr->RemoveAurasDueToSpell(EP_AllianceBuffs[i]); + player->RemoveAurasDueToSpell(EP_AllianceBuffs[i]); if (m_AllianceTowersControlled && m_AllianceTowersControlled < 5) - plr->CastSpell(plr, EP_AllianceBuffs[m_AllianceTowersControlled-1], true); + player->CastSpell(player, EP_AllianceBuffs[m_AllianceTowersControlled-1], true); } } for (PlayerSet::iterator itr = m_players[1].begin(); itr != m_players[1].end(); ++itr) { - Player* plr = *itr; + Player* player = *itr; { for (int i = 0; i < 4; ++i) - plr->RemoveAurasDueToSpell(EP_HordeBuffs[i]); + player->RemoveAurasDueToSpell(EP_HordeBuffs[i]); if (m_HordeTowersControlled && m_HordeTowersControlled < 5) - plr->CastSpell(plr, EP_HordeBuffs[m_HordeTowersControlled-1], true); + player->CastSpell(player, EP_HordeBuffs[m_HordeTowersControlled-1], true); } } } @@ -727,37 +727,37 @@ void OutdoorPvPEP::FillInitialWorldStates(WorldPacket & data) } } -void OutdoorPvPEP::SendRemoveWorldStates(Player* plr) +void OutdoorPvPEP::SendRemoveWorldStates(Player* player) { - plr->SendUpdateWorldState(EP_UI_TOWER_COUNT_A, 0); - plr->SendUpdateWorldState(EP_UI_TOWER_COUNT_H, 0); - plr->SendUpdateWorldState(EP_UI_TOWER_SLIDER_DISPLAY, 0); - plr->SendUpdateWorldState(EP_UI_TOWER_SLIDER_POS, 0); - plr->SendUpdateWorldState(EP_UI_TOWER_SLIDER_N, 0); + player->SendUpdateWorldState(EP_UI_TOWER_COUNT_A, 0); + player->SendUpdateWorldState(EP_UI_TOWER_COUNT_H, 0); + player->SendUpdateWorldState(EP_UI_TOWER_SLIDER_DISPLAY, 0); + player->SendUpdateWorldState(EP_UI_TOWER_SLIDER_POS, 0); + player->SendUpdateWorldState(EP_UI_TOWER_SLIDER_N, 0); - plr->SendUpdateWorldState(EP_EWT_A, 0); - plr->SendUpdateWorldState(EP_EWT_H, 0); - plr->SendUpdateWorldState(EP_EWT_N, 0); - plr->SendUpdateWorldState(EP_EWT_N_A, 0); - plr->SendUpdateWorldState(EP_EWT_N_H, 0); + player->SendUpdateWorldState(EP_EWT_A, 0); + player->SendUpdateWorldState(EP_EWT_H, 0); + player->SendUpdateWorldState(EP_EWT_N, 0); + player->SendUpdateWorldState(EP_EWT_N_A, 0); + player->SendUpdateWorldState(EP_EWT_N_H, 0); - plr->SendUpdateWorldState(EP_PWT_A, 0); - plr->SendUpdateWorldState(EP_PWT_H, 0); - plr->SendUpdateWorldState(EP_PWT_N, 0); - plr->SendUpdateWorldState(EP_PWT_N_A, 0); - plr->SendUpdateWorldState(EP_PWT_N_H, 0); + player->SendUpdateWorldState(EP_PWT_A, 0); + player->SendUpdateWorldState(EP_PWT_H, 0); + player->SendUpdateWorldState(EP_PWT_N, 0); + player->SendUpdateWorldState(EP_PWT_N_A, 0); + player->SendUpdateWorldState(EP_PWT_N_H, 0); - plr->SendUpdateWorldState(EP_NPT_A, 0); - plr->SendUpdateWorldState(EP_NPT_H, 0); - plr->SendUpdateWorldState(EP_NPT_N, 0); - plr->SendUpdateWorldState(EP_NPT_N_A, 0); - plr->SendUpdateWorldState(EP_NPT_N_H, 0); + player->SendUpdateWorldState(EP_NPT_A, 0); + player->SendUpdateWorldState(EP_NPT_H, 0); + player->SendUpdateWorldState(EP_NPT_N, 0); + player->SendUpdateWorldState(EP_NPT_N_A, 0); + player->SendUpdateWorldState(EP_NPT_N_H, 0); - plr->SendUpdateWorldState(EP_CGT_A, 0); - plr->SendUpdateWorldState(EP_CGT_H, 0); - plr->SendUpdateWorldState(EP_CGT_N, 0); - plr->SendUpdateWorldState(EP_CGT_N_A, 0); - plr->SendUpdateWorldState(EP_CGT_N_H, 0); + player->SendUpdateWorldState(EP_CGT_A, 0); + player->SendUpdateWorldState(EP_CGT_H, 0); + player->SendUpdateWorldState(EP_CGT_N, 0); + player->SendUpdateWorldState(EP_CGT_N_A, 0); + player->SendUpdateWorldState(EP_CGT_N_H, 0); } class OutdoorPvP_eastern_plaguelands : public OutdoorPvPScript diff --git a/src/server/scripts/OutdoorPvP/OutdoorPvPEP.h b/src/server/scripts/OutdoorPvP/OutdoorPvPEP.h index 54235bbb569..f196e65725c 100755 --- a/src/server/scripts/OutdoorPvP/OutdoorPvPEP.h +++ b/src/server/scripts/OutdoorPvP/OutdoorPvPEP.h @@ -197,8 +197,8 @@ class OPvPCapturePointEP_EWT : public OPvPCapturePoint void FillInitialWorldStates(WorldPacket & data); // used when player is activated/inactivated in the area - bool HandlePlayerEnter(Player* plr); - void HandlePlayerLeave(Player* plr); + bool HandlePlayerEnter(Player* player); + void HandlePlayerLeave(Player* player); protected: @@ -228,8 +228,8 @@ class OPvPCapturePointEP_NPT : public OPvPCapturePoint void FillInitialWorldStates(WorldPacket & data); // used when player is activated/inactivated in the area - bool HandlePlayerEnter(Player* plr); - void HandlePlayerLeave(Player* plr); + bool HandlePlayerEnter(Player* player); + void HandlePlayerLeave(Player* player); protected: @@ -259,8 +259,8 @@ class OPvPCapturePointEP_CGT : public OPvPCapturePoint void FillInitialWorldStates(WorldPacket & data); // used when player is activated/inactivated in the area - bool HandlePlayerEnter(Player* plr); - void HandlePlayerLeave(Player* plr); + bool HandlePlayerEnter(Player* player); + void HandlePlayerLeave(Player* player); protected: @@ -290,8 +290,8 @@ class OPvPCapturePointEP_PWT : public OPvPCapturePoint void FillInitialWorldStates(WorldPacket & data); // used when player is activated/inactivated in the area - bool HandlePlayerEnter(Player* plr); - void HandlePlayerLeave(Player* plr); + bool HandlePlayerEnter(Player* player); + void HandlePlayerLeave(Player* player); protected: @@ -319,14 +319,14 @@ class OutdoorPvPEP : public OutdoorPvP bool SetupOutdoorPvP(); - void HandlePlayerEnterZone(Player* plr, uint32 zone); - void HandlePlayerLeaveZone(Player* plr, uint32 zone); + void HandlePlayerEnterZone(Player* player, uint32 zone); + void HandlePlayerLeaveZone(Player* player, uint32 zone); bool Update(uint32 diff); void FillInitialWorldStates(WorldPacket &data); - void SendRemoveWorldStates(Player* plr); + void SendRemoveWorldStates(Player* player); void BuffTeams(); diff --git a/src/server/scripts/OutdoorPvP/OutdoorPvPHP.cpp b/src/server/scripts/OutdoorPvP/OutdoorPvPHP.cpp index 2224f4cbf4f..2d26ab80e5c 100755 --- a/src/server/scripts/OutdoorPvP/OutdoorPvPHP.cpp +++ b/src/server/scripts/OutdoorPvP/OutdoorPvPHP.cpp @@ -81,34 +81,34 @@ bool OutdoorPvPHP::SetupOutdoorPvP() return true; } -void OutdoorPvPHP::HandlePlayerEnterZone(Player* plr, uint32 zone) +void OutdoorPvPHP::HandlePlayerEnterZone(Player* player, uint32 zone) { // add buffs - if (plr->GetTeam() == ALLIANCE) + if (player->GetTeam() == ALLIANCE) { if (m_AllianceTowersControlled >=3) - plr->CastSpell(plr, AllianceBuff, true); + player->CastSpell(player, AllianceBuff, true); } else { if (m_HordeTowersControlled >=3) - plr->CastSpell(plr, HordeBuff, true); + player->CastSpell(player, HordeBuff, true); } - OutdoorPvP::HandlePlayerEnterZone(plr, zone); + OutdoorPvP::HandlePlayerEnterZone(player, zone); } -void OutdoorPvPHP::HandlePlayerLeaveZone(Player* plr, uint32 zone) +void OutdoorPvPHP::HandlePlayerLeaveZone(Player* player, uint32 zone) { // remove buffs - if (plr->GetTeam() == ALLIANCE) + if (player->GetTeam() == ALLIANCE) { - plr->RemoveAurasDueToSpell(AllianceBuff); + player->RemoveAurasDueToSpell(AllianceBuff); } else { - plr->RemoveAurasDueToSpell(HordeBuff); + player->RemoveAurasDueToSpell(HordeBuff); } - OutdoorPvP::HandlePlayerLeaveZone(plr, zone); + OutdoorPvP::HandlePlayerLeaveZone(player, zone); } bool OutdoorPvPHP::Update(uint32 diff) @@ -131,20 +131,20 @@ bool OutdoorPvPHP::Update(uint32 diff) return changed; } -void OutdoorPvPHP::SendRemoveWorldStates(Player* plr) +void OutdoorPvPHP::SendRemoveWorldStates(Player* player) { - plr->SendUpdateWorldState(HP_UI_TOWER_DISPLAY_A, 0); - plr->SendUpdateWorldState(HP_UI_TOWER_DISPLAY_H, 0); - plr->SendUpdateWorldState(HP_UI_TOWER_COUNT_H, 0); - plr->SendUpdateWorldState(HP_UI_TOWER_COUNT_A, 0); - plr->SendUpdateWorldState(HP_UI_TOWER_SLIDER_N, 0); - plr->SendUpdateWorldState(HP_UI_TOWER_SLIDER_POS, 0); - plr->SendUpdateWorldState(HP_UI_TOWER_SLIDER_DISPLAY, 0); + player->SendUpdateWorldState(HP_UI_TOWER_DISPLAY_A, 0); + player->SendUpdateWorldState(HP_UI_TOWER_DISPLAY_H, 0); + player->SendUpdateWorldState(HP_UI_TOWER_COUNT_H, 0); + player->SendUpdateWorldState(HP_UI_TOWER_COUNT_A, 0); + player->SendUpdateWorldState(HP_UI_TOWER_SLIDER_N, 0); + player->SendUpdateWorldState(HP_UI_TOWER_SLIDER_POS, 0); + player->SendUpdateWorldState(HP_UI_TOWER_SLIDER_DISPLAY, 0); for (int i = 0; i < HP_TOWER_NUM; ++i) { - plr->SendUpdateWorldState(HP_MAP_N[i], 0); - plr->SendUpdateWorldState(HP_MAP_A[i], 0); - plr->SendUpdateWorldState(HP_MAP_H[i], 0); + player->SendUpdateWorldState(HP_MAP_N[i], 0); + player->SendUpdateWorldState(HP_MAP_A[i], 0); + player->SendUpdateWorldState(HP_MAP_H[i], 0); } } @@ -301,34 +301,34 @@ void OPvPCapturePointHP::FillInitialWorldStates(WorldPacket &data) } } -bool OPvPCapturePointHP::HandlePlayerEnter(Player* plr) +bool OPvPCapturePointHP::HandlePlayerEnter(Player* player) { - if (OPvPCapturePoint::HandlePlayerEnter(plr)) + if (OPvPCapturePoint::HandlePlayerEnter(player)) { - plr->SendUpdateWorldState(HP_UI_TOWER_SLIDER_DISPLAY, 1); + player->SendUpdateWorldState(HP_UI_TOWER_SLIDER_DISPLAY, 1); uint32 phase = (uint32)ceil((m_value + m_maxValue) / (2 * m_maxValue) * 100.0f); - plr->SendUpdateWorldState(HP_UI_TOWER_SLIDER_POS, phase); - plr->SendUpdateWorldState(HP_UI_TOWER_SLIDER_N, m_neutralValuePct); + player->SendUpdateWorldState(HP_UI_TOWER_SLIDER_POS, phase); + player->SendUpdateWorldState(HP_UI_TOWER_SLIDER_N, m_neutralValuePct); return true; } return false; } -void OPvPCapturePointHP::HandlePlayerLeave(Player* plr) +void OPvPCapturePointHP::HandlePlayerLeave(Player* player) { - plr->SendUpdateWorldState(HP_UI_TOWER_SLIDER_DISPLAY, 0); - OPvPCapturePoint::HandlePlayerLeave(plr); + player->SendUpdateWorldState(HP_UI_TOWER_SLIDER_DISPLAY, 0); + OPvPCapturePoint::HandlePlayerLeave(player); } -void OutdoorPvPHP::HandleKillImpl(Player* plr, Unit* killed) +void OutdoorPvPHP::HandleKillImpl(Player* player, Unit* killed) { if (killed->GetTypeId() != TYPEID_PLAYER) return; - if (plr->GetTeam() == ALLIANCE && killed->ToPlayer()->GetTeam() != ALLIANCE) - plr->CastSpell(plr, AlliancePlayerKillReward, true); - else if (plr->GetTeam() == HORDE && killed->ToPlayer()->GetTeam() != HORDE) - plr->CastSpell(plr, HordePlayerKillReward, true); + if (player->GetTeam() == ALLIANCE && killed->ToPlayer()->GetTeam() != ALLIANCE) + player->CastSpell(player, AlliancePlayerKillReward, true); + else if (player->GetTeam() == HORDE && killed->ToPlayer()->GetTeam() != HORDE) + player->CastSpell(player, HordePlayerKillReward, true); } class OutdoorPvP_hellfire_peninsula : public OutdoorPvPScript diff --git a/src/server/scripts/OutdoorPvP/OutdoorPvPHP.h b/src/server/scripts/OutdoorPvP/OutdoorPvPHP.h index 3cbc1296b99..da18f639ba1 100755 --- a/src/server/scripts/OutdoorPvP/OutdoorPvPHP.h +++ b/src/server/scripts/OutdoorPvP/OutdoorPvPHP.h @@ -98,8 +98,8 @@ class OPvPCapturePointHP : public OPvPCapturePoint void FillInitialWorldStates(WorldPacket & data); // used when player is activated/inactivated in the area - bool HandlePlayerEnter(Player* plr); - void HandlePlayerLeave(Player* plr); + bool HandlePlayerEnter(Player* player); + void HandlePlayerLeave(Player* player); private: @@ -116,16 +116,16 @@ class OutdoorPvPHP : public OutdoorPvP bool SetupOutdoorPvP(); - void HandlePlayerEnterZone(Player* plr, uint32 zone); - void HandlePlayerLeaveZone(Player* plr, uint32 zone); + void HandlePlayerEnterZone(Player* player, uint32 zone); + void HandlePlayerLeaveZone(Player* player, uint32 zone); bool Update(uint32 diff); void FillInitialWorldStates(WorldPacket &data); - void SendRemoveWorldStates(Player* plr); + void SendRemoveWorldStates(Player* player); - void HandleKillImpl(Player* plr, Unit* killed); + void HandleKillImpl(Player* player, Unit* killed); private: diff --git a/src/server/scripts/OutdoorPvP/OutdoorPvPNA.cpp b/src/server/scripts/OutdoorPvP/OutdoorPvPNA.cpp index 44b939f806f..03b50f77a50 100755 --- a/src/server/scripts/OutdoorPvP/OutdoorPvPNA.cpp +++ b/src/server/scripts/OutdoorPvP/OutdoorPvPNA.cpp @@ -29,15 +29,15 @@ OutdoorPvPNA::OutdoorPvPNA() m_TypeId = OUTDOOR_PVP_NA; } -void OutdoorPvPNA::HandleKillImpl(Player* plr, Unit* killed) +void OutdoorPvPNA::HandleKillImpl(Player* player, Unit* killed) { - if (killed->GetTypeId() == TYPEID_PLAYER && plr->GetTeam() != killed->ToPlayer()->GetTeam()) + if (killed->GetTypeId() == TYPEID_PLAYER && player->GetTeam() != killed->ToPlayer()->GetTeam()) { - plr->KilledMonsterCredit(NA_CREDIT_MARKER, 0); // 0 guid, btw it isn't even used in killedmonster function :S - if (plr->GetTeam() == ALLIANCE) - plr->CastSpell(plr, NA_KILL_TOKEN_ALLIANCE, true); + player->KilledMonsterCredit(NA_CREDIT_MARKER, 0); // 0 guid, btw it isn't even used in killedmonster function :S + if (player->GetTeam() == ALLIANCE) + player->CastSpell(player, NA_KILL_TOKEN_ALLIANCE, true); else - plr->CastSpell(plr, NA_KILL_TOKEN_HORDE, true); + player->CastSpell(player, NA_KILL_TOKEN_HORDE, true); } } @@ -178,23 +178,23 @@ void OPvPCapturePointNA::FactionTakeOver(uint32 team) UpdateWyvernRoostWorldState(NA_ROOST_E); } -bool OPvPCapturePointNA::HandlePlayerEnter(Player* plr) +bool OPvPCapturePointNA::HandlePlayerEnter(Player* player) { - if (OPvPCapturePoint::HandlePlayerEnter(plr)) + if (OPvPCapturePoint::HandlePlayerEnter(player)) { - plr->SendUpdateWorldState(NA_UI_TOWER_SLIDER_DISPLAY, 1); + player->SendUpdateWorldState(NA_UI_TOWER_SLIDER_DISPLAY, 1); uint32 phase = (uint32)ceil((m_value + m_maxValue) / (2 * m_maxValue) * 100.0f); - plr->SendUpdateWorldState(NA_UI_TOWER_SLIDER_POS, phase); - plr->SendUpdateWorldState(NA_UI_TOWER_SLIDER_N, m_neutralValuePct); + player->SendUpdateWorldState(NA_UI_TOWER_SLIDER_POS, phase); + player->SendUpdateWorldState(NA_UI_TOWER_SLIDER_N, m_neutralValuePct); return true; } return false; } -void OPvPCapturePointNA::HandlePlayerLeave(Player* plr) +void OPvPCapturePointNA::HandlePlayerLeave(Player* player) { - plr->SendUpdateWorldState(NA_UI_TOWER_SLIDER_DISPLAY, 0); - OPvPCapturePoint::HandlePlayerLeave(plr); + player->SendUpdateWorldState(NA_UI_TOWER_SLIDER_DISPLAY, 0); + OPvPCapturePoint::HandlePlayerLeave(player); } OPvPCapturePointNA::OPvPCapturePointNA(OutdoorPvP *pvp) : @@ -220,19 +220,19 @@ bool OutdoorPvPNA::SetupOutdoorPvP() return true; } -void OutdoorPvPNA::HandlePlayerEnterZone(Player* plr, uint32 zone) +void OutdoorPvPNA::HandlePlayerEnterZone(Player* player, uint32 zone) { // add buffs - if (plr->GetTeam() == m_obj->m_ControllingFaction) - plr->CastSpell(plr, NA_CAPTURE_BUFF, true); - OutdoorPvP::HandlePlayerEnterZone(plr, zone); + if (player->GetTeam() == m_obj->m_ControllingFaction) + player->CastSpell(player, NA_CAPTURE_BUFF, true); + OutdoorPvP::HandlePlayerEnterZone(player, zone); } -void OutdoorPvPNA::HandlePlayerLeaveZone(Player* plr, uint32 zone) +void OutdoorPvPNA::HandlePlayerLeaveZone(Player* player, uint32 zone) { // remove buffs - plr->RemoveAurasDueToSpell(NA_CAPTURE_BUFF); - OutdoorPvP::HandlePlayerLeaveZone(plr, zone); + player->RemoveAurasDueToSpell(NA_CAPTURE_BUFF); + OutdoorPvP::HandlePlayerLeaveZone(player, zone); } void OutdoorPvPNA::FillInitialWorldStates(WorldPacket &data) @@ -292,36 +292,36 @@ void OPvPCapturePointNA::FillInitialWorldStates(WorldPacket &data) data << NA_MAP_HALAA_ALLIANCE << uint32(bool(m_HalaaState & HALAA_A)); } -void OutdoorPvPNA::SendRemoveWorldStates(Player* plr) +void OutdoorPvPNA::SendRemoveWorldStates(Player* player) { - plr->SendUpdateWorldState(NA_UI_HORDE_GUARDS_SHOW, 0); - plr->SendUpdateWorldState(NA_UI_ALLIANCE_GUARDS_SHOW, 0); - plr->SendUpdateWorldState(NA_UI_GUARDS_MAX, 0); - plr->SendUpdateWorldState(NA_UI_GUARDS_LEFT, 0); - plr->SendUpdateWorldState(NA_UI_TOWER_SLIDER_DISPLAY, 0); - plr->SendUpdateWorldState(NA_UI_TOWER_SLIDER_POS, 0); - plr->SendUpdateWorldState(NA_UI_TOWER_SLIDER_N, 0); - plr->SendUpdateWorldState(NA_MAP_WYVERN_NORTH_NEU_H, 0); - plr->SendUpdateWorldState(NA_MAP_WYVERN_NORTH_NEU_A, 0); - plr->SendUpdateWorldState(NA_MAP_WYVERN_NORTH_H, 0); - plr->SendUpdateWorldState(NA_MAP_WYVERN_NORTH_A, 0); - plr->SendUpdateWorldState(NA_MAP_WYVERN_SOUTH_NEU_H, 0); - plr->SendUpdateWorldState(NA_MAP_WYVERN_SOUTH_NEU_A, 0); - plr->SendUpdateWorldState(NA_MAP_WYVERN_SOUTH_H, 0); - plr->SendUpdateWorldState(NA_MAP_WYVERN_SOUTH_A, 0); - plr->SendUpdateWorldState(NA_MAP_WYVERN_WEST_NEU_H, 0); - plr->SendUpdateWorldState(NA_MAP_WYVERN_WEST_NEU_A, 0); - plr->SendUpdateWorldState(NA_MAP_WYVERN_WEST_H, 0); - plr->SendUpdateWorldState(NA_MAP_WYVERN_WEST_A, 0); - plr->SendUpdateWorldState(NA_MAP_WYVERN_EAST_NEU_H, 0); - plr->SendUpdateWorldState(NA_MAP_WYVERN_EAST_NEU_A, 0); - plr->SendUpdateWorldState(NA_MAP_WYVERN_EAST_H, 0); - plr->SendUpdateWorldState(NA_MAP_WYVERN_EAST_A, 0); - plr->SendUpdateWorldState(NA_MAP_HALAA_NEUTRAL, 0); - plr->SendUpdateWorldState(NA_MAP_HALAA_NEU_A, 0); - plr->SendUpdateWorldState(NA_MAP_HALAA_NEU_H, 0); - plr->SendUpdateWorldState(NA_MAP_HALAA_HORDE, 0); - plr->SendUpdateWorldState(NA_MAP_HALAA_ALLIANCE, 0); + player->SendUpdateWorldState(NA_UI_HORDE_GUARDS_SHOW, 0); + player->SendUpdateWorldState(NA_UI_ALLIANCE_GUARDS_SHOW, 0); + player->SendUpdateWorldState(NA_UI_GUARDS_MAX, 0); + player->SendUpdateWorldState(NA_UI_GUARDS_LEFT, 0); + player->SendUpdateWorldState(NA_UI_TOWER_SLIDER_DISPLAY, 0); + player->SendUpdateWorldState(NA_UI_TOWER_SLIDER_POS, 0); + player->SendUpdateWorldState(NA_UI_TOWER_SLIDER_N, 0); + player->SendUpdateWorldState(NA_MAP_WYVERN_NORTH_NEU_H, 0); + player->SendUpdateWorldState(NA_MAP_WYVERN_NORTH_NEU_A, 0); + player->SendUpdateWorldState(NA_MAP_WYVERN_NORTH_H, 0); + player->SendUpdateWorldState(NA_MAP_WYVERN_NORTH_A, 0); + player->SendUpdateWorldState(NA_MAP_WYVERN_SOUTH_NEU_H, 0); + player->SendUpdateWorldState(NA_MAP_WYVERN_SOUTH_NEU_A, 0); + player->SendUpdateWorldState(NA_MAP_WYVERN_SOUTH_H, 0); + player->SendUpdateWorldState(NA_MAP_WYVERN_SOUTH_A, 0); + player->SendUpdateWorldState(NA_MAP_WYVERN_WEST_NEU_H, 0); + player->SendUpdateWorldState(NA_MAP_WYVERN_WEST_NEU_A, 0); + player->SendUpdateWorldState(NA_MAP_WYVERN_WEST_H, 0); + player->SendUpdateWorldState(NA_MAP_WYVERN_WEST_A, 0); + player->SendUpdateWorldState(NA_MAP_WYVERN_EAST_NEU_H, 0); + player->SendUpdateWorldState(NA_MAP_WYVERN_EAST_NEU_A, 0); + player->SendUpdateWorldState(NA_MAP_WYVERN_EAST_H, 0); + player->SendUpdateWorldState(NA_MAP_WYVERN_EAST_A, 0); + player->SendUpdateWorldState(NA_MAP_HALAA_NEUTRAL, 0); + player->SendUpdateWorldState(NA_MAP_HALAA_NEU_A, 0); + player->SendUpdateWorldState(NA_MAP_HALAA_NEU_H, 0); + player->SendUpdateWorldState(NA_MAP_HALAA_HORDE, 0); + player->SendUpdateWorldState(NA_MAP_HALAA_ALLIANCE, 0); } bool OutdoorPvPNA::Update(uint32 diff) @@ -329,7 +329,7 @@ bool OutdoorPvPNA::Update(uint32 diff) return m_obj->Update(diff); } -bool OPvPCapturePointNA::HandleCustomSpell(Player* plr, uint32 spellId, GameObject* /*go*/) +bool OPvPCapturePointNA::HandleCustomSpell(Player* player, uint32 spellId, GameObject* /*go*/) { std::vector nodes; nodes.resize(2); @@ -339,33 +339,33 @@ bool OPvPCapturePointNA::HandleCustomSpell(Player* plr, uint32 spellId, GameObje case NA_SPELL_FLY_NORTH: nodes[0] = FlightPathStartNodes[NA_ROOST_N]; nodes[1] = FlightPathEndNodes[NA_ROOST_N]; - plr->ActivateTaxiPathTo(nodes); - plr->SetFlag(PLAYER_FLAGS, PLAYER_FLAGS_IN_PVP); - plr->UpdatePvP(true, true); + player->ActivateTaxiPathTo(nodes); + player->SetFlag(PLAYER_FLAGS, PLAYER_FLAGS_IN_PVP); + player->UpdatePvP(true, true); retval = true; break; case NA_SPELL_FLY_SOUTH: nodes[0] = FlightPathStartNodes[NA_ROOST_S]; nodes[1] = FlightPathEndNodes[NA_ROOST_S]; - plr->ActivateTaxiPathTo(nodes); - plr->SetFlag(PLAYER_FLAGS, PLAYER_FLAGS_IN_PVP); - plr->UpdatePvP(true, true); + player->ActivateTaxiPathTo(nodes); + player->SetFlag(PLAYER_FLAGS, PLAYER_FLAGS_IN_PVP); + player->UpdatePvP(true, true); retval = true; break; case NA_SPELL_FLY_WEST: nodes[0] = FlightPathStartNodes[NA_ROOST_W]; nodes[1] = FlightPathEndNodes[NA_ROOST_W]; - plr->ActivateTaxiPathTo(nodes); - plr->SetFlag(PLAYER_FLAGS, PLAYER_FLAGS_IN_PVP); - plr->UpdatePvP(true, true); + player->ActivateTaxiPathTo(nodes); + player->SetFlag(PLAYER_FLAGS, PLAYER_FLAGS_IN_PVP); + player->UpdatePvP(true, true); retval = true; break; case NA_SPELL_FLY_EAST: nodes[0] = FlightPathStartNodes[NA_ROOST_E]; nodes[1] = FlightPathEndNodes[NA_ROOST_E]; - plr->ActivateTaxiPathTo(nodes); - plr->SetFlag(PLAYER_FLAGS, PLAYER_FLAGS_IN_PVP); - plr->UpdatePvP(true, true); + player->ActivateTaxiPathTo(nodes); + player->SetFlag(PLAYER_FLAGS, PLAYER_FLAGS_IN_PVP); + player->UpdatePvP(true, true); retval = true; break; default: @@ -383,7 +383,7 @@ bool OPvPCapturePointNA::HandleCustomSpell(Player* plr, uint32 spellId, GameObje int32 count = 10; uint32 itemid = 24538; // bomb id count - uint8 msg = plr->CanStoreNewItem(NULL_BAG, NULL_SLOT, dest, itemid, count, &noSpaceForCount); + uint8 msg = player->CanStoreNewItem(NULL_BAG, NULL_SLOT, dest, itemid, count, &noSpaceForCount); if (msg != EQUIP_ERR_OK) // convert to possible store amount count -= noSpaceForCount; @@ -392,11 +392,11 @@ bool OPvPCapturePointNA::HandleCustomSpell(Player* plr, uint32 spellId, GameObje return true; } - Item* item = plr->StoreNewItem(dest, itemid, true); + Item* item = player->StoreNewItem(dest, itemid, true); if (count > 0 && item) { - plr->SendNewItem(item, count, true, false); + player->SendNewItem(item, count, true, false); } return true; @@ -404,9 +404,9 @@ bool OPvPCapturePointNA::HandleCustomSpell(Player* plr, uint32 spellId, GameObje return false; } -int32 OPvPCapturePointNA::HandleOpenGo(Player* plr, uint64 guid) +int32 OPvPCapturePointNA::HandleOpenGo(Player* player, uint64 guid) { - int32 retval = OPvPCapturePoint::HandleOpenGo(plr, guid); + int32 retval = OPvPCapturePoint::HandleOpenGo(player, guid); if (retval >= 0) { const go_type * gos = NULL; diff --git a/src/server/scripts/OutdoorPvP/OutdoorPvPNA.h b/src/server/scripts/OutdoorPvP/OutdoorPvPNA.h index 81812f30402..7ddb47ac437 100755 --- a/src/server/scripts/OutdoorPvP/OutdoorPvPNA.h +++ b/src/server/scripts/OutdoorPvP/OutdoorPvPNA.h @@ -267,12 +267,12 @@ class OPvPCapturePointNA : public OPvPCapturePoint void FillInitialWorldStates(WorldPacket & data); // used when player is activated/inactivated in the area - bool HandlePlayerEnter(Player* plr); - void HandlePlayerLeave(Player* plr); + bool HandlePlayerEnter(Player* player); + void HandlePlayerLeave(Player* player); - bool HandleCustomSpell(Player* plr, uint32 spellId, GameObject* go); + bool HandleCustomSpell(Player* player, uint32 spellId, GameObject* go); - int32 HandleOpenGo(Player* plr, uint64 guid); + int32 HandleOpenGo(Player* player, uint64 guid); uint32 GetAliveGuardsCount(); @@ -320,16 +320,16 @@ class OutdoorPvPNA : public OutdoorPvP bool SetupOutdoorPvP(); - void HandlePlayerEnterZone(Player* plr, uint32 zone); - void HandlePlayerLeaveZone(Player* plr, uint32 zone); + void HandlePlayerEnterZone(Player* player, uint32 zone); + void HandlePlayerLeaveZone(Player* player, uint32 zone); bool Update(uint32 diff); void FillInitialWorldStates(WorldPacket &data); - void SendRemoveWorldStates(Player* plr); + void SendRemoveWorldStates(Player* player); - void HandleKillImpl(Player* plr, Unit* killed); + void HandleKillImpl(Player* player, Unit* killed); private: diff --git a/src/server/scripts/OutdoorPvP/OutdoorPvPSI.cpp b/src/server/scripts/OutdoorPvP/OutdoorPvPSI.cpp index fa058f1d4ec..73c94a0d593 100755 --- a/src/server/scripts/OutdoorPvP/OutdoorPvPSI.cpp +++ b/src/server/scripts/OutdoorPvP/OutdoorPvPSI.cpp @@ -41,11 +41,11 @@ void OutdoorPvPSI::FillInitialWorldStates(WorldPacket &data) data << SI_SILITHYST_MAX << SI_MAX_RESOURCES; } -void OutdoorPvPSI::SendRemoveWorldStates(Player* plr) +void OutdoorPvPSI::SendRemoveWorldStates(Player* player) { - plr->SendUpdateWorldState(SI_GATHERED_A, 0); - plr->SendUpdateWorldState(SI_GATHERED_H, 0); - plr->SendUpdateWorldState(SI_SILITHYST_MAX, 0); + player->SendUpdateWorldState(SI_GATHERED_A, 0); + player->SendUpdateWorldState(SI_GATHERED_H, 0); + player->SendUpdateWorldState(SI_SILITHYST_MAX, 0); } void OutdoorPvPSI::UpdateWorldState() @@ -67,29 +67,29 @@ bool OutdoorPvPSI::Update(uint32 /*diff*/) return false; } -void OutdoorPvPSI::HandlePlayerEnterZone(Player* plr, uint32 zone) +void OutdoorPvPSI::HandlePlayerEnterZone(Player* player, uint32 zone) { - if (plr->GetTeam() == m_LastController) - plr->CastSpell(plr, SI_CENARION_FAVOR, true); - OutdoorPvP::HandlePlayerEnterZone(plr, zone); + if (player->GetTeam() == m_LastController) + player->CastSpell(player, SI_CENARION_FAVOR, true); + OutdoorPvP::HandlePlayerEnterZone(player, zone); } -void OutdoorPvPSI::HandlePlayerLeaveZone(Player* plr, uint32 zone) +void OutdoorPvPSI::HandlePlayerLeaveZone(Player* player, uint32 zone) { // remove buffs - plr->RemoveAurasDueToSpell(SI_CENARION_FAVOR); - OutdoorPvP::HandlePlayerLeaveZone(plr, zone); + player->RemoveAurasDueToSpell(SI_CENARION_FAVOR); + OutdoorPvP::HandlePlayerLeaveZone(player, zone); } -bool OutdoorPvPSI::HandleAreaTrigger(Player* plr, uint32 trigger) +bool OutdoorPvPSI::HandleAreaTrigger(Player* player, uint32 trigger) { switch(trigger) { case SI_AREATRIGGER_A: - if (plr->GetTeam() == ALLIANCE && plr->HasAura(SI_SILITHYST_FLAG)) + if (player->GetTeam() == ALLIANCE && player->HasAura(SI_SILITHYST_FLAG)) { // remove aura - plr->RemoveAurasDueToSpell(SI_SILITHYST_FLAG); + player->RemoveAurasDueToSpell(SI_SILITHYST_FLAG); ++ m_Gathered_A; if (m_Gathered_A >= SI_MAX_RESOURCES) { @@ -101,20 +101,20 @@ bool OutdoorPvPSI::HandleAreaTrigger(Player* plr, uint32 trigger) } UpdateWorldState(); // reward player - plr->CastSpell(plr, SI_TRACES_OF_SILITHYST, true); + player->CastSpell(player, SI_TRACES_OF_SILITHYST, true); // add 19 honor - plr->RewardHonor(NULL, 1, 19); + player->RewardHonor(NULL, 1, 19); // add 20 cenarion circle repu - plr->GetReputationMgr().ModifyReputation(sFactionStore.LookupEntry(609), 20); + player->GetReputationMgr().ModifyReputation(sFactionStore.LookupEntry(609), 20); // complete quest - plr->KilledMonsterCredit(SI_TURNIN_QUEST_CM_A, 0); + player->KilledMonsterCredit(SI_TURNIN_QUEST_CM_A, 0); } return true; case SI_AREATRIGGER_H: - if (plr->GetTeam() == HORDE && plr->HasAura(SI_SILITHYST_FLAG)) + if (player->GetTeam() == HORDE && player->HasAura(SI_SILITHYST_FLAG)) { // remove aura - plr->RemoveAurasDueToSpell(SI_SILITHYST_FLAG); + player->RemoveAurasDueToSpell(SI_SILITHYST_FLAG); ++ m_Gathered_H; if (m_Gathered_H >= SI_MAX_RESOURCES) { @@ -126,25 +126,25 @@ bool OutdoorPvPSI::HandleAreaTrigger(Player* plr, uint32 trigger) } UpdateWorldState(); // reward player - plr->CastSpell(plr, SI_TRACES_OF_SILITHYST, true); + player->CastSpell(player, SI_TRACES_OF_SILITHYST, true); // add 19 honor - plr->RewardHonor(NULL, 1, 19); + player->RewardHonor(NULL, 1, 19); // add 20 cenarion circle repu - plr->GetReputationMgr().ModifyReputation(sFactionStore.LookupEntry(609), 20); + player->GetReputationMgr().ModifyReputation(sFactionStore.LookupEntry(609), 20); // complete quest - plr->KilledMonsterCredit(SI_TURNIN_QUEST_CM_H, 0); + player->KilledMonsterCredit(SI_TURNIN_QUEST_CM_H, 0); } return true; } return false; } -bool OutdoorPvPSI::HandleDropFlag(Player* plr, uint32 spellId) +bool OutdoorPvPSI::HandleDropFlag(Player* player, uint32 spellId) { if (spellId == SI_SILITHYST_FLAG) { // if it was dropped away from the player's turn-in point, then create a silithyst mound, if it was dropped near the areatrigger, then it was dispelled by the outdoorpvp, so do nothing - switch(plr->GetTeam()) + switch(player->GetTeam()) { case ALLIANCE: { @@ -152,18 +152,18 @@ bool OutdoorPvPSI::HandleDropFlag(Player* plr, uint32 spellId) if (atEntry) { // 5.0f is safe-distance - if (plr->GetDistance(atEntry->x, atEntry->y, atEntry->z) > 5.0f + atEntry->radius) + if (player->GetDistance(atEntry->x, atEntry->y, atEntry->z) > 5.0f + atEntry->radius) { // he dropped it further, summon mound GameObject* go = new GameObject; - Map * map = plr->GetMap(); + Map * map = player->GetMap(); if (!map) { delete go; return true; } - if (!go->Create(sObjectMgr->GenerateLowGuid(HIGHGUID_GAMEOBJECT), SI_SILITHYST_MOUND, map, plr->GetPhaseMask(), plr->GetPositionX(), plr->GetPositionY(), plr->GetPositionZ(), plr->GetOrientation(), 0, 0, 0, 0, 100, GO_STATE_READY)) + if (!go->Create(sObjectMgr->GenerateLowGuid(HIGHGUID_GAMEOBJECT), SI_SILITHYST_MOUND, map, player->GetPhaseMask(), player->GetPositionX(), player->GetPositionY(), player->GetPositionZ(), player->GetOrientation(), 0, 0, 0, 0, 100, GO_STATE_READY)) { delete go; } @@ -182,17 +182,17 @@ bool OutdoorPvPSI::HandleDropFlag(Player* plr, uint32 spellId) if (atEntry) { // 5.0f is safe-distance - if (plr->GetDistance(atEntry->x, atEntry->y, atEntry->z) > 5.0f + atEntry->radius) + if (player->GetDistance(atEntry->x, atEntry->y, atEntry->z) > 5.0f + atEntry->radius) { // he dropped it further, summon mound GameObject* go = new GameObject; - Map * map = plr->GetMap(); + Map * map = player->GetMap(); if (!map) { delete go; return true; } - if (!go->Create(sObjectMgr->GenerateLowGuid(HIGHGUID_GAMEOBJECT), SI_SILITHYST_MOUND, map, plr->GetPhaseMask() , plr->GetPositionX(), plr->GetPositionY(), plr->GetPositionZ(), plr->GetOrientation(), 0, 0, 0, 0, 100, GO_STATE_READY)) + if (!go->Create(sObjectMgr->GenerateLowGuid(HIGHGUID_GAMEOBJECT), SI_SILITHYST_MOUND, map, player->GetPhaseMask() , player->GetPositionX(), player->GetPositionY(), player->GetPositionZ(), player->GetOrientation(), 0, 0, 0, 0, 100, GO_STATE_READY)) { delete go; } @@ -211,11 +211,11 @@ bool OutdoorPvPSI::HandleDropFlag(Player* plr, uint32 spellId) return false; } -bool OutdoorPvPSI::HandleCustomSpell(Player* plr, uint32 spellId, GameObject* go) +bool OutdoorPvPSI::HandleCustomSpell(Player* player, uint32 spellId, GameObject* go) { if (!go || spellId != SI_SILITHYST_FLAG_GO_SPELL) return false; - plr->CastSpell(plr, SI_SILITHYST_FLAG, true); + player->CastSpell(player, SI_SILITHYST_FLAG, true); if (go->GetGOInfo()->entry == SI_SILITHYST_MOUND) { // despawn go diff --git a/src/server/scripts/OutdoorPvP/OutdoorPvPSI.h b/src/server/scripts/OutdoorPvP/OutdoorPvPSI.h index 1b3bf284fff..fe696dea7e4 100755 --- a/src/server/scripts/OutdoorPvP/OutdoorPvPSI.h +++ b/src/server/scripts/OutdoorPvP/OutdoorPvPSI.h @@ -59,20 +59,20 @@ class OutdoorPvPSI : public OutdoorPvP bool SetupOutdoorPvP(); - void HandlePlayerEnterZone(Player* plr, uint32 zone); - void HandlePlayerLeaveZone(Player* plr, uint32 zone); + void HandlePlayerEnterZone(Player* player, uint32 zone); + void HandlePlayerLeaveZone(Player* player, uint32 zone); bool Update(uint32 diff); void FillInitialWorldStates(WorldPacket &data); - void SendRemoveWorldStates(Player* plr); + void SendRemoveWorldStates(Player* player); - bool HandleAreaTrigger(Player* plr, uint32 trigger); + bool HandleAreaTrigger(Player* player, uint32 trigger); - bool HandleDropFlag(Player* plr, uint32 spellId); + bool HandleDropFlag(Player* player, uint32 spellId); - bool HandleCustomSpell(Player* plr, uint32 spellId, GameObject* go); + bool HandleCustomSpell(Player* player, uint32 spellId, GameObject* go); void UpdateWorldState(); diff --git a/src/server/scripts/OutdoorPvP/OutdoorPvPTF.cpp b/src/server/scripts/OutdoorPvP/OutdoorPvPTF.cpp index d822c65cd5e..666df2360c5 100755 --- a/src/server/scripts/OutdoorPvP/OutdoorPvPTF.cpp +++ b/src/server/scripts/OutdoorPvP/OutdoorPvPTF.cpp @@ -67,29 +67,29 @@ void OutdoorPvPTF::FillInitialWorldStates(WorldPacket &data) } } -void OutdoorPvPTF::SendRemoveWorldStates(Player* plr) +void OutdoorPvPTF::SendRemoveWorldStates(Player* player) { - plr->SendUpdateWorldState(TF_UI_TOWER_SLIDER_POS, uint32(0)); - plr->SendUpdateWorldState(TF_UI_TOWER_SLIDER_N, uint32(0)); - plr->SendUpdateWorldState(TF_UI_TOWER_SLIDER_DISPLAY, uint32(0)); + player->SendUpdateWorldState(TF_UI_TOWER_SLIDER_POS, uint32(0)); + player->SendUpdateWorldState(TF_UI_TOWER_SLIDER_N, uint32(0)); + player->SendUpdateWorldState(TF_UI_TOWER_SLIDER_DISPLAY, uint32(0)); - plr->SendUpdateWorldState(TF_UI_TOWER_COUNT_H, uint32(0)); - plr->SendUpdateWorldState(TF_UI_TOWER_COUNT_A, uint32(0)); - plr->SendUpdateWorldState(TF_UI_TOWERS_CONTROLLED_DISPLAY, uint32(0)); + player->SendUpdateWorldState(TF_UI_TOWER_COUNT_H, uint32(0)); + player->SendUpdateWorldState(TF_UI_TOWER_COUNT_A, uint32(0)); + player->SendUpdateWorldState(TF_UI_TOWERS_CONTROLLED_DISPLAY, uint32(0)); - plr->SendUpdateWorldState(TF_UI_LOCKED_TIME_MINUTES_FIRST_DIGIT, uint32(0)); - plr->SendUpdateWorldState(TF_UI_LOCKED_TIME_MINUTES_SECOND_DIGIT, uint32(0)); - plr->SendUpdateWorldState(TF_UI_LOCKED_TIME_HOURS, uint32(0)); + player->SendUpdateWorldState(TF_UI_LOCKED_TIME_MINUTES_FIRST_DIGIT, uint32(0)); + player->SendUpdateWorldState(TF_UI_LOCKED_TIME_MINUTES_SECOND_DIGIT, uint32(0)); + player->SendUpdateWorldState(TF_UI_LOCKED_TIME_HOURS, uint32(0)); - plr->SendUpdateWorldState(TF_UI_LOCKED_DISPLAY_NEUTRAL, uint32(0)); - plr->SendUpdateWorldState(TF_UI_LOCKED_DISPLAY_HORDE, uint32(0)); - plr->SendUpdateWorldState(TF_UI_LOCKED_DISPLAY_ALLIANCE, uint32(0)); + player->SendUpdateWorldState(TF_UI_LOCKED_DISPLAY_NEUTRAL, uint32(0)); + player->SendUpdateWorldState(TF_UI_LOCKED_DISPLAY_HORDE, uint32(0)); + player->SendUpdateWorldState(TF_UI_LOCKED_DISPLAY_ALLIANCE, uint32(0)); for (int i = 0; i < TF_TOWER_NUM; ++i) { - plr->SendUpdateWorldState(uint32(TFTowerWorldStates[i].n), uint32(0)); - plr->SendUpdateWorldState(uint32(TFTowerWorldStates[i].h), uint32(0)); - plr->SendUpdateWorldState(uint32(TFTowerWorldStates[i].a), uint32(0)); + player->SendUpdateWorldState(uint32(TFTowerWorldStates[i].n), uint32(0)); + player->SendUpdateWorldState(uint32(TFTowerWorldStates[i].h), uint32(0)); + player->SendUpdateWorldState(uint32(TFTowerWorldStates[i].a), uint32(0)); } } @@ -100,23 +100,23 @@ void OPvPCapturePointTF::UpdateTowerState() m_PvP->SendUpdateWorldState(uint32(TFTowerWorldStates[m_TowerType].a), uint32(bool(m_TowerState & TF_TOWERSTATE_A))); } -bool OPvPCapturePointTF::HandlePlayerEnter(Player* plr) +bool OPvPCapturePointTF::HandlePlayerEnter(Player* player) { - if (OPvPCapturePoint::HandlePlayerEnter(plr)) + if (OPvPCapturePoint::HandlePlayerEnter(player)) { - plr->SendUpdateWorldState(TF_UI_TOWER_SLIDER_DISPLAY, 1); + player->SendUpdateWorldState(TF_UI_TOWER_SLIDER_DISPLAY, 1); uint32 phase = (uint32)ceil((m_value + m_maxValue) / (2 * m_maxValue) * 100.0f); - plr->SendUpdateWorldState(TF_UI_TOWER_SLIDER_POS, phase); - plr->SendUpdateWorldState(TF_UI_TOWER_SLIDER_N, m_neutralValuePct); + player->SendUpdateWorldState(TF_UI_TOWER_SLIDER_POS, phase); + player->SendUpdateWorldState(TF_UI_TOWER_SLIDER_N, m_neutralValuePct); return true; } return false; } -void OPvPCapturePointTF::HandlePlayerLeave(Player* plr) +void OPvPCapturePointTF::HandlePlayerLeave(Player* player) { - plr->SendUpdateWorldState(TF_UI_TOWER_SLIDER_DISPLAY, 0); - OPvPCapturePoint::HandlePlayerLeave(plr); + player->SendUpdateWorldState(TF_UI_TOWER_SLIDER_DISPLAY, 0); + OPvPCapturePoint::HandlePlayerLeave(player); } bool OutdoorPvPTF::Update(uint32 diff) @@ -186,26 +186,26 @@ bool OutdoorPvPTF::Update(uint32 diff) return changed; } -void OutdoorPvPTF::HandlePlayerEnterZone(Player* plr, uint32 zone) +void OutdoorPvPTF::HandlePlayerEnterZone(Player* player, uint32 zone) { - if (plr->GetTeam() == ALLIANCE) + if (player->GetTeam() == ALLIANCE) { if (m_AllianceTowersControlled >= TF_TOWER_NUM) - plr->CastSpell(plr, TF_CAPTURE_BUFF, true); + player->CastSpell(player, TF_CAPTURE_BUFF, true); } else { if (m_HordeTowersControlled >= TF_TOWER_NUM) - plr->CastSpell(plr, TF_CAPTURE_BUFF, true); + player->CastSpell(player, TF_CAPTURE_BUFF, true); } - OutdoorPvP::HandlePlayerEnterZone(plr, zone); + OutdoorPvP::HandlePlayerEnterZone(player, zone); } -void OutdoorPvPTF::HandlePlayerLeaveZone(Player* plr, uint32 zone) +void OutdoorPvPTF::HandlePlayerLeaveZone(Player* player, uint32 zone) { // remove buffs - plr->RemoveAurasDueToSpell(TF_CAPTURE_BUFF); - OutdoorPvP::HandlePlayerLeaveZone(plr, zone); + player->RemoveAurasDueToSpell(TF_CAPTURE_BUFF); + OutdoorPvP::HandlePlayerLeaveZone(player, zone); } bool OutdoorPvPTF::SetupOutdoorPvP() diff --git a/src/server/scripts/OutdoorPvP/OutdoorPvPTF.h b/src/server/scripts/OutdoorPvP/OutdoorPvPTF.h index 874aafd6352..f8257e490f6 100755 --- a/src/server/scripts/OutdoorPvP/OutdoorPvPTF.h +++ b/src/server/scripts/OutdoorPvP/OutdoorPvPTF.h @@ -136,8 +136,8 @@ class OPvPCapturePointTF : public OPvPCapturePoint void FillInitialWorldStates(WorldPacket & data); // used when player is activated/inactivated in the area - bool HandlePlayerEnter(Player* plr); - void HandlePlayerLeave(Player* plr); + bool HandlePlayerEnter(Player* player); + void HandlePlayerLeave(Player* player); void UpdateTowerState(); @@ -158,14 +158,14 @@ class OutdoorPvPTF : public OutdoorPvP bool SetupOutdoorPvP(); - void HandlePlayerEnterZone(Player* plr, uint32 zone); - void HandlePlayerLeaveZone(Player* plr, uint32 zone); + void HandlePlayerEnterZone(Player* player, uint32 zone); + void HandlePlayerLeaveZone(Player* player, uint32 zone); bool Update(uint32 diff); void FillInitialWorldStates(WorldPacket &data); - void SendRemoveWorldStates(Player* plr); + void SendRemoveWorldStates(Player* player); private: diff --git a/src/server/scripts/OutdoorPvP/OutdoorPvPZM.cpp b/src/server/scripts/OutdoorPvP/OutdoorPvPZM.cpp index a3a07e3ddd6..792fe450990 100755 --- a/src/server/scripts/OutdoorPvP/OutdoorPvPZM.cpp +++ b/src/server/scripts/OutdoorPvP/OutdoorPvPZM.cpp @@ -52,23 +52,23 @@ void OPvPCapturePointZM_Beacon::UpdateTowerState() m_PvP->SendUpdateWorldState(uint32(ZMBeaconInfo[m_TowerType].map_tower_h), uint32(bool(m_TowerState & ZM_TOWERSTATE_H))); } -bool OPvPCapturePointZM_Beacon::HandlePlayerEnter(Player* plr) +bool OPvPCapturePointZM_Beacon::HandlePlayerEnter(Player* player) { - if (OPvPCapturePoint::HandlePlayerEnter(plr)) + if (OPvPCapturePoint::HandlePlayerEnter(player)) { - plr->SendUpdateWorldState(ZMBeaconInfo[m_TowerType].slider_disp, 1); + player->SendUpdateWorldState(ZMBeaconInfo[m_TowerType].slider_disp, 1); uint32 phase = (uint32)ceil((m_value + m_maxValue) / (2 * m_maxValue) * 100.0f); - plr->SendUpdateWorldState(ZMBeaconInfo[m_TowerType].slider_pos, phase); - plr->SendUpdateWorldState(ZMBeaconInfo[m_TowerType].slider_n, m_neutralValuePct); + player->SendUpdateWorldState(ZMBeaconInfo[m_TowerType].slider_pos, phase); + player->SendUpdateWorldState(ZMBeaconInfo[m_TowerType].slider_n, m_neutralValuePct); return true; } return false; } -void OPvPCapturePointZM_Beacon::HandlePlayerLeave(Player* plr) +void OPvPCapturePointZM_Beacon::HandlePlayerLeave(Player* player) { - plr->SendUpdateWorldState(ZMBeaconInfo[m_TowerType].slider_disp, 0); - OPvPCapturePoint::HandlePlayerLeave(plr); + player->SendUpdateWorldState(ZMBeaconInfo[m_TowerType].slider_disp, 0); + OPvPCapturePoint::HandlePlayerLeave(player); } void OPvPCapturePointZM_Beacon::ChangeState() @@ -139,29 +139,29 @@ bool OutdoorPvPZM::Update(uint32 diff) return changed; } -void OutdoorPvPZM::HandlePlayerEnterZone(Player* plr, uint32 zone) +void OutdoorPvPZM::HandlePlayerEnterZone(Player* player, uint32 zone) { - if (plr->GetTeam() == ALLIANCE) + if (player->GetTeam() == ALLIANCE) { if (m_GraveYard->m_GraveYardState & ZM_GRAVEYARD_A) - plr->CastSpell(plr, ZM_CAPTURE_BUFF, true); + player->CastSpell(player, ZM_CAPTURE_BUFF, true); } else { if (m_GraveYard->m_GraveYardState & ZM_GRAVEYARD_H) - plr->CastSpell(plr, ZM_CAPTURE_BUFF, true); + player->CastSpell(player, ZM_CAPTURE_BUFF, true); } - OutdoorPvP::HandlePlayerEnterZone(plr, zone); + OutdoorPvP::HandlePlayerEnterZone(player, zone); } -void OutdoorPvPZM::HandlePlayerLeaveZone(Player* plr, uint32 zone) +void OutdoorPvPZM::HandlePlayerLeaveZone(Player* player, uint32 zone) { // remove buffs - plr->RemoveAurasDueToSpell(ZM_CAPTURE_BUFF); + player->RemoveAurasDueToSpell(ZM_CAPTURE_BUFF); // remove flag - plr->RemoveAurasDueToSpell(ZM_BATTLE_STANDARD_A); - plr->RemoveAurasDueToSpell(ZM_BATTLE_STANDARD_H); - OutdoorPvP::HandlePlayerLeaveZone(plr, zone); + player->RemoveAurasDueToSpell(ZM_BATTLE_STANDARD_A); + player->RemoveAurasDueToSpell(ZM_BATTLE_STANDARD_H); + OutdoorPvP::HandlePlayerLeaveZone(player, zone); } OutdoorPvPZM::OutdoorPvPZM() @@ -190,15 +190,15 @@ bool OutdoorPvPZM::SetupOutdoorPvP() return true; } -void OutdoorPvPZM::HandleKillImpl(Player* plr, Unit* killed) +void OutdoorPvPZM::HandleKillImpl(Player* player, Unit* killed) { if (killed->GetTypeId() != TYPEID_PLAYER) return; - if (plr->GetTeam() == ALLIANCE && killed->ToPlayer()->GetTeam() != ALLIANCE) - plr->CastSpell(plr, ZM_AlliancePlayerKillReward, true); - else if (plr->GetTeam() == HORDE && killed->ToPlayer()->GetTeam() != HORDE) - plr->CastSpell(plr, ZM_HordePlayerKillReward, true); + if (player->GetTeam() == ALLIANCE && killed->ToPlayer()->GetTeam() != ALLIANCE) + player->CastSpell(player, ZM_AlliancePlayerKillReward, true); + else if (player->GetTeam() == HORDE && killed->ToPlayer()->GetTeam() != HORDE) + player->CastSpell(player, ZM_HordePlayerKillReward, true); } bool OPvPCapturePointZM_GraveYard::Update(uint32 /*diff*/) @@ -208,12 +208,12 @@ bool OPvPCapturePointZM_GraveYard::Update(uint32 /*diff*/) return retval; } -int32 OPvPCapturePointZM_GraveYard::HandleOpenGo(Player* plr, uint64 guid) +int32 OPvPCapturePointZM_GraveYard::HandleOpenGo(Player* player, uint64 guid) { - int32 retval = OPvPCapturePoint::HandleOpenGo(plr, guid); + int32 retval = OPvPCapturePoint::HandleOpenGo(player, guid); if (retval >= 0) { - if (plr->HasAura(ZM_BATTLE_STANDARD_A) && m_GraveYardState != ZM_GRAVEYARD_A) + if (player->HasAura(ZM_BATTLE_STANDARD_A) && m_GraveYardState != ZM_GRAVEYARD_A) { if (m_GraveYardState == ZM_GRAVEYARD_H) sWorld->SendZoneText(ZM_GRAVEYARD_ZONE, sObjectMgr->GetTrinityStringForDBCLocale(LANG_OPVP_ZM_LOSE_GY_H)); @@ -223,10 +223,10 @@ int32 OPvPCapturePointZM_GraveYard::HandleOpenGo(Player* plr, uint64 guid) sObjectMgr->RemoveGraveYardLink(ZM_GRAVEYARD_ID, ZM_GRAVEYARD_ZONE, HORDE); // rem gy sObjectMgr->AddGraveYardLink(ZM_GRAVEYARD_ID, ZM_GRAVEYARD_ZONE, ALLIANCE, false); // add gy m_PvP->TeamApplyBuff(TEAM_ALLIANCE, ZM_CAPTURE_BUFF); - plr->RemoveAurasDueToSpell(ZM_BATTLE_STANDARD_A); + player->RemoveAurasDueToSpell(ZM_BATTLE_STANDARD_A); sWorld->SendZoneText(ZM_GRAVEYARD_ZONE, sObjectMgr->GetTrinityStringForDBCLocale(LANG_OPVP_ZM_CAPTURE_GY_A)); } - else if (plr->HasAura(ZM_BATTLE_STANDARD_H) && m_GraveYardState != ZM_GRAVEYARD_H) + else if (player->HasAura(ZM_BATTLE_STANDARD_H) && m_GraveYardState != ZM_GRAVEYARD_H) { if (m_GraveYardState == ZM_GRAVEYARD_A) sWorld->SendZoneText(ZM_GRAVEYARD_ZONE, sObjectMgr->GetTrinityStringForDBCLocale(LANG_OPVP_ZM_LOSE_GY_A)); @@ -236,7 +236,7 @@ int32 OPvPCapturePointZM_GraveYard::HandleOpenGo(Player* plr, uint64 guid) sObjectMgr->RemoveGraveYardLink(ZM_GRAVEYARD_ID, ZM_GRAVEYARD_ZONE, ALLIANCE); // rem gy sObjectMgr->AddGraveYardLink(ZM_GRAVEYARD_ID, ZM_GRAVEYARD_ZONE, HORDE, false); // add gy m_PvP->TeamApplyBuff(TEAM_HORDE, ZM_CAPTURE_BUFF); - plr->RemoveAurasDueToSpell(ZM_BATTLE_STANDARD_H); + player->RemoveAurasDueToSpell(ZM_BATTLE_STANDARD_H); sWorld->SendZoneText(ZM_GRAVEYARD_ZONE, sObjectMgr->GetTrinityStringForDBCLocale(LANG_OPVP_ZM_CAPTURE_GY_H)); } UpdateTowerState(); @@ -326,21 +326,21 @@ void OPvPCapturePointZM_GraveYard::SetBeaconState(uint32 controlling_faction) UpdateTowerState(); } -bool OPvPCapturePointZM_GraveYard::CanTalkTo(Player* plr, Creature* c, GossipMenuItems const& /*gso*/) +bool OPvPCapturePointZM_GraveYard::CanTalkTo(Player* player, Creature* c, GossipMenuItems const& /*gso*/) { uint64 guid = c->GetGUID(); std::map::iterator itr = m_CreatureTypes.find(guid); if (itr != m_CreatureTypes.end()) { - if (itr->second == ZM_ALLIANCE_FIELD_SCOUT && plr->GetTeam() == ALLIANCE && m_BothControllingFaction == ALLIANCE && !m_FlagCarrierGUID && m_GraveYardState != ZM_GRAVEYARD_A) + if (itr->second == ZM_ALLIANCE_FIELD_SCOUT && player->GetTeam() == ALLIANCE && m_BothControllingFaction == ALLIANCE && !m_FlagCarrierGUID && m_GraveYardState != ZM_GRAVEYARD_A) return true; - else if (itr->second == ZM_HORDE_FIELD_SCOUT && plr->GetTeam() == HORDE && m_BothControllingFaction == HORDE && !m_FlagCarrierGUID && m_GraveYardState != ZM_GRAVEYARD_H) + else if (itr->second == ZM_HORDE_FIELD_SCOUT && player->GetTeam() == HORDE && m_BothControllingFaction == HORDE && !m_FlagCarrierGUID && m_GraveYardState != ZM_GRAVEYARD_H) return true; } return false; } -bool OPvPCapturePointZM_GraveYard::HandleGossipOption(Player* plr, uint64 guid, uint32 /*gossipid*/) +bool OPvPCapturePointZM_GraveYard::HandleGossipOption(Player* player, uint64 guid, uint32 /*gossipid*/) { std::map::iterator itr = m_CreatureTypes.find(guid); if (itr != m_CreatureTypes.end()) @@ -353,22 +353,22 @@ bool OPvPCapturePointZM_GraveYard::HandleGossipOption(Player* plr, uint64 guid, return true; if (itr->second == ZM_ALLIANCE_FIELD_SCOUT) { - cr->CastSpell(plr, ZM_BATTLE_STANDARD_A, true); - m_FlagCarrierGUID = plr->GetGUID(); + cr->CastSpell(player, ZM_BATTLE_STANDARD_A, true); + m_FlagCarrierGUID = player->GetGUID(); } else if (itr->second == ZM_HORDE_FIELD_SCOUT) { - cr->CastSpell(plr, ZM_BATTLE_STANDARD_H, true); - m_FlagCarrierGUID = plr->GetGUID(); + cr->CastSpell(player, ZM_BATTLE_STANDARD_H, true); + m_FlagCarrierGUID = player->GetGUID(); } UpdateTowerState(); - plr->PlayerTalkClass->SendCloseGossip(); + player->PlayerTalkClass->SendCloseGossip(); return true; } return false; } -bool OPvPCapturePointZM_GraveYard::HandleDropFlag(Player* /*plr*/, uint32 spellId) +bool OPvPCapturePointZM_GraveYard::HandleDropFlag(Player* /*player*/, uint32 spellId) { switch(spellId) { @@ -391,34 +391,34 @@ void OutdoorPvPZM::FillInitialWorldStates(WorldPacket &data) } } -void OutdoorPvPZM::SendRemoveWorldStates(Player* plr) +void OutdoorPvPZM::SendRemoveWorldStates(Player* player) { - plr->SendUpdateWorldState(ZM_UI_TOWER_SLIDER_N_W, 0); - plr->SendUpdateWorldState(ZM_UI_TOWER_SLIDER_POS_W, 0); - plr->SendUpdateWorldState(ZM_UI_TOWER_SLIDER_DISPLAY_W, 0); - plr->SendUpdateWorldState(ZM_UI_TOWER_SLIDER_N_E, 0); - plr->SendUpdateWorldState(ZM_UI_TOWER_SLIDER_POS_E, 0); - plr->SendUpdateWorldState(ZM_UI_TOWER_SLIDER_DISPLAY_E, 0); - plr->SendUpdateWorldState(ZM_WORLDSTATE_UNK_1, 1); - plr->SendUpdateWorldState(ZM_UI_TOWER_EAST_N, 0); - plr->SendUpdateWorldState(ZM_UI_TOWER_EAST_H, 0); - plr->SendUpdateWorldState(ZM_UI_TOWER_EAST_A, 0); - plr->SendUpdateWorldState(ZM_UI_TOWER_WEST_N, 0); - plr->SendUpdateWorldState(ZM_UI_TOWER_WEST_H, 0); - plr->SendUpdateWorldState(ZM_UI_TOWER_WEST_A, 0); - plr->SendUpdateWorldState(ZM_MAP_TOWER_EAST_N, 0); - plr->SendUpdateWorldState(ZM_MAP_TOWER_EAST_H, 0); - plr->SendUpdateWorldState(ZM_MAP_TOWER_EAST_A, 0); - plr->SendUpdateWorldState(ZM_MAP_GRAVEYARD_H, 0); - plr->SendUpdateWorldState(ZM_MAP_GRAVEYARD_A, 0); - plr->SendUpdateWorldState(ZM_MAP_GRAVEYARD_N, 0); - plr->SendUpdateWorldState(ZM_MAP_TOWER_WEST_N, 0); - plr->SendUpdateWorldState(ZM_MAP_TOWER_WEST_H, 0); - plr->SendUpdateWorldState(ZM_MAP_TOWER_WEST_A, 0); - plr->SendUpdateWorldState(ZM_MAP_HORDE_FLAG_READY, 0); - plr->SendUpdateWorldState(ZM_MAP_HORDE_FLAG_NOT_READY, 0); - plr->SendUpdateWorldState(ZM_MAP_ALLIANCE_FLAG_NOT_READY, 0); - plr->SendUpdateWorldState(ZM_MAP_ALLIANCE_FLAG_READY, 0); + player->SendUpdateWorldState(ZM_UI_TOWER_SLIDER_N_W, 0); + player->SendUpdateWorldState(ZM_UI_TOWER_SLIDER_POS_W, 0); + player->SendUpdateWorldState(ZM_UI_TOWER_SLIDER_DISPLAY_W, 0); + player->SendUpdateWorldState(ZM_UI_TOWER_SLIDER_N_E, 0); + player->SendUpdateWorldState(ZM_UI_TOWER_SLIDER_POS_E, 0); + player->SendUpdateWorldState(ZM_UI_TOWER_SLIDER_DISPLAY_E, 0); + player->SendUpdateWorldState(ZM_WORLDSTATE_UNK_1, 1); + player->SendUpdateWorldState(ZM_UI_TOWER_EAST_N, 0); + player->SendUpdateWorldState(ZM_UI_TOWER_EAST_H, 0); + player->SendUpdateWorldState(ZM_UI_TOWER_EAST_A, 0); + player->SendUpdateWorldState(ZM_UI_TOWER_WEST_N, 0); + player->SendUpdateWorldState(ZM_UI_TOWER_WEST_H, 0); + player->SendUpdateWorldState(ZM_UI_TOWER_WEST_A, 0); + player->SendUpdateWorldState(ZM_MAP_TOWER_EAST_N, 0); + player->SendUpdateWorldState(ZM_MAP_TOWER_EAST_H, 0); + player->SendUpdateWorldState(ZM_MAP_TOWER_EAST_A, 0); + player->SendUpdateWorldState(ZM_MAP_GRAVEYARD_H, 0); + player->SendUpdateWorldState(ZM_MAP_GRAVEYARD_A, 0); + player->SendUpdateWorldState(ZM_MAP_GRAVEYARD_N, 0); + player->SendUpdateWorldState(ZM_MAP_TOWER_WEST_N, 0); + player->SendUpdateWorldState(ZM_MAP_TOWER_WEST_H, 0); + player->SendUpdateWorldState(ZM_MAP_TOWER_WEST_A, 0); + player->SendUpdateWorldState(ZM_MAP_HORDE_FLAG_READY, 0); + player->SendUpdateWorldState(ZM_MAP_HORDE_FLAG_NOT_READY, 0); + player->SendUpdateWorldState(ZM_MAP_ALLIANCE_FLAG_NOT_READY, 0); + player->SendUpdateWorldState(ZM_MAP_ALLIANCE_FLAG_READY, 0); } class OutdoorPvP_zangarmarsh : public OutdoorPvPScript diff --git a/src/server/scripts/OutdoorPvP/OutdoorPvPZM.h b/src/server/scripts/OutdoorPvP/OutdoorPvPZM.h index 6b37afce37c..ed181f81bef 100755 --- a/src/server/scripts/OutdoorPvP/OutdoorPvPZM.h +++ b/src/server/scripts/OutdoorPvP/OutdoorPvPZM.h @@ -181,8 +181,8 @@ class OPvPCapturePointZM_Beacon : public OPvPCapturePoint void FillInitialWorldStates(WorldPacket & data); // used when player is activated/inactivated in the area - bool HandlePlayerEnter(Player* plr); - void HandlePlayerLeave(Player* plr); + bool HandlePlayerEnter(Player* player); + void HandlePlayerLeave(Player* player); void UpdateTowerState(); @@ -215,15 +215,15 @@ class OPvPCapturePointZM_GraveYard : public OPvPCapturePoint void UpdateTowerState(); - int32 HandleOpenGo(Player* plr, uint64 guid); + int32 HandleOpenGo(Player* player, uint64 guid); void SetBeaconState(uint32 controlling_team); // not good atm - bool HandleGossipOption(Player* plr, uint64 guid, uint32 gossipid); + bool HandleGossipOption(Player* player, uint64 guid, uint32 gossipid); - bool HandleDropFlag(Player* plr, uint32 spellId); + bool HandleDropFlag(Player* player, uint32 spellId); - bool CanTalkTo(Player* plr, Creature* c, GossipMenuItems const& gso); + bool CanTalkTo(Player* player, Creature* c, GossipMenuItems const& gso); private: @@ -246,16 +246,16 @@ class OutdoorPvPZM : public OutdoorPvP bool SetupOutdoorPvP(); - void HandlePlayerEnterZone(Player* plr, uint32 zone); - void HandlePlayerLeaveZone(Player* plr, uint32 zone); + void HandlePlayerEnterZone(Player* player, uint32 zone); + void HandlePlayerLeaveZone(Player* player, uint32 zone); bool Update(uint32 diff); void FillInitialWorldStates(WorldPacket &data); - void SendRemoveWorldStates(Player* plr); + void SendRemoveWorldStates(Player* player); - void HandleKillImpl(Player* plr, Unit* killed); + void HandleKillImpl(Player* player, Unit* killed); private: diff --git a/src/server/scripts/Outland/BlackTemple/instance_black_temple.cpp b/src/server/scripts/Outland/BlackTemple/instance_black_temple.cpp index 3770dabd8c7..3b8e86293eb 100644 --- a/src/server/scripts/Outland/BlackTemple/instance_black_temple.cpp +++ b/src/server/scripts/Outland/BlackTemple/instance_black_temple.cpp @@ -131,8 +131,8 @@ public: { for (Map::PlayerList::const_iterator itr = players.begin(); itr != players.end(); ++itr) { - if (Player* plr = itr->getSource()) - return plr; + if (Player* player = itr->getSource()) + return player; } } diff --git a/src/server/scripts/Outland/shadowmoon_valley.cpp b/src/server/scripts/Outland/shadowmoon_valley.cpp index c0ac25756f9..070b769e8f9 100644 --- a/src/server/scripts/Outland/shadowmoon_valley.cpp +++ b/src/server/scripts/Outland/shadowmoon_valley.cpp @@ -264,9 +264,9 @@ public: { if (PlayerGUID) { - Unit* plr = Unit::GetUnit((*me), PlayerGUID); - if (plr) - DoCast(plr, SPELL_FORCE_OF_NELTHARAKU, true); + Unit* player = Unit::GetUnit((*me), PlayerGUID); + if (player) + DoCast(player, SPELL_FORCE_OF_NELTHARAKU, true); PlayerGUID = 0; } @@ -288,10 +288,10 @@ public: Tapped = false; if (PlayerGUID) { - Player* plr = Unit::GetPlayer(*me, PlayerGUID); - if (plr && plr->GetQuestStatus(10854) == QUEST_STATUS_INCOMPLETE) + Player* player = Unit::GetPlayer(*me, PlayerGUID); + if (player && player->GetQuestStatus(10854) == QUEST_STATUS_INCOMPLETE) { - DoCast(plr, SPELL_FORCE_OF_NELTHARAKU, true); + DoCast(player, SPELL_FORCE_OF_NELTHARAKU, true); /* float x, y, z; me->GetPosition(x, y, z); @@ -391,9 +391,9 @@ public: { if (PlayerGUID) { - Player* plr = Unit::GetPlayer(*me, PlayerGUID); - if (plr && plr->GetQuestStatus(11020) == QUEST_STATUS_INCOMPLETE) - plr->KilledMonsterCredit(23209, 0); + Player* player = Unit::GetPlayer(*me, PlayerGUID); + if (player && player->GetQuestStatus(11020) == QUEST_STATUS_INCOMPLETE) + player->KilledMonsterCredit(23209, 0); } PoisonTimer = 0; me->DealDamage(me, me->GetHealth(), NULL, DIRECT_DAMAGE, SPELL_SCHOOL_MASK_NORMAL, NULL, false); @@ -761,11 +761,11 @@ public: uint32 NextStep(uint32 Step) { - Unit* plr = Unit::GetUnit((*me), PlayerGUID); + Unit* player = Unit::GetUnit((*me), PlayerGUID); Unit* Illi = Unit::GetUnit((*me), IllidanGUID); - if (!plr || !Illi) + if (!player || !Illi) { EnterEvadeMode(); return 0; @@ -775,9 +775,9 @@ public: { case 0: return 0; break; case 1: me->GetMotionMaster()->MovePoint(0, -5104.41f, 595.297f, 85.6838f); return 9000; break; - case 2: DoScriptText(OVERLORD_YELL_1, me, plr); return 4500; break; - case 3: me->SetInFront(plr); return 3200; break; - case 4: DoScriptText(OVERLORD_SAY_2, me, plr); return 2000; break; + case 2: DoScriptText(OVERLORD_YELL_1, me, player); return 4500; break; + case 3: me->SetInFront(player); return 3200; break; + case 4: DoScriptText(OVERLORD_SAY_2, me, player); return 2000; break; case 5: Illi->SetVisible(true); Illi->RemoveFlag(UNIT_FIELD_FLAGS, UNIT_FLAG_NOT_SELECTABLE); return 350; break; case 6: @@ -789,20 +789,20 @@ public: case 8: me->SetUInt32Value(UNIT_FIELD_BYTES_1, 8); return 2500; break; case 9: DoScriptText(OVERLORD_SAY_3, me); return 6500; break; case 10: DoScriptText(LORD_ILLIDAN_SAY_1, Illi); return 5000; break; - case 11: DoScriptText(OVERLORD_SAY_4, me, plr); return 6000; break; + case 11: DoScriptText(OVERLORD_SAY_4, me, player); return 6000; break; case 12: DoScriptText(LORD_ILLIDAN_SAY_2, Illi); return 5500; break; case 13: DoScriptText(LORD_ILLIDAN_SAY_3, Illi); return 4000; break; case 14: Illi->SetUInt64Value(UNIT_FIELD_TARGET, PlayerGUID); return 1500; break; case 15: DoScriptText(LORD_ILLIDAN_SAY_4, Illi); return 1500; break; case 16: - if (plr) + if (player) { - Illi->CastSpell(plr, SPELL_TWO, true); - plr->RemoveAurasDueToSpell(SPELL_THREE); - plr->RemoveAurasDueToSpell(SPELL_FOUR); + Illi->CastSpell(player, SPELL_TWO, true); + player->RemoveAurasDueToSpell(SPELL_THREE); + player->RemoveAurasDueToSpell(SPELL_FOUR); return 5000; }else{ - CAST_PLR(plr)->FailQuest(QUEST_LORD_ILLIDAN_STORMRAGE); Step = 30; return 100; + CAST_PLR(player)->FailQuest(QUEST_LORD_ILLIDAN_STORMRAGE); Step = 30; return 100; } break; case 17: DoScriptText(LORD_ILLIDAN_SAY_5, Illi); return 5000; break; @@ -821,8 +821,8 @@ public: case 24: me->SetUInt64Value(UNIT_FIELD_TARGET, PlayerGUID); return 5000; break; case 25: DoScriptText(OVERLORD_SAY_6, me); return 2000; break; case 26: - if (plr) - CAST_PLR(plr)->GroupEventHappens(QUEST_LORD_ILLIDAN_STORMRAGE, me); + if (player) + CAST_PLR(player)->GroupEventHappens(QUEST_LORD_ILLIDAN_STORMRAGE, me); return 6000; break; case 27: { @@ -832,16 +832,16 @@ public: return 500; } break; case 28: - plr->RemoveAurasDueToSpell(SPELL_TWO); - plr->RemoveAurasDueToSpell(41519); - plr->CastSpell(plr, SPELL_THREE, true); - plr->CastSpell(plr, SPELL_FOUR, true); + player->RemoveAurasDueToSpell(SPELL_TWO); + player->RemoveAurasDueToSpell(41519); + player->CastSpell(player, SPELL_THREE, true); + player->CastSpell(player, SPELL_FOUR, true); return 1000; break; case 29: { Unit* Yarzill = me->FindNearestCreature(C_YARZILL, 50); if (Yarzill) - DoScriptText(YARZILL_THE_MERC_SAY, Yarzill, plr); + DoScriptText(YARZILL_THE_MERC_SAY, Yarzill, player); return 5000; } break; case 30: @@ -855,7 +855,7 @@ public: { Unit* Yarzill = me->FindNearestCreature(C_YARZILL, 50); if (Yarzill) - Yarzill->CastSpell(plr, 41540, true); + Yarzill->CastSpell(player, 41540, true); return 1000;} break; case 32: me->GetMotionMaster()->MovePoint(0, -5085.77f, 577.231f, 86.6719f); return 5000; break; @@ -1661,15 +1661,15 @@ class go_crystal_prison : public GameObjectScript public: go_crystal_prison() : GameObjectScript("go_crystal_prison") { } - bool OnQuestAccept(Player* plr, GameObject* /*go*/, Quest const* quest) + bool OnQuestAccept(Player* player, GameObject* /*go*/, Quest const* quest) { if (quest->GetQuestId() == QUEST_BATTLE_OF_THE_CRIMSON_WATCH) { - Creature* Illidan = plr->FindNearestCreature(22083, 50); + Creature* Illidan = player->FindNearestCreature(22083, 50); if (Illidan && !CAST_AI(npc_lord_illidan_stormrage::npc_lord_illidan_stormrageAI, Illidan->AI())->EventStarted) { - CAST_AI(npc_lord_illidan_stormrage::npc_lord_illidan_stormrageAI, Illidan->AI())->PlayerGUID = plr->GetGUID(); + CAST_AI(npc_lord_illidan_stormrage::npc_lord_illidan_stormrageAI, Illidan->AI())->PlayerGUID = player->GetGUID(); CAST_AI(npc_lord_illidan_stormrage::npc_lord_illidan_stormrageAI, Illidan->AI())->LiveCount = 0; CAST_AI(npc_lord_illidan_stormrage::npc_lord_illidan_stormrageAI, Illidan->AI())->EventStarted=true; } diff --git a/src/server/scripts/Spells/spell_generic.cpp b/src/server/scripts/Spells/spell_generic.cpp index c1b9b564ad1..a6938122196 100644 --- a/src/server/scripts/Spells/spell_generic.cpp +++ b/src/server/scripts/Spells/spell_generic.cpp @@ -232,13 +232,13 @@ class spell_gen_pet_summoned : public SpellScriptLoader if (caster->GetTypeId() != TYPEID_PLAYER) return; - Player* plr = caster->ToPlayer(); - if (plr->GetLastPetNumber()) + Player* player = caster->ToPlayer(); + if (player->GetLastPetNumber()) { - PetType newPetType = (plr->getClass() == CLASS_HUNTER) ? HUNTER_PET : SUMMON_PET; - if (Pet* newPet = new Pet(plr, newPetType)) + PetType newPetType = (player->getClass() == CLASS_HUNTER) ? HUNTER_PET : SUMMON_PET; + if (Pet* newPet = new Pet(player, newPetType)) { - if (newPet->LoadPetFromDB(plr, 0, plr->GetLastPetNumber(), true)) + if (newPet->LoadPetFromDB(player, 0, player->GetLastPetNumber(), true)) { // revive the pet if it is dead if (newPet->getDeathState() == DEAD) @@ -896,9 +896,9 @@ class spell_generic_clone_weapon : public SpellScriptLoader case SPELL_COPY_WEAPON_2: case SPELL_COPY_WEAPON_3: { - if (Player* plrCaster = caster->ToPlayer()) + if (Player* player = caster->ToPlayer()) { - if (Item* mainItem = plrCaster->GetItemByPos(INVENTORY_SLOT_BAG_0, EQUIPMENT_SLOT_MAINHAND)) + if (Item* mainItem = player->GetItemByPos(INVENTORY_SLOT_BAG_0, EQUIPMENT_SLOT_MAINHAND)) target->SetUInt32Value(UNIT_VIRTUAL_ITEM_SLOT_ID, mainItem->GetEntry()); } else @@ -908,9 +908,9 @@ class spell_generic_clone_weapon : public SpellScriptLoader case SPELL_COPY_OFFHAND: case SPELL_COPY_OFFHAND_2: { - if (Player* plrCaster = caster->ToPlayer()) + if (Player* player = caster->ToPlayer()) { - if (Item* offItem = plrCaster->GetItemByPos(INVENTORY_SLOT_BAG_0, EQUIPMENT_SLOT_OFFHAND)) + if (Item* offItem = player->GetItemByPos(INVENTORY_SLOT_BAG_0, EQUIPMENT_SLOT_OFFHAND)) target->SetUInt32Value(UNIT_VIRTUAL_ITEM_SLOT_ID + 1, offItem->GetEntry()); } else @@ -919,9 +919,9 @@ class spell_generic_clone_weapon : public SpellScriptLoader } case SPELL_COPY_RANGED: { - if (Player* plrCaster = caster->ToPlayer()) + if (Player* player = caster->ToPlayer()) { - if (Item* rangedItem = plrCaster->GetItemByPos(INVENTORY_SLOT_BAG_0, EQUIPMENT_SLOT_RANGED)) + if (Item* rangedItem = player->GetItemByPos(INVENTORY_SLOT_BAG_0, EQUIPMENT_SLOT_RANGED)) target->SetUInt32Value(UNIT_VIRTUAL_ITEM_SLOT_ID + 2, rangedItem->GetEntry()); } else -- cgit v1.2.3