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

• 

  • Reply
  • #7

  sparky02 Mar 06, 2013 at 21:56 UTC - 0 likes

  Just wanted to say "Thank you" for the updated fix to Skillet. :) Much appreciated - seems to be working perfectly again! :D

• 

  • Reply
  • #6

  sapu94 Mar 06, 2013 at 18:08 UTC - 0 likes

  Looks like this bug has already been fixed by yossa, but regarding what Speedwaystar posted below, if you use that code somewhere, be sure you're not polluting the global table (what I use in my addon is slightly different and doesn't pollute).

  Last edited Mar 06, 2013 by sapu94

  Lead Developer and Director of TradeSkillMaster | @Sapu94 on Twitter

  "A good programmer is someone who always looks both ways before crossing a one-way street." Doug Linder

• 

  • Reply
  • #5

  Speedwaystar Mar 06, 2013 at 10:35 UTC - 1 like

  temporary fix/kludge

  open SkilletData.lua in your favourite code editor and scroll to line 2032, which reads:

      local reagentLink = API.GetReagentlink(i,j)
  

  delete or comment out this line, and replace it with:

      tooltip = CreateFrame("GameTooltip", "SkilletDataScanTooltip", UIParent, "GameTooltipTemplate")
      tooltip:SetOwner(UIParent, "ANCHOR_NONE")
      tooltip:ClearLines()
      tooltip:SetTradeSkillItem(i,j)
      _, reagentLink = tooltip:GetItem()
  

  kudos to sapu94 (the author of TradeSkillManager) for this fix

  Last edited Mar 06, 2013 by Speedwaystar
• 

  • Reply
  • #4

  tabeecat Mar 06, 2013 at 01:15 UTC - 0 likes

  i have the same problem. wow 5.2 and skillet r307 Date: 2013-03-05 16:52:11 ID: 2 Error occured in: Global Count: 1 Message: ..\AddOns\Skillet\SkilletData.lua line 2048: attempt to concatenate local 'reagentID' (a nil value) Debug: [C]: ? Skillet\SkilletData.lua:2048: ScanTrade() Skillet\SkilletData.lua:1691: RescanTrade() Skillet\SkilletData.lua:1575: RescanTrade() Skillet\Skillet.lua:882: ?() ...ce\AddOns\Skillet\Libs\AceTimer-3.0\AceTimer-3.0.lua:41: ...ce\AddOns\Skillet\Libs\AceTimer-3.0\AceTimer-3.0.lua:36

• 

  • Reply
  • #3

  Ravenslair38 Mar 06, 2013 at 00:38 UTC - 0 likes

  I get the following error when trying to open skillet:

  1x Skillet-r307\UI\MainFrame.lua:1271: attempt to get length of field "reagentData" (a nil value) Skillet-r307\UI\MainFrame.lua:1271: in function <Skillet\UI\MainFrame.lua:914> (tail call): ? Skillet\Skillet-r307.lua:1122: in function <Skillet\Skillet.lua:1108> (tail call): ? Skillet\Skillet-r307.lua:888: in function "?" Ace3-Release-r1083\AceTimer-3.0\AceTimer-3.0-16.lua:41: in function <Ace3\AceTimer-3.0\AceTimer-3.0.lua:36>

  Locals: (*temporary) = <func> =[C]:-1

• 

  • Reply
  • #2

  eggplant37 Mar 06, 2013 at 00:05 UTC - 0 likes

  Concur with this. Just finished MoP 5.2 upgrade and logged in WoW, no trade skill windows work with r307, similar error message.

• 

  • Reply
  • #1

  sparky02 Mar 05, 2013 at 22:30 UTC - 0 likes

  My apologies- for the question of "Which version are you using?", I may have misunderstood the question. I downloaded your newest update, revision 307 of Skillet, for the newest 5.2 Patch of MoP. Thank you.