Core/Misc: Minor code style fixes and cleanups

This commit is contained in:
tobmaps
2011-06-28 15:51:21 +07:00
parent 79368d4393
commit ffe301b927
3 changed files with 48 additions and 41 deletions

View File

@@ -471,4 +471,3 @@ INSERT INTO `gameobject` (`guid`,`id`,`map`,`spawnMask`,`phaseMask`,`position_x`
(@OGUID+53,193163,571,1,128,5378.068,2813.61719,409.239166,1.55334139,0,0,0,0,120,0,1),
(@OGUID+54,193164,571,1,128,5378.921,2805.43677,409.239166,1.53588688,0,0,0,0,120,0,1),
(@OGUID+55,193165,571,1,128,5378.452,2876.67456,409.239166,1.54461825,0,0,0,0,120,0,1);

View File

@@ -280,12 +280,13 @@ UPDATE `gossip_menu_option` SET `action_script_id`=990401 WHERE `menu_id` IN (99
UPDATE `gossip_menu_option` SET `action_script_id`=990402 WHERE `menu_id` IN (9904) AND `id`=2;
UPDATE `gossip_menu_option` SET `action_script_id`=990403 WHERE `menu_id` IN (9923) AND `id`=2;
-- Add create vehicle Spells to gossip scripts
DELETE FROM `gossip_scripts` WHERE `id` IN (990400,990401,990402);
-- already done by engineer's script
/*DELETE FROM `gossip_scripts` WHERE `id` IN (990400,990401,990402);
INSERT INTO `gossip_scripts` (`id`,`delay`,`command`,`datalong`,`datalong2`) VALUES
(990400,0,33,0,0),(990400,0,15,56663,1),
(990401,0,33,0,0),(990401,0,15,56575,1),
(990402,0,33,0,0),(990402,0,15,61408,1),
(990403,0,33,0,0),(990403,0,15,56661,1);
(990403,0,33,0,0),(990403,0,15,56661,1);*/
-- NPC talk text insert from sniff
DELETE FROM `creature_text` WHERE `entry`=15214 AND `groupid` BETWEEN 0 AND 30;
@@ -335,7 +336,3 @@ INSERT INTO `creature_text` (`entry`,`groupid`,`id`,`text`,`type`,`language`,`pr
(31108,0,0, 'Stop the Horde from retrieving the embers, $n. We cannot risk them having the advantage when the battle resumes!',0,7,100,0,0,0, 'Siege Master Stouthandle'),
(31109,0,0, 'Destroy their foul machines of war, $n!',0,7,100,0,0,0, 'Senior Demolitionist Legoso'),
(34924,0,0, 'The gates have been breached! Defend the keep!',1,0,100,0,0,0, 'High Commander Halford Wyrmbane');

View File

@@ -38,6 +38,19 @@ enum eWGqueuenpctext
WG_NPCQUEUE_TEXTOPTION_JOIN = -1850507,
};
enum eWGdata
{
// engineer spells
SPELL_BUILD_CATAPULT = 56663,
SPELL_BUILD_DEMOLISHER = 56575,
SPELL_BUILD_SIEGE_ENGINE = 61408,
SPELL_BUILD_SIEGE_ENGINE2 = 56661, // does it's really needed here?
SPELL_ACTIVATE_ROBOTIC_ARMS = 49899,
// teleporter spells
SPELL_VEHICLE_TELEPORT = 49759,
}
class npc_demolisher_engineerer : public CreatureScript
{
public:
@@ -45,12 +58,12 @@ class npc_demolisher_engineerer : public CreatureScript
{
}
bool OnGossipHello(Player * pPlayer, Creature * pCreature)
bool OnGossipHello(Player* pPlayer, Creature* pCreature)
{
if (pCreature->isQuestGiver())
pPlayer->PrepareQuestMenu(pCreature->GetGUID());
BattlefieldWG *BfWG = (BattlefieldWG *) sBattlefieldMgr.GetBattlefieldByBattleId(1);
BattlefieldWG* BfWG = (BattlefieldWG *) sBattlefieldMgr.GetBattlefieldByBattleId(1);
if (!BfWG)
return true;
@@ -74,11 +87,11 @@ class npc_demolisher_engineerer : public CreatureScript
return true;
}
bool OnGossipSelect(Player * pPlayer, Creature * pCreature, uint32 /*uiSender */ , uint32 uiAction)
bool OnGossipSelect(Player* pPlayer, Creature* pCreature, uint32 /*uiSender */ , uint32 uiAction)
{
pPlayer->CLOSE_GOSSIP_MENU();
BattlefieldWG *BfWG = (BattlefieldWG *) sBattlefieldMgr.GetBattlefieldByBattleId(1);
BattlefieldWG* BfWG = (BattlefieldWG *) sBattlefieldMgr.GetBattlefieldByBattleId(1);
if (!BfWG)
return true;
@@ -89,19 +102,19 @@ class npc_demolisher_engineerer : public CreatureScript
switch (uiAction - GOSSIP_ACTION_INFO_DEF)
{
case 0:
pPlayer->CastSpell(pPlayer, 56663, false, NULL, NULL, pCreature->GetGUID());
pPlayer->CastSpell(pPlayer, SPELL_BUILD_CATAPULT, false, NULL, NULL, pCreature->GetGUID());
break;
case 1:
pPlayer->CastSpell(pPlayer, 56575, false, NULL, NULL, pCreature->GetGUID());
pPlayer->CastSpell(pPlayer, SPELL_BUILD_DEMOLISHER, false, NULL, NULL, pCreature->GetGUID());
break;
case 2:
pPlayer->CastSpell(pPlayer, pPlayer->GetTeamId()? 61408 : 56661, false, NULL, NULL, pCreature->GetGUID());
pPlayer->CastSpell(pPlayer, pPlayer->GetTeamId() ? SPELL_BUILD_SIEGE_ENGINE : SPELL_BUILD_SIEGE_ENGINE2, false, NULL, NULL, pCreature->GetGUID());
break;
}
//spell 49899 Emote : 406 from sniff
//INSERT INTO `spell_scripts` (`id`, `delay`, `command`, `datalong`, `datalong2`, `dataint`, `x`, `y`, `z`, `o`) VALUES ('49899', '0', '1', '406', '0', '0', '0', '0', '0', '0');
if (Creature * creature = pCreature->FindNearestCreature(27852, 30.0f, true))
creature->CastSpell(creature, 49899, true);
if (Creature* creature = pCreature->FindNearestCreature(27852, 30.0f, true))
creature->CastSpell(creature, SPELL_ACTIVATE_ROBOTIC_ARMS, true);
}
return true;
}
@@ -114,12 +127,12 @@ class npc_wg_spiritguide : public CreatureScript
{
}
bool OnGossipHello(Player * pPlayer, Creature * pCreature)
bool OnGossipHello(Player* pPlayer, Creature* pCreature)
{
if (pCreature->isQuestGiver())
pPlayer->PrepareQuestMenu(pCreature->GetGUID());
BattlefieldWG *BfWG = (BattlefieldWG *) sBattlefieldMgr.GetBattlefieldByBattleId(BATTLEFIELD_BATTLEID_WG);
BattlefieldWG* BfWG = (BattlefieldWG *) sBattlefieldMgr.GetBattlefieldByBattleId(BATTLEFIELD_BATTLEID_WG);
if (BfWG)
{
GraveYardVect gy = BfWG->GetGraveYardVect();
@@ -137,11 +150,11 @@ class npc_wg_spiritguide : public CreatureScript
return true;
}
bool OnGossipSelect(Player * pPlayer, Creature * /*pCreature */ , uint32 /*uiSender */ , uint32 uiAction)
bool OnGossipSelect(Player* pPlayer, Creature* /*pCreature */ , uint32 /*uiSender */ , uint32 uiAction)
{
pPlayer->CLOSE_GOSSIP_MENU();
BattlefieldWG *BfWG = (BattlefieldWG *) sBattlefieldMgr.GetBattlefieldByBattleId(BATTLEFIELD_BATTLEID_WG);
BattlefieldWG* BfWG = (BattlefieldWG *) sBattlefieldMgr.GetBattlefieldByBattleId(BATTLEFIELD_BATTLEID_WG);
if (BfWG)
{
GraveYardVect gy = BfWG->GetGraveYardVect();
@@ -149,7 +162,7 @@ class npc_wg_spiritguide : public CreatureScript
{
if (uiAction - GOSSIP_ACTION_INFO_DEF == i && gy[i]->GetControlTeamId() == pPlayer->GetTeamId())
{
const WorldSafeLocsEntry *ws = sWorldSafeLocsStore.LookupEntry(gy[i]->GetGraveYardId());
WorldSafeLocsEntry const* ws = sWorldSafeLocsStore.LookupEntry(gy[i]->GetGraveYardId());
pPlayer->TeleportTo(ws->map_id, ws->x, ws->y, ws->z, 0);
}
}
@@ -165,12 +178,12 @@ class npc_wg_dalaran_queue : public CreatureScript
{
}
bool OnGossipHello(Player * pPlayer, Creature * pCreature)
bool OnGossipHello(Player* pPlayer, Creature* pCreature)
{
if (pCreature->isQuestGiver())
pPlayer->PrepareQuestMenu(pCreature->GetGUID());
BattlefieldWG *BfWG = (BattlefieldWG *) sBattlefieldMgr.GetBattlefieldByBattleId(BATTLEFIELD_BATTLEID_WG);
BattlefieldWG* BfWG = (BattlefieldWG *) sBattlefieldMgr.GetBattlefieldByBattleId(BATTLEFIELD_BATTLEID_WG);
if (BfWG)
{
@@ -186,22 +199,22 @@ class npc_wg_dalaran_queue : public CreatureScript
if (uiTime < 15 * MINUTE)
{
pPlayer->ADD_GOSSIP_ITEM(GOSSIP_ICON_CHAT, sObjectMgr->GetTrinityStringForDBCLocale(WG_NPCQUEUE_TEXTOPTION_JOIN), GOSSIP_SENDER_MAIN, GOSSIP_ACTION_INFO_DEF);
pPlayer->SEND_GOSSIP_MENU(BfWG->GetDefenderTeam()? WG_NPCQUEUE_TEXT_H_QUEUE : WG_NPCQUEUE_TEXT_A_QUEUE, pCreature->GetGUID());
pPlayer->SEND_GOSSIP_MENU(BfWG->GetDefenderTeam() ? WG_NPCQUEUE_TEXT_H_QUEUE : WG_NPCQUEUE_TEXT_A_QUEUE, pCreature->GetGUID());
}
else
{
pPlayer->SEND_GOSSIP_MENU(BfWG->GetDefenderTeam()? WG_NPCQUEUE_TEXT_H_NOWAR : WG_NPCQUEUE_TEXT_A_NOWAR, pCreature->GetGUID());
pPlayer->SEND_GOSSIP_MENU(BfWG->GetDefenderTeam() ? WG_NPCQUEUE_TEXT_H_NOWAR : WG_NPCQUEUE_TEXT_A_NOWAR, pCreature->GetGUID());
}
}
}
return true;
}
bool OnGossipSelect(Player * pPlayer, Creature * /*pCreature */ , uint32 /*uiSender */ , uint32 /*uiAction */ )
bool OnGossipSelect(Player* pPlayer, Creature* /*pCreature */ , uint32 /*uiSender */ , uint32 /*uiAction */ )
{
pPlayer->CLOSE_GOSSIP_MENU();
BattlefieldWG *BfWG = (BattlefieldWG *) sBattlefieldMgr.GetBattlefieldByBattleId(BATTLEFIELD_BATTLEID_WG);
BattlefieldWG* BfWG = (BattlefieldWG *) sBattlefieldMgr.GetBattlefieldByBattleId(BATTLEFIELD_BATTLEID_WG);
if (BfWG)
{
if (BfWG->IsWarTime())
@@ -221,8 +234,6 @@ class npc_wg_dalaran_queue : public CreatureScript
const uint32 Vehicules[4] = { 32627, 28312, 28094, 27881 };
#define SPELL_VEHICLE_TELEPORT 49759
class go_wintergrasp_teleporter : public GameObjectScript
{
public:
@@ -232,7 +243,7 @@ class go_wintergrasp_teleporter : public GameObjectScript
struct go_wintergrasp_teleporterAI : public GameObjectAI
{
go_wintergrasp_teleporterAI(GameObject * g) : GameObjectAI(g)
go_wintergrasp_teleporterAI(GameObject* g) : GameObjectAI(g)
{
uiCheckTimer = 1000;
}
@@ -242,19 +253,19 @@ class go_wintergrasp_teleporter : public GameObjectScript
if (uiCheckTimer <= diff)
{
for (uint8 i = 0; i < 4; i++)
if (Creature * pVehicle = go->FindNearestCreature(Vehicules[i], 3.0f, true))
if (Creature* pVehicle = go->FindNearestCreature(Vehicules[i], 3.0f, true))
if (!pVehicle->HasAura(SPELL_VEHICLE_TELEPORT))
{
if (pVehicle->GetVehicle())
{
if (Unit * player = pVehicle->GetVehicle()->GetPassenger(0))
if (Unit* player = pVehicle->GetVehicle()->GetPassenger(0))
{
uint32 gofaction = go->GetUInt32Value(GAMEOBJECT_FACTION);
uint32 plfaction = player->getFaction();
if (gofaction == plfaction)
{
pVehicle->CastSpell(pVehicle, SPELL_VEHICLE_TELEPORT, true);
if (Creature * TargetTeleport = pVehicle->FindNearestCreature(23472, 100.0f, true))
if (Creature* TargetTeleport = pVehicle->FindNearestCreature(23472, 100.0f, true))
{
float x, y, z, o;
TargetTeleport->GetPosition(x, y, z, o);
@@ -270,10 +281,10 @@ class go_wintergrasp_teleporter : public GameObjectScript
uiCheckTimer -= diff;
}
private:
uint32 uiCheckTimer;
uint32 uiCheckTimer;
};
GameObjectAI *GetAI(GameObject * go) const
GameObjectAI *GetAI(GameObject* go) const
{
return new go_wintergrasp_teleporterAI(go);
}
@@ -286,12 +297,12 @@ class npc_wintergrasp_quest_giver : public CreatureScript
{
}
bool OnGossipHello(Player * pPlayer, Creature * pCreature)
bool OnGossipHello(Player* pPlayer, Creature* pCreature)
{
if (pCreature->isQuestGiver())
pPlayer->PrepareQuestMenu(pCreature->GetGUID());
BattlefieldWG *BfWG = (BattlefieldWG *) sBattlefieldMgr.GetBattlefieldByBattleId(BATTLEFIELD_BATTLEID_WG);
BattlefieldWG* BfWG = (BattlefieldWG *) sBattlefieldMgr.GetBattlefieldByBattleId(BATTLEFIELD_BATTLEID_WG);
if (BfWG)
{
if (pCreature->isQuestGiver())
@@ -322,7 +333,7 @@ class npc_wintergrasp_quest_giver : public CreatureScript
switch (quest_id)
{
// Horde attacker
// Horde attacker
case 13193:
case 13202:
case 13180:
@@ -339,7 +350,7 @@ class npc_wintergrasp_quest_giver : public CreatureScript
qm.AddMenuItem(quest_id, 2);
}
break;
// Horde defender
// Horde defender
case 13199:
case 13192:
case 13178:
@@ -357,7 +368,7 @@ class npc_wintergrasp_quest_giver : public CreatureScript
qm.AddMenuItem(quest_id, 2);
}
break;
// Alliance attacker
// Alliance attacker
case 13196:
case 13198:
case 13179:
@@ -373,7 +384,7 @@ class npc_wintergrasp_quest_giver : public CreatureScript
qm.AddMenuItem(quest_id, 2);
}
break;
// Alliance defender
// Alliance defender
case 13154:
case 13153:
case 13177: