Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

container id nil error #11

Open
Road-block opened this issue May 19, 2020 · 0 comments
Open

container id nil error #11

Road-block opened this issue May 19, 2020 · 0 comments

Comments

@Road-block
Copy link

Road-block commented May 19, 2020

134x ...\tradeDispenser\tradeDispenser_SlaveFrame.lua:78: attempt to concatenate local 'cID' (a nil value)
...\tradeDispenser\tradeDispenser_SlaveFrame.lua:78: in function <...\tradeDispenser\tradeDispenser_SlaveFrame.lua:3>

From quick code review it looks to me that this is happening due to
tradeDispenser_SlaveFrame.lua#L65

With Blizzard's new chat restrictions if the addon tries to send to SAY while not in an instance it gets blocked, so code never reaches that early return on L69.

What I did on my local copy as a sort of band-aid catch all fix is modify the tradeDispenserMessage function thus.

function tradeDispenserMessage(channel, message)
	channel = strupper(channel)
	if (channel=="WHISPER") then
		SendChatMessage(message, "WHISPER", tD_Loc[UnitFactionGroup("player")], tD_Temp.Target.Name)
	elseif (channel=="RAID" or channel=="PARTY" or channel=="GUILD" or channel=="YELL" or channel=="SAY" or channel=="BATTLEGROUND") then
		local retOK, err = pcall(SendChatMessage, message, channel)
		if not (retOK) then
			tradeDispenserVerbose(0,"Error: cannot send to '"..channel.."'("..tostring(err)..")")
		end
	else
		tradeDispenserVerbose(0,"Error: cannot use channel '"..channel.."'")
	end
end
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant