Tweak to work

[jlrm365]

Hello.

Until recently, an addon called AntiCosmeticAuras existed on CurseForge.
It auto-removed cosmetic buffs / auras, out of combat.

It was removed a while back, but it still functioned.
With the most recent update, it stopped functioning.

Is there any part of this that can be tweaked, to revive it? If so, which?
Thanks!






-- GLOBAL VARIABLES

local anticosmeticaurasframe;



-- FUNCTIONS

function anticosmeticauras_OnEvent(this, event, ...)
if IsInInstance() then return end
for i=1, 40 do
local name = UnitBuff("player", i);

if (name == nil) then
return;
end

if (name == "A Witch!") then
CancelUnitBuff("player", i);
return;
end
end


end -- end onEvent



-- FRAME VARIABLES

-- Create the frame. Register the events that makes it activate, and set scripts on it such that it is good
anticosmeticaurasframe = CreateFrame("Frame", "anticosmeticaurasframe", UIParent);
anticosmeticaurasframe:SetScript("OnEvent", anticosmeticauras_OnEvent);

-- Register the events for the frame
anticosmeticaurasframe:RegisterEvent("UNIT_AURA"); -- Fires when a buff or debuff is either applied to or is removed from the player

[Fizzlemizz]

Try:

local function anticosmeticauras_OnEvent(this, event, ...)
    if IsInInstance() then
        return
    end
    local auraData = C_UnitAuras.GetAuraDataBySpellName("player", "A Witch!")
    if auraData and not InCombatLockdown() then
        CancelSpellByName("A Witch!")
    end
end -- end onEvent
local f = CreateFrame("Frame", "anticosmeticaurasframe", UIParent);
f:SetScript("OnEvent", anticosmeticauras_OnEvent);
f:RegisterEvent("UNIT_AURA"); -- Fires when a buff or debuff is either applied to or is removed from the player

[jlrm365]

Originally Posted by Fizzlemizz Try: local function anticosmeticauras_OnEvent(this, event, ...)     if IsInInstance() then         return     end     local auraData = C_UnitAuras.GetAuraDataBySpellName("player", "A Witch!")     if auraData and not InCombatLockdown() then         CancelSpellByName("A Witch!")     end end -- end onEvent local f = CreateFrame("Frame", "anticosmeticaurasframe", UIParent); f:SetScript("OnEvent", anticosmeticauras_OnEvent); f:RegisterEvent("UNIT_AURA"); -- Fires when a buff or debuff is either applied to or is removed from the player

Wonderful.
I must remember to enter buff names twice, instead of once - as was habit, but that's very slick.
Thanks!

[Fizzlemizz]

Create a table of aura names to check/cancel"

local CancelAuras = { -- add aruas to cancel to the list
    "A Witch!",
    "Arcane Intellect",
    "And Another One",
}
local function anticosmeticauras_OnEvent(this, event, ...)
    if IsInInstance() then
        return
    end
    for _, aura in pairs(CancelAuras) do
        local auraData = C_UnitAuras.GetAuraDataBySpellName("player", aura)
        if auraData then
            CancelSpellByName(aura)
        end
    end
end -- end onEvent
local f = CreateFrame("Frame", "anticosmeticaurasframe", UIParent);
f:SetScript("OnEvent", anticosmeticauras_OnEvent);
f:RegisterEvent("UNIT_AURA"); -- Fires when a buff or debuff is either applied to or is removed from the player

[jlrm365]

Originally Posted by Fizzlemizz Create a table of aura names to check/cancel" local CancelAuras = { -- add aruas to cancel to the list     "A Witch!",     "Something else",     "And Another One", } local function anticosmeticauras_OnEvent(this, event, ...)     if IsInInstance() then         return     end     for _, aura in pairs(CancelAuras) do         local auraData = C_UnitAuras.GetAuraDataBySpellName(aura)         if auraData then             CancelSpellByName(aura)         end     end end -- end onEvent local f = CreateFrame("Frame", "anticosmeticaurasframe", UIParent); f:SetScript("OnEvent", anticosmeticauras_OnEvent); f:RegisterEvent("UNIT_AURA"); -- Fires when a buff or debuff is either applied to or is removed from the player

Better than the original.
Very much appreciated.

[Fizzlemizz]

Left the unit name off the C_UnitAuras.GetAuraDataBySpellName call, fixed in the post above.

[jlrm365]

Originally Posted by Fizzlemizz Left the unit name off the C_UnitAuras.GetAuraDataBySpellName call, fixed in the post above.

Yup.
Added in my extra auras and it does the job.
I'd say your solution improves on the original, with the list as it is and the ability to tweak the LUA so easily.
Thanks again!