b-morgan / Skillet-Classic

World of Warcraft Classic addon
GNU General Public License v3.0
6 stars 6 forks source link

[Bug][Classic 1.14.3] Error loading .lua file #143

Closed apav closed 1 year ago

apav commented 1 year ago

Thank you for this great addon! I'm getting this error message pop up every time.

Error loading Interface\AddOns\Skillet-Classic\Libs\LibPeriodicTable-3.1-90603\LibPeriodicTable-3.1-TradeskillResultMats\LibPeriodicTable-3.1-TradeskillResultMats.lua

I checked and the file is there. The only difference is there is no "-90603" in the folder name, but even after adding that part I was getting the same error message, as well as more error messages. I've tried several versions all the way back to 1.14.2 and am still getting the same issue.

Thanks!

b-morgan commented 1 year ago

I cannot reproduce this error but I am confused because you are reporting the WoW Classic Era version number, not the Skillet-Classic version number. Please delete ...\Interface\Addons\Skillet-Classic, download either Skillet-Classic-1.67-classic or Skillet-Classic-1.68-alpha1-classic, verify the MD5 checksum, and install it.

If you still have the error, then please disable all other addons and verify that the error still exists. Let me know the results and which version of Skillet-Classic you installed.

LibPeriodicTable is an ACE3 library. The way AC3 libraries load is complicated but the end result is that while multiple addons will contain the library code, only one copy will end up in memory and that should be the most recent version. The 90603 is the version number of the library.

apav commented 1 year ago

My apologies, I forgot to include that part. I used the latest 1.6.7 release and even tried going way back to a build like 1.4.7, and I'm still getting the same error. When I disable all addons except for Skillet Classic and Buggrabber/Bugsack (to copy/paste the error, it still happens without them), I get the same error, as well as an additional one that doesn't appear when I have my other addons enabled:

Error loading Interface\AddOns\Skillet-Classic\Libs\LibPeriodicTable-3.1-90603\LibPeriodicTable-3.1-TradeskillLevels\LibPeriodicTable-3.1-TradeskillLevels.lua

Edit: Seems that extra error is happening now even when I turn my other addons on. For extra measure I tried the addon on a fresh install of WoW Classic and the issue still persists.

b-morgan commented 1 year ago

I have no idea what is causing this error. Perhaps there is something in ...\World of Warcraft\_classic_era_\Logs\FrameXML.log or one of the other logs.

apav commented 1 year ago

This is what's in there, so I guess we're at a dead end. As long as it's safe to ignore, I haven't noticed an impact to functionality yet.

4/16 15:44:55.770  Loading add-on Skillet-Classic
4/16 15:44:55.770  ** Loading table of contents Interface\AddOns\Skillet-Classic\Skillet-Classic.toc
4/16 15:44:55.770  Error loading Interface\AddOns\Skillet-Classic\Libs\LibPeriodicTable-3.1\LibPeriodicTable-3.1-TradeskillResultMats\LibPeriodicTable-3.1-TradeskillResultMats.lua
4/16 15:44:55.770  Error loading Interface\AddOns\Skillet-Classic\Libs\LibPeriodicTable-3.1\LibPeriodicTable-3.1-TradeskillLevels\LibPeriodicTable-3.1-TradeskillLevels.lua
b-morgan commented 1 year ago

LibPeriodicTable is an optional dependency so Skillet-Classic will run without it. If you want to eliminate the errors, edit the Skillet-Classic.toc file and comment (add # to the beginning of lines 37-40:

# Libs\LibPeriodicTable-3.1\LibPeriodicTable-3.1.lua
# Libs\LibPeriodicTable-3.1\LibPeriodicTable-3.1-Tradeskill\LibPeriodicTable-3.1-Tradeskill.lua
# Libs\LibPeriodicTable-3.1\LibPeriodicTable-3.1-TradeskillResultMats\LibPeriodicTable-3.1-TradeskillResultMats.lua
# Libs\LibPeriodicTable-3.1\LibPeriodicTable-3.1-TradeskillLevels\LibPeriodicTable-3.1-TradeskillLevels.lua