Authors CurseForge

Quote from Arkayenro

Master and BidNotOpen are globals? any chance these are getting overwritten by another mod? or even blizzard?

you may want to put those debug print statements back in right at the top just to confirm if your hook is being called.

local Master = false;
local BidNotOpen = true;

Quote from chiefcastor

Well they are declared outside of the functions like this...

local Master = false;
local BidNotOpen = true;

Does that make them local to my addon? Because that was my intent. However, I have used debugs before to try to solve this issue. When it occurs my HandleModifiedItemClicks function is not called at all.

edit: Should they be declared DrunkardSK.Master and DrunkardSK.BidNotOpen?

Quote from Arkayenro

mine picks up all the lootable items when you loot and queues them up so ive no need to check for alt clicking on the loot slots themselves but if thats what youre doing it shouldnt be too hard to figure out what the frame is called and hook that as well (if the setitemref hook doesnt work for that)

Quote from Arkayenro

potentially silly question. at any point are you disabling your mod through the interface or via a :Disable( )?

just that ace itself will unhook all your securehooks when your ace based mod is disabled (or at least it used to, not sure if it still does)

edit: stupid bloody smilies.

local mt = getmetatable(_G)

local function newnewindex(t, key, val)
	if key == "HandleModifiedItemClick" then
		print(debugstack(2))
	end
	t[key] = val
end

if mt then
	if mt.__newindex then
		local _mtnewindex = mt.__newindex
		--pre hook newindex
		function mt.__newindex(t, key, ...)
			if key == "HandleModifiedItemClick" then
				print(debugstack(2))
			end
			return _mtnewindex(t, key, ...)
		end
	else
		mt.__newindex = newnewindex
	end
else
	setmetatable(_G, {__newindex = newnewindex})
end

Quote from Slakah

Heres something which should help finding out whats changing the function.

do -- Hook linking buttons
	local O_SetItemRef, O_SpellButton_OnModifiedClick, O_HandleModifiedItemClick, O_CompanionButton_OnModifiedClick =
		SetItemRef, SpellButton_OnModifiedClick, HandleModifiedItemClick, CompanionButton_OnModifiedClick;

...

	function HandleModifiedItemClick(...)
		if IsModifiedClick("CHATLINK") and GetCurrentKeyBoardFocus() == RKCD_EntryMacro and type((...)) == "string" then
			return RKC_HandleLink((...));
		end
		return O_HandleModifiedItemClick(...);
	end
	
...

end

Quote from Arkayenro

do you use AddonLoader? and is your mod setup for it?

are there other mods setup to use it and loading when you zone?

try temporarily moving addonloader to another directory outside of wow (you cant just disable it) and see if your hooks work again when you zone.

Quote from Arrowmaster

/dump issecurevariable("HandleModifiedItemClick")

Quote from chiefcastor

After login...

Dump: value=issecurevariable("HandleModifiedItemClick")
[2]="OPie"