From 5dc1e731435843a9a1822db70f06d0c3955db6c0 Mon Sep 17 00:00:00 2001 From: Nick Clemens Date: Wed, 22 Sep 2021 12:02:43 +0000 Subject: [PATCH] Bug 29076: Fetch the zebraqueue entries when not confirm This patch remove a check for confirm when fetching the entries for possible deletion from the zebraqueue Without this we die when we try to access the entries for counting To test: 1 - perl misc/cronjobs/cleanup_database.pl --sessions --zebraqueue 10 --list-invites --z3950 --logs 180 --mail 375 -v 2 - Dies 3 - Apply patch 4 - Repeat 1 5 - Success! Signed-off-by: Tomas Cohen Arazi Signed-off-by: Marcel de Rooy Signed-off-by: Jonathan Druart Signed-off-by: Kyle M Hall (cherry picked from commit 619e60fd55fefedaea96113525373094c426c880) Signed-off-by: Fridolin Somers --- misc/cronjobs/cleanup_database.pl | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/misc/cronjobs/cleanup_database.pl b/misc/cronjobs/cleanup_database.pl index 062efe6041..0d3a520817 100755 --- a/misc/cronjobs/cleanup_database.pl +++ b/misc/cronjobs/cleanup_database.pl @@ -272,9 +272,7 @@ if ($zebraqueue_days) { WHERE done=1 AND time < date_sub(curdate(), INTERVAL ? DAY) } ); - if ( $confirm ) { - $sth->execute($zebraqueue_days) or die $dbh->errstr; - } + $sth->execute($zebraqueue_days) or die $dbh->errstr; $sth2 = $dbh->prepare(q{ DELETE FROM zebraqueue WHERE id=? }); while ( my $record = $sth->fetchrow_hashref ) { if ( $confirm ) { -- 2.39.5