From c36b04f938338bd12be3ee4ea7042ab0f4c70f5d Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Tue, 9 Sep 2025 11:45:21 +0200 Subject: [PATCH] Bug 30515: Use print if the patron's preference is empty If the patron has not selected a transport type we still want to print. ie. the patrons cannot totally opt out. Signed-off-by: krimsonkharne --- misc/cronjobs/overdue_notices.pl | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/misc/cronjobs/overdue_notices.pl b/misc/cronjobs/overdue_notices.pl index ea5d9e57f8..444136c465 100755 --- a/misc/cronjobs/overdue_notices.pl +++ b/misc/cronjobs/overdue_notices.pl @@ -755,14 +755,16 @@ END_SQL if ( $patronpref && $patronpref->{'transports'} ) { @message_transport_types = keys %{ $patronpref->{'transports'} }; } - } else { - my @message_transport_types = - @{ GetOverdueMessageTransportTypes( $branchcode, $overdue_rules->{categorycode}, $i ) }; - @message_transport_types = - @{ GetOverdueMessageTransportTypes( q{}, $overdue_rules->{categorycode}, $i ) } - unless @message_transport_types; } + @message_transport_types = + @{ GetOverdueMessageTransportTypes( $branchcode, $overdue_rules->{categorycode}, $i ) } + unless @message_transport_types; + + @message_transport_types = + @{ GetOverdueMessageTransportTypes( q{}, $overdue_rules->{categorycode}, $i ) } + unless @message_transport_types; + my $print_sent = 0; # A print notice is not yet sent for this patron for my $mtt (@message_transport_types) { next if $mtt eq 'itiva'; -- 2.39.5