Bug 35518: Check authentication and set userenv before fetching userenv variables
authorNick Clemens <nick@bywatersolutions.com>
Fri, 26 Jan 2024 14:10:01 +0000 (14:10 +0000)
committerMartin Renvoize <martin.renvoize@ptfs-europe.com>
Wed, 14 Feb 2024 13:27:03 +0000 (13:27 +0000)
commit90b6f68616e2ba5ca3fcbbd9698c97ef41a45593
tree477a7e8f9463bc1e7a35cab647d8c792378c80ee
parenta77adbe76e13f79c5b965f9677b0960fb44aabfc
Bug 35518: Check authentication and set userenv before fetching userenv variables

Currently we get the userenv before we have set it correctly for the session

To test:
 1 - Sign in as a user with fast cataloging permission
 2 - Bring up a patron, type gibberish into barcode field to get a fast cataloging link
 3 - Check the link, it should have your current signed in barcode
 4 - Sign in to a different browser with a different user and at a different branch
 5 - Bring up a aptron in circulation and type gibberish into barcode field to get a fast cataloging link
 6 - It may have your branch, but it may also have the other user's branch from the other window
 7 - Keep entering gibberish to get a link until one user has the correct branch
 8 - Then switch to the other browser, and keep entering gibberish, watch the branchcode change
 9 - Apply patch, restart all
10 - Test switching between browsers. generating fast cataloging links
11 - Users should now consistently have the correct branch

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
circ/circulation.pl