Mirror/TrinityCore
1
0
Fork 0
mirror of https://github.com/TrinityCore/TrinityCore.git synced 2026-01-20 01:15:35 +01:00
Code Issues Packages Projects Releases Wiki Activity

Scripts/MagistersTerrace: Fix some crashes in Selin Fireheart. His script is still a shitshow, though. Closes #20614.

Browse Source
This commit is contained in:
Treeston
2017-10-26 11:54:41 +02:00
parent bf81a7b207
commit cbd025f146
1 changed files with 16 additions and 28 deletions
Download Patch File Download Diff File Expand all files Collapse all files

44
src/server/scripts/EasternKingdoms/MagistersTerrace/boss_selin_fireheart.cpp
View File

@@ -65,6 +65,7 @@ enum Misc
ACTION_SWITCH_PHASE = 1
};
// @todo crystals should really be a DB creature summon group, having them in `creature` like this will cause tons of despawn/respawn bugs
class boss_selin_fireheart : public CreatureScript
{
public:
@@ -72,23 +73,15 @@ class boss_selin_fireheart : public CreatureScript
struct boss_selin_fireheartAI : public BossAI
{
boss_selin_fireheartAI(Creature* creature) : BossAI(creature, DATA_SELIN)
{
_scheduledEvents = false;
}
boss_selin_fireheartAI(Creature* creature) : BossAI(creature, DATA_SELIN), _scheduledEvents(false) { }
void Reset() override
{
Crystals.clear();
me->GetCreatureListWithEntryInGrid(Crystals, NPC_FEL_CRYSTAL, 250.0f);
std::list<Creature*> crystals;
me->GetCreatureListWithEntryInGrid(crystals, NPC_FEL_CRYSTAL, 250.0f);
for (Creature* creature : Crystals)
{
if (!creature->IsAlive())
creature->Respawn();
creature->SetFlag(UNIT_FIELD_FLAGS, UNIT_FLAG_NOT_SELECTABLE);
}
for (Creature* creature : crystals)
creature->Respawn(true);
_Reset();
CrystalGUID.Clear();
@@ -115,18 +108,16 @@ class boss_selin_fireheart : public CreatureScript
if (Crystals.empty())
return;
Crystals.sort(Trinity::ObjectDistanceOrderPred(me));
if (Creature* CrystalChosen = Crystals.front())
if (Creature* crystal = me->FindNearestCreature(NPC_FEL_CRYSTAL, 250.0f))
{
Talk(SAY_ENERGY);
Talk(EMOTE_CRYSTAL);
DoCast(CrystalChosen, SPELL_FEL_CRYSTAL_DUMMY);
CrystalGUID = CrystalChosen->GetGUID();
Crystals.remove(CrystalChosen);
DoCast(crystal, SPELL_FEL_CRYSTAL_DUMMY);
CrystalGUID = crystal->GetGUID();
float x, y, z;
CrystalChosen->GetClosePoint(x, y, z, me->GetCombatReach(), CONTACT_DISTANCE);
crystal->GetClosePoint(x, y, z, me->GetCombatReach(), CONTACT_DISTANCE);
events.SetPhase(PHASE_DRAIN);
me->SetWalk(false);
@@ -136,14 +127,11 @@ class boss_selin_fireheart : public CreatureScript
void ShatterRemainingCrystals()
{
if (Crystals.empty())
return;
std::list<Creature*> crystals;
me->GetCreatureListWithEntryInGrid(crystals, NPC_FEL_CRYSTAL, 250.0f);
for (Creature* crystal : Crystals)
{
if (crystal && crystal->IsAlive())
crystal->KillSelf();
}
for (Creature* crystal : crystals)
crystal->KillSelf();
}
void EnterCombat(Unit* /*who*/) override
@@ -259,7 +247,7 @@ class boss_selin_fireheart : public CreatureScript
}
private:
std::list<Creature*> Crystals;
std::list<ObjectGuid> Crystals;
ObjectGuid CrystalGUID;
bool _scheduledEvents;
};