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 Sep 25, 2009 at 14:08 UTC Changed priority from Medium to Low
Dys Sep 13, 2009 at 10:51 UTC Changed status from New to Started
DanoPDX Sep 05, 2009 at 22:54 UTC

Added attachment TradeSkillInfo.jpg

DanoPDX Sep 05, 2009 at 22:54 UTC Create

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

  • 7 comments
  • Avatar of Dys Dys Sep 25, 2009 at 14:08 UTC - 0 likes

    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 Sep 20, 2009 at 12:44 UTC - 0 likes

    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 Sep 13, 2009 at 09:58 UTC - 0 likes

    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 Sep 13, 2009 at 08:51 UTC - 0 likes

    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 Sep 12, 2009 at 05:38 UTC - 0 likes

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

  • Avatar of Dys Dys Sep 06, 2009 at 18:38 UTC - 0 likes

    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 Sep 06, 2009 at 05:52 UTC - 0 likes

    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 8b365fe445bd...
    • TradeskillInfo_2.jpg
    • TradeskillInfo_3.jpg
  • 7 comments

Facts

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

Reported by

Possible assignees