If I access circulation.pl directly, with no patron cardnumber or borrowernumber submitted, I get an error: Can't call method "messages" on an undefined value at /kohadevbox/koha/circ/circulation.pl line 525
Ah bad, some librarians use this to on purpose to get out of patron circ page.
Created attachment 130571 [details] [review] Bug 30099: Fix error when accessing circulation.pl without patron parameter If I access circulation.pl directly, with no patron cardnumber or borrowernumber submitted, I get an error: Can't call method "messages" on an undefined value at /kohadevbox/koha/circ/circulation.pl line 525 This is caused by Bug 29230 which calls $patron outside a "if ($aptron)". Test plan : 1) In staff interface, click in 'check out' search box 2) Hit enter (without a search text) => Without patch you get an error => With patch you see a blank page /cgi-bin/koha/circ/circulation.pl
Created attachment 130573 [details] [review] Bug 30099: Fix error when accessing circulation.pl without patron parameter If I access circulation.pl directly, with no patron cardnumber or borrowernumber submitted, I get an error: Can't call method "messages" on an undefined value at /kohadevbox/koha/circ/circulation.pl line 525 This is caused by Bug 29230 which calls $patron outside a "if ($aptron)". Test plan : 1) In staff interface, click in 'check out' search box 2) Hit enter (without a search text) => Without patch you get an error => With patch you see a blank page /cgi-bin/koha/circ/circulation.pl Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Current patch-complexity: Trivial patch
Created attachment 130591 [details] [review] Bug 30099: Fix error when accessing circulation.pl without patron parameter If I access circulation.pl directly, with no patron cardnumber or borrowernumber submitted, I get an error: Can't call method "messages" on an undefined value at /kohadevbox/koha/circ/circulation.pl line 525 This is caused by Bug 29230 which calls $patron outside a "if ($aptron)". Test plan : 1) In staff interface, click in 'check out' search box 2) Hit enter (without a search text) => Without patch you get an error => With patch you see a blank page /cgi-bin/koha/circ/circulation.pl Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Pushed to master for 22.05, thanks to everybody involved [U+1F984]
Pushed to 21.11.x for 21.11.03
Missing dependency, not backported to 21.05