SLOKnightfall / BetterWardrobe

30 stars 14 forks source link

LUA Error after teleport or something like that #438

Closed the-quality closed 2 months ago

the-quality commented 2 months ago

What version number you are using? 4.22

What game client are you playing? Retail, Wrath, Classic / Live, PTR, Beta Retail

What language is the game client set to? german

Describe the bug

Message: .../ArkInventory/Externals/Ace3/AceHook/AceHook-3.0.lua:180: Attempting to rehook already active hook CollectionsJournal_UpdateSelectedTab.
Time: Sat Jun 22 15:04:34 2024
Count: 5
Stack: .../ArkInventory/Externals/Ace3/AceHook/AceHook-3.0.lua:180: Attempting to rehook already active hook CollectionsJournal_UpdateSelectedTab.
[string "=[C]"]: ?
[string "=[C]"]: ?
[string "@Interface/AddOns/ArkInventory/Externals/Ace3/AceHook/AceHook-3.0.lua"]:180: in function <.../ArkInventory/Externals/Ace3/AceHook/AceHook-3.0.lua:118>
[string "@Interface/AddOns/ArkInventory/Externals/Ace3/AceHook/AceHook-3.0.lua"]:340: in function `SecureHook'
[string "@Interface/AddOns/BetterWardrobe/Modules/Core.lua"]:1413: in function `LoadModules'
[string "@Interface/AddOns/BetterWardrobe/Modules/Core.lua"]:1484: in function <Interface/AddOns/BetterWardrobe/Modules/Core.lua:1484>

Locals: (*temporary) = <function> defined =[C]:-1

To Reproduce Steps to reproduce the behaviour, if possible:

teleport or joining instance or something like that

Screenshots If applicable, add screenshots to help explain your problem. image

image

fubaWoW commented 2 months ago

same here

1x ...lvUI_Libraries/Core/Ace3/AceHook-3.0-9/AceHook-3.0.lua:180: Attempting to rehook already active hook CollectionsJournal_UpdateSelectedTab.
[string "=[C]"]: ?
[string "@ElvUI_Libraries/Core/Ace3/AceHook-3.0-9/AceHook-3.0.lua"]:180: in function <...lvUI_Libraries/Core/Ace3/AceHook-3.0/AceHook-3.0.lua:118>
[string "@ElvUI_Libraries/Core/Ace3/AceHook-3.0-9/AceHook-3.0.lua"]:340: in function `SecureHook'
[string "@BetterWardrobe/Modules/Core.lua"]:1413: in function `LoadModules'
[string "@BetterWardrobe/Modules/Core.lua"]:1484: in function <BetterWardrobe/Modules/Core.lua:1484>

Locals:
(*temporary) = "Attempting to rehook already active hook CollectionsJournal_UpdateSelectedTab."
ESpmdark commented 2 months ago

Not sure how the LoadModules function gets called more than once. That might be the thing to focus on. If intended, then you probably want to call the "IsHooked" function from ace to safeguard from running duplicate hooks.

SLOKnightfall commented 2 months ago

Should be fixed now

the-quality commented 2 months ago

yes, thx