From 5a931e4e93cf706602551423e03b8702166bb41c Mon Sep 17 00:00:00 2001 From: Jesse Weaver 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. Signed-off-by: Lisette Scheer Signed-off-by: Jesse Maseto Signed-off-by: Josef Moravec Signed-off-by: Nick Clemens --- Koha/CirculationRules.pm | 5 ++++- admin/smart-rules.pl | 14 +++++++++++++- .../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 5ddf97af3c..509f0db30d 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 @@   Total current checkouts allowed Total current on-site checkouts allowed + Maximum total holds allowed (count) Hold policy Hold pickup library match Return policy @@ -378,6 +379,10 @@ Defaults[% UNLESS ( default_rules ) %] (not set)[% END %] + + [% SET rule_value = CirculationRules.Get( current_branch, '*', undef, 'max_holds' ) %] + +