Bug 14142 - Holds queue viewer only displays first subtitle from marc keyword mappings
Despite the point of the Keyword to MARC Mappings being to simplify the handling and display of repeated values from multiple subfields, the holds queue viewer will only display the first value found. What it should be doing instead is displaying all fields that match the subtitle keyword. Test Plan: 1) Apply this patch 2) Define multiple Keyword to MARC mappings for the subtitle keyword 3) Place a hold on a record using those subtitle fields 4) View the hold in the holds queue viewer 5) Note that all the subtitles now appear Signed-off-by:Heather Braum <hbraum@nekls.org> Signed-off-by: Jonathan Druart <jonathan.druart@koha-community.org> Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
This commit is contained in:
parent
3cd086b6b6
commit
d76c9f4850
2 changed files with 2 additions and 2 deletions
|
@ -137,7 +137,7 @@ sub GetHoldsQueueItems {
|
|||
while ( my $row = $sth->fetchrow_hashref ){
|
||||
my $record = GetMarcBiblio($row->{biblionumber});
|
||||
if ($record){
|
||||
$row->{subtitle} = GetRecordValue('subtitle',$record,'')->[0]->{subfield};
|
||||
$row->{subtitle} = [ map { $_->{subfield} } @{ GetRecordValue( 'subtitle', $record, '' ) } ];
|
||||
$row->{parts} = GetRecordValue('parts',$record,'')->[0]->{subfield};
|
||||
$row->{numbers} = GetRecordValue('numbers',$record,'')->[0]->{subfield};
|
||||
}
|
||||
|
|
|
@ -68,7 +68,7 @@ $(document).ready(function() {
|
|||
<tbody>[% FOREACH itemsloo IN itemsloop %]
|
||||
<tr>
|
||||
<td class="hq-title">
|
||||
<p><a href="/cgi-bin/koha/catalogue/detail.pl?biblionumber=[% itemsloo.biblionumber %]"><strong>[% itemsloo.title | html %]</strong> [% IF ( itemsloo.subtitle ) %][% itemsloo.subtitle %][% END %]</a></p>
|
||||
<p><a href="/cgi-bin/koha/catalogue/detail.pl?biblionumber=[% itemsloo.biblionumber %]"><strong>[% itemsloo.title | html %]</strong> [% FOREACH s IN itemsloo.subtitle %] [% s %][% END %]</a></p>
|
||||
<p>
|
||||
<div class="hq-biblionumber content_hidden">[% itemsloo.biblionumber %]</div>
|
||||
<div class="hq-author">[% itemsloo.author %]</div>
|
||||
|
|
Loading…
Reference in a new issue