Attachment #50262 for bug #16167

View | Details | Raw Unified | Return to bug 16167
Collapse All | Expand All

Lines 507-513 sub get_template_and_user { Link Here

(-)a/C4/Auth.pm (-1 lines)
507	$template->param(	507	$template->param(
508	OpacAdditionalStylesheet => C4::Context->preference("OpacAdditionalStylesheet"),	508	OpacAdditionalStylesheet => C4::Context->preference("OpacAdditionalStylesheet"),
509	AnonSuggestions => "" . C4::Context->preference("AnonSuggestions"),	509	AnonSuggestions => "" . C4::Context->preference("AnonSuggestions"),
510	AuthorisedValueImages => C4::Context->preference("AuthorisedValueImages"),
511	BranchesLoop => GetBranchesLoop($opac_name),	510	BranchesLoop => GetBranchesLoop($opac_name),
512	BranchCategoriesLoop => $library_categories,	511	BranchCategoriesLoop => $library_categories,
513	opac_name => $opac_name,	512	opac_name => $opac_name,




    return $biblionumber;
}

=head2 get_biblio_authorised_values

find the types and values for all authorised values assigned to this biblio.

parameters:
    biblionumber
    MARC::Record of the bib

returns: a hashref mapping the authorised value to the value set for this biblionumber

  $authorised_values = {
                       'Scent'     => 'flowery',
                       'Audience'  => 'Young Adult',
                       'itemtypes' => 'SER',
                        };

Notes: forlibrarian should probably be passed in, and called something different.

=cut

sub get_biblio_authorised_values {
    my $biblionumber = shift;
    my $record       = shift;

    my $forlibrarian  = 1;                                 # are we in staff or opac?
    my $frameworkcode = GetFrameworkCode($biblionumber);

    my $authorised_values;

    my $tagslib = GetMarcStructure( $forlibrarian, $frameworkcode )
      or return $authorised_values;

    # assume that these entries in the authorised_value table are bibliolevel.
    # ones that start with 'item%' are item level.
    my $query = q(SELECT distinct authorised_value, kohafield
                    FROM marc_subfield_structure
                    WHERE authorised_value !=''
                      AND (kohafield like 'biblio%'
                       OR  kohafield like '') );
    my $bibliolevel_authorised_values = C4::Context->dbh->selectall_hashref( $query, 'authorised_value' );

    foreach my $tag ( keys(%$tagslib) ) {
        foreach my $subfield ( keys( %{ $tagslib->{$tag} } ) ) {

            # warn "checking $subfield. type is: " . ref $tagslib->{ $tag }{ $subfield };
            if ( 'HASH' eq ref $tagslib->{$tag}{$subfield} ) {
                if ( defined $tagslib->{$tag}{$subfield}{'authorised_value'} && exists $bibliolevel_authorised_values->{ $tagslib->{$tag}{$subfield}{'authorised_value'} } ) {
                    if ( defined $record->field($tag) ) {
                        my $this_subfield_value = $record->field($tag)->subfield($subfield);
                        if ( defined $this_subfield_value ) {
                            $authorised_values->{ $tagslib->{$tag}{$subfield}{'authorised_value'} } = $this_subfield_value;
                        }
                    }
                }
            }
        }
    }

    # warn ( Data::Dumper->Dump( [ $authorised_values ], [ 'authorised_values' ] ) );
    return $authorised_values;
}

=head2 CountBiblioInOrders

    $count = &CountBiblioInOrders( $biblionumber);




    return @resultitems;
}

=head3 get_item_authorised_values

find the types and values for all authorised values assigned to this item.

parameters: itemnumber

returns: a hashref malling the authorised value to the value set for this itemnumber

    $authorised_values = {
             'CCODE'      => undef,
             'DAMAGED'    => '0',
             'LOC'        => '3',
             'LOST'       => '0'
             'NOT_LOAN'   => '0',
             'RESTRICTED' => undef,
             'STACK'      => undef,
             'WITHDRAWN'  => '0',
             'branches'   => 'CPL',
             'cn_source'  => undef,
             'itemtypes'  => 'SER',
           };

Notes: see C4::Biblio::get_biblio_authorised_values for a similar method at the biblio level.

=cut

sub get_item_authorised_values {
    my $itemnumber = shift;

    # assume that these entries in the authorised_value table are item level.
    my $query = q(SELECT distinct authorised_value, kohafield
                    FROM marc_subfield_structure
                    WHERE kohafield like 'item%'
                      AND authorised_value != '' );

    my $itemlevel_authorised_values = C4::Context->dbh->selectall_hashref( $query, 'authorised_value' );
    my $iteminfo = GetItem( $itemnumber );
    # warn( Data::Dumper->Dump( [ $itemlevel_authorised_values ], [ 'itemlevel_authorised_values' ] ) );
    my $return;
    foreach my $this_authorised_value ( keys %$itemlevel_authorised_values ) {
        my $field = $itemlevel_authorised_values->{ $this_authorised_value }->{'kohafield'};
        $field =~ s/^items\.//;
        if ( exists $iteminfo->{ $field } ) {
            $return->{ $this_authorised_value } = $iteminfo->{ $field };
        }
    }
    # warn( Data::Dumper->Dump( [ $return ], [ 'return' ] ) );
    return $return;
}

=head3 get_authorised_value_images

find a list of icons that are appropriate for display based on the
authorised values for a biblio.

parameters: listref of authorised values, such as comes from
get_item_authorised_values or
from C4::Biblio::get_biblio_authorised_values

returns: listref of hashrefs for each image. Each hashref looks like this:

      { imageurl => '/intranet-tmpl/prog/img/itemtypeimg/npl/WEB.gif',
        label    => '',
        category => '',
        value    => '', }

Notes: Currently, I put on the full path to the images on the staff
side. This should either be configurable or not done at all. Since I
have to deal with 'intranet' or 'opac' in
get_biblio_authorised_values, perhaps I should be passing it in.

=cut

sub get_authorised_value_images {
    my $authorised_values = shift;

    my @imagelist;

    my $authorised_value_list = GetAuthorisedValues();
    # warn ( Data::Dumper->Dump( [ $authorised_value_list ], [ 'authorised_value_list' ] ) );
    foreach my $this_authorised_value ( @$authorised_value_list ) {
        if ( exists $authorised_values->{ $this_authorised_value->{'category'} }
             && $authorised_values->{ $this_authorised_value->{'category'} } eq $this_authorised_value->{'authorised_value'} ) {
            # warn ( Data::Dumper->Dump( [ $this_authorised_value ], [ 'this_authorised_value' ] ) );
            if ( defined $this_authorised_value->{'imageurl'} ) {
                push @imagelist, { imageurl => C4::Koha::getitemtypeimagelocation( 'intranet', $this_authorised_value->{'imageurl'} ),
                                   label    => $this_authorised_value->{'lib'},
                                   category => $this_authorised_value->{'category'},
                                   value    => $this_authorised_value->{'authorised_value'}, };
            }
        }
    }

    # warn ( Data::Dumper->Dump( [ \@imagelist ], [ 'imagelist' ] ) );
    return \@imagelist;

}

=head1 LIMITED USE FUNCTIONS

The following functions, while part of the public API,

Lines 1906-1912 sub searchResults { Link Here

(-)a/C4/Search.pm (-1 lines)
1906	# add imageurl to itemtype if there is one	1906	# add imageurl to itemtype if there is one
1907	$oldbiblio->{imageurl} = getitemtypeimagelocation( $search_context, $itemtypes{ $oldbiblio->{itemtype} }->{imageurl} );	1907	$oldbiblio->{imageurl} = getitemtypeimagelocation( $search_context, $itemtypes{ $oldbiblio->{itemtype} }->{imageurl} );
1908		1908
1909	$oldbiblio->{'authorised_value_images'} = ($search_context eq 'opac' && C4::Context->preference('AuthorisedValueImages')) \|\| ($search_context eq 'intranet' && C4::Context->preference('StaffAuthorisedValueImages')) ? C4::Items::get_authorised_value_images( C4::Biblio::get_biblio_authorised_values( $oldbiblio->{'biblionumber'}, $marcrecord ) ) : [];
1910	$oldbiblio->{normalized_upc} = GetNormalizedUPC( $marcrecord,$marcflavour);	1909	$oldbiblio->{normalized_upc} = GetNormalizedUPC( $marcrecord,$marcflavour);
1911	$oldbiblio->{normalized_ean} = GetNormalizedEAN( $marcrecord,$marcflavour);	1910	$oldbiblio->{normalized_ean} = GetNormalizedEAN( $marcrecord,$marcflavour);
1912	$oldbiblio->{normalized_oclc} = GetNormalizedOCLCNumber($marcrecord,$marcflavour);	1911	$oldbiblio->{normalized_oclc} = GetNormalizedOCLCNumber($marcrecord,$marcflavour);

Lines 217-223 sub BuildReport { Link Here

(-)a/C4/UsageStats.pm (-2 lines)
217	LetterLog	217	LetterLog
218	ReturnLog	218	ReturnLog
219	SubscriptionLog	219	SubscriptionLog
220	AuthorisedValueImages
221	BiblioDefaultView	220	BiblioDefaultView
222	COinSinOPACResults	221	COinSinOPACResults
223	DisplayOPACiconsXSLT	222	DisplayOPACiconsXSLT
Lines 312-318 sub BuildReport { Link Here
312	SubscriptionHistory	311	SubscriptionHistory
313	Display856uAsImage	312	Display856uAsImage
314	DisplayIconsXSLT	313	DisplayIconsXSLT
315	StaffAuthorisedValueImages
316	template	314	template
317	yuipath	315	yuipath
318	HidePatronName	316	HidePatronName

Lines 446-452 INSERT INTO systempreferences ( `variable`, `value`, `options`, `explanation`, ` Link Here

(-)a/installer/data/mysql/sysprefs.sql (-1 lines)
446	('SpineLabelAutoPrint','0','','If this setting is turned on, a print dialog will automatically pop up for the quick spine label printer.','YesNo'),	446	('SpineLabelAutoPrint','0','','If this setting is turned on, a print dialog will automatically pop up for the quick spine label printer.','YesNo'),
447	('SpineLabelFormat','<itemcallnumber><copynumber>','30\|10','This preference defines the format for the quick spine label printer. Just list the fields you would like to see in the order you would like to see them, surrounded by <>, for example <itemcallnumber>.','Textarea'),	447	('SpineLabelFormat','<itemcallnumber><copynumber>','30\|10','This preference defines the format for the quick spine label printer. Just list the fields you would like to see in the order you would like to see them, surrounded by <>, for example <itemcallnumber>.','Textarea'),
448	('SpineLabelShowPrintOnBibDetails','0','','If turned on, a \"Print Label\" link will appear for each item on the bib details page in the staff interface.','YesNo'),	448	('SpineLabelShowPrintOnBibDetails','0','','If turned on, a \"Print Label\" link will appear for each item on the bib details page in the staff interface.','YesNo'),
449	('StaffAuthorisedValueImages','0',NULL,'','YesNo'),
450	('staffClientBaseURL','',NULL,'Specify the base URL of the staff client','free'),	449	('staffClientBaseURL','',NULL,'Specify the base URL of the staff client','free'),
451	('StaffDetailItemSelection', '1', NULL, 'Enable item selection in record detail page', 'YesNo'),	450	('StaffDetailItemSelection', '1', NULL, 'Enable item selection in record detail page', 'YesNo'),
452	('StaffSearchResultsDisplayBranch','holdingbranch','holdingbranch\|homebranch','Controls the display of the home or holding branch for staff search results','Choice'),	451	('StaffSearchResultsDisplayBranch','holdingbranch','holdingbranch\|homebranch','Controls the display of the home or holding branch for staff search results','Choice'),




              class: multi
            - (separate columns with |)
        -
            - pref: AuthorisedValueImages
              choices:
                  yes: Show
                  no: "Don't show"
            - images for <a href="/cgi-bin/koha/admin/authorised_values.pl">authorized values</a> (such as lost statuses and locations) in search results and item detail pages on the OPAC.
        -
            - Use the image at
            - pref: OpacFavicon
              class: url




                  no: "Don't show"
            - the format, audience, and material type icons in XSLT MARC21 results and detail pages in the staff client.
        -
            - pref: StaffAuthorisedValueImages
              choices:
                  yes: Show
                  no: "Don't show"
            - images for <a href="/cgi-bin/koha/admin/authorised_values.pl">authorized values</a> (such as lost statuses and locations) in search results.
        -
            - 'Display the URI in the 856u field as an image on: '
            - pref: Display856uAsImage
              choices:




                                    </p>
                                    [% IF ( SEARCH_RESULT.searchhighlightblob ) %]<p class="searchhighlightblob">[% SEARCH_RESULT.searchhighlightblob %]</p>[% END %]

                                    [% IF ( SEARCH_RESULT.authorised_value_images ) %]
                                <p>
                                  [% FOREACH authorised_value_image IN SEARCH_RESULT.authorised_value_images %]
                                  <img src="[% authorised_value_image.imageurl %]" alt="[% authorised_value_image.description %]" />
                                  [% END %]
                                </p>
                                [% END %]

                                [% END %]
                                [% END %]
                                  <p class="hold">[% IF ( SEARCH_RESULT.norequests ) %]

Lines 63-69 Link Here

(-)a/koha-tmpl/intranet-tmpl/prog/en/modules/help/admin/authorised_values.tt (-3 lines)
63	</li>	63	</li>
64	<li>Use the Description field for the actual value that will be displayed. If you want something different to show in the OPAC, enter a 'Description (OPAC)'</li>	64	<li>Use the Description field for the actual value that will be displayed. If you want something different to show in the OPAC, enter a 'Description (OPAC)'</li>
65	<li>If you would like to limit this authorized value category to only specific libraries you can choose them from the 'Branches limitation' menu. To have it show for all libraries just choose 'All branches' at the top of the list.</li>	65	<li>If you would like to limit this authorized value category to only specific libraries you can choose them from the 'Branches limitation' menu. To have it show for all libraries just choose 'All branches' at the top of the list.</li>
66	<li>If you have StaffAuthorisedValueImages and/or AuthorisedValueImages set to show images for authorized values you can choose the image under 'Choose an icon'</li>
67	</ul>	66	</ul>
68	</li>	67	</li>
69	<li>Click 'Save'</li>	68	<li>Click 'Save'</li>
Lines 85-92 Link Here
85	</li>	84	</li>
86	<li>Use the Description field for the actual value that will be displayed. If you want something different to show in the OPAC, enter a 'Description (OPAC)'</li>	85	<li>Use the Description field for the actual value that will be displayed. If you want something different to show in the OPAC, enter a 'Description (OPAC)'</li>
87	<li>If you would like to limit this authorized value category to only specific libraries you can choose them from the 'Branches limitation' menu. To have it show for all libraries just choose 'All branches' at the top of the list.</li>	86	<li>If you would like to limit this authorized value category to only specific libraries you can choose them from the 'Branches limitation' menu. To have it show for all libraries just choose 'All branches' at the top of the list.</li>
88	<li>If you have StaffAuthorisedValueImages and/or AuthorisedValueImages set
89	to show images for authorized values you can choose the image under 'Choose an icon'</li>
90	<li>Click 'Save'</li>	87	<li>Click 'Save'</li>
91	<li>The new value will appear in the list along with existing values</li>	88	<li>The new value will appear in the list along with existing values</li>
92	</ul>	89	</ul>

Lines 14-24 not use an API to fetch items that populates item.datedue. Link Here

(-)a/koha-tmpl/opac-tmpl/bootstrap/en/includes/item-status.inc (-5 / +1 lines)
14	[% IF ( av_lib_include ) %]	14	[% IF ( av_lib_include ) %]
15	<span class="item-status lost">[% av_lib_include %]</span>	15	<span class="item-status lost">[% av_lib_include %]</span>
16	[% ELSE %]	16	[% ELSE %]
17	[% IF ( item.lostimageurl ) %]	17	<span class="item-status lost">Item lost</span>
18	<img src="[% item.lostimageurl %]" alt="[% item.lostimagelabel %]" title="[% item.lostimagelabel %]" />
19	[% ELSE %]
20	<span class="item-status lost">Item lost</span>
21	[% END %]
22	[% END %]	18	[% END %]
23	[% END %]	19	[% END %]
24		20




                        [% END # / IF MARCURLS %]
                    [% END # / IF OPACXSLTDetailsDisplay %]

                    [% IF ( AuthorisedValueImages && authorised_value_images ) %]
                        <div class="authorised_value_images">
                            [% FOREACH authorised_value_image IN authorised_value_images %]
                                <img src="[% authorised_value_image.imageurl %]" alt="[% authorised_value_image.label %]" title="[% authorised_value_image.label %]" />
                            [% END %]
                        </div>
                    [% END # / AuthorisedValueImages && authorised_value_images %]

                    <!-- This puts the LTFL reviews in, and if TabbedView is not set, puts the remaining content above the Tabs instead of in them -->
                    [% IF ( LibraryThingForLibrariesID ) %]
                        [% UNLESS ( LibraryThingForLibrariesTabbedView ) %]




                                                </td>
                                            [% END %]

                                            [% # Cell 4: Search result details and controls %]
                                                [% # Cell 4: Show Authorized value image %]
                                                <td class="authvalcol">
                                                    [% FOREACH authorised_value_image IN SEARCH_RESULT.authorised_value_images %]
                                                        [% IF ( authorised_value_image.imageurl ) %]
                                                            <img src="[% authorised_value_image.imageurl %]" alt="[% authorised_value_image.label %]" title="[% authorised_value_image.label %]">
                                                        [% END %]
                                                    [% END %]
                                                </td>
                                            [% END %]

                                            [% # Cell 5: Search result details and controls %]
                                            <td class="bibliocol">
                                                <div class="coverimages">
                                                    <a class="p1" href="/cgi-bin/koha/opac-detail.pl?biblionumber=[% SEARCH_RESULT.biblionumber %]">




$dat->{'count'} = scalar(@items);


my $biblio_authorised_value_images = C4::Context->preference('AuthorisedValueImages')
    ? C4::Items::get_authorised_value_images( C4::Biblio::get_biblio_authorised_values( $biblionumber, $record ) )
    : [];

my (%item_reserves, %priority);
my ($show_holds_count, $show_priority);
for ( C4::Context->preference("OPACShowHoldQueueDetails") ) {

        $itemfields{$_} = 1 if ($itm->{$_});
    }

     # walk through the item-level authorised values and populate some images
     my $item_authorised_value_images = C4::Items::get_authorised_value_images( C4::Items::get_item_authorised_values( $itm->{'itemnumber'} ) );
     # warn( Data::Dumper->Dump( [ $item_authorised_value_images ], [ 'item_authorised_value_images' ] ) );

     if ( $itm->{'itemlost'} ) {
         my $lostimageinfo = List::Util::first { $_->{'category'} eq 'LOST' } @$item_authorised_value_images;
         $itm->{'lostimageurl'}   = $lostimageinfo->{ 'imageurl' };
         $itm->{'lostimagelabel'} = $lostimageinfo->{ 'label' };
     }
     my $reserve_status = C4::Reserves::GetReserveStatus($itm->{itemnumber});
      if( $reserve_status eq "Waiting"){ $itm->{'waiting'} = 1; }
      if( $reserve_status eq "Reserved"){ $itm->{'onhold'} = 1; }

                     itemdata_uri            => $itemfields{uri},
                     itemdata_copynumber     => $itemfields{copynumber},
                     itemdata_itemnotes          => $itemfields{itemnotes},
                     authorised_value_images => $biblio_authorised_value_images,
                     subtitle                => $subtitle,
                     OpacStarRatings         => C4::Context->preference("OpacStarRatings"),
    );

Lines 469-475 sub mocking_systempreferences_to_a_set_value { Link Here

(-)a/t/db_dependent/UsageStats.t (-3 lines)
469	LetterLog	469	LetterLog
470	ReturnLog	470	ReturnLog
471	SubscriptionLog	471	SubscriptionLog
472	AuthorisedValueImages
473	BiblioDefaultView	472	BiblioDefaultView
474	COinSinOPACResults	473	COinSinOPACResults
475	DisplayOPACiconsXSLT	474	DisplayOPACiconsXSLT
Lines 565-571 sub mocking_systempreferences_to_a_set_value { Link Here
565	SubscriptionHistory	564	SubscriptionHistory
566	Display856uAsImage	565	Display856uAsImage
567	DisplayIconsXSLT	566	DisplayIconsXSLT
568	StaffAuthorisedValueImages
569	template	567	template
570	yuipath	568	yuipath
571	HidePatronName	569	HidePatronName
572	-

Return to bug 16167

Lines 3470-3537 sub ModBiblioMarc { Link Here

(-)a/C4/Biblio.pm (-62 lines)
3470	return $biblionumber;	3470	return $biblionumber;
3471	}	3471	}
3472		3472
3473	=head2 get_biblio_authorised_values
3474
3475	find the types and values for all authorised values assigned to this biblio.
3476
3477	parameters:
3478	biblionumber
3479	MARC::Record of the bib
3480
3481	returns: a hashref mapping the authorised value to the value set for this biblionumber
3482
3483	$authorised_values = {
3484	'Scent' => 'flowery',
3485	'Audience' => 'Young Adult',
3486	'itemtypes' => 'SER',
3487	};
3488
3489	Notes: forlibrarian should probably be passed in, and called something different.
3490
3491	=cut
3492
3493	sub get_biblio_authorised_values {
3494	my $biblionumber = shift;
3495	my $record = shift;
3496
3497	my $forlibrarian = 1; # are we in staff or opac?
3498	my $frameworkcode = GetFrameworkCode($biblionumber);
3499
3500	my $authorised_values;
3501
3502	my $tagslib = GetMarcStructure( $forlibrarian, $frameworkcode )
3503	or return $authorised_values;
3504
3505	# assume that these entries in the authorised_value table are bibliolevel.
3506	# ones that start with 'item%' are item level.
3507	my $query = q(SELECT distinct authorised_value, kohafield
3508	FROM marc_subfield_structure
3509	WHERE authorised_value !=''
3510	AND (kohafield like 'biblio%'
3511	OR kohafield like '') );
3512	my $bibliolevel_authorised_values = C4::Context->dbh->selectall_hashref( $query, 'authorised_value' );
3513
3514	foreach my $tag ( keys(%$tagslib) ) {
3515	foreach my $subfield ( keys( %{ $tagslib->{$tag} } ) ) {
3516
3517	# warn "checking $subfield. type is: " . ref $tagslib->{ $tag }{ $subfield };
3518	if ( 'HASH' eq ref $tagslib->{$tag}{$subfield} ) {
3519	if ( defined $tagslib->{$tag}{$subfield}{'authorised_value'} && exists $bibliolevel_authorised_values->{ $tagslib->{$tag}{$subfield}{'authorised_value'} } ) {
3520	if ( defined $record->field($tag) ) {
3521	my $this_subfield_value = $record->field($tag)->subfield($subfield);
3522	if ( defined $this_subfield_value ) {
3523	$authorised_values->{ $tagslib->{$tag}{$subfield}{'authorised_value'} } = $this_subfield_value;
3524	}
3525	}
3526	}
3527	}
3528	}
3529	}
3530
3531	# warn ( Data::Dumper->Dump( [ $authorised_values ], [ 'authorised_values' ] ) );
3532	return $authorised_values;
3533	}
3534
3535	=head2 CountBiblioInOrders	3473	=head2 CountBiblioInOrders
3536		3474
3537	$count = &CountBiblioInOrders( $biblionumber);	3475	$count = &CountBiblioInOrders( $biblionumber);

Lines 1760-1863 sub GetHiddenItemnumbers { Link Here

(-)a/C4/Items.pm (-98 lines)
1760	return @resultitems;	1760	return @resultitems;
1761	}	1761	}
1762		1762
1763	=head3 get_item_authorised_values
1764
1765	find the types and values for all authorised values assigned to this item.
1766
1767	parameters: itemnumber
1768
1769	returns: a hashref malling the authorised value to the value set for this itemnumber
1770
1771	$authorised_values = {
1772	'CCODE' => undef,
1773	'DAMAGED' => '0',
1774	'LOC' => '3',
1775	'LOST' => '0'
1776	'NOT_LOAN' => '0',
1777	'RESTRICTED' => undef,
1778	'STACK' => undef,
1779	'WITHDRAWN' => '0',
1780	'branches' => 'CPL',
1781	'cn_source' => undef,
1782	'itemtypes' => 'SER',
1783	};
1784
1785	Notes: see C4::Biblio::get_biblio_authorised_values for a similar method at the biblio level.
1786
1787	=cut
1788
1789	sub get_item_authorised_values {
1790	my $itemnumber = shift;
1791
1792	# assume that these entries in the authorised_value table are item level.
1793	my $query = q(SELECT distinct authorised_value, kohafield
1794	FROM marc_subfield_structure
1795	WHERE kohafield like 'item%'
1796	AND authorised_value != '' );
1797
1798	my $itemlevel_authorised_values = C4::Context->dbh->selectall_hashref( $query, 'authorised_value' );
1799	my $iteminfo = GetItem( $itemnumber );
1800	# warn( Data::Dumper->Dump( [ $itemlevel_authorised_values ], [ 'itemlevel_authorised_values' ] ) );
1801	my $return;
1802	foreach my $this_authorised_value ( keys %$itemlevel_authorised_values ) {
1803	my $field = $itemlevel_authorised_values->{ $this_authorised_value }->{'kohafield'};
1804	$field =~ s/^items\.//;
1805	if ( exists $iteminfo->{ $field } ) {
1806	$return->{ $this_authorised_value } = $iteminfo->{ $field };
1807	}
1808	}
1809	# warn( Data::Dumper->Dump( [ $return ], [ 'return' ] ) );
1810	return $return;
1811	}
1812
1813	=head3 get_authorised_value_images
1814
1815	find a list of icons that are appropriate for display based on the
1816	authorised values for a biblio.
1817
1818	parameters: listref of authorised values, such as comes from
1819	get_item_authorised_values or
1820	from C4::Biblio::get_biblio_authorised_values
1821
1822	returns: listref of hashrefs for each image. Each hashref looks like this:
1823
1824	{ imageurl => '/intranet-tmpl/prog/img/itemtypeimg/npl/WEB.gif',
1825	label => '',
1826	category => '',
1827	value => '', }
1828
1829	Notes: Currently, I put on the full path to the images on the staff
1830	side. This should either be configurable or not done at all. Since I
1831	have to deal with 'intranet' or 'opac' in
1832	get_biblio_authorised_values, perhaps I should be passing it in.
1833
1834	=cut
1835
1836	sub get_authorised_value_images {
1837	my $authorised_values = shift;
1838
1839	my @imagelist;
1840
1841	my $authorised_value_list = GetAuthorisedValues();
1842	# warn ( Data::Dumper->Dump( [ $authorised_value_list ], [ 'authorised_value_list' ] ) );
1843	foreach my $this_authorised_value ( @$authorised_value_list ) {
1844	if ( exists $authorised_values->{ $this_authorised_value->{'category'} }
1845	&& $authorised_values->{ $this_authorised_value->{'category'} } eq $this_authorised_value->{'authorised_value'} ) {
1846	# warn ( Data::Dumper->Dump( [ $this_authorised_value ], [ 'this_authorised_value' ] ) );
1847	if ( defined $this_authorised_value->{'imageurl'} ) {
1848	push @imagelist, { imageurl => C4::Koha::getitemtypeimagelocation( 'intranet', $this_authorised_value->{'imageurl'} ),
1849	label => $this_authorised_value->{'lib'},
1850	category => $this_authorised_value->{'category'},
1851	value => $this_authorised_value->{'authorised_value'}, };
1852	}
1853	}
1854	}
1855
1856	# warn ( Data::Dumper->Dump( [ \@imagelist ], [ 'imagelist' ] ) );
1857	return \@imagelist;
1858
1859	}
1860
1861	=head1 LIMITED USE FUNCTIONS	1763	=head1 LIMITED USE FUNCTIONS
1862		1764
1863	The following functions, while part of the public API,	1765	The following functions, while part of the public API,

Lines 176-187 OPAC: Link Here

(-)a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/opac.pref (-6 lines)
176	class: multi	176	class: multi
177	- (separate columns with \|)	177	- (separate columns with \|)
178	-	178	-
179	- pref: AuthorisedValueImages
180	choices:
181	yes: Show
182	no: "Don't show"
183	- images for <a href="/cgi-bin/koha/admin/authorised_values.pl">authorized values</a> (such as lost statuses and locations) in search results and item detail pages on the OPAC.
184	-
185	- Use the image at	179	- Use the image at
186	- pref: OpacFavicon	180	- pref: OpacFavicon
187	class: url	181	class: url

Lines 551-564 var holdForPatron = function () { Link Here

(-)a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/results.tt (-8 lines)
551	</p>	551	</p>
552	[% IF ( SEARCH_RESULT.searchhighlightblob ) %]<p class="searchhighlightblob">[% SEARCH_RESULT.searchhighlightblob %]</p>[% END %]	552	[% IF ( SEARCH_RESULT.searchhighlightblob ) %]<p class="searchhighlightblob">[% SEARCH_RESULT.searchhighlightblob %]</p>[% END %]
553		553
554	[% IF ( SEARCH_RESULT.authorised_value_images ) %]
555	<p>
556	[% FOREACH authorised_value_image IN SEARCH_RESULT.authorised_value_images %]
557	<img src="[% authorised_value_image.imageurl %]" alt="[% authorised_value_image.description %]" />
558	[% END %]
559	</p>
560	[% END %]
561
562	[% END %]	554	[% END %]
563	[% END %]	555	[% END %]
564	<p class="hold">[% IF ( SEARCH_RESULT.norequests ) %]	556	<p class="hold">[% IF ( SEARCH_RESULT.norequests ) %]

Lines 363-376 Link Here

(-)a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt (-8 lines)
363	[% END # / IF MARCURLS %]	363	[% END # / IF MARCURLS %]
364	[% END # / IF OPACXSLTDetailsDisplay %]	364	[% END # / IF OPACXSLTDetailsDisplay %]
365		365
366	[% IF ( AuthorisedValueImages && authorised_value_images ) %]
367	<div class="authorised_value_images">
368	[% FOREACH authorised_value_image IN authorised_value_images %]
369	<img src="[% authorised_value_image.imageurl %]" alt="[% authorised_value_image.label %]" title="[% authorised_value_image.label %]" />
370	[% END %]
371	</div>
372	[% END # / AuthorisedValueImages && authorised_value_images %]
373
374	<!-- This puts the LTFL reviews in, and if TabbedView is not set, puts the remaining content above the Tabs instead of in them -->	366	<!-- This puts the LTFL reviews in, and if TabbedView is not set, puts the remaining content above the Tabs instead of in them -->
375	[% IF ( LibraryThingForLibrariesID ) %]	367	[% IF ( LibraryThingForLibrariesID ) %]
376	[% UNLESS ( LibraryThingForLibrariesTabbedView ) %]	368	[% UNLESS ( LibraryThingForLibrariesTabbedView ) %]

Lines 220-237 Link Here

(-)a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results.tt (-12 / +1 lines)
220	</td>	220	</td>
221	[% END %]	221	[% END %]
222		222
223	[% IF ( AuthorisedValueImages ) %]	223	[% # Cell 4: Search result details and controls %]
224	[% # Cell 4: Show Authorized value image %]
225	<td class="authvalcol">
226	[% FOREACH authorised_value_image IN SEARCH_RESULT.authorised_value_images %]
227	[% IF ( authorised_value_image.imageurl ) %]
228	<img src="[% authorised_value_image.imageurl %]" alt="[% authorised_value_image.label %]" title="[% authorised_value_image.label %]">
229	[% END %]
230	[% END %]
231	</td>
232	[% END %]
233
234	[% # Cell 5: Search result details and controls %]
235	<td class="bibliocol">	224	<td class="bibliocol">
236	<div class="coverimages">	225	<div class="coverimages">
237	<a class="p1" href="/cgi-bin/koha/opac-detail.pl?biblionumber=[% SEARCH_RESULT.biblionumber %]">	226	<a class="p1" href="/cgi-bin/koha/opac-detail.pl?biblionumber=[% SEARCH_RESULT.biblionumber %]">

Lines 560-569 foreach my $subscription (@subscriptions) { Link Here

(-)a/opac/opac-detail.pl (-14 lines)
560	$dat->{'count'} = scalar(@items);	560	$dat->{'count'} = scalar(@items);
561		561
562		562
563	my $biblio_authorised_value_images = C4::Context->preference('AuthorisedValueImages')
564	? C4::Items::get_authorised_value_images( C4::Biblio::get_biblio_authorised_values( $biblionumber, $record ) )
565	: [];
566
567	my (%item_reserves, %priority);	563	my (%item_reserves, %priority);
568	my ($show_holds_count, $show_priority);	564	my ($show_holds_count, $show_priority);
569	for ( C4::Context->preference("OPACShowHoldQueueDetails") ) {	565	for ( C4::Context->preference("OPACShowHoldQueueDetails") ) {
Lines 656-670 if ( not $viewallitems and @items > $max_items_to_display ) { Link Here
656	$itemfields{$_} = 1 if ($itm->{$_});	652	$itemfields{$_} = 1 if ($itm->{$_});
657	}	653	}
658		654
659	# walk through the item-level authorised values and populate some images
660	my $item_authorised_value_images = C4::Items::get_authorised_value_images( C4::Items::get_item_authorised_values( $itm->{'itemnumber'} ) );
661	# warn( Data::Dumper->Dump( [ $item_authorised_value_images ], [ 'item_authorised_value_images' ] ) );
662
663	if ( $itm->{'itemlost'} ) {
664	my $lostimageinfo = List::Util::first { $_->{'category'} eq 'LOST' } @$item_authorised_value_images;
665	$itm->{'lostimageurl'} = $lostimageinfo->{ 'imageurl' };
666	$itm->{'lostimagelabel'} = $lostimageinfo->{ 'label' };
667	}
668	my $reserve_status = C4::Reserves::GetReserveStatus($itm->{itemnumber});	655	my $reserve_status = C4::Reserves::GetReserveStatus($itm->{itemnumber});
669	if( $reserve_status eq "Waiting"){ $itm->{'waiting'} = 1; }	656	if( $reserve_status eq "Waiting"){ $itm->{'waiting'} = 1; }
670	if( $reserve_status eq "Reserved"){ $itm->{'onhold'} = 1; }	657	if( $reserve_status eq "Reserved"){ $itm->{'onhold'} = 1; }
Lines 736-742 my $subtitle = GetRecordValue('subtitle', $record, GetFrameworkCode($bib Link Here
736	itemdata_uri => $itemfields{uri},	723	itemdata_uri => $itemfields{uri},
737	itemdata_copynumber => $itemfields{copynumber},	724	itemdata_copynumber => $itemfields{copynumber},
738	itemdata_itemnotes => $itemfields{itemnotes},	725	itemdata_itemnotes => $itemfields{itemnotes},
739	authorised_value_images => $biblio_authorised_value_images,
740	subtitle => $subtitle,	726	subtitle => $subtitle,
741	OpacStarRatings => C4::Context->preference("OpacStarRatings"),	727	OpacStarRatings => C4::Context->preference("OpacStarRatings"),
742	);	728	);