]> git.koha-community.org Git - koha.git/commit
Bug 10029 - CAS authentication fails in OPAC reserve
authorFridolyn SOMERS <fridolyn.somers@biblibre.com>
Thu, 11 Apr 2013 14:46:23 +0000 (16:46 +0200)
committerChris Hall <followingthepath@gmail.com>
Sat, 17 Aug 2013 23:09:29 +0000 (11:09 +1200)
commit2d845f686c49b9f7d4dc6685aac4b902d7a30d0d
treec9736c65c2aa61d50cb755303090029e5eeb4fee
parent76e5ead8cdaf0a80e7c6269c1d203aa8ae5b19f5
Bug 10029 - CAS authentication fails in OPAC reserve

If OPAC reserve page is accessed without being logged-in, login form is displayed as well as a CAS authentication link (if enabled). A click on this link will lead to CAS server but one comming back to Koha, page shows an error : "ERROR: No biblionumber received".
This is because CAS link only contains the query path "/cgi-bin/koha/opac-reserve.pl", not the query parameters.

This patch adds query parameters to URI sent to CAS.

Test plan :
- Enable CAS
- Go to opac without been logged-in
- Try to place hold on a record
=> You get to /cgi-bin/koha/opac-reserve.pl?biblionumber=XXX showing authentication page
=> Check that CAS link contains query param "biblionumber"
- Click on CAS link and log in
=> Check you return well logged-in to reserve page with biblionumber param

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
I have followed the test plan as far as I could and the links
contain the biblionumber now, which they didn't before.
I couldn't check the CAS login, but my normal login worked
as expected.
All tests and the QA script pass.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
(cherry picked from commit 974ab561ef00e76e7329f61b016b5d6b3103dde7)
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
(cherry picked from commit 2dff5177f2d7c542175b28adf1832e201c0ae864)
Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>
(cherry picked from commit 2dff5177f2d7c542175b28adf1832e201c0ae864)
Signed-off-by: Chris Hall <followingthepath@gmail.com>
(cherry picked from commit bd199ff98c8c34bd6587f12bc3a844827ed07aa9)
C4/Auth_with_cas.pm