Detailed changes for version 1.025

added a condition to check if the unit given is not null, as you cannot use gsub with a null argument. If it is null, we return null.

Code:

if unit then
     local playerName, playerRealm = UnitFullName("player");

     local senderName = string.gsub(unit, "-.*","");
     local senderRealm = string.gsub(unit, ".*-","")

     if senderRealm == playerRealm and senderName == playerName then
            return playerName;
      end

     return unit;
end
return unit;

NOTICE : the first return, before end, can be deleted, to be optimized in next version

same idea, new condition to check if the name is not null before doing anything

Code:

if name then
     return string.gsub(name, "-.*","");
     end
     return name;

A lot of changes to make it work in contexts where the unit passed is « mouseover » or « target », and also will only try to parse the full unite name (Name-Realm) if we can find a - .

Code:

-- if the unit passed is a unitID then we use that to determine the name and realm of that unit
     if unit == "mouseover" or unit == "target" or unit == "player" then
          name, realm = UnitName(unit);
     elseif string.find(unit,"-") then
     -- If it's not, we are parsing the unit name to separate the name from the realm
          name = string.gsub(unit, "-.*","");
          realm = string.gsub(unit, ".*-","")
     else
          name = unit;
     end

     -- If the realm is not defined then the unit is from our realm so we define it ourself
     if not realm or realm == "" then
          realm = GetRealmName():gsub("%s+", "");
     end

     if name and realm then
          unit = name.."-"..realm;
     end

     return unit;

New function used to get the full names in chat messages.
Blizzard is being inconsistent… Chat messages in /say and /yell have their sender formatted as Player-Realm, but emotes has they sender sent as Player only if the realm is not necessary in this context. This function uses the GUID given by any message (a unique random identifier for each player) to get the sender’s name and realm.

Code:

-- Added by Ellypse
-- Used for getting the realm of the player via chat messages
function TRP2_UnitNameWithRealmByGUID(guid)
     className, classId, raceName, raceId, gender, name, realm = GetPlayerInfoByGUID(guid);
     if not realm or realm == "" then
          realm = GetRealmName():gsub("%s+", "");
     end
     return name.."-"..realm;
end

added

Code:

arg2 = TRP2_UnitNameWithRealmByGUID(arg12);

We are now using the GUID (unique identifier for a unit) to get the name formatted as Player-Realm. To avoid a lot of changes, I assigned the result to arg2, which is used everywhere as the sender name.

changed

Code:

personnage = personnage..TRP2_GetWithDefaut(TRP2_GetInfo(TRP2_SimplifyNameIfPlayer(perso),"Registre",{}),"Prenom",TRP2_RemoveRealm(perso));


for

Code:

prenom = TRP2_GetWithDefaut(TRP2_GetInfo(perso,"Registre",{}),"Prenom",TRP2_RemoveRealm(perso));
personnage = personnage..prenom;

TRP2_SimplifyNameIfPlayer() is not useful here as in emotes your name is never shown if you are the sender, instead you see « You … »
NOTICE : While bugging I made it a two line instruction, but it might be better to go back to a one line instruction later


replaced

Code:

myChatFrame:AddMessage(TRP2_CTS(personnage.."|r "..string.sub(emote,string.len(perso)+2)),info.r,info.g,info.b,info.id);

for

Code:

myChatFrame:AddMessage(TRP2_CTS(personnage.."|r "..string.sub(emote,string.len(Ambiguate(perso,"none"))+2)),info.r,info.g,info.b,info.id);

Ambiguate() is used to format the sender name so it is interactif (click to whisper, right clic…)