Well that's only a little counter-intuitive (and possibly something you might want to take into consideration if this ever gets committed to the trunk, lots of people might end up with the same error) but after deleting that saved variable file i can open skillet without error - BUT the problems aren't over yet.
With alchemy open, clicking on headings doesn't do anything, but if i click on the little (-) next to a heading name, instead of collapsing the heading and all it's items disappear! I can keep doing this till the entire list is empty. If i switch the grouping drop down to flat, everything is there, but switching back to blizzard they are still gone, and stay gone till i relog or /reloadui. Same happens with Cooking and First Aid.
On a side note, i want to put a vote in to have the 'hide trivial' and 'hide uncraftable' back as checkboxes rather than drop down options... it's much quicker and easier to use them that way, at least for me, and they don't make a lot of sense being under 'grouping' - just a vote, not a demand :).
yeah, well, before i commit to trunk, i hope to have all the saved variable stuff worked out. this is a development branch, so expect a bumpy ride. :)
i found that error with the headers after i fired off this version, but i've already fixed it. for now, if you click rescan i think it'll replace those headers (all that's going on is that collapsed headers not only hide their contents, they hide themselves... d'oh)
the hide trivial/hide uncraftable are in that pulldown because i need to rework their buttons but i didn't want to leave them completely absent from the interface. i'll be doing a slider that lets you filter out trivial/easy/medium and a toggle for hiding items that you don't have on your (maybe with an option to hide the ones you don't have on you nor in your bank).
got the newest one, cleared everything and so far so good. Haven't tried on my engy just yet, but will get to it soon.
So far I have had only one error, though it still went through the process.
Error:
Error: attempt to call method 'ScanInventory' (a nil value)
AddOn: Skillet
File: MerchantWindow.lua
Line: 242
Count: 1
I had a queue for some tendor wold steaks, I had more steaks than soothing spices, it showed that I was -9 short, cool. Went to the cooking vendor, saw the Buy Reagent button, i just now noticed a tooltip there, is that something new or am i just blind? Anyway, I like it. But back to the error. I click on the buy reagents and then it pops up, though I was still able to purchase the reagents that way.
hmm... sounds like a simple syntax error. i'll look into it. hmm.. probably something i added to make sure to update your inventory/shopping list when you purchase stuff from a vendor (otherwise the buy reagents button will keep buying stuff over and over if you keep clicking it). ideally when you buy reagents, the buy button should go away.
ive just logged on on an jewelcrafting alt, and unfortunately skillet doesnt seem to work at all properly. for some reason it seems to ignore some of the crafting recipes altogether,but in a weird way. the show up on the left side of the panel in the list, but when you try to click on them it shows 'unknown' or a recipe thats listed above it in the right side of the panel. i hope that makes sense
EDIT:
I think i know whats causes it. it seems when i learnt a new recipe it put the list out of sync, relogging seems to sort it
ive just logged on on an jewelcrafting alt, and unfortunately skillet doesnt seem to work at all properly. for some reason it seems to ignore some of the crafting recipes altogether,but in a weird way. the show up on the left side of the panel in the list, but when you try to click on them it shows 'unknown' or a recipe thats listed above it in the right side of the panel. i hope that makes sense
EDIT:
I think i know whats causes it. it seems when i learnt a new recipe it put the list out of sync, relogging seems to sort it
hmm... that's not supposed to happen at all. :(
clicking "rescan" should fix it. i'll see if i can reproduce that one. did you happen to learn a bunch of recipes at the same time? cuz i've seen the timing on events be a little bit off when that happens...
edit: okay, i think i understand what's going on. i've changed the system used to display the list of recipes and it appears i've failed to account for skill index changes there (they're accounted for elsewhere).
I have a question.
When I open my enchanters window, it does not change to the enchant button, but stays with the queue buttons. Is this how you intend it to work?
If so, then when you have a enchant in queue, Skillet will not retrieve mats from the bank.
I have a question.
When I open my enchanters window, it does not change to the enchant button, but stays with the queue buttons. Is this how you intend it to work?
yes, this is the intended behavior. tho rare, there are some enchanting recipes that interact with other tradeskills. plus there are more and more items created by enchanting. seems that enchanting should be treated like all the other tradeskills.
If so, then when you have a enchant in queue, Skillet will not retrieve mats from the bank.
Thanks,
can you get mats from the bank with other skills? do you get a shopping list with enchant mats in it?
Yes, I get the shopping list at the bank, but when I click the retrieve button, nothing happens.
After testing on my other characters and with other professions I am having the same problem. I can not retrieve items from the bank( from the shopping list), I don't get any errors just nothing happens.
Also, while testing this with my other professions, the queue all and create all button do not work, just the Queue and create on the right work. ( you may already know about this).
A screen shot will not help much but maybe you can see something.
i created several different flasks yesterday, queued them all up and pressed start.
since some patch in the past we have to press abutton each time we create a different item.
When i finished creating fortificationflasks the button changed to pause, then i pressed this button, it greyed out. to go on producing flasks i had to select the next item in the list and press start again. Why not just pop a " press here to go on" button?
i created several different flasks yesterday, queued them all up and pressed start.
since some patch in the past we have to press abutton each time we create a different item.
When i finished creating fortificationflasks the button changed to pause, then i pressed this button, it greyed out. to go on producing flasks i had to select the next item in the list and press start again. Why not just pop a " press here to go on" button?
yeah, i think i just need to slip an update in there somewhere to refresh those buttons properly. the reason you have to press a button for each item is that blizzard requires hardware input to start a tradeskill task since like patch 2.0.
the really sucky part is that there's no repeatability in enchanting, so if you want to create 5 wizard oils, you have to click go 5 times. :(
This currently breaks Beast Training (hunters training pets new skills); the following error pops up when hitting the ability (and beast training window does not show up):
Message: ..\AddOns\Skillet\Skillet.lua line 718:
attempt to concatenate field 'currentTrade' (a nil value)
Debug:
...ce\AddOns\AutoBar\libs\AceEvent-2.0\AceEvent-2.0.lua:299: TriggerEvent()
...ce\AddOns\AutoBar\libs\AceEvent-2.0\AceEvent-2.0.lua:910:
...ce\AddOns\AutoBar\libs\AceEvent-2.0\AceEvent-2.0.lua:903
[C]: CastSpell()
..\FrameXML\SpellBookFrame.lua:275: SpellButton_OnClick()
[string "*:OnClick"]:4:
[string "*:OnClick"]:1
This currently breaks Beast Training (hunters training pets new skills); the following error pops up when hitting the ability (and beast training window does not show up):
Message: ..\AddOns\Skillet\Skillet.lua line 718:
attempt to concatenate field 'currentTrade' (a nil value)
Debug:
...ce\AddOns\AutoBar\libs\AceEvent-2.0\AceEvent-2.0.lua:299: TriggerEvent()
...ce\AddOns\AutoBar\libs\AceEvent-2.0\AceEvent-2.0.lua:910:
...ce\AddOns\AutoBar\libs\AceEvent-2.0\AceEvent-2.0.lua:903
[C]: CastSpell()
..\FrameXML\SpellBookFrame.lua:275: SpellButton_OnClick()
[string "*:OnClick"]:4:
[string "*:OnClick"]:1
hmm... okay, well that sucks. i'll make sure to fix that...
are there any errors being reported? (make sure you have enabled error logging in the blizzard options)
how are you accessing your tradeskills? normal action bar stuff or some other addon?
Got no errors (I usually get the auctioneer error window whenever I get addon errors so I assume it is enabled), I am accessing through the action bar (Using Bongos 3). No errors, I just don't get any skillet window but the addon is loaded as I can access all it's options and config window, I just moved from the original skillet and that opened fine so I am not sure what the problem is.
I download the skillet zip, unzipped into addon folder and unzipped the Libs zip into the skillet folder so I assume that was correctly done.
Might wanna go into your WTF folder and backup the saved variable files related to the original Skillet, for cya's sake. The delete them, as well as the Skillet addon itself.
That way you install with a fresh version. I had to do that myself inorder for it to work properly.
Good Luck
P.s.- Even though this version is still a little buggy it is way worth it :)
and to LilSparky....still waiting for the button to automatically add 50g to my total everytime I hit it. Need some G for my epic mount lol
The Skillet cannot cope with newly learned recipes very well. Shift-Rescan is the only workaround for this problem in the game. After learning a new Engineering recipe my recipe list becomes desynced with the main view as seen on the attached screenshots.
First screenshot shows Sniper Scope (the new recipe learned) selected, but Deadly Scope displayed...
The second screenshot shows Inlaid Mithril Cylinder Plans selected, but nothing is displayed...
Also to note: the recipe sorting is still behaving wierdly: the "Miscellaneous 2" category in Engineering is jumping around the recipe list - sometimes it is in the middle of the list, sometimes it is the first group displayed.
ATTACHMENTS
WoWScrnShot_051108_032208
WoWScrnShot_051108_032219
Rollback Post to RevisionRollBack
To post a comment, please login or register a new account.
yeah, well, before i commit to trunk, i hope to have all the saved variable stuff worked out. this is a development branch, so expect a bumpy ride. :)
i found that error with the headers after i fired off this version, but i've already fixed it. for now, if you click rescan i think it'll replace those headers (all that's going on is that collapsed headers not only hide their contents, they hide themselves... d'oh)
the hide trivial/hide uncraftable are in that pulldown because i need to rework their buttons but i didn't want to leave them completely absent from the interface. i'll be doing a slider that lets you filter out trivial/easy/medium and a toggle for hiding items that you don't have on your (maybe with an option to hide the ones you don't have on you nor in your bank).
headers fixed, inventory should be much happier, probably some other things i can't remember.
So far I have had only one error, though it still went through the process.
Error:
Error: attempt to call method 'ScanInventory' (a nil value)
AddOn: Skillet
File: MerchantWindow.lua
Line: 242
Count: 1
I had a queue for some tendor wold steaks, I had more steaks than soothing spices, it showed that I was -9 short, cool. Went to the cooking vendor, saw the Buy Reagent button, i just now noticed a tooltip there, is that something new or am i just blind? Anyway, I like it. But back to the error. I click on the buy reagents and then it pops up, though I was still able to purchase the reagents that way.
Thought you'd like to hear about it :)
awesome work!
and yeah, the tooltip was added in my branch.
ive just logged on on an jewelcrafting alt, and unfortunately skillet doesnt seem to work at all properly. for some reason it seems to ignore some of the crafting recipes altogether,but in a weird way. the show up on the left side of the panel in the list, but when you try to click on them it shows 'unknown' or a recipe thats listed above it in the right side of the panel. i hope that makes sense
EDIT:
I think i know whats causes it. it seems when i learnt a new recipe it put the list out of sync, relogging seems to sort it
hmm... that's not supposed to happen at all. :(
clicking "rescan" should fix it. i'll see if i can reproduce that one. did you happen to learn a bunch of recipes at the same time? cuz i've seen the timing on events be a little bit off when that happens...
edit: okay, i think i understand what's going on. i've changed the system used to display the list of recipes and it appears i've failed to account for skill index changes there (they're accounted for elsewhere).
When I open my enchanters window, it does not change to the enchant button, but stays with the queue buttons. Is this how you intend it to work?
If so, then when you have a enchant in queue, Skillet will not retrieve mats from the bank.
Thanks,
yes, this is the intended behavior. tho rare, there are some enchanting recipes that interact with other tradeskills. plus there are more and more items created by enchanting. seems that enchanting should be treated like all the other tradeskills.
can you get mats from the bank with other skills? do you get a shopping list with enchant mats in it?
After testing on my other characters and with other professions I am having the same problem. I can not retrieve items from the bank( from the shopping list), I don't get any errors just nothing happens.
Also, while testing this with my other professions, the queue all and create all button do not work, just the Queue and create on the right work. ( you may already know about this).
A screen shot will not help much but maybe you can see something.
i created several different flasks yesterday, queued them all up and pressed start.
since some patch in the past we have to press abutton each time we create a different item.
When i finished creating fortificationflasks the button changed to pause, then i pressed this button, it greyed out. to go on producing flasks i had to select the next item in the list and press start again. Why not just pop a " press here to go on" button?
hmm.. i've had some other reports of bank retrieval not working, so something must be going south here... i'll take a look.
yeah, i think i just need to slip an update in there somewhere to refresh those buttons properly. the reason you have to press a button for each item is that blizzard requires hardware input to start a tradeskill task since like patch 2.0.
the really sucky part is that there's no repeatability in enchanting, so if you want to create 5 wizard oils, you have to click go 5 times. :(
I can go into the addon configuration window or config it through chat commands but I can't seem to use it as I only get the blizzard frames.
that's odd.
are there any errors being reported? (make sure you have enabled error logging in the blizzard options)
how are you accessing your tradeskills? normal action bar stuff or some other addon?
Message: ..\AddOns\Skillet\Skillet.lua line 718:
attempt to concatenate field 'currentTrade' (a nil value)
Debug:
...ce\AddOns\AutoBar\libs\AceEvent-2.0\AceEvent-2.0.lua:299: TriggerEvent()
...ce\AddOns\AutoBar\libs\AceEvent-2.0\AceEvent-2.0.lua:910:
...ce\AddOns\AutoBar\libs\AceEvent-2.0\AceEvent-2.0.lua:903
[C]: CastSpell()
..\FrameXML\SpellBookFrame.lua:275: SpellButton_OnClick()
[string "*:OnClick"]:4:
[string "*:OnClick"]:1
hmm... okay, well that sucks. i'll make sure to fix that...
Got no errors (I usually get the auctioneer error window whenever I get addon errors so I assume it is enabled), I am accessing through the action bar (Using Bongos 3). No errors, I just don't get any skillet window but the addon is loaded as I can access all it's options and config window, I just moved from the original skillet and that opened fine so I am not sure what the problem is.
I download the skillet zip, unzipped into addon folder and unzipped the Libs zip into the skillet folder so I assume that was correctly done.
Might wanna go into your WTF folder and backup the saved variable files related to the original Skillet, for cya's sake. The delete them, as well as the Skillet addon itself.
That way you install with a fresh version. I had to do that myself inorder for it to work properly.
Good Luck
P.s.- Even though this version is still a little buggy it is way worth it :)
and to LilSparky....still waiting for the button to automatically add 50g to my total everytime I hit it. Need some G for my epic mount lol
First screenshot shows Sniper Scope (the new recipe learned) selected, but Deadly Scope displayed...
The second screenshot shows Inlaid Mithril Cylinder Plans selected, but nothing is displayed...
Also to note: the recipe sorting is still behaving wierdly: the "Miscellaneous 2" category in Engineering is jumping around the recipe list - sometimes it is in the middle of the list, sometimes it is the first group displayed.