Self checkout ignoring patron timeout library setting

[Brent Mills]

* Evergreen 2.5.1
* Debian Squeeze

The library setting "Self Check: Patron Login Timeout (in seconds)" doesn't appear to respect the newer, web-based self check interface at: your.catalog/eg/circ/selfcheck/main. Entries in the library setting for patron timeout aren't being acknowledged and the patron remains logged in at the self check station unless they manually click one of the logout buttons.

This is also mentioned in the Evergreen documentation: http://docs.evergreen-ils.org/2.5/_self_checkout.html where it states that the library setting is not currently supported.

The older, XML based self check respected the patron timeout setting, it would be nice to have the same functionality in the new, web-based self check.

[Holly Brennan (hollyfromhomer)]

While my library does not currently use any self-check method, I am doing the research to get started. The inability to enforce a timeout is a deal breaker, so until that once again works my hands are tied.

[Ben Shum (bshum)]

Hmm, confirming this bug. I guess I hadn't noticed it because I'd been having the web browser for selfcheck machines reset back to the main page after a specified period of time. It would be nice to bring back use of the setting in some way.

[Ben Shum (bshum)]

Hmm, in the regular catalog, the value for page refresh is defined in Open-ILS/src/templates/opac/parts/base.tt2 where it'll find the ctx.authtime value and apply that to the meta tag for refresh directing to the logout page for the catalog.

Perhaps something similar could be employed to apply the setting for use with the selfcheck tt2 interface.

[Ben Shum (bshum)]

Worked on this during the 2014 Hack-A-Way, have some initial code in the working repository. We have to test further to ensure that the reset timer is applied in enough places. Also we need to look at the library settings and ensure that we clean up / verify which settings are used.

Final branch and pullrequest forthcoming.

[Ben Shum (bshum)]

Working branch thus far: user/bshum/lp1306814-selfcheck-timeout

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/bshum/lp1306814-selfcheck-timeout

[Michele Morgan (mmorgan)]

Tested the timeout after the following actions in selfcheck and it worked every time:

Scan a patron barcode
Check out an item
Failed checkout
Renew item
View items out
View holds
View fines
Print items out
Print fines
View items out after checkout
View holds after checkout
View fines after checkout
Checkout item while viewing items out
Checkout item while viewing holds
Checkout item while viewing fines
Return to home after viewing items out
Return to home after viewing holds
Return to home after viewing fines
Checkout item after viewing items out and printing
Renew item after viewing items out

Thanks Ben!

My signoff is here:

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/mmorgan/LP_1306814_selfcheck_timeout_signoff

[Ben Shum (bshum)]

Sweet! Thanks for the comprehensive testing and signoff Michele!

I've pulled in the change to master and backported to rel_2_7, rel_2_6, and rel_2_5.