Bug 10081: add library name to IndependentBranches error message
With IndependentBranches turned on, if you try to check out an item which belongs to another library you will get an error message which is missing the library name. This patch corrects the problem by passing the necessary variable to the template and outputting the library name using the KohaBranchName TT plugin. To test, turn on IndependentBranches and try to check out an item belonging to another library (note that you must test with a staff user who is not a superlibrarian). The error message you see should include the name of the library to which the item belongs: "This item belongs to Nelsonville and cannot be checked out from this location." Checkouts of items belonging to the library should be unaffected. Signed-off-by: Galen Charlton <gmc@esilibrary.com> Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> Signed-off-by: Galen Charlton <gmc@esilibrary.com>
This commit is contained in:
parent
0a6e9c48c5
commit
50ff4f4a94
|
|
@ -896,8 +896,10 @@ sub CanBookBeIssued {
|
|||
if ( C4::Context->preference("IndependentBranches") ) {
|
||||
my $userenv = C4::Context->userenv;
|
||||
if ( ($userenv) && ( $userenv->{flags} % 2 != 1 ) ) {
|
||||
$issuingimpossible{ITEMNOTSAMEBRANCH} = 1
|
||||
if ( $item->{C4::Context->preference("HomeOrHoldingBranch")} ne $userenv->{branch} );
|
||||
if ( $item->{C4::Context->preference("HomeOrHoldingBranch")} ne $userenv->{branch} ){
|
||||
$issuingimpossible{ITEMNOTSAMEBRANCH} = 1;
|
||||
$issuingimpossible{'itemhomebranch'} = $item->{C4::Context->preference("HomeOrHoldingBranch")};
|
||||
}
|
||||
$needsconfirmation{BORRNOTSAMEBRANCH} = GetBranchName( $borrower->{'branchcode'} )
|
||||
if ( $borrower->{'branchcode'} ne $userenv->{branch} );
|
||||
}
|
||||
|
|
|
|||
|
|
@ -1,3 +1,4 @@
|
|||
[% USE KohaBranchName %]
|
||||
[% USE KohaDates %]
|
||||
[% IF ( export_remove_fields OR export_with_csv_profile ) %]
|
||||
[% SET exports_enabled = 1 %]
|
||||
|
|
@ -546,7 +547,7 @@ function validate1(date) {
|
|||
[% END %]
|
||||
|
||||
[% IF ( ITEMNOTSAMEBRANCH ) %]
|
||||
<li>This item belongs to [% itemhomebranch %] and cannot be issued from this location.</li>
|
||||
<li>This item belongs to [% itemhomebranch | $KohaBranchName %] and cannot be checked out from this location.</li>
|
||||
[% END %]
|
||||
|
||||
[% IF ( USERBLOCKEDREMAINING ) %]
|
||||
|
|
|
|||
Loading…
Reference in New Issue