Skip to content

Commit b5cafaa

Browse files
author
mergetest
committed
Click casting: don't drop the zone-in loadout check when it lands in combat
CheckLoadoutProfileSwitch now defers itself via pendingLoadoutCheck in lockdown, so the zone-in+1s call site's own combat guard only served to silently drop the check — the arena-load race the cold-start fix describes. Call it unconditionally and let the function's deferral handle combat.
1 parent eaf31cf commit b5cafaa

1 file changed

Lines changed: 7 additions & 4 deletions

File tree

ClickCasting/Events.lua

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -111,11 +111,14 @@ function CC:RegisterEvents()
111111
-- arena/dungeon of the session, not only after a /reload
112112
CC:ResolveProvisionalMap("zone-in")
113113

114-
-- Check for loadout-based profile on initial load
114+
-- Check for loadout-based profile on initial load. No combat
115+
-- guard here: CheckLoadoutProfileSwitch defers itself via
116+
-- pendingLoadoutCheck in lockdown — the old call-site guard
117+
-- silently DROPPED the check when zone-in+1s landed in combat
118+
-- (the arena-load race), leaving the previous spec's profile
119+
-- active for the whole match.
115120
C_Timer.After(1, function()
116-
if not InCombatLockdown() then
117-
CC:CheckLoadoutProfileSwitch()
118-
end
121+
CC:CheckLoadoutProfileSwitch()
119122
end)
120123
end)
121124
elseif event == "ARENA_PREP_OPPONENT_SPECIALIZATIONS" then

0 commit comments

Comments
 (0)