WoWInterface - Shift middle click an item for GetItemInfo

WoWInterface (https://www.wowinterface.com/forums/index.php)

- Lua/XML Help (https://www.wowinterface.com/forums/forumdisplay.php?f=16)

- - Shift middle click an item for GetItemInfo (https://www.wowinterface.com/forums/showthread.php?t=48964)

Shift middle click an item for GetItemInfo

I would like to be able to Shift Middle click an item in my bags and have it call GetItemInfo on that item, but I'm not sure where to start.

I've done some google-ing and can't find anything myself so any help would be great.

You could hook the OnClick event of the frames.

Lua Code:

for container = 1, 5 do
    for item = 1, 36 do
        _G["ContainerFrame"..container.."Item"..item]:HookScript("OnClick", function(self, button) 
            if IsShiftKeyDown() and button == "MiddleButton" then 
                print(GetItemInfo(GetContainerItemID(self:GetParent():GetID(), self:GetID()))) 
            end 
        end)
    end
end

Another way doing it would be to hook Blizzards ContainerFrameItemButton_OnModifiedClick(self, button).

Like

Lua Code:

hooksecurefunc("ContainerFrameItemButton_OnModifiedClick", function(self, button) 
            if IsShiftKeyDown() and button == "MiddleButton" and self:GetParent():GetID() <= 5 then 
                print(GetItemInfo(GetContainerItemID(self:GetParent():GetID(), self:GetID()))) 
            end 
        end)

I bet Phanx likes this one more because it's using only one hook and not 5x36. ;D

That's exactly what I was looking for! Thanks a bunch ^.^

I do get one error though, it seems only left and right button are passed, for example in the script below...

Lua Code:

for container = 1, 5 do
        for item = 1, 36 do
            _G["ContainerFrame"..container.."Item"..item]:HookScript("OnClick", function(self, button)
                print(button)
                if IsShiftKeyDown() and button == "MiddleButton" then
                    print(GetItemInfo(GetContainerItemID(self:GetParent():GetID(), self:GetID())))
                end
            end)
        end
    end

Only left and right clicks will print anything, this is not a huge deal as I can easily switch the keybinds though, just curious why that is.

I believe you need to call button:RegisterForClicks('MiddleButtonDown') if it's not already watching for that type of click.

I'm not sure. Maybe the middle button is not registered.

I don't have a 3 button mouse available atm and can't test it. :)
Try to register for MiddleButtonDown:

Lua Code:

for container = 1, 5 do
        for item = 1, 36 do
            _G["ContainerFrame"..container.."Item"..item]:RegisterForClicks("MiddleButtonDown")
            _G["ContainerFrame"..container.."Item"..item]:HookScript("OnClick", function(self, button)
                print(button)
                if IsShiftKeyDown() and button == "MiddleButton" then
                    print(GetItemInfo(GetContainerItemID(self:GetParent():GetID(), self:GetID())))
                end
            end)
        end
    end

See http://www.wowwiki.com/API_Button_RegisterForClicks for reference.

Ah, you guys are correct!

Everything is working now, thank you so much for your help. :D

Edit: I guess I should refresh the page before typing my reply when it's been sitting on the reply page for 2 hours because I got distracted by work, but oh well. Leaving this here anyway. :p

On its own, "calling GetItemInfo" won't actually do anything:

Code:

/run GetItemInfo(6948)

You need additional code to catch the return values and do something with those. Let's say, for the sake of example, that you want to print those values to the chat frame, in which case something like this would probably work:

Lua Code:

hooksecurefunc("ContainerFrameItemButton_OnModifiedClick", function(self, button)
    if button ~= "MiddleButton" or not IsShiftKeyDown() then
        return
    end
    local name, link, rarity, level, minLevel, type, subType, stackCount, equipLoc, texture, sellPrice = GetItemInfo(GetContainerItemID(self:GetParent():GetId(), self:GetID())
    print(format("|T%s:0|t %s", texture, link))
    print("   Quality:", _G["ITEM_QUALITY"..rarity.."_DESC"])
    print("   Item Type:", type)
    print("   Item Subtype:", subType)
    if strlen(equipLoc) > 0 then
        print("   Equip Location:",  equipLoc)
    end
    if level > 1 then
        print("   Item Level:", level)
    end
    if minLevel > 0 then
        print("   Requires Level:", minLevel)
    end
    if sellPrice >= 10000 then
        return format("|cffffd700%d|r%s |cffc7c7cf%d|r%s %d%s", abs(sellPrice / 10000), GOLD_AMOUNT_SYMBOL, abs(mod(sellPrice / 100, 100)), SILVER_AMOUNT_SYMBOL, abs(mod(sellPrice, 100)), COPPER_AMOUNT_SYMBOL)
    elseif sellPrice >= 100 then
        return format("|cffc7c7cf%d|r%s %d%s", abs(mod(sellPrice / 100, 100)), SILVER_AMOUNT_SYMBOL, abs(mod(sellPrice, 100)), COPPER_AMOUNT_SYMBOL)
    else
        return format("%d%s", abs(mod(sellPrice, 100)), COPPER_AMOUNT_SYMBOL)
    end
end)

You probably also need to tell the buttons to respond to middle-clicks, instead of just left- and right-clicks:

Lua Code:

for i = 1, 13 do
    for j = 1, 36 do
        _G["ContainerFrame"..i.."Item"..j]::RegisterForClicks("AnyUp")
    end
end

If you're using a third-party bag addon that doesn't reuse the Blizzard item buttons, you'll need to modify its buttons and hook its click-handling function instead.

I had a problem that I wasn't aware of until just recently that I could not sell items normally which I resolved by reading the code you have submitted and registering clicks for AnyUp, the sell part was also very useful, I will definitely be using that. =D

Again, I appreciate the help, thank you. :)