From b1c70a3c7e667137e54c88a2e02b5ca5ad5ee10b Mon Sep 17 00:00:00 2001
From: Owen Leonard <oleonard@myacpl.org>
Date: Thu, 28 Feb 2019 13:12:59 +0000
Subject: [PATCH] Bug 22432: Star ratings plugin replacement missing from a
 couple pages

This patch modifies two templates to conform to the change to the start
ratings plugin: Overdrive search results and RecordedBooks search
results.

On the Overdrive page, the star ratings are now correctly displayed
using the new plugin. On the RecordedBooks page, star ratings code has
been removed because it is unused. RecordedBooks doesn't provide ratings
information in their API.

To test you need credentials for both Overdrive and RecordedBooks. Apply
the patch and enable both services.

 - In the OPAC, perform a search that will return results from Overdrive
   and RecordedBooks.
 - Open the search results for each of those services.
   - On the Overdrive search results page, star ratings should appear
     correctly (in read-only mode).
   - On the RecordedBooks page everything should look the same.

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
---
 .../en/modules/opac-overdrive-search.tt       | 22 ++++++++++++-------
 .../en/modules/opac-recordedbooks-search.tt   | 10 +--------
 2 files changed, 15 insertions(+), 17 deletions(-)

diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-overdrive-search.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-overdrive-search.tt
index 1c2548bcef..62101ac5c6 100644
--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-overdrive-search.tt
+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-overdrive-search.tt
@@ -5,12 +5,6 @@
 <title>[% IF ( LibraryNameTitle ) %][% LibraryNameTitle | html %][% ELSE %]Koha online[% END %] catalog &rsaquo; OverDrive search for '[% q | html %]'</title>
 [% INCLUDE 'doc-head-close.inc' %]
 [% BLOCK cssinclude %]
-[% Asset.css("css/jquery.rating.css") | $raw %]
-<style>
-.actions a.addtocart {
-    display: inline;
-}
-</style>
 [% END %]
 </head>
 [% INCLUDE 'bodytag.inc' bodyid='overdrive-results-page' bodyclass='scrollto' %]
@@ -62,7 +56,6 @@
 [% INCLUDE 'opac-bottom.inc' %]
 [% BLOCK jsinclude %]
 [% Asset.js("js/overdrive.js") | $raw %]
-[% Asset.js("lib/jquery/plugins/jquery.rating.js") | $raw %]
 <script>
 var OD_password_required = [% IF Koha.Preference('OverDrivePasswordRequired') %]1[% ELSE %]0[% END %];
 var querystring = "[% q |replace( "'", "\'" ) |replace( '\n', '\\n' ) |replace( '\r', '\\r' ) | html %]";
@@ -124,11 +117,24 @@ function search( offset ) {
             results.push( '<span class="results_summary mediatype"><span class="label">' + _("Type:") + ' </span>', prod.mediaType, '</span>' );
 
             if ( prod.starRating ) {
+                var rating_value = Math.round( prod.starRating );
                 results.push( '<div class="results_summary ratings">' );
+                results.push( '<div class="br-wrapper br-theme-fontawesome-stars">');
+                results.push( '<div class="br-widget br-readonly">');
                 for ( var rating = 1; rating <= 5; rating++ ) {
-                    results.push( '<input class="star" type="radio" name="rating-' + i + '" value="' + rating + '"' + ( rating == Math.round( prod.starRating ) ? ' checked="checked"' : '' ) + ' disabled="disabled" />' );
+
+                    results.push( '<a href="#"' );
+                    if( rating == rating_value ){
+                        results.push( ' class="br-selected br-current"></a>' );
+                    } else if( rating_value > rating ){
+                        results.push( ' class="br-selected"></a>' );
+                    } else {
+                        results.push( '></a>');
+                    }
                 }
                 results.push( '</div>' );
+                results.push( '</div>' );
+                results.push( '</div>' );
             }
 
             results.push( '</td>' );
diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-recordedbooks-search.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-recordedbooks-search.tt
index 52d8c2c698..64366cd0f3 100644
--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-recordedbooks-search.tt
+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-recordedbooks-search.tt
@@ -5,12 +5,6 @@
 <title>[% IF ( LibraryNameTitle ) %][% LibraryNameTitle | html %][% ELSE %]Koha online[% END %] catalog &rsaquo; RecordedBooks search for '[% q | html %]'</title>
 [% INCLUDE 'doc-head-close.inc' %]
 [% BLOCK cssinclude %]
-[% Asset.css("css/jquery.rating.css") | $raw %]
-<style>
-.actions a.addtocart {
-    display: inline;
-}
-</style>
 [% END %]
 </head>
 [% INCLUDE 'bodytag.inc' bodyid='recordedbooks-results-page' bodyclass='scrollto' %]
@@ -62,8 +56,7 @@
 [% INCLUDE 'opac-bottom.inc' %]
 [% BLOCK jsinclude %]
 [% Asset.js("js/recordedbooks.js") | $raw %]
-[% Asset.js("lib/jquery/plugins/jquery.rating.js") | $raw %]
-<script type="text/javascript">
+<script>
 var querystring = "[% q |replace( "'", "\'" ) |replace( '\n', '\\n' ) |replace( '\r', '\\r' ) |html %]";
 var results_per_page = [% OPACnumSearchResults || 20 | html %];
 
@@ -144,7 +137,6 @@ function search( page ) {
 
         if ( pages.length > 1 ) $( '#top-pages, #bottom-pages' ).find( '.pagination' ).html( '<ul>' + pages.join( '' ) + '</ul>');
 
-        $( '#recordedbooks-results-list input.star' ).rating();
     } );
 }
 
-- 
2.39.5