From cc7f5145c6d91e9069008dc5ec43408d79499781 Mon Sep 17 00:00:00 2001
From: Chris Cormack <chrisc@catalyst.net.nz>
Date: Thu, 5 Apr 2012 15:40:46 +1200
Subject: [PATCH] Bug 6679 :[SIGNED-OFF] Fixing some perlcritic violations in
 the opac

signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
(cherry picked from commit 6f3123bbc65b1087e18bd8212d1ca01ed1087181)
---
 opac/opac-search.pl      | 2 +-
 opac/opac-showmarc.pl    | 2 +-
 opac/opac-showreviews.pl | 6 ++++--
 opac/opac-tags.pl        | 2 +-
 opac/opac-topissues.pl   | 3 ++-
 opac/unapi               | 2 +-
 6 files changed, 10 insertions(+), 7 deletions(-)

diff --git a/opac/opac-search.pl b/opac/opac-search.pl
index a8e9f2f66a..d7b34a4995 100755
--- a/opac/opac-search.pl
+++ b/opac/opac-search.pl
@@ -389,7 +389,7 @@ my @results;
 my $lang = C4::Templates::getlanguage($cgi, 'opac');
 ( $error,$query,$simple_query,$query_cgi,$query_desc,$limit,$limit_cgi,$limit_desc,$stopwords_removed,$query_type) = buildQuery(\@operators,\@operands,\@indexes,\@limits,\@sort_by, 0, $lang);
 
-sub _input_cgi_parse ($) { 
+sub _input_cgi_parse {
     my @elements;
     for my $this_cgi ( split('&',shift) ) {
         next unless $this_cgi;
diff --git a/opac/opac-showmarc.pl b/opac/opac-showmarc.pl
index aec114b333..3638f8869d 100755
--- a/opac/opac-showmarc.pl
+++ b/opac/opac-showmarc.pl
@@ -27,7 +27,7 @@
 use strict;
 use warnings;
 
-use open OUT=>':utf8', ':std';
+use open OUT=> ":encoding(UTF-8)", ':std';
 
 # standard or CPAN modules used
 use CGI;
diff --git a/opac/opac-showreviews.pl b/opac/opac-showreviews.pl
index 74f4684ac7..3477cb985b 100755
--- a/opac/opac-showreviews.pl
+++ b/opac/opac-showreviews.pl
@@ -147,7 +147,8 @@ for my $result (@$reviews){
                     # the page number for this page
                     my $this_page_number = $i;
                     # it should only be highlighted if it's the current page
-                    my $highlight = 1 if ($this_page_number == $current_page_number);
+                    my $highlight;
+            $highlight = 1 if ($this_page_number == $current_page_number);
                     # put it in the array
                     push @page_numbers, { offset => $this_offset, pg => $this_page_number, highlight => $highlight };
 
@@ -159,7 +160,8 @@ for my $result (@$reviews){
                 for ($i=$current_page_number; $i<=($current_page_number + 20 );$i++) {
                     my $this_offset = ((($i-9)*$results_per_page)-$results_per_page);
                     my $this_page_number = $i-9;
-                    my $highlight = 1 if ($this_page_number == $current_page_number);
+                    my $highlight;
+            $highlight = 1 if ($this_page_number == $current_page_number);
                     if ($this_page_number <= $pages) {
                         push @page_numbers, { offset => $this_offset, pg => $this_page_number, highlight => $highlight };
                     }
diff --git a/opac/opac-tags.pl b/opac/opac-tags.pl
index b5ce340b95..983c18e7cd 100755
--- a/opac/opac-tags.pl
+++ b/opac/opac-tags.pl
@@ -53,7 +53,7 @@ my $perBibResults = {};
 # Indexes of @errors that do not apply to a particular biblionumber.
 my @globalErrorIndexes = ();
 
-sub ajax_auth_cgi ($) {     # returns CGI object
+sub ajax_auth_cgi {     # returns CGI object
 	my $needed_flags = shift;
 	my %cookies = fetch CGI::Cookie;
 	my $input = CGI->new;
diff --git a/opac/opac-topissues.pl b/opac/opac-topissues.pl
index 05b7a278d1..6ecc6ad32c 100755
--- a/opac/opac-topissues.pl
+++ b/opac/opac-topissues.pl
@@ -146,7 +146,8 @@ if (!$advanced_search_types or $advanced_search_types eq 'itemtypes') {
 } else {
     my $advsearchtypes = GetAuthorisedValues($advanced_search_types, '', 'opac');
         for my $thisitemtype (@$advsearchtypes) {
-                my $selected = 1 if $thisitemtype->{authorised_value} eq $itemtype;
+                my $selected;
+            $selected = 1 if $thisitemtype->{authorised_value} eq $itemtype;
                 my %row =( value => $thisitemtype->{authorised_value},
                 selected    => $thisitemtype eq $itemtype,
                 description => $thisitemtype->{'lib'},
diff --git a/opac/unapi b/opac/unapi
index fef5f91da5..0f97eced35 100755
--- a/opac/unapi
+++ b/opac/unapi
@@ -44,7 +44,7 @@ use XML::LibXML;
 use XML::LibXSLT;
 
 my $cgi = CGI->new();
-binmode(STDOUT, "utf8"); #output as utf8
+binmode(STDOUT, ":encoding(UTF-8)"); #output as utf8
 
 =head1 VARIABLES
 
-- 
2.39.5