Addons sellings junk/grey items

[kawe]

since early wod almost every auto seller addon/script i used, doesnt work properly anymore. Every time when i have my bags full and open to repair/sell i have to reclick several times to vendor everything in my bag. Is there any addon that does work?

[Yukyuk]

I use Autovendor, and as far as I know its up to date and working.

[JDoubleU00]

I believe this still works http://www.wowinterface.com/download...unkSeller.html. You might need to get a version from his GIT. Tekkub's addons seem to work very well for a long time.

[kawe]

thx for reply's

tried autovendor but it also got stuck half way through selling with my bags full

Lea, i got the new version installed, works as intended

[Nimhfree]

The following code I use to sell junk automatically. It has not failed me.

--	Create a simple frame that allows grey quality items to be sold automatically
local sellFrame = CreateFrame("Frame")
sellFrame:SetScript("OnEvent", function()
		for bag = 0, 4 do
			for slot = 1, GetContainerNumSlots(bag) do
				local _, _, locked, _, _, _, link = GetContainerItemInfo(bag, slot)	-- will return -1 for quality a lot
				if nil ~= link then
					local _, _, realQuality, _, _, _, _, _, _, _, vendorPrice = GetItemInfo(link)
					if 0 == realQuality and 0 ~= vendorPrice and not locked then
						UseContainerItem(bag, slot)
					end
				end
			end
		end
	end)
sellFrame:RegisterEvent("MERCHANT_SHOW")

[VincentSDSH]

Originally Posted by kawe since early wod almost every auto seller addon/script i used, doesnt work properly anymore. Every time when i have my bags full and open to repair/sell i have to reclick several times to vendor everything in my bag. Is there any addon that does work?

Peddler has never failed me. It also visually tags what'll be sold, which I'm particularly fond of.

[SDPhantom]

Originally Posted by VincentSDSH Peddler has never failed me. It also visually tags what'll be sold, which I'm particularly fond of.

The default UI already marks junk items with a coin icon when you're at a vendor.



Found in ContainerFrame.lua line 516:

itemButton.JunkIcon:Hide();
if (quality) then
	if (quality >= LE_ITEM_QUALITY_COMMON and BAG_ITEM_QUALITY_COLORS[quality]) then
		itemButton.IconBorder:Show();
		itemButton.IconBorder:SetVertexColor(BAG_ITEM_QUALITY_COLORS[quality].r, BAG_ITEM_QUALITY_COLORS[quality].g, BAG_ITEM_QUALITY_COLORS[quality].b);
	else
		itemButton.IconBorder:Hide();
	end

	if (quality == LE_ITEM_QUALITY_POOR and not noValue and MerchantFrame:IsShown()) then
		itemButton.JunkIcon:Show();
	end
else
	itemButton.IconBorder:Hide();
end

[VincentSDSH]

Originally Posted by SDPhantom The default UI already marks junk items with a coin icon when you're at a vendor.

The what? Do you have any idea how many years it's been since I've seen the default UI? 'Sides, Peddler's icon is prettier than a coin

[SDPhantom]

Ya, I still have most if not all of the default UI still active, though I've modified quite a bit of it myself.

[sezz]

I use UI_ERROR_MESSAGE to check for ERR_OBJECT_IS_BUSY while selling and retry selling the rest on PLAYER_MONEY, works fine.

[SDPhantom]

This is a more automated version of the script I use on slash command. It uses BAG_UPDATE_DELAYED to retry selling items as it fires once after all BAG_UPDATE events are done.

local EventFrame=CreateFrame("Frame");
EventFrame:RegisterEvent("MERCHANT_SHOW");
EventFrame:RegisterEvent("MERCHANT_CLOSED");
EventFrame:SetScript("OnEvent",function(self,event)
    if event=="MERCHANT_SHOW" then
        self:RegisterEvent("BAG_UPDATE_DELAYED");
    elseif event=="MERCHANT_CLOSED" then
        self:UnregisterEvent("BAG_UPDATE_DELAYED");
        return
    end
 
    local rescan=false;
    ClearCursor();
    for i=0,NUM_BAG_SLOTS do
        for j=1,GetContainerNumSlots(i) do
            local _,_,_,quality,_,_,_,_,novalue=GetContainerItemInfo(i,j);
            if quality==LE_ITEM_QUALITY_POOR and not novalue then
                rescan=true;
                PickupContainerItem(i,j);
                PickupMerchantItem();
            end
        end
    end
 
    if not rescan then self:UnregisterEvent("BAG_UPDATE_DELAYED"); end
end);