brues/SuperCleveRoidMacros
1
1
Fork 0
mirror of https://github.com/brues-code/SuperCleveRoidMacros.git synced 2026-09-16 03:38:00 +00:00
Code Issues 1 Packages Projects Releases Wiki Activity

Fix Lua 5.1 compatibility: remove goto statement from ValidateUnitDebuff

Browse Source 
- Replaced goto/label with shouldSkip flag variable
- Lua 5.1 (used in WoW 1.12) doesn't support goto statements
- Functionality remains identical
This commit is contained in:
Claude
2025-11-17 18:35:32 +00:00
parent b8cc607d90
commit 62c39f71cf
1 changed files with 9 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Conditionals.lua
+9 -7
View File
@@ -819,20 +819,22 @@ function CleveRoids.ValidateUnitDebuff(unit, args)
local effect, _, _, _, _, duration, timeleft, effectCaster = CleveRoids.libdebuff:UnitDebuff(unit, idx, filterCaster)
if not effect then break end
if effect == args.name then
atl = (timeleft and timeleft >= 0) and timeleft or 0
caster = effectCaster
local shouldSkip = false
-- Auto-detect: If args.mine not specified and this is a personal debuff, only match player casts
if args.mine == nil and spellID and CleveRoids.libdebuff.IsPersonalDebuff then
if CleveRoids.libdebuff:IsPersonalDebuff(spellID) and caster ~= "player" then
if CleveRoids.libdebuff:IsPersonalDebuff(spellID) and effectCaster ~= "player" then
-- This is a personal debuff from another player, skip it
atl = nil
goto continue
shouldSkip = true
end
end
break
if not shouldSkip then
atl = (timeleft and timeleft >= 0) and timeleft or 0
caster = effectCaster
break
end
end
::continue::
end
if atl ~= nil then
return cmp[args.operator](atl, args.amount)