Attachment 78679 Details for Bug 18887 – Bug 18887: (follow-up) Fix behavior for default branch/category

[patch] Bug 18887: (follow-up) Fix behavior for default branch/category

Bug-18887-follow-up-Fix-behavior-for-default-branc.patch (text/plain), 4.25 KB, created by Jonathan Druart on 2018-09-14 17:51:14 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2018-09-14 17:51:14 UTC

Size: 4.25 KB

patch

obsolete

>From 481ddaa4cfe82b1349014ae902f4a6e62feab73b Mon Sep 17 00:00:00 2001
>From: Jesse Weaver <pianohacker@gmail.com>
>Date: Wed, 4 Oct 2017 16:50:26 -0600
>Subject: [PATCH] Bug 18887: (follow-up) Fix behavior for default
> branch/category
>
>Also, fix a couple small QA issues.
>---
> Koha/CirculationRules.pm                                   |  5 ++++-
> admin/smart-rules.pl                                       | 14 +++++++++++++-
> .../intranet-tmpl/prog/en/modules/admin/smart-rules.tt     |  5 +++++
> 3 files changed, 22 insertions(+), 2 deletions(-)
>
>diff --git a/Koha/CirculationRules.pm b/Koha/CirculationRules.pm
>index 8004e95196..054eb00858 100644
>--- a/Koha/CirculationRules.pm
>+++ b/Koha/CirculationRules.pm
>@@ -136,7 +136,6 @@ sub set_rule {
> 
> sub set_rules {
>     my ( $self, $params ) = @_;
>-    warn Data::Dumper::Dumper( $params );
> 
>     my $branchcode   = $params->{branchcode};
>     my $categorycode = $params->{categorycode};
>@@ -164,6 +163,10 @@ sub _type {
>     return 'CirculationRule';
> }
> 
>+=head3 object_class
>+
>+=cut
>+
> sub object_class {
>     return 'Koha::CirculationRule';
> }
>diff --git a/admin/smart-rules.pl b/admin/smart-rules.pl
>index ae14a8e8eb..4ff96fcc87 100755
>--- a/admin/smart-rules.pl
>+++ b/admin/smart-rules.pl
>@@ -224,12 +224,15 @@ elsif ($op eq "set-branch-defaults") {
>     my $holdallowed   = $input->param('holdallowed');
>     my $hold_fulfillment_policy = $input->param('hold_fulfillment_policy');
>     my $returnbranch  = $input->param('returnbranch');
>+    my $max_holds = $input->param('max_holds');
>     $maxissueqty =~ s/\s//g;
>     $maxissueqty = undef if $maxissueqty !~ /^\d+/;
>     $maxonsiteissueqty =~ s/\s//g;
>     $maxonsiteissueqty = undef if $maxonsiteissueqty !~ /^\d+/;
>     $holdallowed =~ s/\s//g;
>     $holdallowed = undef if $holdallowed !~ /^\d+/;
>+    $max_holds =~ s/\s//g;
>+    $max_holds = undef if $max_holds !~ /^\d+/;
> 
>     if ($branch eq "*") {
>         my $sth_search = $dbh->prepare("SELECT count(*) AS total
>@@ -265,6 +268,15 @@ elsif ($op eq "set-branch-defaults") {
>             $sth_insert->execute($branch, $maxissueqty, $maxonsiteissueqty, $holdallowed, $hold_fulfillment_policy, $returnbranch);
>         }
>     }
>+    Koha::CirculationRules->set_rule(
>+        {
>+            branchcode   => $branch,
>+            categorycode => '*',
>+            itemtype     => undef,
>+            rule_name    => 'max_holds',
>+            rule_value   => $max_holds,
>+        }
>+    );
> }
> elsif ($op eq "add-branch-cat") {
>     my $categorycode  = $input->param('categorycode');
>@@ -337,7 +349,7 @@ elsif ($op eq "add-branch-cat") {
> 
>             Koha::CirculationRules->set_rule(
>                 {
>-                    branchcode   => undef,
>+                    branchcode   => '*',
>                     categorycode => $categorycode,
>                     itemtype     => undef,
>                     rule_name    => 'max_holds',
>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 440dbef5a6..b584f8d51e 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
>@@ -369,6 +369,7 @@
>                     <th>&nbsp;</th>
>                     <th>Total current checkouts allowed</th>
>                     <th>Total current on-site checkouts allowed</th>
>+                    <th>Maximum total holds allowed (count)</th>
>                     <th>Hold policy</th>
>                     <th>Hold pickup library match</th>
>                     <th>Return policy</th>
>@@ -379,6 +380,10 @@
>                     <td><input type="text" name="maxissueqty" size="3" value="[% default_maxissueqty | html %]"/></td>
>                     <td><input type="text" name="maxonsiteissueqty" size="3" value="[% default_maxonsiteissueqty | html %]"/></td>
>                     <td>
>+                        [% SET rule_value = CirculationRules.Get( current_branch, '*', undef, 'max_holds' ) %]
>+                        <input name="max_holds" size="3" value="[% rule_value %]" />
>+                    </td>
>+                    <td>
>                         <select name="holdallowed">
>                             [% IF ( default_holdallowed_any ) %]
>                             <option value="2" selected="selected">
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 18887: 65233 | 65234 | 65235 | 65236 | 65237 | 67620 | 70986 | 70987 | 70988 | 70989 | 70990 | 70991 | 71031 | 71042 | 72218 | 72219 | 72220 | 72221 | 72222 | 72223 | 72224 | 77080 | 77081 | 77082 | 77083 | 77084 | 77085 | 77086 | 77087 | 77207 | 77208 | 77209 | 77210 | 77211 | 77212 | 77213 | 77214 | 77215 | 77218 | 77238 | 77649 | 77650 | 77651 | 77652 | 77653 | 77654 | 77655 | 77656 | 77657 | 77658 | 77659 | 78346 | 78347 | 78348 | 78349 | 78350 | 78351 | 78352 | 78353 | 78354 | 78355 | 78356 | 78674 | 78675 | 78676 | 78677 | 78678 | 78679 | 78680 | 78681 | 78682 | 78683 | 78684 | 78743 | 78744 | 78745 | 78746 | 78747 | 78748 | 78749 | 78750 | 78751 | 78752 | 78753 | 78879 | 79119 | 79626 | 79627 | 79628 | 79629 | 79630 | 79631 | 79632 | 79633 | 79634 | 79635 | 79636 | 79637 | 79638 | 79639 | 79640 | 79641 | 79761 | 79762 | 79763 | 79764 | 79765 | 79766 | 79767 | 79768 | 79769 | 79770 | 79771 | 79772 | 79773 | 79774 | 79775 | 79776 | 79778 | 79891