From 040f3612af2aecc03f2dca2ce2623543c8072efb Mon Sep 17 00:00:00 2001
From: Kyle M Hall <kyle@bywatersolutions.com>
Date: Tue, 6 Apr 2021 09:16:52 -0400
Subject: [PATCH] Bug 25429: Use filter_by_last_update

---
 misc/cronjobs/cleanup_database.pl | 20 ++++++++++++++------
 1 file changed, 14 insertions(+), 6 deletions(-)

diff --git a/misc/cronjobs/cleanup_database.pl b/misc/cronjobs/cleanup_database.pl
index 254bb7e895..00713c7485 100755
--- a/misc/cronjobs/cleanup_database.pl
+++ b/misc/cronjobs/cleanup_database.pl
@@ -509,14 +509,22 @@ if ($pStatistics) {
 
 if ($return_claims) {
     print "Purging return claims older than $return_claims days.\n" if $verbose;
-    $sth = $dbh->prepare(
-        q{
-            DELETE FROM return_claims
-            WHERE resolved_on < DATE_SUB(CURDATE(), INTERVAL ? DAY)
+
+    my $return_claims = Koha::Checkouts::ReturnClaims->filter_by_last_update(
+        {
+            timestamp_column_name => 'resolved_on',
+            days => $return_claims
         }
     );
-    $sth->execute($return_claims);
-    print "Done with purging return claims.\n" if $verbose;
+
+    my $count = $return_claims->count;
+    $return_claims->delete if $confirm;
+
+    if ($verbose) {
+        say $confirm
+            ? sprintf "Done with purging %d resolved return claims.", $count
+            : sprintf "%d resolved return claims would have been purged.", $count;
+    }
 }
 
 if ($pDeletedCatalog) {
-- 
2.24.3 (Apple Git-128)