Skillet

233 - Skillet Window Won't Open/Doesn't Work-Error

What steps will reproduce the problem?
1. When I attempt to open the Trade Skill Window (any), nothing from Skillet opens up; all I get is an error message.
2.
3.

What is the expected output? What do you see instead?
I expect to open my professions menu, click on a profession, and see the Skillet box open up. Instead, I am receiving an error message and no box.

What version of the product are you using?
I have MoP, with newest Patch 5.2, which just came out today.

Do you have an error log of what happened?
Date: 2013-03-05 16:22:59
ID: 1
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
Locals:

AddOns:

Please provide any additional information below.
(Thank you for your time.)

User When Change
yossa Mar 20, 2013 at 11:36 UTC
sparky02 Mar 05, 2013 at 22:28 UTC Create

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

  • 7 comments
  • Avatar of sparky02 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

  • Avatar of sapu94 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

  • Avatar of Speedwaystar 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
  • Avatar of tabeecat 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

  • Avatar of Ravenslair38 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

  • Avatar of eggplant37 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.

  • Avatar of sparky02 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.

  • 7 comments

Facts

Last updated
Mar 20, 2013
Reported
Mar 05, 2013
Status
Fixed - Developer made requested changes. QA should verify.
Type
Defect - A shortcoming, fault, or imperfection
Priority
Medium - Normal priority.
Votes
2

Reported by

Possible assignees