Bugzilla – Attachment 105500 Details for
Bug 18532
Messaging preferences for auto renewals
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 18532: Add new letter AUTO_RENEWALS_DGST, new Auto_Renewals in message_attributes, and configured message_transports
Bug-18532-Add-new-letter-AUTORENEWALSDGST-new-Auto.patch (text/plain), 6.74 KB, created by
Agustín Moyano
on 2020-06-02 19:54:26 UTC
(
hide
)
Description:
Bug 18532: Add new letter AUTO_RENEWALS_DGST, new Auto_Renewals in message_attributes, and configured message_transports
Filename:
MIME Type:
Creator:
Agustín Moyano
Created:
2020-06-02 19:54:26 UTC
Size:
6.74 KB
patch
obsolete
>From 8cea8d7b701db70fd744620cd950023991816990 Mon Sep 17 00:00:00 2001 >From: Agustin Moyano <agustinmoyano@theke.io> >Date: Tue, 29 Oct 2019 15:27:37 -0300 >Subject: [PATCH] Bug 18532: Add new letter AUTO_RENEWALS_DGST, new > Auto_Renewals in message_attributes, and configured message_transports > >Signed-off-by: tgoat <tgoatley@gmail.com> > >Signed-off-by: Kelly McElligott <kelly@bywatersolutions.com> > >Signed-off-by: Marti Fuerst <mfuerst@hmcpl.org> >--- > .../data/mysql/atomicupdate/bug-18532.perl | 68 +++++++++++++++++++ > .../sample_notices_message_attributes.sql | 3 +- > .../sample_notices_message_transports.sql | 6 +- > .../mysql/pl-PL/mandatory/sample_notices.sql | 21 ++++++ > 4 files changed, 96 insertions(+), 2 deletions(-) > create mode 100644 installer/data/mysql/atomicupdate/bug-18532.perl > >diff --git a/installer/data/mysql/atomicupdate/bug-18532.perl b/installer/data/mysql/atomicupdate/bug-18532.perl >new file mode 100644 >index 0000000000..1e6fb32111 >--- /dev/null >+++ b/installer/data/mysql/atomicupdate/bug-18532.perl >@@ -0,0 +1,68 @@ >+$DBversion = 'XXX'; # will be replaced by the RM >+if( CheckVersion( $DBversion ) ) { >+ # you can use $dbh here like: >+ $dbh->do( qq{ >+ INSERT IGNORE INTO letter (module, code, name, title, content, message_transport_type) VALUES ('circulation', 'AUTO_RENEWALS_DGST', 'notification on auto renewing', 'Auto renewals (Digest)', >+ "Dear [% borrower.firstname %] [% borrower.surname %], >+ [% IF checkout.auto_renew_errors %] >+ There were [% checkout.auto_renew_errors %] items that where not correctly renewed. >+ [% END %] >+ [% IF checkout.auto_renew %] >+ There were [% checkout.auto_renew %] items that where correctly renewed. >+ [% END %] >+ ", 'email'); >+ }); >+ >+ $dbh->do( qq{ >+ INSERT IGNORE INTO `message_attributes` >+ (`message_attribute_id`, message_name, `takes_days`) >+ VALUES (7, 'Auto_Renewals', 0) >+ }); >+ >+ $dbh->do( qq{ >+ INSERT IGNORE INTO `message_transports` >+ (`message_attribute_id`, `message_transport_type`, `is_digest`, `letter_module`, `letter_code`) >+ VALUES (7, 'email', 0, 'circulation', 'AUTO_RENEWALS'), >+ (7, 'sms', 0, 'circulation', 'AUTO_RENEWALS'), >+ (7, 'email', 1, 'circulation', 'AUTO_RENEWALS_DGST'), >+ (7, 'sms', 1, 'circulation', 'AUTO_RENEWALS_DGST') >+ }); >+ >+ $dbh->do( qq{ >+ insert ignore into borrower_message_preferences (categorycode, message_attribute_id, days_in_advance, wants_digest) >+ select c.categorycode, 7, NULL, 1 >+ from categories c >+ left join >+ borrower_message_preferences p >+ on c.categorycode = p.categorycode >+ and p.message_attribute_id = 7 >+ where p.categorycode is null >+ >+ }); >+ >+ $dbh->do( qq{ >+ insert ignore into borrower_message_preferences (borrowernumber, message_attribute_id, days_in_advance, wants_digest) >+ select b.borrowernumber, 7, NULL, 1 >+ from borrowers b >+ left join >+ borrower_message_preferences p >+ on b.borrowernumber = p.borrowernumber >+ and p.message_attribute_id = 7 >+ where p.borrowernumber is null >+ }); >+ >+ $dbh->do( qq{ >+ insert into borrower_message_transport_preferences (borrower_message_preference_id, message_transport_type) >+ select p.borrower_message_preference_id, 'email' >+ from borrower_message_preferences p >+ left join >+ borrower_message_transport_preferences t >+ on p.borrower_message_preference_id = t.borrower_message_preference_id >+ where p.message_attribute_id = 7 >+ and t.borrower_message_preference_id is null; >+ }); >+ >+ # Always end with this (adjust the bug info) >+ SetVersion( $DBversion ); >+ print "Upgrade to $DBversion done (Bug 18532 - Add AUTO_RENEWALS in message_attributes and add AUTO_RENEWALS_DGST letter)\n"; >+} >diff --git a/installer/data/mysql/mandatory/sample_notices_message_attributes.sql b/installer/data/mysql/mandatory/sample_notices_message_attributes.sql >index a7a6032cc7..32ab818d03 100644 >--- a/installer/data/mysql/mandatory/sample_notices_message_attributes.sql >+++ b/installer/data/mysql/mandatory/sample_notices_message_attributes.sql >@@ -5,5 +5,6 @@ values > (2, 'Advance_Notice', 1), > (4, 'Hold_Filled', 0), > (5, 'Item_Check_in', 0), >-(6, 'Item_Checkout', 0); >+(6, 'Item_Checkout', 0), >+(7, 'Auto_Renewals', 0); > >diff --git a/installer/data/mysql/mandatory/sample_notices_message_transports.sql b/installer/data/mysql/mandatory/sample_notices_message_transports.sql >index 5954bafc7b..eecdbdf16a 100644 >--- a/installer/data/mysql/mandatory/sample_notices_message_transports.sql >+++ b/installer/data/mysql/mandatory/sample_notices_message_transports.sql >@@ -17,4 +17,8 @@ values > (5, 'email', 0, 'circulation', 'CHECKIN'), > (5, 'sms', 0, 'circulation', 'CHECKIN'), > (6, 'email', 0, 'circulation', 'CHECKOUT'), >-(6, 'sms', 0, 'circulation', 'CHECKOUT'); >+(6, 'sms', 0, 'circulation', 'CHECKOUT'), >+(7, 'email', 0, 'circulation', 'AUTO_RENEWALS'), >+(7, 'sms', 0, 'circulation', 'AUTO_RENEWALS'), >+(7, 'email', 1, 'circulation', 'AUTO_RENEWALS_DGST'), >+(7, 'sms', 1, 'circulation', 'AUTO_RENEWALS_DGST'); >diff --git a/installer/data/mysql/pl-PL/mandatory/sample_notices.sql b/installer/data/mysql/pl-PL/mandatory/sample_notices.sql >index dbfcc7b24d..f3d4e10933 100644 >--- a/installer/data/mysql/pl-PL/mandatory/sample_notices.sql >+++ b/installer/data/mysql/pl-PL/mandatory/sample_notices.sql >@@ -397,3 +397,24 @@ Your total unpaid fines are too high. > [% ELSE %] > The following item, [% biblio.title %], has correctly been renewed and is now due on [% checkout.date_due | $KohaDates as_due_date => 1 %] > [% END %]", 'email'); >+======= >+You have overdues. >+[% ELSIF checkout.auto_renew_error == 'auto_too_late' %] >+It\'s too late to renew this checkout. >+[% ELSIF checkout.auto_renew_error == 'auto_too_much_oweing' %] >+You have too much unpaid fines. >+[% END %] >+[% ELSE %] >+The following item [% biblio.title %] has correctly been renewed and is now due [% checkout.date_due %] >+[% END %]", 'email'); >+ >+INSERT INTO letter (module, code, name, title, content, message_transport_type) VALUES ('circulation', 'AUTO_RENEWALS_DGST', 'notification on auto renewing', 'Auto renewals (Digest)', >+"Dear [% borrower.firstname %] [% borrower.surname %], >+[% IF <<error>> %] >+There were <<error>> items that where not correctly renewed. >+[% END %] >+[% IF <<success>> %] >+There were <<success>> items that where correctly renewed. >+[% END %] >+", 'email'); >+>>>>>>> Bug 18532: Add new letter AUTO_RENEWALS_DGST, new Auto_Renewals in message_attributes, and configured message_transports:installer/data/mysql/en/mandatory/sample_notices.sql >-- >2.25.0
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 18532
:
95030
|
95031
|
95337
|
95338
|
95456
|
96049
|
96050
|
96487
|
96494
|
96495
|
98078
|
98079
|
99193
|
99194
|
104880
|
104881
|
104882
|
104883
|
105500
|
105501
|
105502
|
105503
|
114316
|
115798
|
115799
|
115800
|
115801
|
115802
|
115829
|
115830
|
115831
|
115832
|
115833
|
118198
|
118199
|
118205
|
118318
|
118329
|
118330
|
118331
|
118356