Bugzilla – Attachment 31124 Details for
Bug 12717
Library no longer receiving Overdue email for patrons without email address
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
[SIGNED OFF] Bug 12717: Library no longer receiving Overdue email for patrons without email address
SIGNED-OFF-Bug-12717-Library-no-longer-receiving-O.patch (text/plain), 2.76 KB, created by
Nick Clemens (kidclamp)
on 2014-08-23 15:58:05 UTC
(
hide
)
Description:
[SIGNED OFF] Bug 12717: Library no longer receiving Overdue email for patrons without email address
Filename:
MIME Type:
Creator:
Nick Clemens (kidclamp)
Created:
2014-08-23 15:58:05 UTC
Size:
2.76 KB
patch
obsolete
>From 6e72d5710c6df4384e5203c51f9510c672d16bba Mon Sep 17 00:00:00 2001 >From: Jonathan Druart <jonathan.druart@biblibre.com> >Date: Wed, 20 Aug 2014 13:26:49 +0200 >Subject: [PATCH] [SIGNED OFF] Bug 12717: Library no longer receiving Overdue > email for patrons without email address > >Bug 10832 changes the fallback behavior if a patron does not have email >address: a print notice is generated into the message_queue table. >But this can cause issue for some libraries. The script should sent an >email and (generated csv, html, text file) with the list of all unsent >notices. > >Test plan: >1/ Add overdue to a patron without email address (or smsalertnumber) >2/ Check email in the overdue rules configuration (or sms) >3/ Launch the overdue_notices.pl cronjob >4/ Verify the message_queue contain a print notice AND an email notice >for the library > >Signed-off-by: Nick Clemens <nick@quecheelibrary.org> >--- > misc/cronjobs/overdue_notices.pl | 17 +++++++++++++++++ > 1 file changed, 17 insertions(+) > >diff --git a/misc/cronjobs/overdue_notices.pl b/misc/cronjobs/overdue_notices.pl >index a0a1254..942cb6a 100755 >--- a/misc/cronjobs/overdue_notices.pl >+++ b/misc/cronjobs/overdue_notices.pl >@@ -692,6 +692,23 @@ END_SQL > if ( ($mtt eq 'email' and not scalar @emails_to_use) or ($mtt eq 'sms' and not $data->{smsalertnumber}) ) { > # email or sms is requested but not exist, do a print. > $mtt = 'print'; >+ push @output_chunks, >+ prepare_letter_for_printing( >+ { letter => $letter, >+ borrowernumber => $borrowernumber, >+ firstname => $data->{'firstname'}, >+ lastname => $data->{'surname'}, >+ address1 => $data->{'address'}, >+ address2 => $data->{'address2'}, >+ city => $data->{'city'}, >+ postcode => $data->{'zipcode'}, >+ country => $data->{'country'}, >+ email => $notice_email, >+ itemcount => $itemcount, >+ titles => $titles, >+ outputformat => defined $csvfilename ? 'csv' : defined $htmlfilename ? 'html' : defined $text_filename ? 'text' : '', >+ } >+ ); > } > unless ( $mtt eq 'print' and $print_sent == 1 ) { > # Just sent a print if not already done. >-- >1.7.10.4
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 12717
:
31022
|
31124
|
31163