Buy from a vendor the first time

[Xrystal]

Excellent.

It is a learning curve. I've been writing addons since 2008 I think, maybe earlier but there are still areas I haven't even looked at ( including the stuff I needed to write this test addon rofl ) and with Blizzard changing stuff most expansions, it's time to learn again rofl.

But, seriously, those links I posted before you will find useful. At least bookmark the wiki.gg link I posted as even us veterans visit it when we need to rofl.

[Hubb777]

I add all the topics with my questions to my browser bookmarks. This is a very good forum, with helpful people.

[Hubb777]

Originally Posted by Xrystal Not quite - I wasn't sure what the following was for so my updated version doesn't have it in if IsShiftKeyDown() then return end Give this a go. As a reminder, event watching only frames technically don't need to be named. I have clearly separated the sections of the code you posted so that it makes it easier to see which items you can put in separate files if and when you need to. At the moment it's small enough in the one file. local addonName, addon = ...   --------------------------- -- Vendor Related Set up -- ---------------------------   local vendors = {     ['Arvik'] = {['Skrog Liver Oil'] = true, },     ['Bukarakikk'] = {['Hunk o\' Blubber'] = true, },     ['Erugosa'] = {['Exquisite Ohn\'ahran Potato'] = true, ['Flaky Pastry Dough'] = true, ['Dark Thaldraszian Cocoa Powder'] = true, ['Four-Cheese Blend'] = true, },     ['Gracus'] = {['Greenberry'] = true, ['Fresh Dragon Fruit'] = true, ['Juicy Bushfruit'] = true, ['Dried Coldsnap Sagittate'] = true, },     ['Hanu'] = {['Eye of Bass'] = true, },     ['Head Chef Stacks'] = {['Rations: Scorpid Surprise'] = true, ['Rations: Undermine Clam Chowder'] = true, ['Rations: Westfall Stew'] = true, ['Rations: Dragonbreath Chili'] = true, },     ['Jinkutuk'] = {['Salted Fish Scraps'] = true, },     ['Junnik'] = {['Thousandbite Piranha Collar'] = true, },     ['Elder Nappa'] = {['Nappa\'s Famous Tea'] = true, },     ['Norukk'] = {['Norukk\'s "All-Purpose" Fish Powder'] = true, },     ['Qariin Dotur'] = {['Seven Spices Bruffalon'] = true, ['Dragonflame Argali'] = true, ['Thrice-Charred Mammoth Ribs'] = true, ['"Volcano" Duck'] = true, },     ['Patchu'] = {['Lunker Bits'] = true, },     ['Rokkutuk'] = {['Deepsquid Ink'] = true, },     ['Tattukiaka'] = {['Fermented Mackerel Paste'] = true, },     ['Tikukk'] = {['Island Crab Jerky'] = true, },     ['Tuukanit'] = {['Piping-Hot Orca Milk'] = true, }, }     ----------------------- -- Utility Functions -- -----------------------   local function PrintMessage(msg)     print("[ZAMESTOTV: Community Feast] " .. msg) end   -------------------------------- -- The actual purchasing part -- --------------------------------   local function BuyItemsFromVendor(vendorName)     local vendor = vendors[vendorName]     if not vendor then return end       local numItems = GetMerchantNumItems()     for i = numItems, 1, -1 do         local name = GetMerchantItemInfo(i)         if vendor[name] then             local success = BuyMerchantItem(i)             if success then                 PrintMessage("Purchased: " .. name)             else                 PrintMessage("Failed to purchase: " .. name)             end         end     end end   ---------------------- -- Event Management -- ----------------------   local function AutoPurchaseSelectedItems()     local targetName = UnitName("target")     if not targetName then return end         BuyItemsFromVendor(targetName) end   local function OnEvent(self, event, ...)     if event == "MERCHANT_SHOW" then         self:RegisterEvent("MERCHANT_UPDATE")     elseif event == "MERCHANT_UPDATE" then         self:UnregisterEvent(event)         AutoPurchaseSelectedItems()     end end   local eventWatcher = CreateFrame("Frame") eventWatcher:RegisterEvent("MERCHANT_SHOW") eventWatcher:SetScript("OnEvent", OnEvent)

Hello. During quick testing, I noticed that when purchasing items, it says “Failed to purchase” on all items.

Option 1) - remove these inscriptions.
Option 2) - so that the inscription is correct.

Any option suits me.

[Xrystal]

It would depend on the reason for failing.

Are the correct items per vendor set up correctly ?
Did you investigate and see which ones it didn't purchase ?
Did you check to see if there was a miss spell somewhere that meant it couldn't find the item to purchase ?
Did you run out of currency to purchase the item ?

Finding out why it happened will put you in the direction of identifying what needs to be done. It could be as simple as removing a non existent item, correct a spelling mistake or coding a stop or skip when the required currency is no longer availablw.

[Hubb777]

Originally Posted by Xrystal It would depend on the reason for failing. Are the correct items per vendor set up correctly ? Did you investigate and see which ones it didn't purchase ? Did you check to see if there was a miss spell somewhere that meant it couldn't find the item to purchase ? Did you run out of currency to purchase the item ? Finding out why it happened will put you in the direction of identifying what needs to be done. It could be as simple as removing a non existent item, correct a spelling mistake or coding a stop or skip when the required currency is no longer availablw.

It’s just that the merchant has only 1 item available for sale (the rest will be blocked and not available)
For example

['Head Chef Stacks'] = {['Rations: Scorpid Surprise'] = true, ['Rations: Undermine Clam Chowder'] = true, ['Rations: Westfall Stew'] = true, ['Rations: Dragonbreath Chili'] = true, },

In this example, the merchant has all the items in stock, but can only buy 1.

[Xrystal]

It should already be ignoring the other items.

Also, looking at your line for BuyMerchantItem - the wiki page doesn't say it returns a value, so it will always be false I suspect. This could be the cause of your problem
https://warcraft.wiki.gg/wiki/API_BuyMerchantItem

local function BuyItemsFromVendor(vendorName)
    --  Create a table of items for this vendor's name
    local vendor = vendors[vendorName]   
    -- No items ?  Then don't bother with the rest of the work
    if not vendor then return end               
 
    local numItems = GetMerchantNumItems()
    for i = numItems, 1, -1 do
        local name = GetMerchantItemInfo(i)
        -- Is this item in the list of items for this vendor we want to buy ?
        if vendor[name] then                      
            local success = BuyMerchantItem(i)
            if success then
                PrintMessage("Purchased: " .. name)
            else
                PrintMessage("Failed to purchase: " .. name)
            end
        end
    end
end