gitea
/
Koha-clms
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Bug 20341: Show authorized value description for withdrawn like damaged and lost 

Actually catalog detail page shows for damaged and lost items the authorized value description if exists.
We must do the same for withdrawn information.

Test plan :
1) Set for items a subfield linked to items.withdrawn and using the authorized value category 'WITHDRAWN'
2) Edit the authorized value category to add value '2' and description 'Eaten by dog'
3) Edit an item to set this withdrawn value
4) Go to staff interface and look at items in record detail page /cgi-bin/koh/catalogue/detail.pl :
=> Without patch you see 'Withdrawn' and with patch you see 'Eaten by dog'
5) Go to OPAc and look at items in record detail page /cgi-bin/koha/opac-detail.pl :
=> Without patch you see 'Withdrawn' and with patch you see 'Eaten by dog'

Signed-off-by: delaye <stephane.delaye@biblibre.com>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
This commit is contained in:
Fridolin Somers 2018-03-06 12:26:19 +01:00 committed by Nick Clemens
parent 6bf4623501
commit 830ce35795
3 changed files with 19 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
catalogue/detail.pl
View File

@ -211,6 +211,10 @@ $mss = Koha::MarcSubfieldStructures->search({ frameworkcode => $fw, kohafield =>
if ( $mss->count ) {
$template->param( itemdamagedloop => GetAuthorisedValues( $mss->next->authorised_value ) );
}
$mss = Koha::MarcSubfieldStructures->search({ frameworkcode => $fw, kohafield => 'items.withdrawn', authorised_value => { not => undef } });
if ( $mss->count ) {
$template->param( itemwithdrawnloop => GetAuthorisedValues( $mss->next->authorised_value) );
}
$mss = Koha::MarcSubfieldStructures->search({ frameworkcode => $fw, kohafield => 'items.materials', authorised_value => { not => undef } });
my %materials_map;

10
koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt
View File

@ -411,7 +411,15 @@
[% END %]
[% IF ( item.withdrawn ) %]
<span class="wdn">Withdrawn</span>
[% IF itemwithdrawnloop %]
[% FOREACH itemwithdrawnloo IN itemwithdrawnloop %]
[% IF itemwithdrawnloo.authorised_value == item.withdrawn %]
<span class="wdn">[% itemwithdrawnloo.lib %]</span>
[% END %]
[% END %]
[% ELSE %]
<span class="wdn">Withdrawn</span>
[% END %]
[% END %]
[% IF ( item.damaged ) %]

7
koha-tmpl/opac-tmpl/bootstrap/en/includes/item-status.inc
View File

@ -53,7 +53,12 @@
[% IF ( item.withdrawn ) %]
[% SET itemavailable = 0 %]
<span class="item-status withdrawn">Item withdrawn</span>
[% av_lib_include = AuthorisedValues.GetByCode( 'WITHDRAWN', item.withdrawn, 1 ) %]
[% IF av_lib_include %]
<span class="item-status withdrawn">[% av_lib_include %]</span>
[% ELSE %]
<span class="item-status withdrawn">Item withdrawn</span>
[% END %]
[% END %]