[Feature Request] Add sell commands

[biship]

Issue type *

Improvement

Addon version

ef393fc

WoW client version

3.3.5a

Server/AzerothCore version

e867443c4ef696e8ee75fe8b027c2f87afeb50ab

Description *

Can you please add the sell commands as button(s):
s *	sell all grey items
s vendor	sell all items that can be sold
https://github.com/mod-playerbots/mod-playerbots/wiki/Playerbot-Commands
Idealy a button telling all bots to do so, as well as buttons for each bot. 
Thank you for your hard work!

Steps to reproduce

Expected behavior *

1. Select vendor
2. Click All Bots Sell button (all bots sell)
3. Click Bot Sell Button (individual bot sells)

Actual behavior / Errors

No response

Crashlog / Gist link (optional)

No response

Screenshots

No response

Pre-submission checklist *

• I searched for existing issues that match mine.
• I tested with the minimum set of addons/changes.
• I provided reproduction steps and/or relevant logs.

Additional context

No response

[Wishmaster117]

I will take a look at that :)

[Wishmaster117]

@biship , done: fbe847d
Please test.
If all ok i add this : "Idealy a button telling all bots to do so, as well as buttons for each bot."

[biship]

Yes, both buttons work as expected, nice work.

I have an item in my server ac items table in the world db, but havent added that item to my clients Item.dbc file.
This prevents the error in your code (i know you code isn't in error, but this covers my scenario).

-- Returns a safe texture path, falling back when missing.
-- If the provided texture name doesn't start with "Interface",
-- it is treated as an Icons name and prefixed with "Interface/Icons/".
-- When nil or empty, use an addon-provided generic icon.
MultiBot.SafeTexture = function(tex)
	if type(tex) ~= "string" or tex == "" then
		return "Interface\\AddOns\\MultiBot\\Icons\\reward.blp"
	end
	if string.sub(tex, 1, 9) ~= "Interface" then
		return "Interface/Icons/" .. tex
	end
	return tex
end

and then change all settexture to be:
button.icon:SetTexture(MultiBot.SafeTexture(pTexture))

[Wishmaster117]

Yes, both buttons work as expected, nice work.

I have an item in my server ac items table in the world db, but havent added that item to my clients Item.dbc file. This prevents the error in your code (i know you code isn't in error, but this covers my scenario).

-- Returns a safe texture path, falling back when missing.
-- If the provided texture name doesn't start with "Interface",
-- it is treated as an Icons name and prefixed with "Interface/Icons/".
-- When nil or empty, use an addon-provided generic icon.
MultiBot.SafeTexture = function(tex)
	if type(tex) ~= "string" or tex == "" then
		return "Interface\\AddOns\\MultiBot\\Icons\\reward.blp"
	end
	if string.sub(tex, 1, 9) ~= "Interface" then
		return "Interface/Icons/" .. tex
	end
	return tex
end

and then change all settexture to be: button.icon:SetTexture(MultiBot.SafeTexture(pTexture))

Is not you are suggesting in the PR #72 ?

[biship]

oh yeah lol. I thought I lost my PR and was to lazy to create another so just put it here.

[Wishmaster117]

oh yeah lol. I thought I lost my PR and was to lazy to create another so just put it here.

Just correct the two things and i can merge it.

[Wishmaster117]

Added Global button:
71208c4