From 029ccc0d512a767255807f21842262f8efcfcf17 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Mon, 4 Apr 2016 14:45:48 +0100 Subject: [PATCH] Bug 16179: Do not crash if "rate me" is clicked and not rate selected MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit If JS is disabled and a user clicks on the "Rate me" button, Koha will crashes with: DBIx::Class::ResultSet::create(): Column 'rating_value' cannot be null at /usr/share/koha/lib/C4/Ratings.pm line 208 To avoid that, opac-ratings.pl will check if a rate has been selected. Test plan: Disable JS On a record detail page, click on the "Rate me" button TESTED PLAN: 1) go to /cgi-bin/koha/opac-ratings.pl?biblionumber=1 -- kaboom as above. 2) apply patch 3) refresh -- either login screen (don't know why) -- or if already logged in, detail page. 4) koha qa test tool Signed-off-by: Mark Tompsett Signed-off-by: Owen Leonard I tested successfully by temporarily removing the modification made by Bug 16210. Signed-off-by: Katrin Fischer Signed-off-by: Brendan Gallagher (cherry picked from commit b679cac96409b7248f8e224e10c73dafa4c82890) Signed-off-by: Julian Maurice (cherry picked from commit a965702c08f84d8d770fb81e09f13bee8e922bba) Signed-off-by: Frédéric Demians --- opac/opac-ratings.pl | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/opac/opac-ratings.pl b/opac/opac-ratings.pl index 2ad0cb0861..23e73daedf 100755 --- a/opac/opac-ratings.pl +++ b/opac/opac-ratings.pl @@ -45,6 +45,13 @@ 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 ) { + print $query->redirect( + "/cgi-bin/koha/opac-detail.pl?biblionumber=$biblionumber"); + exit; +} + if ( !$rating_old_value ) { $rating = AddRating( $biblionumber, $loggedinuser, $rating_value ); } -- 2.39.5