Thumbkin / MissingTradeSkillsList_Classic

Addon for World of Warcraft (Classic) v1.13
3 stars 5 forks source link

MTSL broken as of 14.1 Classic era WOW #4

Open Velarax opened 2 years ago

Velarax commented 2 years ago

Here is the bug report from Bug Sack

19x MissingTradeSkillsList\ui\tools.lua:29: attempt to call method 'SetBackdrop' (a nil value) [string "@MissingTradeSkillsList\ui\tools.lua"]:29: in function CreateBaseFrame' [string "@MissingTradeSkillsList\ui\mainframes\missing_tradeskills_frame.lua"]:36: in functionInitialise' [string "@MissingTradeSkillsList\ui\event_handler.lua"]:34: in function `?' [string "@MissingTradeSkillsList\ui\event_handler.lua"]:310: in function <...eMissingTradeSkillsList\ui\event_handler.lua:307>

Locals: self =

{ CreateDropDown = defined @MissingTradeSkillsList\ui\tools.lua:318 FillDropDown = defined @MissingTradeSkillsList\ui\tools.lua:333 CreateMainFrame = defined @MissingTradeSkillsList\ui\tools.lua:60 CreateWayPoint = defined @MissingTradeSkillsList\ui\tools.lua:292 tomtom_warned = false GetLocalisedLabelSpecialAction = defined @MissingTradeSkillsList\ui\tools.lua:407 CreateLabel = defined @MissingTradeSkillsList\ui\tools.lua:137 GetLocalisedData = defined @MissingTradeSkillsList\ui\tools.lua:415 CreateBaseFrame = defined @MissingTradeSkillsList\ui\tools.lua:22 CreateIconTextButton = defined @MissingTradeSkillsList\ui\tools.lua:159 FillDropDownCheckable = defined @MissingTradeSkillsList\ui\tools.lua:364 SetAddonLocale = defined @MissingTradeSkillsList\ui\tools.lua:276 PrintHelpMessage = defined @MissingTradeSkillsList\ui\tools.lua:255 CreateScrollFrame = defined @MissingTradeSkillsList\ui\tools.lua:219 AddDragToFrame = defined @MissingTradeSkillsList\ui\tools.lua:389 GetLocalisedLabel = defined @MissingTradeSkillsList\ui\

type = "Frame" name = "MTSLUI_MissingTradeSkillsFrame" parent = MTSLUI_ToggleButton { 0 = Text = MTSLUI_ToggleButtonText { } Right = { } Middle = { } Left = { } } template = nil width = 908 height = 497 has_backdrop = true generic_frame = MTSLUI_MissingTradeSkillsFrame { 0 = }} (temporary) =

{ bgFile = "Interface/Tooltips/UI-Tooltip-Background" tileSize = 16 edgeFile = "Interface/Tooltips/UI-Tooltip-Border" tile = true edgeSize = 16 insets =
{ } } (temporary) = "attempt to call method 'SetBackdrop' (a nil value)"

MyTechnoHunter commented 2 years ago

just chiming in that it is the same in Seasons of Mastery with the same SetBackdrop nil error.. if needed i will grab a lua error report..

ty for looking into this!

Velarax commented 2 years ago

Am using the BCC version in Classic. It seems to work fine, other than having BCC recipes in it.

MyTechnoHunter commented 2 years ago

Am using the BCC version in Classic. It seems to work fine, other than having BCC recipes in it.

same, going to see how much of a pain in the hind parts it is to swap the data... wish me luck

umuppetu commented 1 year ago

Make the following 3 changes to get it working on Classic Era: In file: \World of Warcraft_classicera\Interface\AddOns\MissingTradeSkillsList\ui\tools.lua Line 23 should be: local generic_frame = CreateFrame(type, name, parent, "BackdropTemplate")

In file: \World of Warcraft_classicera\Interface\AddOns\MissingTradeSkillsList\ui\common\filter_frame.lua Line 69 should be: self.ui_frame.search_box = CreateFrame("EditBox", self.filter_frame_name .. "_TF", self.ui_frame, "BackdropTemplate")

In file: World of Warcraft_classicera\Interface\AddOns\MissingTradeSkillsList\ui\common\npc_filter_frame.lua Line 74 should be: self.ui_frame.search_box = CreateFrame("EditBox", self.filter_frame_name .. "_TF", self.ui_frame, "BackdropTemplate")