Bazooka

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

You must login to post a comment. Don't have an account? Register to get one!

  • 2 comments
  • Avatar of mitch0 mitch0 Jul 22, 2011 at 07:00 UTC - 0 likes

    Hi,

    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

    Last edited Jul 22, 2011 by mitch0
  • Avatar of mikk mikk Jul 21, 2011 at 21:08 UTC - 0 likes

    Broker_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.

  • 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

Reported by

Possible assignees