This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "Evergreen ILS". The branch, rel_3_14 has been updated via f54b1a9e1844d03d27cc6f60f04b9981eb648374 (commit) via f821b35f9acfc87c621d4cfa86a0b62a8851db1a (commit) from d0dac6d3e9721db0f8dbb7355b0f890955bae694 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- commit f54b1a9e1844d03d27cc6f60f04b9981eb648374 Author: Jane Sandberg <sandbergja@gmail.com> Date: Mon Jun 2 19:12:05 2025 -0700 LP#2112185 follow-up: add a live test We don't want to see this particular regression again! Signed-off-by: Jane Sandberg <sandbergja@gmail.com> diff --git a/Open-ILS/src/perlmods/live_t/lp2112185-fix-precat-checkout.t b/Open-ILS/src/perlmods/live_t/lp2112185-fix-precat-checkout.t new file mode 100644 index 0000000000..267dc8b339 --- /dev/null +++ b/Open-ILS/src/perlmods/live_t/lp2112185-fix-precat-checkout.t @@ -0,0 +1,82 @@ +#!perl +use strict; use warnings; +use Test::More tests => 7; +use OpenILS::Utils::TestUtils; + +diag 'LP2112185: Precat items must be able to check out multiple times'; + +use constant WORKSTATION_NAME => 'BR4-test-lp2112185-fix-precat-checkout.t'; +use constant WORKSTATION_LIB => 7; # BR4 +use constant BR4_PATRON_ID => 14; # Robert Wade +use constant ITEM_BARCODE => 'μῆνιν ἄειδε θεὰ Πηληϊάδεω Ἀχιλῆος'; # This barcode should not exist in the test data hahaha + +my $script = OpenILS::Utils::TestUtils->new; +$script->bootstrap; + +my $authtoken = $script->authenticate({ + username=>'admin', + password=>'demo123', + type=>'staff' +}); +ok( + $script->authtoken, + 'Have an authtoken' +); + +my $ws = $script->find_or_register_workstation(WORKSTATION_NAME, WORKSTATION_LIB); +ok( + ! ref $ws, + 'Found or registered workstation' +); + +# Login again, this time with the appropriate workstation +$script->authenticate({ + username => 'admin', + password => 'demo123', + type => 'staff', + workstation => WORKSTATION_NAME +}); + +my $checkout_resp = $script->do_checkout({ + patron => BR4_PATRON_ID, + barcode => ITEM_BARCODE +}); +is $checkout_resp->{textcode}, + 'ITEM_NOT_CATALOGED', + 'It tells you that the item is not cataloged'; + + +$checkout_resp = $script->do_checkout({ + patron => BR4_PATRON_ID, + barcode => ITEM_BARCODE, + dummy_title => 'my title', + precat => 1 +}); +is $checkout_resp->{textcode}, + 'SUCCESS', + 'It can check out a precat if you pass precat => 1'; + +my $checkin_resp = $script->do_checkin({ + copy_barcode => ITEM_BARCODE +}); +is $checkin_resp->{textcode}, + 'ITEM_NOT_CATALOGED', + 'It lets the client know that it is a precat, so the client can tell the user to route to cataloging'; + +$checkout_resp = $script->do_checkout({ + patron => BR4_PATRON_ID, + barcode => ITEM_BARCODE, + dummy_title => 'my title', + precat => 1 +}); +is $checkout_resp->{textcode}, + 'SUCCESS', + 'You can check out a precat with the same barcode again'; + +$checkin_resp = $script->do_checkin({ + copy_barcode => ITEM_BARCODE +}); +is $checkin_resp->{textcode}, + 'ITEM_NOT_CATALOGED', + 'The second time you check it in, it still lets you know that it is a precat'; + commit f821b35f9acfc87c621d4cfa86a0b62a8851db1a Author: Mike Rylander <mrylander@gmail.com> Date: Fri May 30 10:47:46 2025 -0400 LP#2112185: Bring back core precat behavior The fix proposed and committed for LP bug 2098898 removed all precat event overrides in an attempt to stop blocked patrons from receiving precat items at checkout. Unfortunately, the precat mechanism uses the automatic COPY_NOT_AVAILABLE.override permission check to see if staff are allowed to check out a precat that has circulated in the past. This commit brings back the override check for that specific COPY_NOT_AVAILALBE event, allowing precats to properly circulate again. Signed-off-by: Mike Rylander <mrylander@gmail.com> Signed-off-by: Ruth Frasur Davis <rfrasur@gmail.com> Signed-off-by: Jane Sandberg <sandbergja@gmail.com> diff --git a/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Circulate.pm b/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Circulate.pm index c312d0a592..9755d03868 100644 --- a/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Circulate.pm +++ b/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Circulate.pm @@ -161,7 +161,7 @@ __PACKAGE__->register_method( sub run_method { my( $self, $conn, $auth, $args ) = @_; translate_legacy_args($args); - $args->{override_args} = { all => 1 } unless defined $args->{override_args}; + $args->{override_args} = { all => 1, events => [] } unless defined $args->{override_args}; $args->{new_copy_alerts} ||= $self->api_level > 1 ? 1 : 0; my $api = $self->api_name; @@ -248,6 +248,11 @@ sub run_method { } elsif( $api =~ /checkout.full/ ) { $circulator->skip_permit_key(1); + if ( $circulator->request_precat && $circulator->editor->allowed('CREATE_PRECAT') ) { + $circulator->override(1); + $circulator->override_args->{all} = 0; # precat checkout should only override COPY_NOT_AVAILABLE, and whatever else the client requested + push @{$circulator->override_args->{events}}, 'COPY_NOT_AVAILABLE'; + } $circulator->do_permit(); $circulator->is_checkout(1); unless( $circulator->bail_out ) { ----------------------------------------------------------------------- Summary of changes: .../lib/OpenILS/Application/Circ/Circulate.pm | 7 +- .../live_t/lp2112185-fix-precat-checkout.t | 82 ++++++++++++++++++++++ 2 files changed, 88 insertions(+), 1 deletion(-) create mode 100644 Open-ILS/src/perlmods/live_t/lp2112185-fix-precat-checkout.t hooks/post-receive -- Evergreen ILS