LFG joining/leaving

[Grimsin]

So im trying to make a button to join and leave the lfg channel but so far no luck. This is what i have so far, some may ask why i dont put leave on right mouse button well... simple something else is on right mousebutton. or is it left? well either way need to get the whole function in one click. I know right now their is some sort of error with the way i tried to link the if's, not sure how to put 2 separated if sets in one button click.

The other and more major error at the moment imo is that if i remove it so it only has the one command to SetLookingForGroup it does setup the LFG page and joins LFG but it does not actually join the LFG channel for what ever reason.

Any thoughts? here is the code i have so far.

if button == 'LeftButton' then
		if (GetNumPartyMembers()==0) then
			SetLookingForGroup(3, 5, 1);
			SetLFGComment("LFG-Channel enabled by GrimUI");
			UIErrorsFrame:AddMessage("LookingForGroup On", 0.0, 1.0, 0.0, 53, 3);
		elseif
			(UnitIsPartyLeader("player")==1) then
			SetLookingForMore(5, 1);
			SetLFGComment("LFG-Channel enabled by GrimUI");
			UIErrorsFrame:AddMessage("LookingForGroup On", 0.0, 1.0, 0.0, 53, 3);
		end
	else
		if (UnitIsPartyLeader("player")==nil) then
			ClearLookingForGroup()
			UIErrorsFrame:AddMessage("LookingForGroup Off", 1.0, 0.0, 0.0, 53, 3);
		elseif
			(n > 0) then
			ClearLookingForGroup()
			UIErrorsFrame:AddMessage("LookingForGroup Off", 1.0, 0.0, 0.0, 53, 3);
		end
    end

[Seerah]

Originally Posted by Grimsin So im trying to make a button to join and leave the lfg channel but so far no luck. This is what i have so far, some may ask why i dont put leave on right mouse button well... simple something else is on right mousebutton. or is it left? well either way need to get the whole function in one click. I know right now their is some sort of error with the way i tried to link the if's, not sure how to put 2 separated if sets in one button click.

Well, right now, you're telling it to join on left click and leave on any other click. Let me fix your lines/indentation to show you that:

if button == 'LeftButton' then
          if (GetNumPartyMembers()==0) then
                    SetLookingForGroup(3, 5, 1);
                    SetLFGComment("LFG-Channel enabled by GrimUI");
                    UIErrorsFrame:AddMessage("LookingForGroup On", 0.0, 1.0, 0.0, 53, 3);
          elseif
                    (UnitIsPartyLeader("player")==1) then
                    SetLookingForMore(5, 1);
                    SetLFGComment("LFG-Channel enabled by GrimUI");
                    UIErrorsFrame:AddMessage("LookingForGroup On", 0.0, 1.0, 0.0, 53, 3);
          end
else
          if (UnitIsPartyLeader("player")==nil) then
                    ClearLookingForGroup()
                    UIErrorsFrame:AddMessage("LookingForGroup Off", 1.0, 0.0, 0.0, 53, 3);
          elseif (n > 0) then          --have no clue what this n variable is, but I can guess what you mean to do...
                    ClearLookingForGroup()
                    UIErrorsFrame:AddMessage("LookingForGroup Off", 1.0, 0.0, 0.0, 53, 3);
          end
end

Here's how your code *should* read:

if button == 'LeftButton' then
          if IsInLFGQueue() then          --http://wowprogramming.com/docs/api/IsInLFGQueue
                    ClearLookingForGroup()          --leave LFG if already in it
                    UIErrorsFrame:AddMessage("LookingForGroup Off", 1.0, 0.0, 0.0, 53, 3)
          else          --if not in it, then join
                    if (GetNumPartyMembers()==0) then          --you're not in a party
                              SetLookingForGroup(3, 5, 1);
                              SetLFGComment("LFG-Channel enabled by GrimUI");
                              UIErrorsFrame:AddMessage("LookingForGroup On", 0.0, 1.0, 0.0, 53, 3);
                    elseif (UnitIsPartyLeader("player")==1) then
                              SetLookingForMore(5, 1);
                              SetLFGComment("LFG-Channel enabled by GrimUI");
                              UIErrorsFrame:AddMessage("LookingForGroup On", 0.0, 1.0, 0.0, 53, 3);
                    end
          end
end

Can't double check it atm, but that'll start you off.

[Grimsin]

oh wow. i had not seen that lfgqueue command that might make all the difference right their.

Well that for sure fixed the issues i had with my if statements. still no luck on it actually joining the channel or not which makes no sense. /join LookingForGroup also does not function the way one would think it should...

[Grimsin]

Okay got it to work, its because i had used the /join command at one point so it was not doing anything. It does join it now without a problem. What it does not do is leave the channel or clear the lfg.....

[Grimsin]

OKay so i think the issue might be with IsInLFGQueue, i think that returns a value right? dont we need to return that value somewhere and do the if on th at? so it reads if IsInLFGQueue's returned value is such n such do this?

[Grimsin]

ah nope IsInLFGQueue() is not valid for this.... because you dont actually join the autojoin queue. i think GetLookingForGroup is what i need to base things on.

[Seerah]

Ah, yep. Skimmed past that one. I was trying to type out my post in the, like, 2 minutes before my next student arrived.

[Grimsin]

What do you teach?

and also added this to it ClearLookingForMore()

i have not figured out how to do the calculation yet to make it check the GetLookingForGroup command yet.... it can return a lot of variables.....

[Grimsin]

This is the line i think that does it in leafLFG im trying to understand how it works....

local function GetLFGStatus()
 	local _, _, _, _, _, _, _, _, _, _, lfg, lfm = GetLookingForGroup()
	return lfg or lfm, lfg, lfm
end

I have no idea though how that line there works haha i know it later goes into play with this line...

if GetLFGStatus() then
			playerDisable = GetGroupStatus()
			Leave()

Leave() is defined with the ClearLookingForGroup() ClearLookingForMore() tags.

Attempting to write something similar as we speak....

[Grimsin]

figured it out still curious what you teach?

[Seerah]

I teach piano and voice lessons privately.

 	local _, _, _, _, _, _, _, _, _, _, lfg, lfm = GetLookingForGroup()

With so many return values, and the ones you want being all the way at the end, you could use select() for readability. Would add an extra function call to your code, but I don't think you'll be calling it often enough to matter.

local lfg, lfm = select(11, GetLookingForGroup())

This gives you returns 11+. Also, I don't think this line

	return lfg or lfm, lfg, lfm

does what you think it does. That line there returns 3 values. 1. lfg or lfm (whichever is valid first, left to right), 2. the value of lfg, and 3. the value of lfm. But you're trying to use it as just a basic check. Instead, do this:

local lfg, lfm = select(11, GetLookingForGroup())
if lfg or lfm then
			playerDisable = GetGroupStatus()
			Leave()

/edit: I should note that the addon SaneLFG allows a person to be in both LFG *and* LFM.

[Grimsin]

hmm in both at once does it make a difference? really all im after is the lfg channel toggle. something i just noticed to is that if the party is full the code i used does not allow you to turn on the lfg channel. ill post the entirety of it right now you can take a look. i have not tried using the code you just posted above yet. obviously...

-------------------------
--     LFG HotPoint    --
-------------------------
local function GetGroupStatus()
	local raidnum = GetRealNumRaidMembers()
	local partynum = GetRealNumPartyMembers()
	if raidnum > 0 then
		return 'Raid', (raidnum ~= 40) and IsRaidLeader()
	elseif partynum > 0 then
		return 'Party', (partynum ~= 5) and IsPartyLeader'player'
	else
		return 'Solo', true
	end
end
local function GetLFGStatus()
 	local _, _, _, _, _, _, _, _, _, _, lfg, lfm = GetLookingForGroup()
	return lfg or lfm, lfg, lfm
end

local LFGHotPoint = CreateFrame('Button', "LFGHotPoint", UIParent)
LFGHotPoint:SetPoint('TOPRIGHT', GrimUIcoreArtB1, 'TOPRIGHT', -82, 0)
LFGHotPoint:SetHeight(15)
LFGHotPoint:SetWidth(15)
LFGHotPoint:RegisterForClicks('LeftButtonUp', 'RightButtonUp')
LFGHotPoint:SetNormalTexture("none")
LFGHotPoint:SetScript('OnClick', function(self, button)
	if button == "RightButton" then
		if _G.GetCVar("autoLootDefault") == "0" then
			_G.SetCVar("autoLootDefault", "1")
			UIErrorsFrame:AddMessage("AutoLoot On", 0.0, 1.0, 0.0, 53, 3);
		elseif _G.GetCVar("autoLootDefault") == "1" then
			_G.SetCVar("autoLootDefault", "0")
			UIErrorsFrame:AddMessage("AutoLoot Off", 1.0, 0.0, 0.0, 53, 3);
		end
	end
	if button == 'LeftButton' then
					if GetLFGStatus() then
					playerDisable = GetGroupStatus()
					ClearLookingForGroup()          --leave LFG if already in it
					ClearLookingForMore()        --http://wowprogramming.com/docs/api/IsInLFGQueue
                    
                    UIErrorsFrame:AddMessage("LookingForGroup Off", 1.0, 0.0, 0.0, 53, 3)
          else          --if not in it, then join
                    if (GetNumPartyMembers()==0) then          --you're not in a party
                              SetLookingForGroup(1, 5, 1);
                              SetLFGComment("LFG-Channel enabled by GrimUI");
                              UIErrorsFrame:AddMessage("LookingForGroup On", 0.0, 1.0, 0.0, 53, 3);
                    elseif (UnitIsPartyLeader("player")==1) then
                              SetLookingForMore(5, 1);
                              SetLFGComment("LFG-Channel enabled by GrimUI");
                              UIErrorsFrame:AddMessage("LookingForGroup On", 0.0, 1.0, 0.0, 53, 3);
                    end
          end
	end
end)

local function OnEnter(self)
	if self.tooltip then
		GameTooltip:ClearLines()
	else
		GameTooltip:SetOwner(self, 'ANCHOR_TOP', 200, 4)
	end
	GameTooltip:AddLine("|cffeda55fLeft Click|r toggle LFG channel", 0.2, 1, 0.2)
	GameTooltip:AddLine("|cffeda55fRight Click|r toggle AutoLoot", 0.2, 1, 0.2)
	if not self.tooltip then
		GameTooltip:Show()
		self.tooltip = true
	end
end
LFGHotPoint:SetScript('OnEnter', OnEnter)

LFGHotPoint:SetScript('OnLeave', function(self)
	GameTooltip:Hide()
	self.tooltip = nil
end)