the flow of events goes like this:

Display Picture is changed
five second timer is triggered
after five seconds, the Display Picture is changed back to the original

I have had no problems getting this to work.
HOWEVER, occasionally a few seconds after it switches back to the original, it will switch again, back to the one it switched to.
ie, image A becomes image B for 5 seconds, turns back to image A but then after a few seconds goes back to image B and stays there until changed.

The fact that it only does this sometimes and not all the time leads me to think that it may not be a problem on my side but lag between the client and server and the server is switching it back, but I'm really not sure.

Does anyone know the true cause of this problem and has anyone found a (reasonable) work around for it?

Post your code.

[code=js][/code]

js code:

---

var DelayLength = 5000;
var FolderWithEmotes = "";
var DParray = new Array(
    "",
    "1.gif",
    "2.gif",
    "3.gif"
);
String.prototype.toArray = function(){
    return this.split(' ');
}

function OnEvent_Timer(timerID)
{
    if(timerID == "DPdelay")
    {
        Messenger.MyDisplayPicture = DParray[0];
    }
}

function OnEvent_ChatWndSendMessage(ChatWnd, Message)
{
    var index = 0;
    var msgwords = Message.toArray();
    for (var i=0;i<msgwords.length;i++){

        if(msgwords[i] == "blah"){
            index = 1;
        }
    }   

    if (index > 0) {
        DParray[0] = Messenger.MyDisplayPicture;
        Messenger.MyDisplayPicture = FolderWithEmotes + DParray[index];
        MsgPlus.AddTimer("DPdelay", DelayLength)
    }
    index = 0;
}

---

Is the the complete code? If so, some potential problems will arise. This is because you're using timers. And with timers you need to be extra carefull... always. And cope with a lot more situations than just the obvious ones.

For starters, you don't cancel the timer when some other user logs in to Messenger. It isn't even cancelled when you log out. This will produce errors and at the least unexpected behaviour when such events occur. Remember that timers are not user specific and that scripts start to run the second you start Messenger. So they are running even before you log in and continue to run until you completely quit the Messenger application; they don't stop running when you simply log out. So this is also true for timers. All this is way too often forgotten in extremely many scripts (including many in the official script DB), making them all not work properly!!

Second, whenever a message is send before the timer delay has passed, you will get the exact behaviour as you described as being the problem. This is because the second time the ChatWndSendMessage event is fired the 'original' DP in the array (at index 0) is overwritten with the temporary one from the previous event.

So, what you must do:
1) add the event for when the user signs out. In that event, cancel the timer which might still be running.

2) Add a check to see if the global array contains an 'original' DP at index 0. if so, do not write the current display picture to index 0. This because when this occurs, the current showing DP is not the user's original one but a temporary one from a previous 'word to DP' event.

3) In the timer event, after the original DP is set back, you also need to clear out index 0 of the global array to accomodate proper workings of point 2.


Note: there are limits in how often you can change your DP in a certain timespan. This means that scripts like this will always have some potential problems and limits and might not always work properly; there will always be times and situations when your original DP can not be set back.


Note for point 1: you can make it a bit more complicated in such a way that the timer isn't cancelled but continues to run when a user logs out. But, in that case you need to make the timer user specific (the name can contain the user ID to make it user specific) and check if the same user is still logged in before setting the DP back. This makes that the script will support multiple users and that it can cope with sudden unexpected disconnections and reconnections of Messenger. Ofcourse, you would also need an array per user, or in other words a two dimensional array containing the user's ID and the corresponding original DP, (and the index of the 'Word'-DP for that user).

According to the documentation, the timer is only created once when called and is destroyed after completion, so I don't understand your problem with the timer never going away.
It is created only when a matching word is found, and deletes itself 5 seconds later.

and the problem with changing dps to fast and overwriting the original I am well aware of, however even if I don't say anything at all, it will still sometimes switch back. so I know this is not the problem either.

quote:

---

Originally posted by Genesis Genome
According to the documentation, the timer is only created once when called and is destroyed after completion, so I don't understand your problem with the timer never going away.
It is created only when a matching word is found, and deletes itself 5 seconds later.

---

quote:

---

Originally posted by Genesis Genome
and the problem with changing dps to fast and overwriting the original I am well aware of, however even if I don't say anything at all, it will still sometimes switch back. so I know this is not the problem either.

---

Well, considering the chances of me sending a message that triggers the timer,logging out and logging into a second account I don't have, all within 5 seconds, the issue isn't on the top of my todo list.
but adding a second timer to prevent consecutive changes is a good idea, I'll have a play with that. thanks.
It seems like its just the server rejecting my changes after all, which is what I always suspected.

quote:

---

Originally posted by Genesis Genome
It seems like its just the server rejecting my changes after all, which is what I always suspected.

---

quote:

---

Originally posted by Genesis Genome
According to the documentation, the timer is only created once when called and is destroyed after completion, so I don't understand your problem with the timer never going away.
It is created only when a matching word is found, and deletes itself 5 seconds later.

---

quote:

---

Originally posted by Genesis Genome
and the problem with changing dps to fast and overwriting the original I am well aware of, however even if I don't say anything at all, it will still sometimes switch back. so I know this is not the problem either.

---

quote:

---

Originally posted by Genesis Genome
Well, considering the chances of me sending a message that triggers the timer,logging out and logging into a second account I don't have, all within 5 seconds, the issue isn't on the top of my todo list.

---

quote:

---

Originally posted by Genesis Genome
It seems like its just the server rejecting my changes after all, which is what I always suspected.

---

Woh, I didn't mean to get you so riled up.
I know the script isn't perfect, and I know that the timer cancelling is important and should be there to reduce problems further down the road, but the reason I pushed these aside is because they were not relevant to my original question of "why does it switch back" (the answer being the server refuses to handle multiple quick changes)
I'm grateful of the help you've given me and all and I have fixed up these problems already.
Thanks again,
-Gen

quote:

---

Originally posted by Genesis Genome
Woh, I didn't mean to get you so riled up.

---

quote:

---

Originally posted by Genesis Genome
I know the script isn't perfect, and I know that the timer cancelling is important and should be there to reduce problems further down the road, but the reason I pushed these aside is because they were not relevant to my original question of "why does it switch back" (the answer being the server refuses to handle multiple quick changes)

---

quote:

---

Originally posted by Genesis Genome
I'm grateful of the help you've given me and all and I have fixed up these problems already.
Thanks again,
-Gen

---