search

herotc / hero-lib

WoW Addon - Core Library used by HeroRotation and AethysTools, can also be used by others 3rd-Party Addons.
European Union Public License 1.2
44 stars 35 forks source link

Lua Error #61

Closed BelakJG closed 3 years ago

BelakJG commented 3 years ago

HeroLib\Events\Action.lua:190: attempt to index field 'ElvUIPaging' (a nil value) [string "@HeroLib\Events\Action.lua"]:190: in function <HeroLib\Events\Action.lua:185> [string "@HeroLib\Events\Action.lua"]:226: in function <HeroLib\Events\Action.lua:208> [string "@HeroLib\Events\Action.lua"]:320: in function <HeroLib\Events\Action.lua:288> [string "@HeroLib\Events\Action.lua"]:342: in function `Handler' [string "@HeroLib\Events\Main.lua"]:80: in function <HeroLib\Events\Main.lua:78>

Locals: class = "DEATHKNIGHT" (for index) = 1 (for limit) = 10 (for step) = 1 i = 1 BarNum = "bar1" PagingString = nil (temporary) = nil (temporary) = nil (temporary) = nil (temporary) = nil (temporary) = nil (temporary) = nil (temporary) = nil (temporary) = nil (temporary) = nil (temporary) = nil (temporary) = nil (temporary) = "attempt to index field 'ElvUIPaging' (a nil value)" Cache =

{ GUIDInfo =
{ } Set = defined @HeroCache\Main.lua:268 APLVar =
{ } Enemies =
{ } Get = defined @HeroCache\Main.lua:252 HasBeenReset = true SpellInfo =
{ } UnitInfo =
{ } MiscInfo =
{ } Persistent =
{ } ItemInfo =
{ } Reset = defined @HeroCache\Main.lua:41 }

Cilraaz commented 3 years ago

What version of the addon are you using? If you're updating from CurseForge/Overwolf, they are for whatever reason not catching our newer releases. You may need to grab the latest version from github, available here

If you are using version 9.0.5.04 and still having this issue, please let us know.

BelakJG commented 3 years ago

I am using the latest version, I use the latest version from github.

Cilraaz commented 3 years ago

Hmm, I'll have to investigate this. HeroCache/Main.lua creates the Cache.Persistent.ElvUIPaging structure, so it shouldn't be coming back as nil.

BelakJG commented 3 years ago

Thank you. It seems to be keeping hero rotation from displaying key binds if that helps any.

BelakJG commented 3 years ago

I rolled back to see when the issue started exactly and it started after https://github.com/herotc/hero-lib/commit/de46e4df83363d841665cb78867a42bb4149b63f was committed.

Cilraaz commented 3 years ago

That was the commit that added caching of ElvUI's paging info. I'm still not seeing any reason that an error would be occurring. In game, what do you get if you issue the command /dump HeroCache.Persistent.ElvUIPaging?

BelakJG commented 3 years ago

all it says is empty result

Cilraaz commented 3 years ago

Can you confirm that in your copy of the addon HeroCache/Main.lua, line 37 is:

ElvUIPaging = { PagingString, PagingStrings = {}, PagingBars = {} }

?

BelakJG commented 3 years ago

yes, I have the same line 37

Cilraaz commented 3 years ago

This is the line that defines the structure that you got "empty" in the dump. Something isn't right in your setup. I've logged in a character that I haven't used since BfA, as well as a brand new level 1, and both show the structure properly when performing that dump.

I would suggest completely deleting HeroCache and HeroLib, as well as the HeroCache.lua SavedVariables file, found in WoW\_retail_\WTF\Account\<YourAccount>\SavedVariables. After that, reinstall version 9.0.5.05 (newly created release to incorporate some other changes... no changes relating to this issue).

BelakJG commented 3 years ago

That fixed it, for some reason herocache was updating through github but the symlink had broken so it wasn't updating in my wow addons. Thank you for your patience with this.

Cilraaz commented 3 years ago

No worries. Glad it's fixed!

  • © Githubissues.
  • Githubissues is a development platform for aggregating issues.