Bug 26352 changed the line in circ/circulation.pl that searches for a patron card number in the barcode field from using the $barcode variable to the $findborrower variable. This breaks AutoSwitchPatron.
Created attachment 128232 [details] [review] Proposed fix Test plan: 1. make sure AutoSwitchPatron system preference is on. 2. get two patron card numbers. 3. enter the first card number to check out to the patron. 4. enter the second card number in the item barcode field. 5. observe that the barcode is not found. 6. apply patch 7. enter the second card number in the item barcode field again. 8. observe that the second patron is loaded to check out to.
Created attachment 128240 [details] [review] Bug 29637: AutoSwitchPatron should look for card number in the barcode Bug 26352 changed the line in circ/circulation.pl that searches for a patron card number in the barcode field from using the $barcode variable to the $findborrower variable. This breaks AutoSwitchPatron. Test plan: 1. make sure AutoSwitchPatron system preference is on. 2. get two patron card numbers. 3. enter the first card number to check out to the patron. 4. enter the second card number in the item barcode field. 5. observe that the barcode is not found. 6. apply patch 7. enter the second card number in the item barcode field again. 8. observe that the second patron is loaded to check out to. Signed-off-by: David Nind <david@davidnind.com>
Created attachment 128275 [details] [review] Bug 29637: AutoSwitchPatron should look for card number in the barcode Bug 26352 changed the line in circ/circulation.pl that searches for a patron card number in the barcode field from using the $barcode variable to the $findborrower variable. This breaks AutoSwitchPatron. Test plan: 1. make sure AutoSwitchPatron system preference is on. 2. get two patron card numbers. 3. enter the first card number to check out to the patron. 4. enter the second card number in the item barcode field. 5. observe that the barcode is not found. 6. apply patch 7. enter the second card number in the item barcode field again. 8. observe that the second patron is loaded to check out to. Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Note that bug 26352 no longer seems to work (unless I set things up wrong) - see my note on the bug.
(In reply to David Nind from comment #4) > Note that bug 26352 no longer seems to work (unless I set things up wrong) - > see my note on the bug. I've tested with KitchenSink and pull-request : https://github.com/bywatersolutions/dev-koha-plugin-kitchen-sink/pull/17 All good.
(In reply to Fridolin Somers from comment #5) > All good. Excellent!
Pushed to master for 22.05, thanks to everybody involved [U+1F984]
Pushed to 21.11.x for 21.11.01
Missing dependency, not backported to 21.05.x