Authors CurseForge

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

Quote from lilsparky

i don't have any updates planned.

Not so good. :(

Quote from lilsparky

what i have planned is a new mod, ...

I hope it will have at least all the features that "Skillet" has. I can't live without it anymore... :p

Quote from lilsparky

what i have planned is a new mod, but i haven't quite taken the plunge into coding it yet.

any speculation as to when this might be done? :)

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...

link on first page of thread worked fien for me.


http://www.wowace.com/projects/skillet/repositories/lilsparkys-clone/files/

Hmm, the last time I didn't find anything on the Skillet page under the Repositories link... I have to be blind :)

The version posted 11-17-08 stopped working for me after the last Blizzard patch. Did anyone else have the same problem?

does it completely fail or is it just one feature that is broken?

i really do plan on writing a new ts mod, but i keep having other ideas come up..

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.

LilSparky,

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

yeah. they changed the format of the trade links so now a line of <'s no longer means all bits set. it's now /'s.

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!

Hmmmm - my version has:

local encodedString = string.rep("{",encodingLength)

Should I still do the replacement?

oops, my mistake. "<" is for no bits set and "{" is for all bits set... so yeah, replace the "{" with "/"...

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?

actually, that line should be :

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]

Quote from lilsparky

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

I'm being stupid and have done the above code changes but I'm being spouted with errors.

Any chance to update the latest copy to the repository.

i could probably update things a bit to fix some of the known issues...

That would be awesome and so much appreciated. :)