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

Sporadic LUA error during classic stress test #7

Closed
herrfennessey opened this issue Aug 9, 2019 · 8 comments
Closed

Sporadic LUA error during classic stress test #7

herrfennessey opened this issue Aug 9, 2019 · 8 comments

Comments

@herrfennessey
Copy link

herrfennessey commented Aug 9, 2019

Hey Tercioo!

I ran into this glitch fairly often running Details Damage Meter in Durotar during the classic stress test. I've attached the full LUA dump in a gist for you. I can take a look at it as well, although I have to admit the Portuguese variable names are throwing me for a loop :)

Calthaza is right, it happens on normal mobs too It appears to happen whenever I get a debuff that's a dot on me, so the scorpid poison will fairly reliably set it off

https://gist.github.com/vazheel/25a8843b02f797b98aa2815e6175de91

@herrfennessey
Copy link
Author

Update: I added a simple nil check around this line https://github.com/Tercioo/Details-Damage-Meter/blob/master/functions/spellcache.lua#L237

And it went away -- I don't really understand what this code is trying to accomplish, so I leave it up to you if this would be a reasonable solution

@Calthaza
Copy link

Calthaza commented Aug 9, 2019

10x Details\functions\spellcache.lua:237: attempt to concatenate local 'nome' (a nil value)
Details\functions\spellcache.lua:237: in function SpellIsDot' Details\classes\classe_damage_habilidade.lua:75: in function funcao_de_criacao'
...aceDetails\classes\container_habilidades.lua:71: in function `PegaHabilidade'
Details\core\parser.lua:950: in function <Details\core\parser.lua:419>
(tail call): ?

Locals:
InCombatSkipped

I am killing Mottled Boars in durotar, so it does not have anything with debuff as the other guy mentioned.

@herrfennessey
Copy link
Author

Good point, I noticed that it happened reliably with mobs with dots, and sporadically with non-dot mobs. I updated my original message.

@Tercioo
Copy link
Owner

Tercioo commented Aug 9, 2019

Got it! thanks.

@jsabin69
Copy link

Error continues with latest update:

Message: Interface\AddOns\Details\functions\spellcache.lua:237: attempt to concatenate local 'nome' (a nil value)
Time: Sat Aug 10 06:39:11 2019
Count: 8
Stack: Interface\AddOns\Details\functions\spellcache.lua:237: attempt to concatenate local 'nome' (a nil value)
Interface\AddOns\Details\functions\spellcache.lua:237: in function SpellIsDot' ...\AddOns\Details\classes\classe_damage_habilidade.lua:75: in function funcao_de_criacao'
...ace\AddOns\Details\classes\container_habilidades.lua:71: in function `PegaHabilidade'
Interface\AddOns\Details\core\parser.lua:948: in function <Interface\AddOns\Details\core\parser.lua:419>
(tail call): ?

Locals: self =

{
SetBarSpecIconSettings = defined @interface\AddOns\Details\gumps\janela_principal.lua:4285
APITopics =
{
}
InitializeRunCodeWindow = defined @interface\AddOns\Details\core\windows.lua:4496
EncounterDetailsTempWindow = defined @interface\AddOns\Details_EncounterDetails\frames.lua:49
EntrarEmCombate = defined @interface\AddOns\Details\core\control.lua:313
hex = defined @interface\AddOns\Details\core\util.lua:687
BuildSpellList = defined @interface\AddOns\Details\functions\spellcache.lua:308
opened_windows = 2
RefreshAttributeTextSize = defined @interface\AddOns\Details\gumps\janela_principal.lua:7546
PotionList =
{
}
EndRefresh = defined @interface\AddOns\Details\core\control.lua:1691
FechaJanelaInfo = defined @interface\AddOns\Details\gumps\janela_info.lua:380
GetMode = defined @interface\AddOns\Details\classes\classe_instancia.lua:148
SetMode = defined @interface\AddOns\Details\classes\classe_instancia.lua:2706
RestoreState_CurrentMythicDungeonRun = defined @interface\AddOns\Details\functions\profiles.lua:1478
APIVersion = 140
IsPluginEnabled = defined @interface\AddOns\Details\core\plugins.lua:58
ImportProfile = defined @interface\AddOns\Details\functions\profiles.lua:1629
SetNickname = defined @interface\AddOns\Details\Libs\NickTag-1.0\NickTag-1.0.lua:495
color_by_arena_team = true
SortKeyGroupHeal = defined @interface\AddOns\Details\classes\classe_heal.lua:125
SortKeyGroup = defined @interface\AddOns\Details\classes\classe_damage.lua:133
IsInEncounter = defined @interface\AddOns\Details\core\parser.lua:5314
Check_ZONE_CHANGED_NEW_AREA = defined @interface\AddOns\Details\core\parser.lua:4072
plugin_window_pos =
{
}
gump =
{
}
CaptureRefresh = defined @interface\AddOns\Details\core\parser.lua:3726
NewModule = defined @interface\AddOns\Details\Libs\AceAddon-3.0\AceAddon-3.0.lua:235
InstancesToStoreData =
{
}
name_space_info = defined @interface\AddOns\Details\core\util.lua:1385
ShowDelayMsg = defined @interface\AddOns\Details\boot.lua:658
Resize = defined @interface\AddOns\Details\classes\classe_instancia.lua:1867
GetEncounterActorsName = defined @interface\AddOns\Details\functions\boss.lua:306
RequestCloudData = defined @interface\AddOns\Details\core\network.lua:721
RestauraJanela = defined @interface\AddOns\Details\classes\classe_instancia.lua:1429
default_skin_to_use = "Minimalistic"
GetOpenedWindowsAmount = defined @interface\AddOns\Details\classes\classe_instancia.lua:256
GetInstanceGroup = defined @interface\AddOns\Details\gumps\janela_principal.lua:8007
deadlog_events = 32
Dump = defined @interface\AddOns\Details\core\windows.lua:7132
OpenNewsWindow = defined @interface\AddOns\Details\gumps\janela_news.lua:9
last_instance_id = 0
_detalhes_props =
{
}
atributo_energy =
{
}
UnitDamageOnUnit = defined @interface\AddOns\Details\functions\api2.lua:1123
name_space_generic = defined @interface\AddOns\Det

@Tercioo
Copy link
Owner

Tercioo commented Aug 10, 2019

this issue should be solved with latest version from curse

@Tercioo
Copy link
Owner

Tercioo commented Aug 11, 2019

also, spellIds not working correctly on the client side

@Tercioo
Copy link
Owner

Tercioo commented Aug 17, 2019

Made a new version for classic only, it's available at curse and wow interface.

@Tercioo Tercioo closed this as completed Aug 17, 2019
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

4 participants