Help: string.gmatch

[Sythalin]

Working on autoinvite function for InviteHim!

function IH_AutoInvite(_,_,msg)
	local name
	if strfind(msg, "suggests inviting") then
		if IsRaidLeader() or IsRaidOfficer() or IsPartyLeader() then
			DEFAULT_CHAT_FRAME:AddMessage("is a leader")  --debug
			i = 0
			repeat
				name = string.gmatch(msg, "%a+")
				i = i + 1
				DEFAULT_CHAT_FRAME:AddMessage(name) --debug
			until i == 5
			InviteUnit(name)
		end
	end
	return false
end

local f = CreateFrame("Frame")
f:RegisterEvent("CHAT_MSG_PARTY")
ChatFrame_AddMessageEventFilter("CHAT_MSG_PARTY", IH_AutoInvite)

To clarify what's passing:
msg = <Player> suggests inviting <given name> to the party.

All vars are passing fine and it's popping my debug is a leader, but it doesn't seem to be running the repeat portion at all (name text doesn't appear in chat frame). I need to pull the fourth word (given name) from the passed msg.

[Sythalin]

42 views and NO ONE can help me on this?

[Shirik]

You are using string.gmatch very wrong. It returns an iterator.

Did you mean string.match? I don't even know where to start correcting your code because it makes no sense to use string.gmatch like that, unfortunately.

A typical use of string.gmatch is something like

for name in string.gmatch(msg, "%a+") do
   -- Do something with name
end

[BWarner]

What's that about a g string? O.o

[Animammal]

As long as the format of your msg stays consistent with the only variation being what "given name" would be, and "given name" won't actually have spaces, I think what you want is something like;

name = string.match(msg, "<(%a+)>", 2)

This should return whatever word is in between <> in msg. (the 2 is where in msg to start the search so that it won't return "Player")

And I don't think you need the loop in there at all.

[Vrul]

If you are going to use ChatFrame_AddMessageEventFilter then I don't think you need to create a frame and register an event. That said, I think you are better off creating a frame, registering an event, and setting an OnEvent script for it. ChatFrame_AddMessageEventFilter just seems like it could break your code if someone else made a bad filter.

local frame = CreateFrame('Frame')
frame:RegisterEvent('CHAT_MSG_PARTY')
frame:SetScript('OnEvent', function(_, _, message)
   local name = strmatch(message, "^%a+ suggests inviting (%a+) to the party.$")
   if name and (IsRaidLeader() or IsRaidOfficer() or IsPartyLeader()) then
      InviteUnit(name)
   end
end)

[Slakah]

Why are you registering the event "CHAT_MSG_PARTY" and then using a
ChatFrame_AddMessageEventFilter .

Anyway, I had a look at InviteHim and I guess you want to extract the name from this string

player.." suggests inviting ".. name

?

You would do that using string.match and this pattern should work.

local name = string.match(msg, "[^%s]+ suggests inviting ([^%s]+)")

Hope this helps .


edit: Vrul you have the wrong argument, message is the first arg not the third.

[Vrul]

Originally Posted by Slakah Vrul you have the wrong argument, message is the first arg not the third.

self, event, arg1, ...

[Sythalin]

Originally Posted by Shirik You are using string.gmatch very wrong. It returns an iterator. Did you mean string.match? I don't even know where to start correcting your code because it makes no sense to use string.gmatch like that, unfortunately. A typical use of string.gmatch is something like for name in string.gmatch(msg, "%a+") do    -- Do something with name end

First time I'm really playing with string.crap and I was already pretty sure I was going to be screwing it up from the get-go.

Originally Posted by Slakah Why are you registering the event "CHAT_MSG_PARTY" and then using a ChatFrame_AddMessageEventFilter .

Cause I'm a noob and thought I needed both.

Originally Posted by Slakah Anyway, I had a look at InviteHim and I guess you want to extract the name from this string player.." suggests inviting ".. name ? You would do that using string.match and this pattern should work. local name = string.match(msg, "[^%s]+ suggests inviting ([^%s]+)") Hope this helps .

This is exactly what I'm trying to do. Though I'm not exactly sure how this particular example works, I'll give it a go.

Originally Posted by Slakah edit: Vrul you have the wrong argument, message is the first arg not the third.

Vrul is correct. It was changed.

[Slakah]

Originally Posted by Vrul self, event, arg1, ...

sorry I'm just being dense, I really need to take a look at things twice before saying stuff :P.

Chaos inc:
"[^%s]+" is basically a pattern which means not a space, and I think (if I'm not being silly again) will work better than "%a+" as "%a+" will only work for standard letters and none of the funky ones with accents etc.

Hope this helps .