From 5cd402af49e93908f5cd652563aa66fc71fc9fbd Mon Sep 17 00:00:00 2001 From: Michele Morgan Date: Thu, 12 Sep 2019 16:30:47 -0400 Subject: [PATCH] LP#1842431 Make renewal failure reason more appropriate for patron notifications Changes the failure reason for the autorenewal to include only the desc, Removes the textcode of the event, which shouldn't be included in a patron notice. Signed-off-by: Michele Morgan Signed-off-by: Jason Stephenson Signed-off-by: Jason Boyer --- .../lib/OpenILS/Application/Trigger/Reactor/Circ/AutoRenew.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Open-ILS/src/perlmods/lib/OpenILS/Application/Trigger/Reactor/Circ/AutoRenew.pm b/Open-ILS/src/perlmods/lib/OpenILS/Application/Trigger/Reactor/Circ/AutoRenew.pm index 09dc0b18ae..6c6de46b4e 100644 --- a/Open-ILS/src/perlmods/lib/OpenILS/Application/Trigger/Reactor/Circ/AutoRenew.pm +++ b/Open-ILS/src/perlmods/lib/OpenILS/Application/Trigger/Reactor/Circ/AutoRenew.pm @@ -70,7 +70,7 @@ sub handler { my %user_data = ( copy => $_->target_copy(), is_renewed => $is_renewed, - reason => !$is_renewed ? sprintf("%s : %s", $evt->{textcode}, substr($evt->{desc}, 0, 140)) : '', + reason => !$is_renewed ? $evt->{desc} : '', new_due_date => $is_renewed ? $evt->{payload}->{circ}->due_date : '', old_due_date => !$is_renewed ? $_->due_date() : '', ); -- 2.43.2