Sometimes when I craft an item or when I am scrolling though my profession UI, I get a looooong list of errors in my chat. (Sorry dont know the exact error). Can you please not report those errors though chat, because I cant scroll up anymore after this bug comes up.
Can you please tell me the error and what version you are running when you got the errors.
My bad. Had an illegal default value for frame strata. I've commited a fix. You may have to manually edit TradeskillInfo.lua in WTF if you keep getting the error. Value for FrameStrata should be 1 to 3. (Low, Medium, High)
Thank you for TradeskillInfo r29175. That fixed it for me.
It is spammed in my main chat window, which is very irritating because you can no longer read what was in the chat before the spam. I would be very happy if this was fixed.
It is spammed in my main chat window, which is very irritating because you can no longer read what was in the chat before the spam. I would be very happy if this was fixed.
You are running an old version of TradeskillInfo. Newer versions report more information, but only one line.
It is spammed in my main chat window, which is very irritating because you can no longer read what was in the chat before the spam. I would be very happy if this was fixed.
You are running an old version of TradeskillInfo. Newer versions report more information, but only one line.
I found 2 typos which cause this errors:
In line 422 and 454 you can find somethink like this:
... +numSkills);
You have to change this to -numSkills.
Recipe for the
Dark Iron Pulverizer (BS 265)
requires Weaponsmith.
I've fixed this for my own, but just so author and everybody know...
P.S. Great mod, but would be greater if in recipe links were not just item names, but item links.
Good luck.
Recipe for the
Dark Iron Pulverizer (BS 265)
requires Weaponsmith.
I've fixed this for my own, but just so author and everybody know...
P.S. Great mod, but would be greater if in recipe links were not just item names, but item links.
Good luck.
Thanks. I'll update Dark Iron Pulverizer.
The reason the Recipe Link doesn't contain the item links is that it will get to many characters to insert into the chatframe in one line. (All the special characters for the link takes alot of space).
Edit:
Strange. Dark Iron Pulverizer is one of the old Weaponsmithing recipes. According to a post by blizzard all those should no longer require specialization. Wonder if this is a bug by blizzard.
Yeah it seems that's Blizzard's fault. Anyways ty for the fix.
One more question, any chance that TSI set-ups will be separated for profiles? Now if i set Smithing for my main character, smithing appears for all alt characters, and if i remove it for alt it is removed for main.
Tradeskillinfo does this because it wants to give info about your other char's professions as well, the UI reports that the recipe is known by "Name" which is most likely your other character.
Yeah it seems that's Blizzard's fault. Anyways ty for the fix.
One more question, any chance that TSI set-ups will be separated for profiles? Now if i set Smithing for my main character, smithing appears for all alt characters, and if i remove it for alt it is removed for main.
I don't understand what you mean by "i set Smithing for my main character", but all configuration is done on a per profile basis, but there is no GUI to change profile. For that you need to use the slash command /tsi profile change {char/class/realm/default}
I'm currently looking at a different configuration GUI since it takes to long to add new things the way it's done now.
There's something not right about my TSI installation. I just installed it ten minutes ago, so I know I have the most updated version. I have both TSI and TSI UI installed.
>> When I use /TSI config, I turned off the combine cost, but it still shows in my tradeskill windows.
>> The skill requirement numbers are all messed up. For example, my Heavy Wool Bandage lists this: 0/0/145/115.
Yeah it seems that's Blizzard's fault. Anyways ty for the fix.
One more question, any chance that TSI set-ups will be separated for profiles? Now if i set Smithing for my main character, smithing appears for all alt characters, and if i remove it for alt it is removed for main.
I don't understand what you mean by "i set Smithing for my main character", but all configuration is done on a per profile basis, but there is no GUI to change profile. For that you need to use the slash command /tsi profile change {char/class/realm/default}
I'm currently looking at a different configuration GUI since it takes to long to add new things the way it's done now.
Well... Damn my bad english.
My main character knows BS and Cooking, (from the TS list of TSI). And i set TSI to show these professions for me. My alt character is Jewelcrafter, but when i switch on it TSI window still shows BS and Cooking, and if i change BS to JC, same changes apply for the main character. What i wish is split set-ups for different characters.
There's something not right about my TSI installation. I just installed it ten minutes ago, so I know I have the most updated version. I have both TSI and TSI UI installed.
>> When I use /TSI config, I turned off the combine cost, but it still shows in my tradeskill windows.
>> The skill requirement numbers are all messed up. For example, my Heavy Wool Bandage lists this: 0/0/145/115.
Everything else seems to work as advertised. :p
The turn off of the 'combine cost' actually works. It just seems like it don't :P Let me explain. The text is added the the "Reagents:" text label, but when it is turned off nothing is updating that text label leaving it as it was when the feature was turned off. I earlier had it reset to the original text, but that prevented other addons from adding info to the text label. I'll see if I can make a better solution. But for now if you disable 'Combine Cost' and then do a ReloadUI or when you log in the next time you should not see any combine cost from TradeskillInfo.
The skill requirement numbers are not supplied with the addon, but are learned. I haven't found a reliable source for them. (And don't come dragging with thottbot because there numbers are often way off.)
Well... Damn my bad english.
My main character knows BS and Cooking, (from the TS list of TSI). And i set TSI to show these professions for me. My alt character is Jewelcrafter, but when i switch on it TSI window still shows BS and Cooking, and if i change BS to JC, same changes apply for the main character. What i wish is split set-ups for different characters.
Ah. My bad. The configuration for TradeskillInfoUI is stored with TradeskillInfoUI which don't have a chat command. So changing profile for TradeskillInfo will do nothing for the configuration of TradeskillInfoUI. I'll have to make a fix for that.
The skill requirement numbers are not supplied with the addon, but are learned. I haven't found a reliable source for them. (And don't come dragging with thottbot because there numbers are often way off.)
hhtp://wwww.wowhead.com (maybe?) I haven't looked into them for skills, but they're proving out as a reliable information site for me so far.
Rollback Post to RevisionRollBack
To post a comment, please login or register a new account.
-
View User Profile
-
View Posts
-
Send Message
Curse PremiumCan you please tell me the error and what version you are running when you got the errors.
Thank you for TradeskillInfo r29175. That fixed it for me.
I have tried to reproduce the error again, but failed in doing so. If it occurs again (it used to occur very often), I will post the exact error here.
UpdateKnownTradeRecipes id=nil, link=nil type=nil
UpdateKnownTradeRecipes id=nil, link=nil type=nil
UpdateKnownTradeRecipes id=nil, link=nil type=nil
UpdateKnownTradeRecipes id=nil, link=nil type=nil
UpdateKnownTradeRecipes id=nil, link=nil type=nil
(and a lot more)
It is spammed in my main chat window, which is very irritating because you can no longer read what was in the chat before the spam. I would be very happy if this was fixed.
-
View User Profile
-
View Posts
-
Send Message
Curse PremiumYou are running an old version of TradeskillInfo. Newer versions report more information, but only one line.
My pc freezes for a couple of minutes btw :(
-
View User Profile
-
View Posts
-
Send Message
Curse PremiumI found 2 typos which cause this errors:
In line 422 and 454 you can find somethink like this:
... +numSkills);
You have to change this to -numSkills.
Takika
-
View User Profile
-
View Posts
-
Send Message
Curse PremiumThanks.
I'll update the SVN with this change as soon as i've verified it works.
Dark Iron Pulverizer (BS 265)
requires Weaponsmith.
I've fixed this for my own, but just so author and everybody know...
P.S. Great mod, but would be greater if in recipe links were not just item names, but item links.
Good luck.
-
View User Profile
-
View Posts
-
Send Message
Curse PremiumThanks. I'll update Dark Iron Pulverizer.
The reason the Recipe Link doesn't contain the item links is that it will get to many characters to insert into the chatframe in one line. (All the special characters for the link takes alot of space).
Edit:
Strange. Dark Iron Pulverizer is one of the old Weaponsmithing recipes. According to a post by blizzard all those should no longer require specialization. Wonder if this is a bug by blizzard.
One more question, any chance that TSI set-ups will be separated for profiles? Now if i set Smithing for my main character, smithing appears for all alt characters, and if i remove it for alt it is removed for main.
-
View User Profile
-
View Posts
-
Send Message
Curse Premium-
View User Profile
-
View Posts
-
Send Message
Curse PremiumI don't understand what you mean by "i set Smithing for my main character", but all configuration is done on a per profile basis, but there is no GUI to change profile. For that you need to use the slash command /tsi profile change {char/class/realm/default}
I'm currently looking at a different configuration GUI since it takes to long to add new things the way it's done now.
>> When I use /TSI config, I turned off the combine cost, but it still shows in my tradeskill windows.
>> The skill requirement numbers are all messed up. For example, my Heavy Wool Bandage lists this: 0/0/145/115.
Everything else seems to work as advertised. :p
Well... Damn my bad english.
My main character knows BS and Cooking, (from the TS list of TSI). And i set TSI to show these professions for me. My alt character is Jewelcrafter, but when i switch on it TSI window still shows BS and Cooking, and if i change BS to JC, same changes apply for the main character. What i wish is split set-ups for different characters.
-
View User Profile
-
View Posts
-
Send Message
Curse PremiumThe turn off of the 'combine cost' actually works. It just seems like it don't :P Let me explain. The text is added the the "Reagents:" text label, but when it is turned off nothing is updating that text label leaving it as it was when the feature was turned off. I earlier had it reset to the original text, but that prevented other addons from adding info to the text label. I'll see if I can make a better solution. But for now if you disable 'Combine Cost' and then do a ReloadUI or when you log in the next time you should not see any combine cost from TradeskillInfo.
The skill requirement numbers are not supplied with the addon, but are learned. I haven't found a reliable source for them. (And don't come dragging with thottbot because there numbers are often way off.)
-
View User Profile
-
View Posts
-
Send Message
Curse PremiumAh. My bad. The configuration for TradeskillInfoUI is stored with TradeskillInfoUI which don't have a chat command. So changing profile for TradeskillInfo will do nothing for the configuration of TradeskillInfoUI. I'll have to make a fix for that.