Attachment 14217 Details for Bug 9151 – Bug 9151 - Renewal notice according to patron alert preferences - QA Followup - Fix DB Revs & Grammar

[patch] Bug 9151 - Renewal notice according to patron alert preferences - QA Followup - Fix DB Revs & Grammar

Bug-9151---Renewal-notice-according-to-patron-aler.patch (text/plain), 5.18 KB, created by Paul Poulain on 2012-12-20 17:39:15 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Paul Poulain

Created: 2012-12-20 17:39:15 UTC

Size: 5.18 KB

patch

obsolete

>From 83c5355b6d0889d1e0caa4638e5b5b067c2bfde3 Mon Sep 17 00:00:00 2001
>From: Kyle M Hall <kyle@bywatersolutions.com>
>Date: Fri, 7 Dec 2012 07:48:05 -0500
>Subject: [PATCH] Bug 9151 - Renewal notice according to patron alert
> preferences - QA Followup - Fix DB Revs & Grammar
>
>---
> .../data/mysql/en/mandatory/sample_notices.sql     |    1 +
> installer/data/mysql/updatedatabase.pl             |   14 +++-----------
> .../en/modules/admin/preferences/circulation.pref  |    4 ++--
> 3 files changed, 6 insertions(+), 13 deletions(-)
>
>diff --git a/installer/data/mysql/en/mandatory/sample_notices.sql b/installer/data/mysql/en/mandatory/sample_notices.sql
>index 30b7e1f..77fb501 100644
>--- a/installer/data/mysql/en/mandatory/sample_notices.sql
>+++ b/installer/data/mysql/en/mandatory/sample_notices.sql
>@@ -7,6 +7,7 @@ VALUES ('circulation','ODUE','Overdue Notice','Item Overdue','Dear <<borrowers.f
> ('circulation','DUEDGST','Item Due Reminder (Digest)','Item Due Reminder','You have <<count>> items due'), 
> ('circulation','PREDUE','Advance Notice of Item Due','Advance Notice of Item Due','Dear <<borrowers.firstname>> <<borrowers.surname>>,\r\n\r\nThe following item will be due soon:\r\n\r\n<<biblio.title>>, <<biblio.author>> (<<items.barcode>>)'), 
> ('circulation','PREDUEDGST','Advance Notice of Item Due (Digest)','Advance Notice of Item Due','You have <<count>> items due soon'),
>+('circulation','RENEWAL','Item Renewals','Item Renewals','The following items have been renewed:\r\n----\r\n<<biblio.title>>\r\n----\r\nThank you for visiting <<branches.branchname>>.');
> ('reserves', 'HOLD', 'Hold Available for Pickup', 'Hold Available for Pickup at <<branches.branchname>>', 'Dear <<borrowers.firstname>> <<borrowers.surname>>,\r\n\r\nYou have a hold available for pickup as of <<reserves.waitingdate>>:\r\n\r\nTitle: <<biblio.title>>\r\nAuthor: <<biblio.author>>\r\nCopy: <<items.copynumber>>\r\nLocation: <<branches.branchname>>\r\n<<branches.branchaddress1>>\r\n<<branches.branchaddress2>>\r\n<<branches.branchaddress3>>\r\n<<branches.branchcity>> <<branches.branchzip>>'),
> ('reserves', 'HOLD_PRINT', 'Hold Available for Pickup (print notice)', 'Hold Available for Pickup (print notice)', '<<branches.branchname>>\r\n<<branches.branchaddress1>>\r\n<<branches.branchaddress2>>\r\n\r\n\r\nChange Service Requested\r\n\r\n\r\n\r\n\r\n\r\n\r\n\r\n<<borrowers.firstname>> <<borrowers.surname>>\r\n<<borrowers.address>>\r\n<<borrowers.city>> <<borrowers.zipcode>>\r\n\r\n\r\n\r\n\r\n\r\n\r\n\r\n\r\n\r\n\r\n<<borrowers.firstname>> <<borrowers.surname>> <<borrowers.cardnumber>>\r\n\r\nYou have a hold available for pickup as of <<reserves.waitingdate>>:\r\n\r\nTitle: <<biblio.title>>\r\nAuthor: <<biblio.author>>\r\nCopy: <<items.copynumber>>\r\n'),
> ('circulation','CHECKIN','Item Check-in (Digest)','Check-ins','The following items have been checked in:\r\n----\r\n<<biblio.title>>\r\n----\r\nThank you.'),
>diff --git a/installer/data/mysql/updatedatabase.pl b/installer/data/mysql/updatedatabase.pl
>index 18ab83e..f256c13 100755
>--- a/installer/data/mysql/updatedatabase.pl
>+++ b/installer/data/mysql/updatedatabase.pl
>@@ -6298,25 +6298,17 @@ if (C4::Context->preference("Version") < TransformToNum($DBversion)) {
>     SetVersion ($DBversion);
> }
> 
>-
>-$DBversion = "XXXX";
>+$DBversion = "3.11.00.XXX";
> if ( C4::Context->preference("Version") < TransformToNum($DBversion) ) {
>+    $dbh->do("INSERT INTO systempreferences (variable,value,options,explanation,type) VALUES('RenewalSendNotice', '0', '', NULL, 'YesNo')");
>     $dbh->do(q{
>         INSERT INTO `letter` (`module`, `code`, `name`, `title`, `content`) VALUES
>-        ('circulation','RENEWAL','Item Renewal','Renewals','The following items have been renew:\r\n----\r\n<<biblio.title>>\r\n----\r\nThank you for visiting <<branches.branchname>>.');
>+        ('circulation','RENEWAL','Item Renewals','Item Renewals','The following items have been renewed:\r\n----\r\n<<biblio.title>>\r\n----\r\nThank you for visiting <<branches.branchname>>.');
>     });
>     print "Upgrade to $DBversion done (Bug 9151 - Renewal notice according to patron alert preferences)\n";
>     SetVersion($DBversion);
> }
> 
>-$DBversion = "XXXX";
>-if ( C4::Context->preference('Version') < TransformToNum($DBversion) ) {
>-    $dbh->do("INSERT INTO systempreferences (variable,value,options,explanation,type) VALUES('RenewalSendNotice', '0', '', NULL, 'YesNo')");
>-    print "Upgrade to $DBversion done (Bug 9151 - System preference to controll the renewal notice sending)\n";
>-    SetVersion($DBversion);
>-}
>-
>-
> =head1 FUNCTIONS
> 
> =head2 TableExists($table)
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref
>index f121133..f32b5b1 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref
>@@ -227,8 +227,8 @@ Circulation:
>               choices:
>                   date_due: the old due date of the checkout.
>                   now: the current date.
>-	-
>-	    - pref: RenewalSendNotice
>+        -
>+            - pref: RenewalSendNotice
>               choices:
>                   yes: Send
>                   no: "Don't send"
>-- 
>1.7.9.5

Actions: View | Diff | Splinter Review

Attachments on bug 9151: 13704 | 13705 | 13913 | 13914 | 13915 | 14215 | 14216 | 14217 | 14218 | 14477 | 14638 | 14639