I'll see about the vendor item patch. I've been pretty busy and have slipped out of heavy modding lately :)
The vials thing confuses me. I specifically wrote Hemlock to use excess vials (and other reagents that you have in your bags). It properly consumes excess vials for me. The only thing I could think of would be if you bought reagents for 10 Instants, made 8, stopped the process, and started making Deadly. When Hemlock buys reagents for you, it "locks" those reagents so that they can only be used for the poison you bought them for. These locks are released upon closing the vendor window, but it should only ignore materials in that one particular case. If you need 8 instants and Hemlock buys reagents for 8 instants, it should only lock 8 vials. That is, when you're done making all your poisons, you should never have more than four vials left over in your inventory. The four is a byproduct of the 5-minimum purchase quantity, and can't be worked around, but if you're ending up with more than four, there's a problem.
Regarding lower-level poisons, I don't have any plans to allow for lower-level poisons at the moment. Hemlock was written specifically to not require the user to have to think about ranks - I wanted it to be a fire-and-forget, no configuration or setup needed addon. What annoys me most about most other poison addons is that you have to spend 10 minutes setting them up before you can use them.
To this end, Hemlock specifically looks for the highest rank of every poison you have. It wouldn't be difficult to make it look for lower ranks, but it wouldn't be a 30-second fix either.
I've since found a more elegant solution to the vial thing, adding this line under the tomake>0 check around line 503
toMake = floor(toMake/5) * 5 + 5
Basically without this it has a bad habit of buying too many vials because they only sell in 5s, for instance you have 16 instants with a desired of 20, it would buy 5 vials, but 4 for everything else, so you end up with an extra vial in your inventory. With this change it will buy/make a full 5 every time, even if it goes over your desired, I would sooner have the extra poison or 2 I can use, than vials. I'm sure it could be implemented as a toggle, I just didn't get around to thinking it that far in.
With this change it will buy/make a full 5 every time, even if it goes over your desired, I would sooner have the extra poison or 2 I can use, than vials. I'm sure it could be implemented as a toggle, I just didn't get around to thinking it that far in.
Any chance this could be made an option in the core mod? Seems to be nice functionality (I'd use it for example).
Also is the ItemLink fix in place? If so I can start using WowAceUpdater for Hemlock again.
Hemlock stopped working for me as well. /hemlock brings up the command list, but /hemlock scan won't work; it won't even let me press Enter, which is extremely weird.
And I get the same error message others already reported.
I love Hemlock, but on my end atleast, the TOC is still 20000, and that was a few patches ago..
Sure, the addon works fine by turning on "Load out of date addons", but that means non-working out of date addons are also loaded..
Keeping the TOC up to date means the author has verified that the addon is still working, and that the author is still maintaining it.
atm when ever i log into the game hemlock asks me to scan.
when im at the poison vendor no buttons show up to buy
any help would be great
Do you happen to run the CharacterProfiler mod from www.rpgoutfitter.com ? If so, that mod also scans your poisons and conflicts with Hemlock.
Workaround: Open your poison screen and wait for a message from CharacterProfiler that is has scanned succesfully. After that, run /hemlock scan manually and it will be fixed.