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)
committerFridolin Somers <fridolin.somers@biblibre.com>
Thu, 15 Feb 2024 09:48:08 +0000 (10:48 +0100)
commit26722f2a08af99b9e3cb4eb50398df896085f527
treede688fc96b68c89eb77887f4192ca132684ab503
parent5c640913648d4bed48628fa52428661281eaeda0
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>
(cherry picked from commit 90b6f68616e2ba5ca3fcbbd9698c97ef41a45593)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
circ/circulation.pl