From 9aa5399538313828faaa54e6cd0dd07d38644bd1 Mon Sep 17 00:00:00 2001 From: Martin Renvoize Date: Mon, 11 Apr 2022 17:20:11 +0100 Subject: [PATCH] Bug 29346: FIXME: Highlight double enqueue This patch highlights a possible issue with the triggers.. though it may not actually matter in reality. I appear to already see test failures before this patch with a double enqueue of the rebuild for both AddIssue and AddReturn.. I couldn't spot what was causing that but whilst digging I did find another case where it could happen as highlighted here... --- t/db_dependent/Circulation_holdsqueue.t | 1 + 1 file changed, 1 insertion(+) diff --git a/t/db_dependent/Circulation_holdsqueue.t b/t/db_dependent/Circulation_holdsqueue.t index f8a82ffd19..8c9e22292e 100755 --- a/t/db_dependent/Circulation_holdsqueue.t +++ b/t/db_dependent/Circulation_holdsqueue.t @@ -41,6 +41,7 @@ subtest 'AddIssue() and AddReturn() real-time holds queue tests' => sub { my $item = $builder->build_sample_item({ library => $library->id }); t::lib::Mocks::mock_userenv({ branchcode => $library->id }); + t::lib::Mocks::mock_preference( 'UpdateTotalIssuesOnCirc', 1 ); my $action; -- 2.20.1