Question 50720 in Javascript Question posted in
A very good W3school tutorial can be found here.

Javascript clearTimeout Not Working Properly

I have written this function which calls a .net method if there is no user activity on the page for sometime.

function IdleSessionLogout(dotnetHelper) {
    var timer;
    document.onmousemove = resetTimer;
    document.onkeypress = resetTimer;

    function resetTimer() {
        console.log("I am in Reset Timer Function");
        clearTimeout(timer);
        timer = setTimeout(Logout, 5000);
    }
    function Logout() {

        dotnetHelper.invokeMethodAsync("Logout");
    }
}

I have checked function is triggered but the clearTimeout does not work and user is still logged out even there is some activity from user. Can please tell me what I am doing wrong here?

Tags:

2

Answers


  1. 0

    I created the timeout outside of the reset timer, if the user moves the mouse clearTimeout will invoke and log out operation will cancel. Try this;

    function IdleSessionLogout(dotnetHelper) {
    var timer = setTimeout(Logout, 5000); 
    document.onmousemove = resetTimer;
    document.onkeypress = resetTimer;

    function resetTimer() {
        console.log("I am in Reset Timer Function");
        clearTimeout(timer);
    }
    function Logout() {
        dotnetHelper.invokeMethodAsync("Logout");
    }
}
    Login or Signup to reply.
  2. 0

    Your code works as-is:

    function IdleSessionLogout(dotnetHelper) {
    var timer;
    document.onmousemove = resetTimer;
    document.onkeypress = resetTimer;

    function resetTimer() {
        clearTimeout(timer);
        timer = setTimeout(Logout, 5000);
    }
    function Logout() {

        console.log("no action for 5 seconds");
    }
}

IdleSessionLogout();

    So then the issue (assuming dotnetHelper.invokeMethodAsync("Logout") works as expected) is that you’re setting an event listener with = instead of .addEventListener. Only one event listener can be enabled at a time if you use =, so if you have an = somewhere after this, the event listeners will be removed.

    You should also initialize the timer so that if there is never any mouse movement or keypress that it properly calls the Logout function.

    function IdleSessionLogout(dotnetHelper) {
    var timer = setTimeout(Logout, 5000);         
    window.addEventListener("mousemove", resetTimer);
    window.addEventListener("keypress", resetTimer);

    function resetTimer() {
        clearTimeout(timer);
        timer = setTimeout(Logout, 5000);
    }
    function Logout() {

        console.log("no action for 5 seconds");
    }
}

IdleSessionLogout();
    Login or Signup to reply.
Please signup or login to give your own answer.

Back To Top
Search