Bug 30588: (QA follow-up) Auth - remove two warns and second logout
authorMarcel de Rooy <m.de.rooy@rijksmuseum.nl>
Fri, 9 Sep 2022 09:18:10 +0000 (09:18 +0000)
committerTomas Cohen Arazi <tomascohen@theke.io>
Fri, 21 Oct 2022 14:37:00 +0000 (11:37 -0300)
commit316355e8d535514a052964b33defd2c8db252d42
treead1d467bc207197fb0bbaf5e13beb0c2cfa8084b
parent17c504f54f65128636cbf2a039afbc93fd7dba0e
Bug 30588: (QA follow-up) Auth - remove two warns and second logout

Resolves:
Use of uninitialized value $request_method in string eq at /usr/share/koha/C4/Auth.pm line 1122.
Use of uninitialized value $return in numeric gt (>) at /usr/share/koha/C4/Auth.pm line 1155.

We also remove the double logout from Auth.t

Test plan:
Run t/db_dependent/Auth.t
Check if you do not see the warns anymore.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
C4/Auth.pm
t/db_dependent/Auth.t