Retrieve item name to disenchant

[Benalish]

My intent is to retrirve the name of the item I'm about to disenchant, so as to warn if I'm disenchanting a piece I want to keep. How can I accomplish all this?

[Benalish]

Ok, this script works, and is based on the GameTooltip. But what about if GameTooltip is hidden for some reason?

local mt = {
    __index = {
        isvalue = function(t, value)
            local is = false
            for k, entry in ipairs(t) do
                if (entry == value) then
                    is = true
                    break
                end
            end
            return is
        end
    }
}
 
 
local protected = { "item1", "item2", "item3", "etch." } -- items I want to protect
setmetatable(protected, mt)
 
local antidisenchant = CreateFrame("Frame")
 
antidisenchant:RegisterEvent("UNIT_SPELLCAST_SENT")
 
antidisenchant:SetScript("OnEvent", function(self, event, ...)
    if (event == "UNIT_SPELLCAST_SENT") then
        local item, link = GameTooltip:GetItem()
        if (arg2 == "Disenchant") and (protected:isvalue(item)) then
            DEFAULT_CHAT_FRAME:AddMessage("WARNING! You're disenchanting "..link,1,0,0)
        end
    end
end)

[fatrog]

I'm not sure if I can help you, but have you tried to get the target of the spell itself?
spell id of disenchant is 13262

[SDPhantom]

This hooks calls to target an item and store the ItemID in an upvalue. I added support for Classic Era, which still has UseContainerItem() in the global table instead of C_Container. Feel free to remove this support if you don't plan on dealing with that client. There are other situations that'll overwrite the value, but when read immediately when the spellcast starts, it should be fine.

local C_Container=C_Container or _G;--  C_Container doesn't exist in Classic Era (added to Wrath Classic in 3.4.1)
local TargetItemID;
 
--  Runs from ContainerFrameItemButton_OnClick() (also SecureActionButton "target-bag"/"target-slot" attributes for container items)
hooksecurefunc(C_Container,"UseContainerItem",function(bag,slot)
    TargetItemID=C_Container.GetContainerItemID(bag,slot);
end);
 
--  Runs from SecureActionButton "target-slot" attribute for inventory items
hooksecurefunc("UseInventoryItem",function(slot)
    TargetItemID=GetInventoryItemID("player",slot);
end);
 
--  Runs from SecureActionButton "target-item" attribute
hooksecurefunc("SpellTargetItem",function(item)
    TargetItemID=GetItemInfoInstant(item);--    Accepts ItemID, ItemLink, or ItemName; Returns ItemID as first value
end);

PS: I have concerns over reading from the arg2 global. I had thought Blizzard stopped using arg# as globals a long time ago.

[Benalish]

Since I'm unfamiliar with hooksecurefuncs, how could I trace the item using these functions?

[SDPhantom]

hooksecurefunc() lets you post-hook a function without tainting it. As used in the code I posted, it takes the args passed into each function that could be triggered by clicking an item to disenchant it and it records the ItemID into an upvalue named TargetItemID. All you need to do outside of this is check TargetItemID when you detect that the Disenchant spell casts and use that as a reference.

[Benalish]

Thank you very much!