TradeskillInfo

20 - Will be able to Learn Error

What steps will reproduce the problem?
1. Crsr over a Glyph

What is the expected output? What do you see instead?

It should not say "Will be able to learn" if I already know it.

What version of the product are you using?

r331

Do you have an error log of what happened?

Please provide any additional information below.

  • TradeSkillInfo.jpg
User When Change
Dys Fri, 25 Sep 2009 14:08:32 Changed priority from Medium to Low
Dys Sun, 13 Sep 2009 10:51:32 Changed status from New to Started
DanoPDX Sat, 05 Sep 2009 22:54:39

Added attachment TradeSkillInfo.jpg

DanoPDX Sat, 05 Sep 2009 22:54:16 Create

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

  • 7 comments
  • Avatar of Dys Dys Fri, 25 Sep 2009 14:08:06

    Thanks for validating it.

    I will leave the ticket open until I can find a way to check if the skill requirement numbers are valid or not. However, I will move the priority to "low", because I don't have the time to do the research for it these days.

  • Avatar of DanoPDX DanoPDX Sun, 20 Sep 2009 12:44:12

    Fix seems to work just fine ^_^  Thanks

    Hopefully you can catch the incorrect/unknown skill level requirements to add colors to the tooltip for mats used for crafting.  It would be helpfull.

  • Avatar of Dys Dys Sun, 13 Sep 2009 09:58:35

    I've downloaded the updated data and added it to TSI. Although it did contain a fix for the example you have given, there were still a lot of combines with incorrect/unknown skill level requirements. So, I've also modified the code to never show both "known by" and "will be learnable by" at the same time.

    Could you check if r334 fixes the problem for you?

  • Avatar of Dys Dys Sun, 13 Sep 2009 08:51:48

    Thanks for the saved variables file. It indicated one TSI somehow thinks you know the recipes for Colorful Kilt and Stylish Green Shirt. I didn't see any other obvious problems with it.

    As for the other problem (recipe known and will be able to learn at the same time) it is a problem with incorrect data for difficulty levels in TSI itself. It looks like wowhead now has more accurate info on when you will be able to learn these glyphs. I will do a sweep soon to update the information.

  • Avatar of DanoPDX DanoPDX Sat, 12 Sep 2009 05:38:56

    I attached it below but I am gonna delete it I am not really worried about loosing anything.

  • Avatar of Dys Dys Sun, 06 Sep 2009 18:38:40

    Looks like some sort of corruption in the saved variables file - either due to an old profession, or something else entirely. Could you attach a copy of the file <WoW>\WTF\Account\<AccountName>\SavedVariables\TradeskillInfo.lua? That way I can diagnose the problem. <WoW> is where the game is installed, and <AccountName> is the name you use to login to game.

    However, an easy fix would be to delete this file, and scan you characters' professions if you don't want to wait for me to have a look (this will have the unfortunate side-effect of erasing your settings). Just be sure to make a backup copy, in case you want to go back.

  • Avatar of DanoPDX DanoPDX Sun, 06 Sep 2009 05:52:08

    A related error is that I did not know the pattern, but it said I did and that an alt was able to learn it.

    Name Size MD5
    TradeskillInfo.lua 88.3 KiB 8b365fe445bde406d42cf3eec9eb71fb
    • TradeskillInfo_2.jpg
    • TradeskillInfo_3.jpg
  • 7 comments

Facts

Last updated on
25 Sep 2009
Reported on
05 Sep 2009
Status
Started - Work on this issue has begun.
Type
Defect - A shortcoming, fault, or imperfection
Priority
Low - Might slip to a later milestone.

Reported by

Possible assignees