Jonathan Druart
d019f86f67
The subroutine C4::Koha::GetAuthValCode returned the authorised value category for a given kohafield. This can be acchieve easily using a new Koha::AuthorisedValues->search_by_koha_field method which will mimic search_by_marc_field. Test plan: Confirm that the description is correctly displayed on the following pages: - detail and moredetail of a bibliographic page (itemlost, damaged, materials) - Set AcqCreateItem=ordering and receiving items. The description for notforloan, restricted, location, ccode, etc. field should be displayed. - Items search form - On the checkout list from the circulation.pl and returns.pl pages, the description for "materials" should be displayed Note that GetKohaAuthorisedValuesMapping is going to be removed on bug 17251. Followed test plan, works as expected. Signed-off-by: Marc Véron <veron@veron.ch> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
142 lines
6.3 KiB
Perl
142 lines
6.3 KiB
Perl
#!/usr/bin/perl
|
|
|
|
use Modern::Perl;
|
|
use Test::More tests => 15;
|
|
|
|
use C4::Context;
|
|
use Koha::AuthorisedValue;
|
|
use Koha::AuthorisedValues;
|
|
use Koha::AuthorisedValueCategories;
|
|
use Koha::MarcSubfieldStructures;
|
|
|
|
my $dbh = C4::Context->dbh;
|
|
$dbh->{AutoCommit} = 0;
|
|
$dbh->{RaiseError} = 1;
|
|
|
|
$dbh->do("DELETE FROM authorised_values");
|
|
$dbh->do("DELETE FROM authorised_value_categories");
|
|
|
|
# insert
|
|
Koha::AuthorisedValueCategory->new({ category_name => 'av_for_testing' })->store;
|
|
Koha::AuthorisedValueCategory->new({ category_name => 'aaav_for_testing' })->store;
|
|
my $av1 = Koha::AuthorisedValue->new(
|
|
{
|
|
category => 'av_for_testing',
|
|
authorised_value => 'value 1',
|
|
lib => 'display value 1',
|
|
lib_opac => 'opac display value 1',
|
|
imageurl => 'image1.png',
|
|
}
|
|
)->store();
|
|
|
|
my $av2 = Koha::AuthorisedValue->new(
|
|
{
|
|
category => 'av_for_testing',
|
|
authorised_value => 'value 2',
|
|
lib => 'display value 2',
|
|
lib_opac => 'opac display value 2',
|
|
imageurl => 'image2.png',
|
|
}
|
|
)->store();
|
|
|
|
my $av3 = Koha::AuthorisedValue->new(
|
|
{
|
|
category => 'av_for_testing',
|
|
authorised_value => 'value 3',
|
|
lib => 'display value 3',
|
|
lib_opac => 'opac display value 3',
|
|
imageurl => 'image2.png',
|
|
}
|
|
)->store();
|
|
|
|
my $av4 = Koha::AuthorisedValue->new(
|
|
{
|
|
category => 'aaav_for_testing',
|
|
authorised_value => 'value 4',
|
|
lib => 'display value 4',
|
|
lib_opac => 'opac display value 4',
|
|
imageurl => 'image4.png',
|
|
}
|
|
)->store();
|
|
|
|
ok( $av1->id(), 'AV 1 is inserted' );
|
|
ok( $av2->id(), 'AV 2 is inserted' );
|
|
ok( $av3->id(), 'AV 3 is inserted' );
|
|
ok( $av4->id(), 'AV 4 is inserted' );
|
|
|
|
is( $av3->opac_description, 'opac display value 3', 'Got correction opac description if lib_opac is set' );
|
|
$av3->lib_opac('');
|
|
is( $av3->opac_description, 'display value 3', 'Got correction opac description if lib_opac is *not* set' );
|
|
|
|
my @authorised_values =
|
|
Koha::AuthorisedValues->new()->search( { category => 'av_for_testing' } );
|
|
is( @authorised_values, 3, "Get correct number of values" );
|
|
|
|
my $branches_rs = Koha::Database->new()->schema()->resultset('Branch')->search();
|
|
my $branch1 = $branches_rs->next();
|
|
my $branchcode1 = $branch1->branchcode();
|
|
my $branch2 = $branches_rs->next();
|
|
my $branchcode2 = $branch2->branchcode();
|
|
|
|
$av1->add_branch_limitation( $branchcode1 );
|
|
|
|
@authorised_values = Koha::AuthorisedValues->new()->search( { category => 'av_for_testing', branchcode => $branchcode1 } );
|
|
is( @authorised_values, 3, "Search including value with a branch limit ( branch can use the limited value ) gives correct number of results" );
|
|
|
|
@authorised_values = Koha::AuthorisedValues->new()->search( { category => 'av_for_testing', branchcode => $branchcode2 } );
|
|
is( @authorised_values, 2, "Search including value with a branch limit ( branch *cannot* use the limited value ) gives correct number of results" );
|
|
|
|
$av1->del_branch_limitation( $branchcode1 );
|
|
@authorised_values = Koha::AuthorisedValues->new()->search( { category => 'av_for_testing', branchcode => $branchcode2 } );
|
|
is( @authorised_values, 3, "Branch limitation deleted successfully" );
|
|
|
|
$av1->add_branch_limitation( $branchcode1 );
|
|
$av1->branch_limitations( [ $branchcode1, $branchcode2 ] );
|
|
|
|
my $limits = $av1->branch_limitations;
|
|
is( @$limits, 2, 'branch_limitations functions correctly both as setter and getter' );
|
|
|
|
my @categories = Koha::AuthorisedValues->new->categories;
|
|
is( @categories, 2, 'There should have 2 categories inserted' );
|
|
is( $categories[0], $av4->category, 'The first category should be correct (ordered by category name)' );
|
|
is( $categories[1], $av1->category, 'The second category should be correct (ordered by category name)' );
|
|
|
|
subtest 'search_by_*_field' => sub {
|
|
plan tests => 2;
|
|
my $loc_cat = Koha::AuthorisedValueCategories->find('LOC');
|
|
$loc_cat->delete if $loc_cat;
|
|
my $mss = Koha::MarcSubfieldStructures->search( { tagfield => 952, tagsubfield => 'c', frameworkcode => '' } );
|
|
$mss->delete if $mss;
|
|
$mss = Koha::MarcSubfieldStructures->search( { tagfield => 952, tagsubfield => 'd', frameworkcode => '' } );
|
|
$mss->delete if $mss;
|
|
Koha::AuthorisedValueCategory->new( { category_name => 'LOC' } )->store;
|
|
Koha::AuthorisedValueCategory->new( { category_name => 'ANOTHER_4_TESTS' } )->store;
|
|
Koha::MarcSubfieldStructure->new( { tagfield => 952, tagsubfield => 'c', frameworkcode => '', authorised_value => 'LOC', kohafield => 'items.location' } )->store;
|
|
Koha::MarcSubfieldStructure->new( { tagfield => 952, tagsubfield => 'c', frameworkcode => 'ACQ', authorised_value => 'LOC', kohafield => 'items.location' } )->store;
|
|
Koha::MarcSubfieldStructure->new( { tagfield => 952, tagsubfield => 'd', frameworkcode => '', authorised_value => 'ANOTHER_4_TESTS', kohafield => 'items.another_field' } )->store;
|
|
Koha::AuthorisedValue->new( { category => 'LOC', authorised_value => 'location_1' } )->store;
|
|
Koha::AuthorisedValue->new( { category => 'LOC', authorised_value => 'location_2' } )->store;
|
|
Koha::AuthorisedValue->new( { category => 'LOC', authorised_value => 'location_3' } )->store;
|
|
Koha::AuthorisedValue->new( { category => 'ANOTHER_4_TESTS', authorised_value => 'an_av' } )->store;
|
|
Koha::AuthorisedValue->new( { category => 'ANOTHER_4_TESTS', authorised_value => 'another_av' } )->store;
|
|
subtest 'search_by_marc_field' => sub {
|
|
plan tests => 4;
|
|
my $avs;
|
|
$avs = Koha::AuthorisedValues->search_by_marc_field();
|
|
is ( $avs, undef );
|
|
$avs = Koha::AuthorisedValues->search_by_marc_field({ frameworkcode => '' });
|
|
is ( $avs, undef );
|
|
$avs = Koha::AuthorisedValues->search_by_marc_field({ tagfield => 952, tagsubfield => 'c'});
|
|
is( $avs->count, 3, 'default fk');
|
|
is( $avs->next->authorised_value, 'location_1', );
|
|
};
|
|
subtest 'search_by_koha_field' => sub {
|
|
plan tests => 3;
|
|
my $avs;
|
|
$avs = Koha::AuthorisedValues->search_by_koha_field();
|
|
is ( $avs, undef );
|
|
$avs = Koha::AuthorisedValues->search_by_koha_field( { kohafield => 'items.location', tagfield => 952, tagsubfield => 'c' } );
|
|
is( $avs->count, 3, );
|
|
is( $avs->next->authorised_value, 'location_1', );
|
|
};
|
|
};
|