[Koha-bugs] [Bug 15642] Batch patron deletion should not delete patrons with issues

[bugzilla-daemon at bugs.koha-community.org]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15642

--- Comment #6 from Jonathan Druart <jonathan.druart at bugs.koha-community.org> ---
What's the point of this _skip_borrowers_with_nonzero_balance subroutine
actually? :)

>From my understanding, it removes the patrons with charges.

In misc/cronjobs/delete_patrons.pl it's done with:

 68     my $flags = C4::Members::patronflags( $member );
 69     if ( my $charges = $flags->{CHARGES}{amount} ) {
 70         say "Failed to delete patron $borrowernumber: patron has $charges
in fines";
 71         next;
 72     }

Should not we simply move this check to the GetBorrowersToExpunge subroutine?

-- 
You are receiving this mail because:
You are watching all bug changes.