search

Bunny67 / WeakAuras-WotLK

World of Warcraft addon that provides a powerful framework to display customizable graphics on your screen.
GNU General Public License v2.0
161 stars 105 forks source link

Lua Error with automatic icon and cooldown progress (item) #27

Closed Zidras closed 4 years ago

Zidras commented 4 years ago

If I use this test aura with both triggers, it gives this lua error.

Message: Interface\AddOns\WeakAuras\GenericTrigger.lua:1989: GetSpellTexture(): Invalid spell slot
Time: 06/16/20 20:06:29
Count: 1
Stack: [C]: ?
[C]: in function `GetSpellTexture'
Interface\AddOns\WeakAuras\GenericTrigger.lua:1989: in function <Interface\AddOns\WeakAuras\GenericTrigger.lua:1973>
[C]: in function `UseInventoryItem'
Interface\FrameXML\ChatFrame.lua:1007: in function `SecureCmdUseItem'
Interface\FrameXML\ChatFrame.lua:1034: in function `?'
Interface\FrameXML\ChatFrame.lua:4031: in function `ChatEdit_ParseText'
Interface\FrameXML\ChatFrame.lua:3660: in function `ChatEdit_SendText'
Interface\FrameXML\ChatFrame.lua:2477: in function <Interface\FrameXML\ChatFrame.lua:2470>
[C]: in function `UseAction'
Interface\FrameXML\SecureTemplates.lua:310: in function `handler'
Interface\FrameXML\SecureTemplates.lua:529: in function <Interface\FrameXML\SecureTemplates.lua:478>
[C]: ?
Interface\FrameXML\SecureHandlers.lua:319: in function <Interface\FrameXML\SecureHandlers.lua:316>
[C]: ?
Interface\FrameXML\SecureHandlers.lua:349: in function <Interface\FrameXML\SecureHandlers.lua:332>
(tail call): ?

Locals:

Test string with Sindra HC Trinket:

!TAvBZPXnq4FnDA7hINdWj29JGZrmtWhUOZXPnPce3P7qjcj6jDGXFW)27URoGZKMonZ4zyolTsR2N9TN18o8uoBfNLJ)GLEB2gzLtzn8KEr9II4mrTFPTAYApi0b3XZzfvIvsMVs4f8KoCwM1QZTBn8b4ATTcUgih)1L(2J(Eo(fu)HjffoPh21ncvWKRA9472F6RU8cW6MmW63Avg4ORItsJNYzvYs4(P7wl5mfOFZxW61kWt(ZKbD)qNL9T96rW3gWvaJX5Ls(cqE2EBYCErLpyCLr1SQaw6wsRHD(kvzjeycow6bb0schI6krxaaoz8dWBZwl16r5OLLBKi4Vwk0(LG1QxqsUTswOEGZy3gpEmioOqtqOgXbBTwSdSXbvy1fKkZUQplDglT)uW8gix0OfIb6Du5niLDrN307s8SC5cq7qm764X3o8UXbdD0tqpfsy)BEheI81o86nEdfwrVDwP0iRuzSL2TtOCWZfWCWF7R3k25(U(o(mkVCvc4kTIyx1KYEA(Tv2YkPZ908FzeCVFfChWvXeiN15hkADWmjVoQ3BopyBs)bNgo(pCeQMix5(sTbkJ2GPFZUqn1gzA4fUXMdw5vDIW7UvLd5EOLIZE0AxXtGfFvkx3hsxz(PORqfLg1QGB1QSSjfySg4Ru4WopPPKEVJbIznjkj2p5cxm9OIagwjuMxWN7qdYl0dIDFqZ7NE77)RNM)PbARnhwaDK5vIsRt8Zq2FOwSvt1bdfMYNMNgZWCQXlRmc9h2ZC193GqPE9Ya7uGcziYzfmi7QPXXjiw0fNWSaGRqvUNlaPcjYcOKAkX50Iaq(GFMBPakq)4EgROgX4ho7NCp7wx1IymIigJiIXOJeJHBFev9VlDsJuSFdlnYUNQKy9P(p6iOixMheNGv0b)9pofuFP25vf7AXIAHMAzkH1KjjXh61RxeCasT)h85buyrepSs94tZ)9ArooCasqpps9nGARTk)(kXAo7(dROd2OCQfAk4h8)2ZbgLKepD2GjPPtUz6O3DDAlmWupkXjl7JdFtYbVuOoyYDPJhb(9EwVsTD7Wk5Fxlnz7afoR7Rdcrua8eVtVplUiip8mlQ9ERzcuSaeqHdg3u)dn)0(2(DP(51cFVG6(CbbasRpEYRKUuL9vd0oqL5OewMqlpSBScoKNCzy3aialrOtTAlLQYL(aJeqKHCNJOjPJWMPcrM8Z4E3NPbzZyu6BwFJxDJOe5ebPCM2kO)TbKefNmqrgpGbtZu0mTWfMXQmLa0yVnUF61VpjK0wvR9Q9JyX3iC9bhNiajd1AjngT922K3z2vlei9nocnJF6Z6W6HtesJdDWenOivsreg0R29SoxCgKX3abWZ7Ijb()8
Zidras commented 4 years ago

(edit: removed specifics on how to reproduce. the error was tied to cooldown progress item spell icon and GCD)

Test string - gives error during a global cooldown:

!TwvutQTnq4FnDA7d3ngUCjPpcxm5ycXqr(6LojvGWETr3jKOwYWX9a)27UsgWqM2PpKzymsR2D1UF73UI3HNYzsoBO2bvfIm4BdZmA73K6nZEc2cQQDZCvs9ZGBw0B5mlN196oV76B5SvCwo9dDGZKTbQSsJMNCt0nrrCMqNT0unOsScs3Tg4m2DtJJt4mLr4TP2cZ0aAgFb6aHc0ouSJZYucRL0Wk1Lk0Ype3l9(pLm8J3JcxvRCsVIUGpcQ3pSr6GvWFxlxd5KO2BtUn6M3(MgBmRwiCKg21qg)s3ALVcxkeflQDyknETdZtBtSAuMkCzsh)VU(V34)(g6lQYlJlkSGd31fHLDh2D17F3bqAIrI5o7U4K04PCwfuIxqa0KyXavl7WvYSorvdovi1s7YWACP7q0JMuildYTGQ4c3NzmQCZwDCEjqqV)kqKaRYLLyrmKnPhf4jiioMGvYZaXsqdvYm2sZ2XEpCUaMf)VNARyhYz0OXnqg5lKeqac6OCyrDrrizVpE0KbpmcVz)wmvD1OT26finr7yOIYx4Sz31JLoJL2BA6PIUp6Arc83aTNag2ALyhMju1wPgM3ejEVIatdGSF(KktzfyT7N)ldrF8Ry4vxjiSNZ6q(QXcKiyofxtQaFCXMepAuic83AFpdv3cg)paOZqfFzmxAFQwJv(naru2fObBG0G)(Sjhlhx1jYRlyrScd1qfL6L1c1FCOLS7V1eZ9jpjxfsPwSjgiSawWbDPBjrGBkbAJgobcZAQlaYyqm0Bu6j9W4yLqQ)b6UJm8FqoKyF4iRV(Hp9x7N)1(kJjhxG4vELO0yf)mw5hOeBvEoWaHUezc4DIeHxnMv8eSbUGMOHHIWjO(e2wzUpKWJEgG19ObkUPu8qLIA6(M2ncEv9CNetbvCOjL(6esGM671B1ZbV4Mzxkq(4xomPiQrm9HZ(j7zADxRbqr(bqr(bqrNgaf0(0GGEpKoUrkrki2q2J(SG1ZtS9hHuAipioH64dJQ(ZldQNQTozXUwtVmyBlK6J1KXjXh7MRxesaVz)pMBgIcdfXdQKVUF(VxlYjyF)80ZrQVlO2AQYFSsSMZE84k)bBKw5cLh8d5F75VdtsINoR)400XFEA4XMJXaJEraNGFah(UIdPuGRn(H0rdX82rPf10RmBhuHVcb6SDObx392wycDQFMXhvhkMlc2e82IANZOhJCgCew4GrnDbyRVFF70VuDoL4FbB93AZ98LlmNLHVf7P2(OyPm7zn2pCuYijULN8(WU(ia7Fc33DTeKLlDHUbQjuE2tLHXS(HqQ1lfnXYgAKxxAj)Fo
Bunny67 commented 4 years ago

It happened after https://github.com/Bunny67/WeakAuras-WotLK/commit/96fe4a0e96ed64465b74659a79f7eefc3806b89d ?

Bunny67 commented 4 years ago

I can not reproduce

Zidras commented 4 years ago

Yes, for me it's somehow linked to the global cooldown. I can try and revert that change to check if it's linked to that commit. I'll post my findings here

Zidras commented 4 years ago

Yeah, without that commit, I no longer have the error