ffzg/recall_notices.pl: added --interval and --dedup

[koha.git] / opac / opac-ratings.pl

diff --git a/opac/opac-ratings.pl b/opac/opac-ratings.pl
index 7ce681c..38c50d6 100755 (executable)
--- a/opac/opac-ratings.pl
+++ b/opac/opac-ratings.pl
@@ -25,13 +25,11 @@ note: there is currently no 'delete rating' functionality in this script
 
 =cut
 
-use strict;
-use warnings;
+use Modern::Perl;
 use CGI qw ( -utf8 );
 
 use C4::Auth;
 use C4::Context;
-use C4::Ratings;
 use C4::Debug;
 
 use Koha::Ratings;
@@ -45,7 +43,6 @@ my $loggedinuser = C4::Context->userenv->{'number'};
 my $biblionumber     = $query->param('biblionumber');
 my $rating_old_value = $query->param('rating_value');
 my $rating_value     = $query->param('rating');
-my $rating;
 
 # If JS is disabled and a user click on "Rate me" without selecting a rate
 unless ( $biblionumber and $rating_value ) {
@@ -55,10 +52,12 @@ unless ( $biblionumber and $rating_value ) {
 }
 
 if ( !$rating_old_value ) {
-    Koha::Rating->new( { biblionumber => $biblionumber, borrowernumber => $loggedinuser, rating_value => $rating_value, })->store;
+    my $rating = Koha::Rating->new( { biblionumber => $biblionumber, borrowernumber => $loggedinuser, rating_value => $rating_value, });
+    $rating->store if $rating;
 }
 else {
-    $rating = ModRating( $biblionumber, $loggedinuser, $rating_value );
+    my $rating = Koha::Ratings->find( { biblionumber => $biblionumber, borrowernumber => $loggedinuser });
+    $rating->rating_value($rating_value)->store if $rating;
 }
 print $query->redirect(
     "/cgi-bin/koha/opac-detail.pl?biblionumber=$biblionumber");