Kyle M Hall
e2e0c32701
Some libraries would like to allow arbitrary lists to be used by all librarians with the 'manage_patron_lists' permission. Test Plan: 1) Apply this patch 2) Run updatedatabase.pl 3) Create or find two patrons with the manage_patron_lists permission 4) Using the first patron, create two new lists, mark one of them as shared 5) Log in as the second patron, browse to the patron lists page 6) Note the second patron can view, add and remove patrons from the shared list owned by the first patron Signed-off-by: George Williams <george@nekls.org> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
113 lines
3.6 KiB
Perl
Executable file
113 lines
3.6 KiB
Perl
Executable file
#!/usr/bin/perl
|
|
|
|
# This file is part of Koha.
|
|
#
|
|
# Koha is free software; you can redistribute it and/or modify it
|
|
# under the terms of the GNU General Public License as published by
|
|
# the Free Software Foundation; either version 3 of the License, or
|
|
# (at your option) any later version.
|
|
#
|
|
# Koha is distributed in the hope that it will be useful, but
|
|
# WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
# GNU General Public License for more details.
|
|
#
|
|
# You should have received a copy of the GNU General Public License
|
|
# along with Koha; if not, see <http://www.gnu.org/licenses>.
|
|
|
|
use Modern::Perl;
|
|
|
|
use Test::More tests => 9;
|
|
use t::lib::TestBuilder;
|
|
|
|
use Koha::Database;
|
|
use Koha::List::Patron
|
|
qw( AddPatronList AddPatronsToList DelPatronList DelPatronsFromList GetPatronLists ModPatronList );
|
|
|
|
my $schema = Koha::Database->schema;
|
|
$schema->storage->txn_begin;
|
|
|
|
my $builder = t::lib::TestBuilder->new;
|
|
|
|
C4::Context->_new_userenv('DUMMY SESSION');
|
|
C4::Context->set_userenv( 0 ); # Koha::List::Patron only needs a number
|
|
|
|
# Create 10 sample borrowers
|
|
my @borrowers = ();
|
|
foreach (1..10) {
|
|
push @borrowers, $builder->build({ source => 'Borrower' });
|
|
}
|
|
|
|
my $owner = $borrowers[0]->{borrowernumber};
|
|
my $owner2 = $borrowers[1]->{borrowernumber};
|
|
|
|
my @lists = GetPatronLists( { owner => $owner } );
|
|
my $list_count_original = @lists;
|
|
|
|
my $list1 = AddPatronList( { name => 'Test List 1', owner => $owner } );
|
|
is( $list1->name(), 'Test List 1', 'AddPatronList works' );
|
|
|
|
my $list2 = AddPatronList( { name => 'Test List 2', owner => $owner } );
|
|
|
|
ModPatronList(
|
|
{
|
|
patron_list_id => $list2->patron_list_id(),
|
|
name => 'Test List 3',
|
|
owner => $owner
|
|
}
|
|
);
|
|
$list2->discard_changes();
|
|
is( $list2->name(), 'Test List 3', 'ModPatronList works' );
|
|
|
|
AddPatronsToList(
|
|
{ list => $list1, cardnumbers => [ map { $_->{cardnumber} } @borrowers ] }
|
|
);
|
|
is(
|
|
scalar @borrowers,
|
|
$list1->patron_list_patrons()->search_related('borrowernumber')->all(),
|
|
'AddPatronsToList works for cardnumbers'
|
|
);
|
|
|
|
AddPatronsToList(
|
|
{
|
|
list => $list2,
|
|
borrowernumbers => [ map { $_->{borrowernumber} } @borrowers ]
|
|
}
|
|
);
|
|
is(
|
|
scalar @borrowers,
|
|
$list2->patron_list_patrons()->search_related('borrowernumber')->all(),
|
|
'AddPatronsToList works for borrowernumbers'
|
|
);
|
|
|
|
my @ids =
|
|
$list1->patron_list_patrons()->get_column('patron_list_patron_id')->all();
|
|
DelPatronsFromList(
|
|
{
|
|
list => $list1,
|
|
patron_list_patrons => \@ids,
|
|
}
|
|
);
|
|
$list1->discard_changes();
|
|
is( $list1->patron_list_patrons()->count(), 0, 'DelPatronsFromList works.' );
|
|
|
|
@lists = GetPatronLists( { owner => $owner } );
|
|
is( scalar @lists, $list_count_original + 2, 'GetPatronLists works' );
|
|
|
|
my $list3 = AddPatronList( { name => 'Test List 3', owner => $owner2, shared => 0 } );
|
|
@lists = GetPatronLists( { owner => $owner } );
|
|
is( scalar @lists, $list_count_original + 2, 'GetPatronLists does not return non-shared list' );
|
|
|
|
my $list4 = AddPatronList( { name => 'Test List 4', owner => $owner2, shared => 1 } );
|
|
@lists = GetPatronLists( { owner => $owner } );
|
|
is( scalar @lists, $list_count_original + 3, 'GetPatronLists does return shared list' );
|
|
|
|
DelPatronList( { patron_list_id => $list1->patron_list_id(), owner => $owner } );
|
|
DelPatronList( { patron_list_id => $list2->patron_list_id(), owner => $owner } );
|
|
|
|
@lists =
|
|
GetPatronLists( { patron_list_id => $list1->patron_list_id(), owner => $owner } );
|
|
is( scalar @lists, 0, 'DelPatronList works' );
|
|
|
|
$schema->storage->txn_rollback;
|
|
|