Bug 15839: [QA Follow-up] Error checking in opac-review.pl
authorMarcel de Rooy <m.de.rooy@rijksmuseum.nl>
Fri, 29 Jul 2016 08:24:44 +0000 (10:24 +0200)
committerKyle M Hall <kyle@bywatersolutions.com>
Fri, 9 Sep 2016 10:31:06 +0000 (10:31 +0000)
commit177542bf5208e5e053ac9b1500d2536e1eff5ae2
tree6b50dfb55ccbc068289da4ddcb9fe43c93faed35
parentb5c3b1c9747693c284c3c0e8fdb2df94d256088e
Bug 15839: [QA Follow-up] Error checking in opac-review.pl

[1] Adds a check on biblionumber. (Prevents a DBIx error.)
[2] If you have a reviewid, search on that and check results.
    Add an unauthorized error in template.
[3] If you add a new review, check that there is no review yet.
    If so, edit the existing one.
    This supports the added FIXME on a unique constraint.

Note: This script could receive further attention.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Tested all crud ops with opac-review.pl (incl URL manipulation).

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-review.tt
opac/opac-review.pl