@@ -, +, @@ HighlightOwnItemsOnOPACWhich --- C4/UsageStats.pm | 1 - .../data/mysql/atomicupdate/bug_12561.perl | 7 ++++ .../en/modules/admin/preferences/opac.pref | 11 ------ .../prog/en/modules/catalogue/detail.tt | 6 +--- .../bootstrap/en/modules/opac-detail.tt | 6 +--- .../bootstrap/en/modules/opac-results.tt | 7 +--- opac/opac-detail.pl | 34 ------------------ opac/opac-search.pl | 35 ------------------- t/db_dependent/UsageStats.t | 1 - 9 files changed, 10 insertions(+), 98 deletions(-) create mode 100644 installer/data/mysql/atomicupdate/bug_12561.perl --- a/C4/UsageStats.pm +++ a/C4/UsageStats.pm @@ -230,7 +230,6 @@ sub BuildReport { COinSinOPACResults DisplayOPACiconsXSLT hidelostitems - HighlightOwnItemsOnOPAC OpacAddMastheadLibraryPulldown OPACDisplay856uAsImage OpacHighlightedWords --- a/installer/data/mysql/atomicupdate/bug_12561.perl +++ a/installer/data/mysql/atomicupdate/bug_12561.perl @@ -0,0 +1,7 @@ +$DBversion = 'XXX'; # will be replaced by the RM +if( CheckVersion( $DBversion ) ) { + + $dbh->do( q{ DELETE FROM systempreferences WHERE variable IN ('HighlightOwnItemsOnOPAC', 'HighlightOwnItemsOnOPACWhich')} ); + + NewVersion( $DBversion, 12561, "Remove system preferences HighlightOwnItemsOnOPAC and HighlightOwnItemsOnOPACWhich"); +} --- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/opac.pref +++ a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/opac.pref @@ -42,17 +42,6 @@ OPAC: 0: Disable - "Koha OPAC as public. Private OPAC requires authentication before accessing the OPAC. " - 'Note: This does not affect the public API, see the RESTPublicAnonymousRequests to control the API.' - - - - pref: HighlightOwnItemsOnOPAC - choices: - 1: Emphasize - 0: "Don't emphasize" - - "results from the " - - pref: HighlightOwnItemsOnOPACWhich - choices: - PatronBranch: "patron's home library" - OpacURLBranch: "OPAC's library via the URL" - - " by moving the results to the front and increasing the size or highlighting the rows for those results (non-XSLT only)." - - "Show star-ratings on" - pref: OpacStarRatings --- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt +++ a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt @@ -107,11 +107,7 @@ [% END %] [% IF ( AmazonCoverImages || LocalCoverImages || AdlibrisEnabled || IntranetCoce || (Koha.Preference('CustomCoverImages') && Koha.Preference('CustomCoverImagesURL')) ) %] - [% IF ( XSLTDetailsDisplay ) %] -
- [% ELSE %] -
- [% END %] +
[% ELSE %]
[% END %] --- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt +++ a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt @@ -1263,11 +1263,7 @@ [% FOREACH ITEM_RESULT IN items %] - [% IF ITEM_RESULT.this_branch %] - - [% ELSE %] - - [% END %] + [% IF Koha.Preference('OPACLocalCoverImages') && ( tab == 'holdings' && itemloop_has_images || tab == 'otherholdings' && otheritemloop_has_images ) %] --- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results.tt +++ a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results.tt @@ -501,12 +501,7 @@ [% IF ( SEARCH_RESULT.available_items_loop.size() ) %] Items available: [% FOREACH available_items_loo IN SEARCH_RESULT.available_items_loop %] - [% IF available_items_loo.this_branch %] - - [% ELSE %] - - [% END %] - + [% IF ( singleBranchMode ) %] [% available_items_loo.location | html %] [% ELSE %] --- a/opac/opac-detail.pl +++ a/opac/opac-detail.pl @@ -558,40 +558,6 @@ if ($hideitems) { @items = @all_items; } -my $branch = ''; -if (C4::Context->userenv){ - $branch = C4::Context->userenv->{branch}; -} -if ( C4::Context->preference('HighlightOwnItemsOnOPAC') ) { - if ( - ( ( C4::Context->preference('HighlightOwnItemsOnOPACWhich') eq 'PatronBranch' ) && $branch ) - || - C4::Context->preference('HighlightOwnItemsOnOPACWhich') eq 'OpacURLBranch' - ) { - my $branchcode; - if ( C4::Context->preference('HighlightOwnItemsOnOPACWhich') eq 'PatronBranch' ) { - $branchcode = $branch; - } - elsif ( C4::Context->preference('HighlightOwnItemsOnOPACWhich') eq 'OpacURLBranch' ) { - $branchcode = $ENV{'BRANCHCODE'}; - } - - my @our_items; - my @other_items; - - foreach my $item ( @items ) { - if ( $item->{branchcode} eq $branchcode ) { - $item->{'this_branch'} = 1; - push( @our_items, $item ); - } else { - push( @other_items, $item ); - } - } - - @items = ( @our_items, @other_items ); - } -} - my $dat = &GetBiblioData($biblionumber); my $HideMARC = $record_processor->filters->[0]->should_hide_marc( { --- a/opac/opac-search.pl +++ a/opac/opac-search.pl @@ -834,41 +834,6 @@ for (my $i=0;$i<@servers;$i++) { } $template->param(results_per_page => $results_per_page); my $hide = ($hidingrules) ? 1 : 0; - my $branch = ''; - if (C4::Context->userenv){ - $branch = C4::Context->userenv->{branch}; - } - if ( C4::Context->preference('HighlightOwnItemsOnOPAC') ) { - if ( - ( ( C4::Context->preference('HighlightOwnItemsOnOPACWhich') eq 'PatronBranch' ) && $branch ) - || - C4::Context->preference('HighlightOwnItemsOnOPACWhich') eq 'OpacURLBranch' - ) { - my $branchcode; - if ( C4::Context->preference('HighlightOwnItemsOnOPACWhich') eq 'PatronBranch' ) { - $branchcode = $branch; - } - elsif ( C4::Context->preference('HighlightOwnItemsOnOPACWhich') eq 'OpacURLBranch' ) { - $branchcode = $ENV{'BRANCHCODE'}; - } - - foreach my $res ( @newresults ) { - my @new_loop; - my @top_loop; - my @old_loop = @{$res->{'available_items_loop'}}; - foreach my $item ( @old_loop ) { - if ( $item->{'branchcode'} eq $branchcode ) { - $item->{'this_branch'} = 1; - push( @top_loop, $item ); - } else { - push( @new_loop, $item ); - } - } - my @complete_loop = ( @top_loop, @new_loop ); - $res->{'available_items_loop'} = \@complete_loop; - } - } - } $template->param( SEARCH_RESULTS => \@newresults, --- a/t/db_dependent/UsageStats.t +++ a/t/db_dependent/UsageStats.t @@ -491,7 +491,6 @@ sub mocking_systempreferences_to_a_set_value { COinSinOPACResults DisplayOPACiconsXSLT hidelostitems - HighlightOwnItemsOnOPAC OpacAddMastheadLibraryPulldown OPACDisplay856uAsImage OpacHighlightedWords --