Bugzilla – Attachment 75685 Details for
Bug 20837
CanItemBeReserved should follow ReservesControlBranch and not CircControl
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 20837: Use ReservesControlBranch to determine holdability of items
Bug-20837-Use-ReservesControlBranch-to-determine-h.patch (text/plain), 2.68 KB, created by
Nick Clemens (kidclamp)
on 2018-05-30 17:32:01 UTC
(
hide
)
Description:
Bug 20837: Use ReservesControlBranch to determine holdability of items
Filename:
MIME Type:
Creator:
Nick Clemens (kidclamp)
Created:
2018-05-30 17:32:01 UTC
Size:
2.68 KB
patch
obsolete
>From c7720bf96a6ab257e8dd8367685f4cb6022a877b Mon Sep 17 00:00:00 2001 >From: Nick Clemens <nick@bywatersolutions.com> >Date: Wed, 30 May 2018 16:05:04 +0000 >Subject: [PATCH] Bug 20837: Use ReservesControlBranch to determine holdability > of items > >To test: > 1 - Set a default holds plicy for an item type as 'from home library' > 2 - Set CircControl to 'logged in library' > 3 - Log in as staff from a library without the rule set > 4 - Note you can set holds for patrons of any library on items of type >above > 5 - Toggle ReservesControlBranch and note that nothing changes > 6 - Toggle CircControl and note some combinations don't allow holds > 7 - Apply patch > 8 - Toggle ReservesControlBranch and note that holdability is affected > 9 - Toggle CircControl and note holdability does not change >10 - Read the note on admin/smartrules.pl and confirm it makes sense >--- > C4/Reserves.pm | 2 +- > koha-tmpl/intranet-tmpl/prog/en/modules/admin/smart-rules.tt | 3 ++- > 2 files changed, 3 insertions(+), 2 deletions(-) > >diff --git a/C4/Reserves.pm b/C4/Reserves.pm >index ff29b5f..b5ae24d 100644 >--- a/C4/Reserves.pm >+++ b/C4/Reserves.pm >@@ -410,7 +410,7 @@ sub CanItemBeReserved { > } > > my $circ_control_branch = >- C4::Circulation::_GetCircControlBranch( $item->unblessed(), $borrower ); >+ GetReservesControlBranch( $item->unblessed(), $borrower ); > my $branchitemrule = > C4::Circulation::GetBranchItemRule( $circ_control_branch, $item->itype ); > >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 86226cb..9c57963 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 >@@ -1,4 +1,5 @@ > [% USE Asset %] >+[% USE Koha %] > [% USE Branches %] > [% SET footerjs = 1 %] > [% INCLUDE 'doc-head-open.inc' %] >@@ -628,7 +629,7 @@ > <li><strong>No holds allowed:</strong> No patron may put this book on hold.</li> > </ul> > <p><strong>Note: </strong>If the system preference 'AllowHoldPolicyOverride' is enabled, these policies can be overridden by your circulation staff.</br /> >- <strong>Important: </strong>The policies are based on the patron's home library, not the library where the hold is being placed. >+ <strong>Important: </strong>The policies are applied based on the ReservesControlBranch system preference which is set to <a href="/cgi-bin/koha/admin/preferences.pl?op=search&searchfield=ReservesControlBranch">[% Koha.Preference('ReservesControlBranch') %]</a>. > </p> > > <form method="post" action="/cgi-bin/koha/admin/smart-rules.pl"> >-- >2.1.4
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 20837
:
75684
|
75685
|
87021
|
87022
|
87023
|
87024
|
87025
|
87409
|
87410
|
87673
|
87674
|
87675
|
88699
|
88700
|
88701