Allow param "bornum" as in template, not just "borrowernumber". Added FIXMEs.
authorJoe Atzberger <joe.atzberger@liblime.com>
Tue, 8 Apr 2008 22:43:51 +0000 (17:43 -0500)
committerJoshua Ferraro <jmf@liblime.com>
Wed, 9 Apr 2008 00:49:06 +0000 (19:49 -0500)
Signed-off-by: Joshua Ferraro <jmf@liblime.com>
opac/opac-renew.pl

index 0832b5e..195e827 100755 (executable)
@@ -21,13 +21,15 @@ my ( $template, $borrowernumber, $cookie ) = get_template_and_user(
        }
 ); 
 my $itemnumber     = $query->param('item');
-my $borrowernumber = $query->param("borrowernumber");
+my $borrowernumber = $query->param('borrowernumber') || $query->param('bornum');
 
 my ($status,$error) = CanBookBeRenewed( $borrowernumber, $itemnumber );
 if ( $status == 1 ) {
     AddRenewal( $borrowernumber, $itemnumber );
 }
+# FIXME: else return ERROR to user!!
 
 if ( $query->param('from') eq 'opac_user' ) {
     print $query->redirect("/cgi-bin/koha/opac-user.pl");
-}
+} 
+# FIXME: ELSE WHAT?  No response at all.  Not very robust.