Attachment 65130 Details for Bug 15486 – Bug 15486: Make circ rules UI handle holds_per_day

[patch] Bug 15486: Make circ rules UI handle holds_per_day

Bug-15486-Make-circ-rules-UI-handle-holdsperday.patch (text/plain), 4.20 KB, created by Tomás Cohen Arazi (tcohen) on 2017-07-19 22:43:26 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Tomás Cohen Arazi (tcohen)

Created: 2017-07-19 22:43:26 UTC

Size: 4.20 KB

patch

obsolete

>From 115d7fa6a7c12caf3b5a9a9a1b6423b71cc6313f Mon Sep 17 00:00:00 2001
>From: Tomas Cohen Arazi <tomascohen@theke.io>
>Date: Wed, 19 Jul 2017 15:57:52 -0300
>Subject: [PATCH] Bug 15486: Make circ rules UI handle holds_per_day
>
>This patch makes the staff UI correctly handle the holds_per_day
>configuration.
>---
> admin/smart-rules.pl                                         |  2 ++
> koha-tmpl/intranet-tmpl/prog/en/modules/admin/smart-rules.tt | 10 +++++++---
> 2 files changed, 9 insertions(+), 3 deletions(-)
>
>diff --git a/admin/smart-rules.pl b/admin/smart-rules.pl
>index 6a55b7bde4..b72a70a707 100755
>--- a/admin/smart-rules.pl
>+++ b/admin/smart-rules.pl
>@@ -143,6 +143,7 @@ elsif ($op eq 'add') {
>     $no_auto_renewal_after_hard_limit = output_pref( { dt => $no_auto_renewal_after_hard_limit, dateonly => 1, dateformat => 'iso' } ) if ( $no_auto_renewal_after_hard_limit );
>     my $reservesallowed  = $input->param('reservesallowed');
>     my $holds_per_record  = $input->param('holds_per_record');
>+    my $holds_per_day = $input->param('holds_per_day');
>     my $onshelfholds     = $input->param('onshelfholds') || 0;
>     $maxissueqty =~ s/\s//g;
>     $maxissueqty = undef if $maxissueqty !~ /^\d+/;
>@@ -182,6 +183,7 @@ elsif ($op eq 'add') {
>         no_auto_renewal_after_hard_limit => $no_auto_renewal_after_hard_limit,
>         reservesallowed               => $reservesallowed,
>         holds_per_record              => $holds_per_record,
>+        holds_per_day                 => $holds_per_day,
>         issuelength                   => $issuelength,
>         lengthunit                    => $lengthunit,
>         hardduedate                   => $hardduedate,
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/smart-rules.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/smart-rules.tt
>index 0283d80fe9..4c0402dc25 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/smart-rules.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/smart-rules.tt
>@@ -187,7 +187,8 @@ $(document).ready(function() {
>                 <th>Automatic renewal</th>
>                 <th>No automatic renewal after</th>
>                 <th>No automatic renewal after (hard limit)</th>
>-                <th>Holds allowed (count)</th>
>+                <th>Holds allowed (total)</th>
>+                <th>Holds allowed (daily)</th>
>                 <th>Holds per record (count)</th>
>                 <th>On shelf holds allowed</th>
>                 <th>Item level holds</th>
>@@ -270,6 +271,7 @@ $(document).ready(function() {
>                             <td>[% rule.no_auto_renewal_after %]</td>
>                             <td>[% rule.no_auto_renewal_after_hard_limit %]</td>
> 							<td>[% rule.reservesallowed %]</td>
>+                            <td>[% rule.holds_per_day %]</td>
>                                                         <td>[% rule.holds_per_record %]</td>
>                                                         <td>
>                                                             [% IF rule.onshelfholds == 1 %]
>@@ -361,6 +363,7 @@ $(document).ready(function() {
>                         <div class="hint">[% INCLUDE 'date-format.inc' %]</div>
>                     </td>
>                     <td><input type="text" name="reservesallowed" id="reservesallowed" size="2" /></td>
>+                    <td><input type="text" name="holds_per_day" id="holds_per_day" size="2" /></td>
>                     <td><input type="text" name="holds_per_record" id="holds_per_record" size="2" /></td>
>                     <td>
>                         <select name="onshelfholds" id="onshelfholds">
>@@ -413,8 +416,9 @@ $(document).ready(function() {
>                       <th>No renewal before</th>
>                       <th>Automatic renewal</th>
>                       <th>No automatic renewal after</th>
>-                       <th>No automatic renewal after (hard limit)</th>
>-                      <th>Holds allowed (count)</th>
>+                      <th>No automatic renewal after (hard limit)</th>
>+                      <th>Holds allowed (total)</th>
>+                      <th>Holds allowed (daily)</th>
>                       <th>Holds per record (count)</th>
>                       <th>On shelf holds allowed</th>
>                       <th>Item level holds</th>
>-- 
>2.13.3

Actions: View | Diff | Splinter Review

Attachments on bug 15486: 65128 | 65129 | 65130 | 65131 | 65132 | 65133 | 67202 | 67203 | 67204 | 67205 | 67206 | 67207 | 67337 | 67840 | 67841 | 67842 | 67843 | 67844 | 67845 | 68728 | 68729 | 68730 | 68731 | 68732 | 68733 | 68734 | 68735 | 68750 | 68751 | 68752 | 68791 | 71314 | 71315 | 71316 | 71317 | 71318 | 71319 | 72709 | 79318 | 79319 | 79320 | 79321 | 79322 | 79323 | 79324 | 79696 | 79697 | 79698 | 79699 | 79700 | 79701 | 79702 | 80478 | 80479 | 80480 | 80481 | 80482 | 80483 | 80484 | 81087