ansaurus

Question

Answer 1

A:

<body onload="timerStart()">

...

function timerStart(){
  setTimeout(confirmStay, 60000)   // 1 minute
}

function confirmStay() {
  if (confirm("Press OK to stay here.")) {
    // user pressed OK
    timerStart();   // restart timer
  } else {
    // user pressed Cancel
    location.href="http://example.com";  //redirect to homepage
  }
}

Zed 2009-08-10 05:23:50

'setTimeout("confirmStay()", 60000)' shouldn't that just be "confirmStay" for the 1st argument?

alex 2009-08-10 05:28:21

I think both versions (string and a function reference) are supported.

finpingvin 2009-08-10 05:47:22

I have always put there a string containing a js statement... but I'm not catching up with all the new fashions, so it might be outdated.

Zed 2009-08-10 05:59:55

Both string and function reference are supported, however string version uses eval to parse the string so it's theoretically less secure.

RaYell 2009-08-10 06:25:29

Never use the string version, as it invokes "eval()" unnecessarily. Instead, pass a function reference or an anonymous function. So "setTimeout("confirmStay()", 60000)" becomes "setTimeout(confirmStay, 60000);" (also note the semi-colon—every statement should end with one).

Steve Harrison 2009-08-10 07:18:48

Thanks guys for the comments, I have changed it to function reference. Will also try to remember this next time I use setTimeout :)

Zed 2009-08-10 07:59:14

Answer 2

A:

I'm assuming that this has something to do with refreshing a session to preserve a login. If that is the case then you should keep in mind that the user may not respond to your dialog for some time after it pops up. Meaning, your popup may come up after 10 minutes and your session may last for 15 minutes, but if it takes longer than 5 minutes for the user to respond then the session will expire and the refresh of the page will not help you.

<body onload="setTimeout(refreshLogin, 600000)">
    <script>
        function refreshLogin() {
          if (confirm("Press OK to refresh this page and renew your login.")) {
            window.location.reload();   // refresh the page
          } else {
            window.location.href="http://example.com"  //redirect to somewhere
          }
        }
    </script>
</body>

Prestaul 2009-08-10 06:05:32

Answer 3

+2 A:

Yet another approach:

Usage:

var timer = reloadTimer({  seconds:5, logoutURL: '/logout',
                           message:'Do you want to stay logged in?'});
    timer.start();

Implementation:

var reloadTimer = function (options) {
  var seconds = options.seconds || 0,
      logoutURL = options.logoutURL,
      message = options.message;

  this.start = function () {
    setTimeout(function (){
      if ( confirm(message) ) {
        window.location.reload(true);
      } else {
        window.location.href = logoutURL;
      }
    }, seconds * 1000);
  }
  return this;
};

CMS 2009-08-10 06:21:03

ansaurus

tags:

views:

answers:

JavaScript timeout script

related questions