1 - Fix to prevent MacroTalk.lua from throwing Lua error when trying to use secure command with /opt or /rnd
What does the provided patch do?
Currently, MacroTalk 2.0.1 will throw an error on line 18 of MacroTalk.lua when an attempt to use a secure command via /opt or /rnd is made, due to the variable L not being defined. Simply adding the line
local L = MacroTalk_Locals
at the top of the file (above or below the "local print =" line) will fix the issue.
Please provide any additional information below.
Discovered when I came back to WoW after an extended hiatus and tested out one of my old buttons that switched between outfits (using the now secure command "/equipset") and two scripts to toggle helm or cloak on and off.
It's a little bit nicer to have MacroTalk print the intended simple "oops" message "You cannot use /equipset with /opt or /rnd" than to get the big scary Lua error window. :)
| User | When | Change |
|---|---|---|
| Djidam | Dec 01, 2011 at 18:44 UTC | Changed status from New to Fixed |
| Boccanegra | Sep 23, 2011 at 00:43 UTC | Create |
- 1 comment
- 1 comment
Facts
- Last updated
- Dec 01, 2011
- Reported
- Sep 23, 2011
- Status
- Fixed - Developer made requested changes. QA should verify.
- Type
- Patch - Source code patch for review
- Priority
- Medium - Normal priority.
- Votes
- 0
- Reply
- #1
Djidam Dec 01, 2011 at 18:44 UTC - 0 likesHey, sorry I just saw your ticket...
With wow 4.3 I got an issue, so I was looking and then, I saw your ticket !
It's funny because I had the same issue with /opt and /equipset, which did not work, but I forgot to turn on lua error messages to see what was wrong...
So thank you, i have added your line, and the little change with a constant (that does not exist anymore).
It should work just fine :)