Bug 31157: (QA follow-up) Update descriptions and wording of configuration options
authorKatrin Fischer <katrin.fischer@bsz-bw.de>
Fri, 22 Jul 2022 15:57:25 +0000 (15:57 +0000)
committerTomas Cohen Arazi <tomascohen@theke.io>
Fri, 22 Jul 2022 18:14:43 +0000 (15:14 -0300)
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
installer/data/mysql/atomicupdate/bug_31157.pl
installer/data/mysql/mandatory/sysprefs.sql
koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref
misc/cronjobs/overdue_notices.pl

index 0165a43..a3be6dd 100644 (file)
@@ -9,7 +9,7 @@ return {
 
         $dbh->do(q{
             INSERT IGNORE INTO systempreferences ( `variable`, `value`, `options`, `explanation`, `type` ) VALUES
-                ('OverdueNoticeFrom', 'cron', 'cron|item-issuebranch|item-homebranch', 'Use the choice as the from address when generating overdue notices', 'Choice')
+                ('OverdueNoticeFrom', 'cron', 'cron|item-issuebranch|item-homebranch', 'Organize and send overdue notices by item home library or checkout library', 'Choice')
         });
     },
 };
index a94351c..259d7f0 100644 (file)
@@ -521,7 +521,7 @@ INSERT INTO systempreferences ( `variable`, `value`, `options`, `explanation`, `
 ('OverDrivePasswordRequired','0',NULL,'Does the library require passwords for OverDrive SIP authentication','YesNo'),
 ('OverDriveUsername','cardnumber','cardnumber|userid','Which patron information should be passed as OverDrive username','Choice'),
 ('OverdueNoticeCalendar',0,NULL,'Take the calendar into consideration when generating overdue notices','YesNo'),
-('OverdueNoticeFrom', 'cron', 'cron|item-issuebranch|item-homebranch', 'Use the choice as the from address when generating overdue notices', 'Choice'),
+('OverdueNoticeFrom', 'cron', 'cron|item-issuebranch|item-homebranch', 'Organize and send overdue notices by item home library or checkout library', 'Choice'),
 ('OverduesBlockCirc','noblock','noblock|confirmation|block','When checking out an item should overdues block checkout, generate a confirmation dialogue, or allow checkout','Choice'),
 ('OverduesBlockRenewing','allow','allow|blockitem|block','If any of patron checked out documents is late, should renewal be allowed, blocked only on overdue items or blocked on whatever checked out document','Choice'),
 ('PassItemMarcToXSLT','0',NULL,'If enabled, item fields in the MARC record will be made avaiable to XSLT sheets. Otherwise they will be removed.','YesNo'),
index cd3fc97..a67236b 100644 (file)
@@ -408,13 +408,13 @@ Circulation:
                   0: "Ignore calendar"
             - "when working out the period for overdue notices."
         -
-            - "Use the"
+            - "Organize and send overdue notices using the"
             - pref: OverdueNoticeFrom
               choices:
-                  cron: "system default"
+                  cron: "command-line option"
                   item-issuebranch: "checkout library"
                   item-homebranch: "item home library"
-            - "as the from address when generating overdue notices."
+            - .
         -
             - Include up to
             - pref: PrintNoticesMaxLines
index 28b748e..86a0cb0 100755 (executable)
@@ -69,7 +69,9 @@ overdue_notices.pl
    --date         <yyyy-mm-dd>     Emulate overdues run for this date.
    --email        <email_type>     Type of email that will be used.
                                    Can be 'email', 'emailpro' or 'B_email'. Repeatable.
-   --frombranch                    Set the from address for the notice to one of 'item-homebranch' or 'item-issuebranch'.
+   --frombranch                    Organize and send overdue notices by home library (item-homebranch) or checkout library (item-issuebranch).
+                                   This option is only used, if the OverdueNoticeFrom system preference is set to 'command-line option'.
+                                   Defaults to item-issuebranch.
 
 =head1 OPTIONS
 
@@ -180,9 +182,9 @@ Allows to specify which type of email will be used. Can be email, emailpro or B_
 
 =item B<--frombranch>
 
-Use the address information from the item homebranch library instead of the issuing library.
-
-Defaults to 'item-issuebranch'
+Organize overdue notices either by checkout library (item-issuebranch) or item home library (item-homebranch).
+This option is only used, if the OverdueNoticeFrom system preference is set to use 'command-line option'.
+Defaults to checkout library (item-issuebranch).
 
 =back