From a98c622ab810c4e7a34b58c408a3f6bb2b679569 Mon Sep 17 00:00:00 2001
From: Joe Atzberger <joe.atzberger@liblime.com>
Date: Mon, 21 Apr 2008 16:22:42 -0500
Subject: [PATCH] Remove extraneous code. Note: this is another example of
 unchecked input, yet unfixed.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
---
 opac/opac-review.pl | 30 +++++-------------------------
 1 file changed, 5 insertions(+), 25 deletions(-)

diff --git a/opac/opac-review.pl b/opac/opac-review.pl
index 3e6864915a..9f9722e27a 100755
--- a/opac/opac-review.pl
+++ b/opac/opac-review.pl
@@ -31,7 +31,7 @@ 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 $reviewid     = $query->param('reviewid');
 my ( $template, $borrowernumber, $cookie ) = get_template_and_user(
     {
         template_name   => "opac-review.tmpl",
@@ -47,38 +47,18 @@ my $savedreview = getreview( $biblionumber, $borrowernumber );
 if ( $type eq 'save' ) {
     savereview( $biblionumber, $borrowernumber, $review );
 }
-if ( $type eq 'update' ) {
+elsif ( $type eq 'update' ) {
     updatereview( $biblionumber, $borrowernumber, $review );
 }
-if ($savedreview) {
-    $type = "update";
-}
-else {
-    $type = "save";
-}
-my $reviewdata = $savedreview->{'review'};
+$type = ($savedreview) ? "update" : "save";
 $template->param(
     'biblionumber'   => $biblionumber,
     'borrowernumber' => $borrowernumber,
     'type'           => $type,
-    'review'         => $reviewdata,
-	'reviewid'  => $reviewid,
+    'review'         => $savedreview->{'review'},
+	'reviewid'       => $reviewid,
     'title'          => $biblio->{'title'},
 );
 
-# get the record
-my $order  = $query->param('order');
-my $order2 = $order;
-if ( $order2 eq '' ) {
-    $order2 = "date_due desc";
-}
-my $limit = $query->param('limit');
-if ( $limit eq 'full' ) {
-    $limit = 0;
-}
-else {
-    $limit = 50;
-}
-
 output_html_with_http_headers $query, $cookie, $template->output;
 
-- 
2.20.1