Bug 30377: Fix two CGI::param called in list context-warnings
authorMarcel de Rooy <m.de.rooy@rijksmuseum.nl>
Mon, 28 Mar 2022 12:07:43 +0000 (12:07 +0000)
committerFridolin Somers <fridolin.somers@biblibre.com>
Mon, 4 Apr 2022 14:23:46 +0000 (16:23 +0200)
Trivial fix.

Test plan:
Delete two lines from opac search history.
Verify results. Check plack-opac-error.log.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
opac/opac-search-history.pl
serials/claims.pl

index 019fdc6..8ef2306 100755 (executable)
@@ -102,7 +102,7 @@ unless ( $loggedinuser ) {
             C4::Search::History::delete(
                 {
                     userid => $loggedinuser,
-                    id     => [ $cgi->param('id') ],
+                    id     => [ @id ],
                 }
             );
         } else {
index a16d840..5b5479f 100755 (executable)
@@ -59,7 +59,7 @@ my @serialnums=$input->multi_param('serialid');
 if (@serialnums) { # i.e. they have been flagged to generate claims
     my $err;
     eval {
-        $err = SendAlerts('claimissues',\@serialnums,$input->param("letter_code"));
+        $err = SendAlerts( 'claimissues', \@serialnums, scalar $input->param("letter_code") );
         if ( not ref $err or not exists $err->{error} ) {
             C4::Serials::updateClaim( \@serialnums );
         }