Attachment 154328 Details for Bug 34505 – Bug 34505: Patron invalid age in search_for_data_inconsistencies.pl skip expired patrons

[patch] Bug 34505: Patron invalid age in search_for_data_inconsistencies.pl skip expired patrons

Bug-34505-Patron-invalid-age-in-searchfordataincon.patch (text/plain), 1.51 KB, created by Fridolin Somers on 2023-08-09 07:47:00 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Fridolin Somers

Created: 2023-08-09 07:47:00 UTC

Size: 1.51 KB

patch

obsolete

>From 63c60926089eb32dde71de7dad9629b7e5672613 Mon Sep 17 00:00:00 2001
>From: Fridolin Somers <fridolin.somers@biblibre.com>
>Date: Tue, 8 Aug 2023 21:44:24 -1000
>Subject: [PATCH] Bug 34505: Patron invalid age in
> search_for_data_inconsistencies.pl skip expired patrons
>
>Bug 26311 added patron invalid age in search_for_data_inconsistencies.pl
>But this is not relevant for expired patrons, especially in child categories.
>Check should skip expired patrons
>
>1) Set a non-expired patron with invalid age
>2) Run misc/maintenance/search_for_data_inconsistencies.pl
>=> Check you see the patron
>3) Edit patron to be expired
>4) Run misc/maintenance/search_for_data_inconsistencies.pl
>=> Check you do not see the patron
>---
> misc/maintenance/search_for_data_inconsistencies.pl | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/misc/maintenance/search_for_data_inconsistencies.pl b/misc/maintenance/search_for_data_inconsistencies.pl
>index 4fd9502c36..947655c827 100755
>--- a/misc/maintenance/search_for_data_inconsistencies.pl
>+++ b/misc/maintenance/search_for_data_inconsistencies.pl
>@@ -310,7 +310,7 @@ use C4::Biblio qw( GetMarcFromKohaField );
>     );
>     my @invalid_patrons;
>     while ( my $aging_patron = $aging_patrons->next ) {
>-        push @invalid_patrons, $aging_patron unless $aging_patron->is_valid_age;
>+        push @invalid_patrons, $aging_patron unless $aging_patron->is_valid_age or $aging_patron->is_expired;
>     }
>     if (@invalid_patrons) {
>         new_section("Patrons with invalid age for category");
>-- 
>2.41.0

Actions: View | Diff | Splinter Review

Attachments on bug 34505: 154328 | 154330 | 154790