DoTradeSkill(index, amount) not working for me

[Himar]

Can anyone figure out why this function would be ignoring the value of Amount which is passed to DoTradeSkill()? It makes one potion correctly then just stops, the solution eludes me.

if the slash command given is

/mix 5 greater mana
then
Amount = 5
Potion = 'greater mana'

SlashCmdList['MIX'] = function(msg) MixPotion(msg) end

function MixPotion(msg)
    CastSpellByName('Alchemy')
    Amount, Potion = GetCmd(msg)
    for i = 1, GetNumTradeSkills() do
        if strfind(strlower(GetTradeSkillInfo(i)), strlower(Potion)) then
            DEFAULT_CHAT_FRAME:AddMessage('Mixing '..Amount..'x '..GetTradeSkillInfo(i)..'.')
            DoTradeSkill(i, Amount)
            break
        end
    end
    CloseTradeSkill()
end

function GetCmd(msg) -- thanks to Tigerheart for this code from wowwikki
 	if msg then
 		local a, b, c = strfind(msg, '(%S+)') -- contiguous string of non-space characters
 		if a then return c, strsub(msg, b + 2)
 		else return ""
 		end
 	end
end

[wmrojer]

It's because you close the tradeskill window. Try it manually by makeing 5 and closeing the window before it's done. It will make the one in progress, the quit.

[Himar]

You sir, are my new favourite person. Bang on... I was looking at everything under the sun except that CloseTradeSkill. Thank you!

[MadjackMcMad]

wmrojer: good find!

You may want to actually move the CloseTradeskillWindow() call to right above CastSpellByName('Alchemy'). If you don't, your macro will fumble if the window is already open; the cast will close the window.