missed that thread I was temporarily disabling skillet much this days due to some suspect activities of the nomore updated version, thanks a lot for maintaining this great project ! thumbup
lilsparky, is your branch available somewhere? nogudnik's version seems to be broken right now; I've found Cauldron on Curse which seems to work but lilsparky's Workshop doesn't support it...
The window appears and is empty of recipes (I think I only tried alchemy and engineering - my two 80's). Clicking other trades (the buttons at the top of the Skillet window) on the same toon does absoultely nothing.
Edit: I may have been mistaken. I think what I saw was the problems this version has on initial characters with no saved vars. When I reloaded it just now on my JC the window opened and performed flawlessly. I think I may have wiped my SV files after the patch and caused my confusion. I'm eager to try it again on my other toons now.
What is happening on my end, with both your Skillet clone and LSW installed, is that the "other" professions, that my current character does not have, no longer show up when selecting them. This began rather recently, but not sure of the particular patch; other functions seem to be fine, though.
Will watch more closely to see if there is an actual error.
Another needed change is in skillet.lua around line 1286, from
local _,_,tradeString = string.find(link, "(trade:%d+:%d+:%d+:[0-9a-fA-F]+:[<-{]+)")
to
local _,_,tradeString = string.find(link, "(trade:%d+:%d+:%d+:[0-9a-fA-F]+:[<-{/]+)")
Not sure if the curly braces are still needed or not. :P
Also, some 'All data' links like Inscription and leatherworking are still broken. I've drilled down the problem to Skillet using an incorrect number of '/'s in the link (thank god for GYP saved variables. :D), still reading through the code...
Edit: Ah. so it's the TradeSkillRecipeList table in SkilletData.lua that's causing the recipe counts to be broken... is there any easy way to quickly update this table without digging through 3.0.x patch notes to copy the newly added recipe links?
local _,_,tradeString = string.find(link, "(trade:%d+:%d+:%d+:[0-9a-fA-F]+:[a-zA-Z0-9+/]+)")
they changed their encoding scheme to use different characters entirely.
and for that matter, it should probably be:
local tradeString = string.match(link, "(trade:%d+:%d+:%d+:[0-9a-fA-F]+:[a-zA-Z0-9+/]+)")
since i'm tossing the indices...
and yeah, the data i'm using for the TradeSkillRecipeList is probably outdated enough to cause issues for certain trades.
the good news is that it's an identical format to what i'm using in gyp (Config.spellList in the savedvars file = TradeSkillRecipeList in skillet). gyp doesn't record data for runeforging, tho. so no digging thru notes.
i'm seriously considering writing a tradeskill lib that puts all this stuff into one place.
[FONT=Times New Roman]Lilsparky,[/FONT]
[FONT=Times New Roman][/FONT]
[FONT=Times New Roman]Appreciate the input, will give the suggested changes a try. [/FONT][FONT=Times New Roman]Have not had the time to log in or anything for a couple of days, as likely happens on your end, too. =)[/FONT]
[FONT=Times New Roman][/FONT]
[FONT=Times New Roman]As to any "crazy code", lord knows we have enough in our stuff![/FONT]
[FONT=Times New Roman][/FONT]
[FONT=Times New Roman]Looking forward to when you have the time to craft your TS addon; you seem to have a tight grip on what everyone from the occasional to professional crafter needs.[/FONT]
[FONT=Times New Roman]Thank you,[/FONT]
[FONT=Times New Roman][/FONT]
[FONT=Times New Roman]Nikk :cool:[/FONT]
i don't have any updates planned. what i have planned is a new mod, but i haven't quite taken the plunge into coding it yet.
Since Skillet comes up with messages like "Error in LibPossesions. The addon will not be used" or something close to this (BugGrabber does not show any errors, its just a chat message), I wonder if the new mod is still on your table? ;D
Cheers
Not so good. :(
I hope it will have at least all the features that "Skillet" has. I can't live without it anymore... :p
any speculation as to when this might be done? :)
http://www.wowace.com/projects/skillet/repositories/lilsparkys-clone/files/
i really do plan on writing a new ts mod, but i keep having other ideas come up..
Edit: I may have been mistaken. I think what I saw was the problems this version has on initial characters with no saved vars. When I reloaded it just now on my JC the window opened and performed flawlessly. I think I may have wiped my SV files after the patch and caused my confusion. I'm eager to try it again on my other toons now.
What is happening on my end, with both your Skillet clone and LSW installed, is that the "other" professions, that my current character does not have, no longer show up when selecting them. This began rather recently, but not sure of the particular patch; other functions seem to be fine, though.
Will watch more closely to see if there is an actual error.
Thank you,
Nikk
in SkilletData.lua around line 1050 or so, look for:
local encodedString = string.rep("<",encodingLength)
and replace it with
local encodedString = string.rep("/",encodingLength)
that should work for now.
oh, and try not to look around too much or you might need to take a shower.
i'd update the svn, but my local copy has been hacked on a bit for some other things...
i promise i'll be writing a new ts mod soon, tho!
local encodedString = string.rep("{",encodingLength)
Should I still do the replacement?
to
Not sure if the curly braces are still needed or not. :P
Also, some 'All data' links like Inscription and leatherworking are still broken. I've drilled down the problem to Skillet using an incorrect number of '/'s in the link (thank god for GYP saved variables. :D), still reading through the code...
Edit: Ah. so it's the TradeSkillRecipeList table in SkilletData.lua that's causing the recipe counts to be broken... is there any easy way to quickly update this table without digging through 3.0.x patch notes to copy the newly added recipe links?
local _,_,tradeString = string.find(link, "(trade:%d+:%d+:%d+:[0-9a-fA-F]+:[a-zA-Z0-9+/]+)")
they changed their encoding scheme to use different characters entirely.
and for that matter, it should probably be:
local tradeString = string.match(link, "(trade:%d+:%d+:%d+:[0-9a-fA-F]+:[a-zA-Z0-9+/]+)")
since i'm tossing the indices...
and yeah, the data i'm using for the TradeSkillRecipeList is probably outdated enough to cause issues for certain trades.
the good news is that it's an identical format to what i'm using in gyp (Config.spellList in the savedvars file = TradeSkillRecipeList in skillet). gyp doesn't record data for runeforging, tho. so no digging thru notes.
i'm seriously considering writing a tradeskill lib that puts all this stuff into one place.
[FONT=Times New Roman][/FONT]
[FONT=Times New Roman]Appreciate the input, will give the suggested changes a try. [/FONT][FONT=Times New Roman]Have not had the time to log in or anything for a couple of days, as likely happens on your end, too. =)[/FONT]
[FONT=Times New Roman][/FONT]
[FONT=Times New Roman]As to any "crazy code", lord knows we have enough in our stuff![/FONT]
[FONT=Times New Roman][/FONT]
[FONT=Times New Roman]Looking forward to when you have the time to craft your TS addon; you seem to have a tight grip on what everyone from the occasional to professional crafter needs.[/FONT]
[FONT=Times New Roman]Thank you,[/FONT]
[FONT=Times New Roman][/FONT]
[FONT=Times New Roman]Nikk :cool:[/FONT]
Since Skillet comes up with messages like "Error in LibPossesions. The addon will not be used" or something close to this (BugGrabber does not show any errors, its just a chat message), I wonder if the new mod is still on your table? ;D
Cheers
Any chance to update the latest copy to the repository.