WowAce.com

Analyst

15 - Error displaying tooltip for "current session only"

I have the analyst info being displayed on titan panel, for the current session only. When I hover over the analyst plug-in, I get the following error:

Error occured in: Global
Count: 1
Message: ..\AddOns\Analyst\Analyst.lua line 320:
   'for' initial value must be a number
Debug:
   (tail call): ?
   [C]: ?
   Analyst\Analyst.lua:320: OnTooltipShow()
   Analyst\Analyst.lua:136: func()
   Titan\LDBToTitan.lua:137: TitanLDBSetTooltip()
   Titan\LDBToTitan.lua:194:
      Titan\LDBToTitan.lua:192

It seems that after Analyst:OnTooltipShow() calls GetPeriodStartFinish(), it doesn't handle the case where the return values are nil (which seems to be a valid return value for GetPeriodStartFinish() if looking at current session data).
Analyst:UpdateReport() seems to handle this correctly, however. Adding the same check for nil and else clause to OnTooltipShow() seemed to resolve the issue for me (as far as I can tell, anyway).

User When Change
Bethink Sun, 08 Nov 2009 23:55:00 Changed status from Accepted to Fixed
Bethink Mon, 02 Nov 2009 23:02:27 Changed status from New to Accepted
orgevo Thu, 15 Oct 2009 20:17:29 Create

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

  • 1 comment
  • Avatar of Bethink Bethink Sun, 08 Nov 2009 23:54:49

    Your analysis is right on point. Thanks. Fixed in 1.0.10.

  • 1 comment

Facts

Last updated on
08 Nov 2009
Reported on
15 Oct 2009
Status
Fixed - Developer made requested changes. QA should verify.
Type
Defect - A shortcoming, fault, or imperfection
Priority
Medium - Normal priority.

Reported by

Possible assignees