Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bad argument #6

Closed
bygnub opened this issue Apr 16, 2020 · 6 comments
Closed

bad argument #6

bygnub opened this issue Apr 16, 2020 · 6 comments

Comments

@bygnub
Copy link

bygnub commented Apr 16, 2020

Message: Interface\AddOns\SpellCalc\system/buffs.lua:418: bad argument #1 to 'ipairs' (table expected, got nil)
Time: Thu Apr 16 11:48:45 2020
Count: 1
Stack: Interface\AddOns\SpellCalc\system/buffs.lua:418: bad argument #1 to 'ipairs' (table expected, got nil)
[C]: ?
[C]: in function ipairs' Interface\AddOns\SpellCalc\system/buffs.lua:418: in function UpdateTalents'
Interface\AddOns\SpellCalc\system/stats.lua:284: in function FullUpdate' Interface\AddOns\SpellCalc\events.lua:20: in function ?'
Interface\AddOns\SpellCalc\events.lua:109: in function <Interface\AddOns\SpellCalc\events.lua:108>

Locals:

@coolmodi
Copy link
Owner

coolmodi commented Apr 17, 2020

Are you trying to use this as a warrior, rogue or hunter? If so, as stated in the readme:

Warrior, Hunter and Rogue have no support at all.

I may add a check so the addon doesn't load when on those classes, because I probably won't get to finish the rewrite on the core system to finish melee support for some time.

If you're also using it on a supported class then you should disable it on the not supported characters to not get the error there. Sorry about that.

@bygnub
Copy link
Author

bygnub commented Apr 19, 2020 via email

@coolmodi
Copy link
Owner

Can't reproduce it with a lvl 22 paladin. Also can't see how it's even possible to happen tbh. Does it only happen with that char?

@bygnub
Copy link
Author

bygnub commented Apr 28, 2020 via email

@coolmodi
Copy link
Owner

I found another error with level 1 paladins and fixed it, but I still can't reproduce the error you got. Maybe try the newest version and see if you still get it. Maybe the error message you got was just misleading.

@bygnub
Copy link
Author

bygnub commented Apr 30, 2020

latest version looks good :)

@bygnub bygnub closed this as completed Apr 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants