Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve BRD arena event #171

Closed
wants to merge 1 commit into from
Closed

Improve BRD arena event #171

wants to merge 1 commit into from

Conversation

cala
Copy link
Contributor

@cala cala commented Jun 22, 2015

Now the crowd of spectators will turn neutral once the arena event is defeated

Now the crowd of spectators will turn neutral once the arena event is defeated
{
for (GuidSet::const_iterator itr = m_sArenaCrowdNpcGuids.begin(); itr != m_sArenaCrowdNpcGuids.end(); ++itr)
{
Creature* pSpectator = NULL;
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops, that line should be moved above the for block... My mistake...

@Grz3s
Copy link

Grz3s commented Jun 22, 2015

👍

@@ -87,8 +96,20 @@ enum
SPELL_STONED = 10255, // Aura of Warbringer Constructs in Relict Vault

FACTION_DWARF_HOSTILE = 754, // Hostile faction for the Tomb of the Seven dwarfs
FACTION_ARENA_NEUTRAL = 674, // Neutral faction for NPC in top of Arena after event complete
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Tobschinski provided me data a few minutes ago saying that faction should instead be 15 and not 674.

[25] UpdateType: Values
[25] Object Guid: Full: 0x1C16FC1CC008B000003A08000308899B Creature/0 R1471/S14856 Map: 230 Entry: 8896 Low: 50891163
[25] UNIT_FIELD_FACTIONTEMPLATE: 15

@xfurry
Copy link
Member

xfurry commented Jun 23, 2015

Looks good.
However you are not handling the server reset scenario. I will add the missing code when pushing.

@evil-at-wow
Copy link
Contributor

Yup, this is pretty much what I had in mind. I haven't verified that those entries are correct, and I didn't check the cylinder data either, but I trust you on that. It's easy enough to tweak/adjust anyway.

Good job, +1 from me too 👍

@xfurry xfurry closed this in 7624584 Jun 23, 2015
@cala cala deleted the BRD branch June 24, 2015 12:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants