[3.3.5] Core/Naxxramas boss_sapphiron cast spell without cooldown #18765

Open
bahajan95 opened this Issue Jan 8, 2017 · 22 comments

Projects

None yet

6 participants

@bahajan95
bahajan95 commented Jan 8, 2017 edited

Description:
http://wowhead.com/spell=55699
https://cloud.githubusercontent.com/assets/13211961/21746100/4b62851c-d55c-11e6-8778-541ba3bf0be5.png
Current behaviour: (Tell us what happens.)
Sapphiron casts http://wowhead.com/spell=28560 without cooldown
Expected behaviour: (Tell us what should happen instead.)
should cast spell with cooldown
Steps to reproduce the problem:
go to sapph(10 or 25), start fight
1.
2.
3.

Branch(es): 3.3.5

TC rev. hash/commit: b1fa8ff

TDB version: TDB 62 + all updates

Operating system: Debian 8 x64

@TrinityCoreBot

This issue is not considered valid because of the following reasons:

  • No proper commit hash (revision)

Please read http://bit.ly/tc-issuetracker-and-you and add the missing information to this issue. Thanks.

@Keader
Member
Keader commented Jan 8, 2017

Based on script, Blizzard has "cooldown"
Look: https://github.com/TrinityCore/TrinityCore/blob/3.3.5/src/server/scripts/Northrend/Naxxramas/boss_sapphiron.cpp#L267

Btw summoning @Treeston, he know better about timers of this boss ^^

@Treeston
Member
Treeston commented Jan 8, 2017 edited

Detailed repro steps please. Script has a cooldown for this spell unless somebody fucked around with it.
Is this unmodded core?

@Keader can you try to repro ingame I don't have a core on hand right now.

@ccrs
Contributor
ccrs commented Jan 8, 2017

ummm taking a wild guess, this seems like the npc misses flags and melee attacks

going to test right now

@ccrs
Contributor
ccrs commented Jan 8, 2017 edited

the problem seems to be that the debuffs stack on 25 man mode (up to 4 appear), on 10 normal only one appears.

@Treeston
Member
Treeston commented Jan 8, 2017

If you're standing in multiple blizzards?

@bahajan95

sorry for the bad English, the boss hangs on a single player single stack if a lot of players, but the spells cooldown is 10 seconds and it needs to end then knit castes should go, and then casted spell without cooldown

@bahajan95

with kelthuzad almost the same problem only he spawned mobs without Cooldown

@Treeston
Member
Treeston commented Jan 8, 2017

Summon Blizzard uses spellmgr override to limit target count to 1. Maybe 25man has difficulty entry that doesn't have it?

@offl
offl commented Jan 8, 2017

Maybe this happens because Blizzard uses this script?

@Treeston
Member
Treeston commented Jan 8, 2017

That is intended.

@offl
offl commented Jan 8, 2017

With this script he will cast Chill every 3 seconds. Instead of 10.
Or not?

@Treeston
Member
Treeston commented Jan 8, 2017

It ticks every 3 sec.

@Treeston
Member
Treeston commented Jan 8, 2017 edited

Hm or no the aura ticks every so often but should cast every 10.

Is swing timer different on 10 vs 25?

@Treeston
Member
Treeston commented Jan 8, 2017 edited

Ok, so.
Summon Blizzard spawns a Blizzard beacon npc. That beacon npc chases players and casts Chill every 3 seconds.
The Chill spell drops a blizzard dynobj (the actual visible damaging area) when cast.
Script is fine.

@ccrs
Contributor
ccrs commented Jan 9, 2017 edited

Confirmed, you can see multiple blizzard npcs being spawned, one at a time.
It can go up to 4 npcs following random players.

DoCastAOE(SPELL_SUMMON_BLIZZARD);
events.ScheduleEvent(EVENT_BLIZZARD, RAID_MODE(Seconds(20), Seconds(7)), 0, PHASE_GROUND);

Seems like the timer is too low, that or the script needs to be updated so that players that are already followed by another blizzard cant have a new one chasing.

Also it might be a good idea to check the number of players facing the boss, so only one active blizzard is possible if there are only 3-2 players fighting, for example.

And one last point, when air phase begins is it intended to have blizzards still active? Hiding all together while a killing rain is chasing you seems kinda off to me xd

@bahajan95

I will make a video and show exactly what the bug can do so even easier to find a fix

@Treeston
Member
Treeston commented Jan 9, 2017

@ccrs multiple blizzards on one target, you might be on to something there

blizzard during air is intended i think, iirc they time out shortly after phase starts due to how timers line up

@bahajan95

if you want to look or test I can give realmlist my server.

@Treeston
Member
Treeston commented Jan 9, 2017

iirc the blizzard dummy casts a spell on players to select its chase target, @ccrs check if that applies a hidden aura or something.

@bahajan95

any news?

@bahajan95

Any news?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment