Skip to content

Commit

Permalink
Fix spell ordering on CC break alert
Browse files Browse the repository at this point in the history
Commit 8606a9c flipped the order in the CC break alert
due to SpamMultiRemoved behaving in an unexpected manner. This corrects SpamMultiRemoved
to behave in a more predictable manner and adds a note in the call from MassDispel
that spellOutput/extraSpellOutput parameters are backwards.
  • Loading branch information
Arrhae committed Feb 11, 2024
1 parent 464cebd commit 30022f9
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions modules/Alerts.lua
Original file line number Diff line number Diff line change
Expand Up @@ -384,7 +384,7 @@ do
local index = source..spell..extra
if not targets[index] then
targets[index] = {}
self:ScheduleTimer(spam, 0.2, key, index, L["%s on %s removed by %s's %s"], extra, targets[index], source, spell)
self:ScheduleTimer(spam, 0.2, key, index, L["%s on %s removed by %s's %s"], spell, targets[index], source, extra)
end
tinsert(targets[index], dest)
end
Expand Down Expand Up @@ -645,7 +645,7 @@ function module:Dispel(srcOutput, dstOutput, spellOutput, extraSpellOuput)
end

function module:MassDispel(srcOutput, dstOutput, spellOutput, extraSpellOuput)
self:SpamMultiRemoved("dispel", srcOutput, dstOutput, spellOutput, extraSpellOuput)
self:SpamMultiRemoved("dispel", srcOutput, dstOutput, extraSpellOuput, spellOutput) -- Mass Dispel has the spell/extra spell flipped
end

function module:Interrupt(srcOutput, dstOutput, spellOutput, extraSpellOuput)
Expand Down

0 comments on commit 30022f9

Please sign in to comment.