Text Display for Inscription/Tailoring/Enchanting #55

  • New
  • Defect
Open
Assigned to
View _ForgeUser22265's Profile
_ForgeUser22265
  • View barackstar's Profile
    barackstar created this issue Nov 22, 2012

    This is the error I receive when trying to display the cooldown for Scroll of Wisdom, Imperial Cloth, or Sha Crystal:

     1
 2
 3
 4
 5
 6
 7
 8
 9
10
11
12
13
14
15
16
17
    8x Broker_TradeCooldowns\Broker_TradeCooldowns-r20121101234205.lua:83: attempt to concatenate local "text" (a boolean value)
Broker_TradeCooldowns\Broker_TradeCooldowns-r20121101234205.lua:83: in function <Broker_TradeCooldowns\Broker_TradeCooldowns.lua:83>
Broker_TradeCooldowns\Broker_TradeCooldowns-r20121101234205.lua:567: in function "UpdateDataText"
Broker_TradeCooldowns\Broker_TradeCooldowns-r20121101234205.lua:618: in function <Broker_TradeCooldowns\Broker_TradeCooldowns.lua:578>
(tail call): ?
<in C code>
<string>:"safecall Dispatcher[1]":9: in function <string>:"safecall Dispatcher[1]":5
(tail call): ?
Ace3-r1066\AceAddon-3.0\AceAddon-3.0-11.lua:543: in function "EnableAddon"
Ace3-r1066\AceAddon-3.0\AceAddon-3.0-11.lua:636: in function <Ace3\AceAddon-3.0\AceAddon-3.0.lua:621>
<in C code>
FrameXML\UIParent.lua:299: in function "UIParentLoadAddOn"
FrameXML\UIParent.lua:322: in function "CombatLog_LoadUI"
FrameXML\UIParent.lua:692: in function <FrameXML\UIParent.lua:657>

Locals:
nil
  • View barackstar's Profile
    barackstar     added the tags New Defect Nov 22, 2012
  • View speedwayst4r's Profile
    speedwayst4r posted a comment May 5, 2013

    Here's my fix:

    in Broker_TradeCooldowns.lua, find the function private.UpdateDataText(), and change the line which reads:

    launcher.text = _G.SUBTITLE_FORMAT:format(Yellow(COOLDOWN_SPELL_MAP[spell_id]), time_text)

    to

    launcher.text = _G.SUBTITLE_FORMAT:format(Yellow(private.CooldownDisplayName(spell_id)), time_text)

    this sanity check is necessary because COOLDOWN_SPELL_MAP[spell_id] may contain either a boolean value, or the name of the spell itself.

    Edited May 5, 2013

To post a comment, please login or register a new account.