https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29939
Bug ID: 29939
Summary: Improve auth check in opac-ratings-ajax.pl
Change sponsored?: ---
Product: Koha
Version: master
Hardware: All
OS: All
Status: NEW
Severity: normal
Priority: P5 - low
Component: Architecture, internals, and plumbing
Assignee: koha-bugs@lists.koha-community.org
Reporter: m.de.r...@rijksmuseum.nl
QA Contact: testo...@bugs.koha-community.org
This script certainly needs a better authentication check but it is less
trivial than the others.
The block with Koha::Ratings calls should be within a check on loggedinuser.
The whole script is a weird mix between an ajax script and a regular one.
This statement will crash on an undefined borrowernumber (SQL constraint):
Koha::Rating->new( { biblionumber => $biblionumber, borrowernumber =>
$loggedinuser, rating_value => $rating_value, })->store;
--
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
_______________________________________________
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
