Transforming an existing addon

[Dzib]

Doesnt work for me....

[Xubera]

post your .toc and your .lua file

you could just copy and paste it, but put the toc and lua in different [code] tags.

[Phanx]

You should have:

AddOns
  - NameOfAddOn
      - NameOfAddOn.toc
      - SomeFileName.lua

Remember that your folder name and TOC file name must match exactly. You can change "NameOfAddOn" to whatever you want, as long as it matches. The name of the Lua file does not matter, as long as you call it correctly in your TOC file.

Your TOC file should look like this:

## Interface: 40000
## Title: Name Of Addon

SomeFileName.lua

Your Lua file should look like this:

local channelsToProcess = {
	["GUILD"] = true,
	["SAY"] = true,
	["YELL"] = true,
}

local old = SendChatMessage

local new = function(message, channel, ...)
	if channel and channelsToProcess[channel] and message:len() > 0 and not message:match("^/") then
		message = message:replace("r", "rr"):replace("rrrr", "rr"):replace("R", "RR"):replace("RRRR", "RR")
	end
	old(message, channel, ...)
end

SendChatMessage = new

SLASH_RRR1 = "/rrr"

SlashCmdList.RRR = function()
	if SendChatMessage == new then
		SendChatMessage = old
		DEFAULT_CHAT_FRAME:AddMessage("RRR addon disabled.")
	else
		SendChatMessage = new
		DEFAULT_CHAT_FRAME:AddMessage("RRR addon enabled.")
	end
end

If that still doesn't work, install the addon BugSack and post the error message(s) being triggered.

[Dzib]

Got it:

local new = function(message, channel, ...)
	if channel and channelsToProcess[channel] and message:len() > 0 and not message:match("^/") then
		message = message:replace("r", "rr"):replace("rrrr", "rr"):replace("R", "RR"):replace("RRRR", "RR")
	end
	old(message, ...)
end

Thanks Bugsack and thanks to all of you

One last question: you think it would be possible to leave the text between () or ** unchanged? (for RP purpose )


Oups: there's a mistake when I hit enter to send a message:

1x SendChatMessage(): Unknown chat type:
<in C code>: ?
<in C code>: ?
Rrr\Rrr.lua:13: in function `SendChatMessage'
Interface\FrameXML\ChatFrame.lua:3763: in function `ChatEdit_SendText':
Interface\FrameXML\ChatFrame.lua:3775: in function `ChatEdit_OnEnterPressed':
<string>:"*:OnEnterPressed":1: in function <[string "*:OnEnterPressed"]:1>

Locals:
(*temporary) = "bonjourr"
(*temporary) = "Orc"

[Xubera]

the 2nd temp value is "Orc"... is the language not called "Orcish"?

[Dzib]

Orc in french, Orcish in english

[Xubera]

oh i see

old(message, ...)

its suppose to be
old(message,channel,...)

its passing "Orc" as the chatType instead of channel.

[Dzib]

Yes, it works
Thanks a lot

you think it would be possible to leave the text between () or ** unchanged? (for RP purpose )

[Phanx]

Originally Posted by Dzib you think it would be possible to leave the text between () or ** unchanged? (for RP purpose )

If your RP messages always begin with a ( or a *, then that's easy:

local new = function(message, channel, ...)
	if channel and channelsToProcess[channel] and message:len() > 0 and not message:match("^[%(*/]") then
		message = message:replace("r", "rr"):replace("rrrr", "rr"):replace("R", "RR"):replace("RRRR", "RR")
	end
	old(message, channel, ...)
end

If you're inserting () and ** wrapped text in the middle of messages, that's a bit trickier...

[Ketho]

Originally Posted by Phanx If your RP messages always begin with a ( or a *, then that's easy: not message:match("^[%(*/]") If you're inserting () and ** wrapped text in the middle of messages, that's a bit trickier...

.. everytime I see one of those patterns, I get the feeling pattern matching is the craziest thing about Lua itself.
It looks like leetspeak to me but then for Lua gods ><

[Xubera]

don't fret, Regular Expressions was invented by the devil himself to torture anyone who looks at it. If you look directly at it, rumor has it you turn to stone.


anyways...
"^[%(%*/]"
lets break it down

^ is a character that says match this only if its at the BEGINNING of the string.
so "^a" would match apple, but not bear

"%" is an escape character, you usually see stuff like "%d" for integers or "%w" for alphanumerics. but some characters are special like () and *, and they need to be escaped to match them, %( and %* mean they are actually looking for those characters.
"%(hello%)" would match (hello) but not hello

"[ ]" are character sets, as in, the character must match only one of the characters in here. As earlier in this example [Rr] means that it must either be a "R" or a "r"
"[AB][pe][pa][lr]" would match "Appl" and "Bear"

so finally, we mush it all together
"^[%(%*/]" means if either ( or * or / are found at the beginning of the string, we found a match!
"/help" finds a match
"*me" finds a match
"() Hello" finds a match
"The equation is 12/4" does not find a match because the '/' was not at the beginning

[Dzib]

Originally Posted by Phanx If you're inserting () and ** wrapped text in the middle of messages, that's a bit trickier...

That was the idea but anyway, it's already very kind of you to help me so much, Thanks

[Ketho]

Originally Posted by Xubera don't fret, Regular Expressions was invented by the devil himself to torture anyone who looks at it. If you look directly at it, rumor has it you turn to stone. anyways... "^[%(%*/]" lets break it down [...]

Thanks Xub, for the clear explanation on the pattern/regex! (n_n)
Still wondering though, why Phanx included the slash /, is it maybe also used for Out of Character messages?

[Xubera]

Originally Posted by Ketho Thanks Xub, for the clear explanation on the pattern/regex! (n_n) Still wondering though, why Phanx included the slash /, is it maybe also used for Out of Character messages?

because it would turn

"/reload" into "/rreload" and then the chat frame would be like "no such command"

"/e turns around" still gets turned to "/e turrns arround" because when you type "/e " you dont send the "/e" it gets parsed out automatically while typing (it changes the font to emote color (orange by default) and changes the chatType to emote.

[Dzib]

Hi there

After quitting WoW for a long time, I went back a few weeks ago and wanted to use this little addon but it doesn't work anymore.... I don't really understand why. Here's the code used:

local channelsToProcess = {
	["GUILD"] = true,
	["SAY"] = true,
	["YELL"] = true,
}

local old = SendChatMessage

local new = function(message, channel, ...)
	if channel and channelsToProcess[channel] and message:len() > 0 and not message:match("^[%(*/]") then
		message = message:replace("r", "rr"):replace("rrrr", "rr"):replace("R", "RR"):replace("RRRR", "RR")
	end
	old(message, channel, ...)
end

SendChatMessage = new

SLASH_RRR1 = "/rrr"

SlashCmdList.RRR = function()
	if SendChatMessage == new then
		SendChatMessage = old
		DEFAULT_CHAT_FRAME:AddMessage("RRR addon disabled.")
	else
		SendChatMessage = new
		DEFAULT_CHAT_FRAME:AddMessage("RRR addon enabled.")
	end
end

And here's the error message bugsack gives me:

5x Teuto\teuto.lua:11: attempt to call method 'replace' (a nil value)
Interface\FrameXML\ChatFrame.lua:3864: in function `ChatEdit_SendText':
Interface\FrameXML\ChatFrame.lua:3876: in function `ChatEdit_OnEnterPressed':
<string>:"*:OnEnterPressed":1: in function <[string "*:OnEnterPressed"]:1>

nil

Locals:
editBox = ChatFrame1EditBox {
 0 = <userdata>
 at3matches = <table> {
 }
 focusLeft = ChatFrame1EditBoxFocusLeft {
 }
 focusRight = ChatFrame1EditBoxFocusRight {
 }
 chatFrame = ChatFrame1 {
 }
 addSpaceToAutoComplete = true
 lDrag = <unnamed> {
 }
 focusMid = ChatFrame1EditBoxFocusMid {
 }
 MAParent = "ChatEditBoxesMover"
 at3curMatch = 0
 text = "/"
 hookedByAceTab3 = true
 setText = 0
 header = ChatFrame1EditBoxHeader {
 }
 tabCompleteIndex = 1
 headerSuffix = ChatFrame1EditBoxHeaderSuffix {
 }
 frame = <unnamed> {
 }
 language = "Commun"
 rDrag = <unnamed> {
 }
}
addHistory = 1
type = "SAY"
text = "bonjour"

[Nibelheim]

The replace function was removed in 4.2, so someone will have to rewrite that particular line for you

[Haleth]

I think it was replaced (derp) by gsub, though I'm not sure if it does exactly the same.

[Dzib]

Works with gsub... thanks

[Dzib]

I'd like to have the addon working also on a "RP" channel (which is number 7), so I've add

local channelsToProcess = {
["GUILD"] = true,
["SAY"] = true,
["YELL"] = true,
["RP"] = true,

but that doesn't work....

[Nibelheim]

Originally Posted by Dzib I'd like to have the addon working also on a "RP" channel (which is number 7), so I've add local channelsToProcess = { ["GUILD"] = true, ["SAY"] = true, ["YELL"] = true, ["RP"] = true, but that doesn't work....

It probably relies on proper channel names (GUILD, SAY, PARTY, etc), or channel numbers for non-recognized channels. Would have to look at how the addon does it.