The original Auto Restocker mod is abandoned and is stuck in Classic Era, this project updates it for TBC, adds bug fixes.
Restocker
Restocks items on the list automatically (if checked) Will only add items by name that are cached by the client (you must have seen the item during your play session). Only true for adding items, not for items already added. Supports adding ANY item by itemID
Profiles
Full support for profiles. Useful for pve/pvp differences where items you wish to put into bank will need to be set to 0.
Example: PVE profile has all potions/elixirs and their respective number to restock. PVP profile still need to have the potions but with the amount set to 0. When you visit the bank with the PVP profile the addon will put all potions/elixirs into bank and restock on your PVP profile items.
Restocks from
- Vendor
- Bank (Hold shift when opening bank to temp disable restocking)
Full poisons support!
- Will restock poisons from bank.
- Will restock poison reagents if you have no poisons in the bank and below the set number.
/rs or /restocker to toggle window.
Randomly on the latest version it'll not want to stack a full stack of something like food (20) it'll just say "Couldn't split those items" in wow error text.
In reply to kvakvs:
This is happening at the bank, max stack size:
https://media.discordapp.net/attachments/493910348454297610/855609109167341608/unknown.png
In that screenshot it's stuck on the healing potion injector, last night it was stuck on elixir of major defense (but today it stacked them fine), previously it was stuck on blackened sporefish.
19 in bags, 20 in bank of the health injectors. If I push the 20 stack from the bank into my bags, it then throws 19 back into the bank and stops being stuck.
I can't reproduce it reliably, but if I put random stacks of something in both the bags and the bank it'll fetch up from time to time.
Interesting glitch with this version, if I set something greater than 5, it just bounces them between the bank and inventory.
Nice find. I am going to look at this later today. The bug already been filed by you or another person? https://github.com/kvakvs/wow-Restocker/issues/5
Hi.
Can you open the tickets functionality in github, to make bugs easier to follow ?
Done
But please try the beta I uploaded 6 hours ago. 2021.5.3
Published 2 beta versions for Classic and TBC, please try in different languages (tested in German)
version 2021.5.3
I can't open tickets on the github page, so i put it there.
Lua error : (while opening a poison vendor window)
1x RestockerClassic\Poisons.lua:122: bad argument #1 to 'pairs' (table expected, got nil)
[C]: in function `pairs'
RestockerClassic\Poisons.lua:122: in function `getPoisonReagents'
RestockerClassic\Events.lua:95: in function `?'
RestockerClassic\Events.lua:31: in function <RestockerClassic\Events.lua:30>
Locals:
(*temporary) = nil
(*temporary) = "table expected, got nil"
= <function> defined =[C]:-1
In reply to UristMacFrench:
(and it's not buying anything)
tested with bank, it's working fine.
and sometimes it's working fine too with poison vendor, no idea why.
Uploaded a new version please give it a try
In reply to kvakvs:
I got a different error. Weirdly enough, i don't have anything to buy at the poison vendor: my stacks are full. Nonetheless, i got the error.
1x RestockerClassic\Poisons.lua:109: bad argument #1 to 'pairs' (table expected, got nil)
[C]: in function `pairs'
RestockerClassic\Poisons.lua:109: in function `getPoisonReagents'
RestockerClassic\Events.lua:99: in function `?'
RestockerClassic\Events.lua:31: in function <RestockerClassic\Events.lua:30>
Locals:
(*temporary) = nil
(*temporary) = "table expected, got nil"
= <function> defined =[C]:-1
In reply to UristMacFrench:
Actually, i have this error at every merchant in the game.
Edit: i did some more tests, and can confirm: the "bank" part of the addon works fine, the "merchant" part is 100% broken.
In reply to UristMacFrench:
And i did some more test: that's weird. it 100% fail on my rogue, its not failing on my other characters.
Will do some more tests soon on another rogue.
Edit: it's not failing on my other rogue.
Same computer, same addons.
So you have added some poison that is not listed in the poisons for Restocker, first i will fix the actual code to allow such situation to happen if the player wants to restock something that has word "poison" in it which is not a rogue poison. And second i'd like to add the poison you were trying to restock to the list.
What was your poison name? Are you playing non-English game? Because original Restocker mod reacts to English item names and this will be fixed in a future release, it is not ready yet. I had plans to make this work for any language.
In reply to kvakvs:
I play in English.
I add poisons by creating them first, with the components i buy from the poison vendor, then adding them (the poison directly, not the components) in restocker, so they can't be "non-existant".
Also, it works fine on a rogue, and not on the other one... i have no clue why.
I'll try deleting them from restocker and add them again, but TBH, i'm lost so far ^^
In reply to kvakvs:
So, i got news :)
Elixir of poison resistance (it's not a poison, it's a potion) is the culprit... more or less.
If i delete him from restocker, i don't have the error anymore.
But i have it too in restocker on my other rogue, and it don't trigger the error for him.
In reply to UristMacFrench:
Another bit of information: if i have enough "elixir of poison resistance" in my bag, it also don't trigger the error.
Old restocker code reacted to "poison" in the item name. This is bad code and i have changed it yesterday and uploaded a Beta version, please get it and have a look.
[feature request][Classic ERA] no message at login
I use a ton of addons, so each login is a spam fest.
Maybe put the greetings message as an option, if you want to keep it ?
Edit: it seem the greetings message only happen once, at installation. So it's fine with me :)