1010101110 / Auctionator-classic

forked from wow Auctionator on classic release.
11 stars 8 forks source link

Auctionator Shopping List errors #35

Open b-morgan opened 3 years ago

b-morgan commented 3 years ago

I got the following two errors opening the Interface, Addons, Auctionator, Shopping List, Edit (of an existing shopping list). Also it would be nice if Manage Shopping Lists opened to the proper place in the Interface, Addons frame. I believe the following change in AuctionatorShop.lua will fix it...

function Atr_MngSLists_OnClick ()

  InterfaceOptionsFrame_Show()
  InterfaceOptionsFrame_OpenToCategory ("Shopping Lists");
2x [string "Atr_ShpList_Edit_Frame:OnShow"]:1: attempt to call method 'SetBackdrop' (a nil value)
[string "*:OnShow"]:1: in function <[string "*:OnShow"]:1>
[string "=[C]"]: in function 'Show'
[string "@Auctionator\AuctionatorConfig.lua"]:1240: in function <Auctionator\AuctionatorConfig.lua:1209>

Locals:
self = Atr_ShpList_Edit_Frame {
 0 = <userdata>
}
(*temporary) = nil
(*temporary) = Atr_ShpList_Edit_Frame {
 0 = <userdata>
}
(*temporary) = <table> {
 bgFile = "MERCHANTFRAME\UI-BuyBack-TopLeft"
 tileSize = 256
 edgeFile = "DialogFrame\UI-DialogBox-Border"
 tile = true
 edgeSize = 32
 insets = <table> {
 }
}
(*temporary) = "attempt to call method 'SetBackdrop' (a nil value)"
2x [string "Atr_ShpList_Edit_FrameScrollFrame:OnShow"]:1: attempt to call method 'SetBackdrop' (a nil value)
[string "*:OnShow"]:1: in function <[string "*:OnShow"]:1>
[string "=[C]"]: in function 'Show'
[string "@Auctionator\AuctionatorConfig.lua"]:1240: in function <Auctionator\AuctionatorConfig.lua:1209>

Locals:
self = Atr_ShpList_Edit_FrameScrollFrame {
 ScrollBar = Atr_ShpList_Edit_FrameScrollFrameScrollBar {
 }
 offset = 0
 0 = <userdata>
}
(*temporary) = nil
(*temporary) = Atr_ShpList_Edit_FrameScrollFrame {
 ScrollBar = Atr_ShpList_Edit_FrameScrollFrameScrollBar {
 }
 offset = 0
 0 = <userdata>
}
(*temporary) = <table> {
 insets = <table> {
 }
 bgFile = "CharacterFrame\UI-Party-Background"
 tileSize = 32
 tile = true
}
(*temporary) = "attempt to call method 'SetBackdrop' (a nil value)"
b-morgan commented 3 years ago

Add the following three lines to the scripts for the above two frames in AuctionatorConfig.xml

        if not self.SetBackdrop then
            Mixin(self, BackdropTemplateMixin)
        end
b-morgan commented 3 years ago

Here's a patch file with the above changes.

AuctionatorClassicFix.zip