fix minor warning issues in SCO

* test for NO_MORE_RENEWALS now does expect
  that hash key will always exist
* removed unconditional warn

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

diff --git a/opac/sco/sco-main.pl b/opac/sco/sco-main.pl
index 984aded..9b15ac5 100755 (executable)
--- a/opac/sco/sco-main.pl
+++ b/opac/sco/sco-main.pl
@@ -80,7 +80,7 @@ elsif ( $op eq "checkout" ) {
     if (scalar keys %$impossible) {
 
         #  warn "impossible: numkeys: " . scalar (keys(%$impossible));
-        warn join " ", keys %$impossible;
+        #warn join " ", keys %$impossible;
         my $issue_error = (keys %$impossible)[0];
 
         # FIXME  we assume only one error.
@@ -147,7 +147,7 @@ if ($borrower->{cardnumber}) {
     my ($issueslist) = GetPendingIssues( $borrower->{'borrowernumber'} );
     foreach my $it (@$issueslist) {
         my ($renewokay, $renewerror) = CanBookBeIssued($borrower, $it->{'barcode'},'','');
-        $it->{'norenew'} = 1 if $renewokay->{'NO_MORE_RENEWALS'} == 1;
+        $it->{'norenew'} = 1 if $renewokay->{'NO_MORE_RENEWALS'};
         push @issues, $it;
     }