Quote:
If you don't mind, ofcourse. |
Yeah the last one attached was the version that didn't seem to crash anymore so give it a whirl again. The version I have been tested just has the changes for the temporary chat frame to test .. but sure go ahead and use what's here. Let me know if something else goes gaga on it or you spot something that isn't doing what you expected. I'll update the thread once I have a proper addon up and running that's been properly tested.
|
Quote:
|
Hmm, my battle tag friends haven't logged in for months .. only have a few real id friends so that's all I've been able to test with. And yeah, I can hear the fireworks outside already. My friends are in the states so I usually get to see them later in the day but today probably not much at all. I'll try and find some battle tag friends amongst the regular guildies and look into that element.
Does it show Battle Tags as the name when chatting to them ? All my friends have names that display so never have seen the battle tags appearing. |
Quote:
If I whisper someone with the battle tag MrExample, it'll show as Quote:
EDIT: Why write it when a picture says a thousand words? EDIT#2: A RealID friend logged in, and I tested it further. I get the same result with RealID friends, no classcolor on the name. The login message is classcolored, through. |
2 Attachment(s)
Hmm, very weird. I don't think I made any more changes since the last post to the messaging. I guess more tests and screenshots. No friends online at the moment so I can't test yet.
Well here's the latest I have with the added attempt at it working in the Temporary Chat Frame as well which the last test it didn't work but tests against 2 of my realID friends after I last posted the files showed their real name/character and realm names were all colored with that character's class. And here's the last screenshot I took when I last uploaded that file. Just in case it is affecting it .. are you using a chat addon that formats the output ? Perhaps it is overriding my changes with its own. |
Xrystal, the main problem with your implmentation (aside from questionable coding practices) is that you're spending CPU and memory re-coloring the same name with the same color every time you get a message. It's more efficient to just update a static list whenever a friend's status changes, so you can just do a simple table lookup for each message, instead of needing to call functions and format strings.
Quote:
Quote:
|
Thanks Phanx,
I appreciate your input regarding my programming practices. I found that usage somewhere in a lua tutorial or another addon somewhere during a hunt to investigate something a few months back and found it useful for my event argument tracking. http://www.lua.org/pil/5.1.html Quote:
At the time when I saw that being demonstrated I felt that 1 line holding the values of whatever arguments that current event needed would be a lot better than several lines of specific values being used. In this instance, however, at least so far, it doesn't look to be needed, and will if it stays that way be rectified. So please don't look at one piece of code and question my programming practice on what is essentially a beta version when you come to think of it. I agree with your notice of the fact I am recoloring every time a chat is sent and received, in my case that is minimal but in the instance of someone playing one toon and chatting significantly I can see the flaw in it. Will look into changing that ready for my next test session. |
Quote:
|
I wouldn't say that it's unreasonable to expect a beginning programmer (who doesn't event know the phrase "best practices") to assume that the official documentation is exemplary in both what one can do and how one should do it. Lord knows I wasn't filtering examples through any kind of sanity filter when I was learning BASIC, HTML and JavaScript back in middle school and high school -- I didn't have a programming sanity filter at that point.
|
3 Attachment(s)
Thanks pelf. I'm not a novice programmer but the only lua programming I have done is with wow, and programming I did 20 years ago is somewhat outdated now, and definitely rusty. The way lua uses tables is different to the way that other programming languages I have used. Hence, why I usually make some changes when I am pointed away from it.
Some of the coding I have seen in addon's blow my head away but I use those to find out how they do what they do which then makes me realise that some of my addons are due a rewrite rofl. Anyway, thanks to Cooper's ( on these forums ) assistance I have managed to rewrite the addon so that as much as possible is done when the player logs in or their friends logs in so that the AddMessage function just has to print it. Please find the new version with all the changes including the temporary chat window part with a couple of screenshots. Let me know if this is working for you now Wimpface. |
It seems to be working fine! I'll do more extensive testing and report back if I find any issues.
Thanks a ton, Xrystal. Saved me from many misstells. :D EDIT: Found a conflict, haste's iiLinks. |
1 Attachment(s)
What happens when that addon is active ? Does what it do get ignored because mine overrides it or does mine get ignored because it overrides mine ?
A quick look at their code it sounds like what is probably happening is that by the time my addon gets to using it's version of AddMessage iiLinks version has changed what message is now there which means the pattern matching and formatting I have is now broken. This probably explains why there are no small addons that do little bits and pieces of the chatframe as it is more likely to be overwritten by others that are also installed and changing the same functionality. Will have to monitor and check to see what happens when someone logs in .. Edit: Well a real id friend logging in message was not affected by the ilinks addon so far. Here is a picture showing the iLinks addon working on an item link and the playerBNLink apparently working fine when my friend logs in. But can see that messages sent messes up ... then I get logged out .. weird. |
Quote:
|
Looks like I need to reverse this block of code :
local AddMessage = function(self, text,...) if(type(text) == "string") then text = text:gsub("|H(.-)|h%[(.-)%]|h", "|H%1|h%2|h") end return origs[self](self, text, ...) end so that my addon is back to using the default player link or add in a check so that it ignores BNPlayer links. And again my friends log off, so will take a break for now and see who is online later before I head off to bed. |
Quote:
|
3 Attachment(s)
Okay, thankfully I have a couple of cool Battle Net friends that are quite happy with me sending them test messages while they run dungeons and stuff rofl, so finally got it figured out and with the help of this thread : http://www.wowinterface.com/forums/s...ad.php?t=38821 I learnt what pattern change I needed to make.
It now works whether iiLinks is loaded or not. New version and screenshots attached. Let me know if this works now for you Wimpface. |
Sorry for the late response, I'll try it out together with iiLinks as soon as a friend logs on.
Don't know how to thank you! EDIT: It works! I found a teeny tiny bug if you're a perfectionist, but it's barely noticeable. I'm super happy right now. The login-message has brackets and isn't classcolored. I'm going to try with different options turned off and on. God what a wonderful person you are. |
Yeah I noticed the login message had stopped being coloured and added it to the list of things to figure out.
I also noticed that for some reason when my friend was toon hopping it wasn't recognising the realmName which was somehow "" rather than nil or a value so it displayed the "-" separator but not the realm itself so changed that, but now it seems that it doesn't show the realm at all, at least for toon hoppers. And the brackets are probably an after affect of the iiLinks addon removing them and me adding them to make my addon work, was hoping iiLinks would remove the ones I added after making my change but maybe not. I didn't notice them at my end but will check it out over the next week or so. Let me know if something crops up over the next few days or so and I'll add it to my list of things to figure out. |
Quote:
Quote:
|
All times are GMT -6. The time now is 04:42 PM. |
vBulletin © 2024, Jelsoft Enterprises Ltd
© 2004 - 2022 MMOUI