X-Git-Url: http://git.rot13.org/?a=blobdiff_plain;f=opac%2Fopac-review.pl;h=16ed56af963da53c24df39513452b2c517d61f21;hb=9c5e40e4923179bf0b2b630d3e09797dc4c1fdb0;hp=9f9722e27a83af71b24d29966d6dcc2839c668fb;hpb=a98c622ab810c4e7a34b58c408a3f6bb2b679569;p=koha.git

diff --git a/opac/opac-review.pl b/opac/opac-review.pl
index 9f9722e27a..16ed56af96 100755
--- a/opac/opac-review.pl
+++ b/opac/opac-review.pl
@@ -13,50 +13,68 @@
 # WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR
 # A PARTICULAR PURPOSE.  See the GNU General Public License for more details.
 #
-# You should have received a copy of the GNU General Public License along with
-# Koha; if not, write to the Free Software Foundation, Inc., 59 Temple Place,
-# Suite 330, Boston, MA  02111-1307 USA
+# You should have received a copy of the GNU General Public License along
+# with Koha; if not, write to the Free Software Foundation, Inc.,
+# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
 
 use strict;
-require Exporter;
+use warnings;
 use CGI;
 use C4::Auth;
 use C4::Koha;
 use C4::Output;
-use C4::Circulation;
 use C4::Review;
 use C4::Biblio;
+use C4::Scrubber;
+use C4::Debug;
 
 my $query        = new CGI;
 my $biblionumber = $query->param('biblionumber');
-my $type         = $query->param('type');
 my $review       = $query->param('review');
-my $reviewid     = $query->param('reviewid');
 my ( $template, $borrowernumber, $cookie ) = get_template_and_user(
     {
         template_name   => "opac-review.tmpl",
         query           => $query,
         type            => "opac",
-        authnotrequired => 1,
+        authnotrequired => ( C4::Context->preference("OpacPublic") ? 1 : 0 ),
     }
 );
 
-my $biblio = GetBiblioData( $biblionumber);
+# FIXME: need to allow user to delete their own comment(s)
 
-my $savedreview = getreview( $biblionumber, $borrowernumber );
-if ( $type eq 'save' ) {
-    savereview( $biblionumber, $borrowernumber, $review );
+my $biblio = GetBiblioData($biblionumber);
+my $savedreview = getreview($biblionumber,$borrowernumber);
+my ($clean, @errors);
+if (defined $review) {
+	if ($review !~ /\S/) {
+		push @errors, {empty=>1};
+	} else {
+		$clean = C4::Scrubber->new('comment')->scrub($review);
+		if ($clean !~ /\S/) {
+			push @errors, {scrubbed_all=>1};
+		} else {
+			if ($clean ne $review) {
+				push @errors, {scrubbed=>$clean};
+			}
+			my $js_ok_review = $clean;
+			$js_ok_review =~ s/"/"/g;	# probably redundant w/ TMPL ESCAPE=JS
+			$template->param(clean_review=>$js_ok_review);
+			if ($savedreview) {
+    			updatereview($biblionumber, $borrowernumber, $clean);
+			} else {
+    			savereview($biblionumber, $borrowernumber, $clean);
+			}
+			unless (@errors){ $template->param(WINDOW_CLOSE=>1); }
+		}
+	}
 }
-elsif ( $type eq 'update' ) {
-    updatereview( $biblionumber, $borrowernumber, $review );
-}
-$type = ($savedreview) ? "update" : "save";
+(@errors   ) and $template->param(   ERRORS=>\@errors);
+($cgi_debug) and $template->param(cgi_debug=>1       );
 $template->param(
     'biblionumber'   => $biblionumber,
     'borrowernumber' => $borrowernumber,
-    'type'           => $type,
-    'review'         => $savedreview->{'review'},
-	'reviewid'       => $reviewid,
+    'review'         => $clean || $savedreview->{'review'},
+	'reviewid'       => $query->param('reviewid') || 0,
     'title'          => $biblio->{'title'},
 );