31 - Problem with BE and Bazooka
What steps will reproduce the problem?
1. Hover mouse over icon to show tooltip
What is the expected output? What do you see instead?
Don't see a tooltip
What version of the product are you using?
r207
Do you have an error log of what happened?
1x AceLocale-3.0-5 (Ace3):49: assertion failed!
AceLocale-3.0-5 (Ace3):49: in function <...nterface\AddOns\Ace3\AceLocale-3.0\AceLocale-3.0.lua:49
Broker_Everything-0.1.5\modules\gps.lua:89 in function <Broker_Everything\modules\gps.lua:85
Broker_Everything-0.1.5\modules\gps.lua:125 in function `OnEnter'
Bazooka-Bazooka-r207\Bazooka.lua:1393 in function `showTip'
Bazooka-Bazooka-r207\Bazooka.lua:1221 in function <Bazooka\Bazooka.lua:1212
Please provide any additional information below.
Bazooka.toc states v2.1.3 - Bazooka.lua states r207 - confusing, which is why the error above says r207
| User | When | Change |
|---|---|---|
| mitch0 | Aug 14, 2011 at 19:31 UTC | Changed status from New to Invalid |
| onaforeignshore | Jul 21, 2011 at 19:36 UTC | Create |
- 2 comments
- 2 comments
Facts
- Last updated
- Aug 14, 2011
- Reported
- Jul 21, 2011
- Status
- Invalid - This was not a valid report.
- Type
- Defect - A shortcoming, fault, or imperfection
- Priority
- Medium - Normal priority.
- Votes
- 0
- Reply
- #2
mitch0 Jul 22, 2011 at 07:00 UTC - 0 likesHi,
As Mikk said below, the problem is caused by BE using a global L variable that probably gets overwritten by some other (also badly written) addon that's also using a global L... BE leaks some other globals, too, that can lead to similar problems. I suggest you contact BE's developer and ask him/her to clean up the code a bit (most importantly with regards to global pollution).
I'll fix the version string generation so as not to confuse users with two different version in case of an error though (Bazooka.lua in v2.1.3 is the same as it was in r207, I just hat to bump the interface version in Bazooka.toc without touching any other files to make Bazooka "WoW-4.2 ready", that's why you see the "old" revision number in the trace).
cheers, mitch
- Reply
- #1
mikk Jul 21, 2011 at 21:08 UTC - 0 likesBroker_Everything uses a global "L" AceLocale variable. Very very bad idea. It can get replaced by any other addon at any time. Or by its own locale files depending on load order.