Selfcheck warn on patron auto-logout, reset activity timer on checkout

[Bill Erickson]

Evergreen 2.8, likely affects all versions.

Setting values for the org unit settings "circ.selfcheck.patron_password_required" and "circ.selfcheck.patron_login_timeout" means that patrons must log in to the selfcheck UI and that their session will be terminated after a period of inactivity. I have 2 usability improvements to suggest for these.

1. Show a warning dialog just before terminating a selfcheck session. The standard "you will be logged out in X seconds" should suffice. As it stands, the session simply ends with no warning.

2. Reset the inactivity timer with each checkout attempt. Currently, the timer is only reset by navigation and printing. On a system with an aggressive timeout, you could be kicked out while you are in the middle of checking out a large pile of items.

[Bill Erickson (berick)]

Correction: the "circ.selfcheck.patron_password_required" org unit setting has no bearing on this. Password or not, the login session will timeout if a timeout is configured.

Pushed a possible fix here:

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/berick/lp1476370-selfcheck-auth-timer-warning

It adds a warning dialog w/ hard-coded 30 second timeout, effectively increasing the overall timeout by 30 seconds. (I was really hoping to avoid YAOUS for this). If we stick w/ this, we'll need a blurb in the release/upgrade notes about explaining this.

[Bill Erickson (berick)]

Force pushed an update. Lowered the dialog time to 20 seconds and instead of adding to the overall timeout, the warning now appears 20 seconds before the configured timeout occurs. No changes needed to retain the current timeout. Now with release notes.

[Ben Shum (bshum)]

Works for me. Pushed to master for 2.9 awesomeness. Thanks Bill!!!