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

(-)a/opac/opac-ratings-ajax.pl (-3 / +5 lines)
Lines 72-78 my $rating; Link Here
72
$rating_value //= '';
72
$rating_value //= '';
73
73
74
if ( $rating_value eq '' ) {
74
if ( $rating_value eq '' ) {
75
    Koha::Ratings->find( { biblionumber => $biblionumber, borrowernumber => $loggedinuser } )->delete;
75
    my $rating = Koha::Ratings->find( { biblionumber => $biblionumber, borrowernumber => $loggedinuser } );
76
    $rating->delete if $rating;
76
}
77
}
77
78
78
elsif ( $rating_value and !$rating_old_value ) {
79
elsif ( $rating_value and !$rating_old_value ) {
Lines 80-86 elsif ( $rating_value and !$rating_old_value ) { Link Here
80
}
81
}
81
82
82
elsif ( $rating_value ne $rating_old_value ) {
83
elsif ( $rating_value ne $rating_old_value ) {
83
    Koha::Ratings->find( { biblionumber => $biblionumber, borrowernumber => $loggedinuser })->rating_value($rating_value)->store;
84
    my $rating = Koha::Ratings->find( { biblionumber => $biblionumber, borrowernumber => $loggedinuser });
85
    $rating->rating_value($rating_value)->store if $rating
84
}
86
}
85
87
86
my $ratings = Koha::Ratings->search({ biblionumber => $biblionumber });
88
my $ratings = Koha::Ratings->search({ biblionumber => $biblionumber });
Lines 91-97 my %js_reply = ( Link Here
91
    rating_total   => $ratings->count,
93
    rating_total   => $ratings->count,
92
    rating_avg     => $avg,
94
    rating_avg     => $avg,
93
    rating_avg_int => sprintf("%.0f", $avg),
95
    rating_avg_int => sprintf("%.0f", $avg),
94
    rating_value   => $my_rating->rating_value,
96
    rating_value   => $my_rating ? $my_rating->rating_value : undef,
95
    auth_status    => $auth_status,
97
    auth_status    => $auth_status,
96
98
97
);
99
);
(-)a/opac/opac-ratings.pl (-3 / +4 lines)
Lines 53-62 unless ( $biblionumber and $rating_value ) { Link Here
53
}
53
}
54
54
55
if ( !$rating_old_value ) {
55
if ( !$rating_old_value ) {
56
    Koha::Rating->new( { biblionumber => $biblionumber, borrowernumber => $loggedinuser, rating_value => $rating_value, })->store;
56
    my $rating = Koha::Rating->new( { biblionumber => $biblionumber, borrowernumber => $loggedinuser, rating_value => $rating_value, });
57
    $rating->store if $rating;
57
}
58
}
58
else {
59
else {
59
    Koha::Ratings->find( { biblionumber => $biblionumber, borrowernumber => $loggedinuser })->rating_value($rating_value)->store;
60
    my $rating = Koha::Ratings->find( { biblionumber => $biblionumber, borrowernumber => $loggedinuser });
61
    $rating->rating_value($rating_value)->store if $rating;
60
}
62
}
61
print $query->redirect(
63
print $query->redirect(
62
    "/cgi-bin/koha/opac-detail.pl?biblionumber=$biblionumber");
64
    "/cgi-bin/koha/opac-detail.pl?biblionumber=$biblionumber");
63
- 

Return to bug 17089