Skip to content

Handle IsForbidden() frames and GameTooltip:IsUnit() crash #2

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.

Sign up for GitHub

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

Open
Ketho wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Handle IsForbidden() frames and GameTooltip:IsUnit() crash #2

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
48ea3e7
Handle IsForbidden() frames
Ketho Aug 12, 2019
ed8abf7
Fixed GameTooltip:IsUnit() crash
Ketho Sep 1, 2019
97953e1
Additional IsForbidden check
Ketho Jul 25, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 7 additions & 4 deletions Spew.lua
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@

---@diagnostic disable: undefined-field, unbalanced-assignments
local myname, ns = ...

local TABLEITEMS, TABLEDEPTH = 5, 1
Expand Down Expand Up @@ -44,7 +44,9 @@ local function pretty_tostring(value, depth)
if t == "string" then return '|cff00ff00"'..value:gsub("|", "||"):gsub("([\001-\031\128-\255])", escape)..'"|r'
elseif t == "table" then
if depth > TABLEDEPTH then return "|cff9f9f9f{...}|r"
elseif type(rawget(value, 0)) == "userdata" and type(value.GetObjectType) == "function" then return "|cffffea00<"..value:GetObjectType()..":"..(value:GetName() or "(anon)")..">|r"
elseif type(rawget(value, 0)) == "userdata" and type(value.GetObjectType) == "function" then
if value.IsForbidden and value:IsForbidden() then return "|cffff0000<Forbidden:(unknown)>|r"
else return "|cffffea00<"..value:GetObjectType()..":"..(value:GetName() or "(anon)")..">|r" end
else return "|cff9f9f9f"..string.join(", ", TableToString(value, nil, nil, depth+1)).."|r" end
elseif colors[t] then return colors[t]..tostring(value).."|r"
else return tostring(value) end
Expand Down Expand Up @@ -86,14 +88,15 @@ function Spew(input, a1, ...)
if type(a1) == "table" then
if type(rawget(a1, 0)) == "userdata" and type(a1.GetObjectType) == "function" then
-- We've got a frame!
Print("|cffffea00<"..a1:GetObjectType()..":"..(a1:GetName() or input.."(anon)").."|r")
if a1.IsForbidden and a1:IsForbidden() then Print("|cffff0000<Forbidden:"..input.."|r")
else Print("|cffffea00<"..a1:GetObjectType()..":"..(a1:GetName() or input.."(anon)").."|r") end
local sorttable = {}
for i in pairs(a1) do table.insert(sorttable, i) end
for i in pairs(getmetatable(a1).__index) do table.insert(sorttable, i) end
table.sort(sorttable, downcasesort)
for _,i in ipairs(sorttable) do
local v, output = a1[i]
if type(v) == "function" and type(i) == "string" and not blist[i] and (i:find("^Is") or i:find("^Can") or i:find("^Get")) then
if type(v) == "function" and type(i) == "string" and not blist[i] and ((i:find("^Is") and i ~= "IsUnit") or i:find("^Can") or i:find("^Get")) then
output = pcallhelper(pcall(v, a1))
end
if output then Print(" |cff7fd5ff"..tostring(i).."|r => "..output)
Expand Down