if text and SCCN_InChatHighlight == 1 and arg8 ~= 3 and arg8 ~= 4 and string.sub(event, 1 , 8) == "CHAT_MSG" and string.sub(event, 1 , 15) ~= "CHAT_MSG_COMBAT" then
local rWord = "";
for rWord in string.gmatch(text, "%w%w%w+") do
if text:find("%[[%w%s]*"..rWord) == nil and rWord ~= arg2 and string.find(rWord,"|") == nil and string.find(rWord,"^%x*$") == nil and string.find(rWord,"^%a*$") ~= nil then


Changes are:
Searching for “%w%w%w+” makes the “and string.len(rWord) > 3″ redundant.
New condition “text:find(“%[[%w%s]*”..rWord) == nil” makes “SCCN_URLFOUND == nil” and “SCCN_INVITEFOUND == nil” checks redundant (I think, tho I may have broken the invite thing), as well as my “text:find(“%[%w+%s%w”) == nil”. In addition, since this check happens inside the loop, it doesn’t prevent toon name highlighting when items or URLs appear.

I do think the checks after “rWord ~= arg2″ may be redundant, but I’m guessing you have put them in for a reason.

@Morgulo: Class color – as long as you “know” the char – also work on the trade channels. But As said you need to know him. To can all persons in a trade channel can cause hughe problems So I decided to leave this out.

I added the following argument to line 460 in ChatMOD.lua:

and text:find("%[%w+%s%w") == nil

This takes advantage of the fact that whitespace is not allowed in character names, but is almost always present in item names (there’s a few exceptions, like “Glowcap”, but these have much lower incidence).

This change does mean that, if an item is linked along with a toon name, that toon name will not have the highlighting.

——

Very nice addon! One question for the “class clolor”. Does class coloring not work for Trade and LFG channels? Is it possible to add this in a future version? Or is it possible to force a class lookup in any way?

http://solariz.de/gaming/wow-chatmod.htm

I actually arrived at that page first (via a search engine) and thought that the most recent update was from last year!

In any case, thanks for the hard work, keep up the great job!