Koha-community/Koha
13
7
Fork 0
Code Issues Releases Wiki Activity

Bug 29955: Fix method import issues

Browse source 
This patch fixes this:

$ prove t/db_dependent/Utils/Datatables_Virtualshelves.t
t/db_dependent/Utils/Datatables_Virtualshelves.t .. 2/13 Use of inherited AUTOLOAD for non-method Koha::Virtualshelf::haspermission() is no longer allowed at /kohadevbox/koha/Koha/Virtualshelf.pm line 248.

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
This commit is contained in:
Tomás Cohen Arazi 2022-08-22 13:06:14 -03:00
parent 20108c837e
commit 7fcda17eab
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
Koha/Virtualshelf.pm
View file

@ -18,7 +18,7 @@ package Koha::Virtualshelf;
use Modern::Perl;
use C4::Auth qw( haspermission );
use C4::Auth;
use Koha::Patrons;
use Koha::Database;
@ -234,7 +234,7 @@ sub can_be_deleted {
my $patron = Koha::Patrons->find( $borrowernumber ) or return 0;
return 1 if $self->is_public and haspermission( $patron->userid, { lists => 'delete_public_lists' } );
return 1 if $self->is_public and C4::Auth::haspermission( $patron->userid, { lists => 'delete_public_lists' } );
return 0;
}
@ -246,7 +246,7 @@ sub can_be_managed {
my $patron = Koha::Patrons->find( $borrowernumber ) or return 0;
return 1
if $self->is_public and haspermission( $patron->userid, { lists => 'edit_public_lists' } );
if $self->is_public and C4::Auth::haspermission( $patron->userid, { lists => 'edit_public_lists' } );
return 0;
}
@ -311,7 +311,7 @@ sub cannot_be_transferred {
if( $self->public ) {
my $by_patron = Koha::Patrons->find($by);
return 'unauthorized_transfer'
if !$by_patron || !haspermission( $by_patron->userid, { lists => 'edit_public_lists' });
if !$by_patron || !C4::Auth::haspermission( $by_patron->userid, { lists => 'edit_public_lists' });
} else {
return 'unauthorized_transfer' if !$self->can_be_managed($by);
}