Bugzilla – Attachment 51307 Details for
Bug 16259
More: Replace CGI->param with CGI->multi_param in list context
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
[PASSED QA] Bug 16259: Replace CGI->param with CGI->multi_param in list context - part 2
PASSED-QA-Bug-16259-Replace-CGI-param-with-CGI-mul.patch (text/plain), 8.39 KB, created by
Katrin Fischer
on 2016-05-08 11:18:48 UTC
(
hide
)
Description:
[PASSED QA] Bug 16259: Replace CGI->param with CGI->multi_param in list context - part 2
Filename:
MIME Type:
Creator:
Katrin Fischer
Created:
2016-05-08 11:18:48 UTC
Size:
8.39 KB
patch
obsolete
>From 291eb1e24663f1a1753e3fe5ccf70735caf21972 Mon Sep 17 00:00:00 2001 >From: Jonathan Druart <jonathan.druart@bugs.koha-community.org> >Date: Fri, 15 Apr 2016 09:00:28 +0100 >Subject: [PATCH] [PASSED QA] Bug 16259: Replace CGI->param with > CGI->multi_param in list context - part 2 > >This patch is a follow-up of bug 16154. > >It removes the warning "CGI::param called in list context" in the >following scripts: >admin/branches.pl >admin/categories.pl >admin/patron-attr-types.pl >admin/preferences.pl >catalogue/image.pl >circ/circulation.pl >patroncards/add_user_search.pl >serials/add_user_search.pl >tools/marc_modification_templates.pl >virtualshelves/shelves.pl > >Note that the warning from catalogue/itemsearch.pl still exists (the >call to CGI->param is done from the template). > >Test plan: >- Add/modify a library, patron category, patron attr type >- Update a syspref >- Set localcoverimage and call catalogue/image.pl?biblionumber=XXX >- Search for patrons in the patron cards or serials module >- Add a marc modification templates >- Add a list (shelves) > >You should not get the warning in the log after all these actions. > >Signed-off-by: Owen Leonard <oleonard@myacpl.org> > >Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> >--- > C4/Form/MessagingPreferences.pm | 4 ++-- > admin/branches.pl | 8 ++++---- > admin/patron-attr-types.pl | 4 ++-- > catalogue/image.pl | 2 +- > circ/circulation.pl | 2 +- > svc/config/systempreferences | 2 +- > svc/members/search | 2 +- > svc/virtualshelves/search | 2 +- > tools/marc_modification_templates.pl | 6 +++--- > 9 files changed, 16 insertions(+), 16 deletions(-) > >diff --git a/C4/Form/MessagingPreferences.pm b/C4/Form/MessagingPreferences.pm >index cbee804..b98015f 100644 >--- a/C4/Form/MessagingPreferences.pm >+++ b/C4/Form/MessagingPreferences.pm >@@ -81,11 +81,11 @@ sub handle_form_action { > message_attribute_id => $option->{'message_attribute_id'} }; > > # find the desired transports >- @{$updater->{'message_transport_types'}} = $query->param( $option->{'message_attribute_id'} ); >+ @{$updater->{'message_transport_types'}} = $query->multi_param( $option->{'message_attribute_id'} ); > next OPTION unless $updater->{'message_transport_types'}; > > if ( $option->{'has_digest'} ) { >- if ( List::Util::first { $_ == $option->{'message_attribute_id'} } $query->param( 'digest' ) ) { >+ if ( List::Util::first { $_ == $option->{'message_attribute_id'} } $query->multi_param( 'digest' ) ) { > $updater->{'wants_digest'} = 1; > } > } >diff --git a/admin/branches.pl b/admin/branches.pl >index 3626b67..abe3e5a 100755 >--- a/admin/branches.pl >+++ b/admin/branches.pl >@@ -87,7 +87,7 @@ if ( $op eq 'add_form' ) { > if ($is_a_modif) { > my $library = Koha::Libraries->find($branchcode); > for my $field (@fields) { >- $library->$field( $input->param($field) ); >+ $library->$field( scalar $input->param($field) ); > } > $library->update_categories( \@categories ); > >@@ -168,9 +168,9 @@ if ( $op eq 'add_form' ) { > if ($is_a_modif) { > my $category = Koha::LibraryCategories->find($categorycode); > for my $field (@fields) { >- $category->$field( $input->param($field) ); >+ $category->$field( scalar $input->param($field) ); > } >- $category->show_in_pulldown( $input->param('show_in_pulldown') eq 'on' ); >+ $category->show_in_pulldown( scalar $input->param('show_in_pulldown') eq 'on' ); > eval { $category->store; }; > if ($@) { > push @messages, { type => 'alert', code => 'error_on_update_category' }; >@@ -183,7 +183,7 @@ if ( $op eq 'add_form' ) { > ( map { $_ => scalar $input->param($_) || undef } @fields ) > } > ); >- $category->show_in_pulldown( $input->param('show_in_pulldown') eq 'on' ); >+ $category->show_in_pulldown( scalar $input->param('show_in_pulldown') eq 'on' ); > eval { $category->store; }; > if ($@) { > push @messages, { type => 'alert', code => 'error_on_insert_category' }; >diff --git a/admin/patron-attr-types.pl b/admin/patron-attr-types.pl >index c99394b..ceb1184 100755 >--- a/admin/patron-attr-types.pl >+++ b/admin/patron-attr-types.pl >@@ -166,8 +166,8 @@ sub add_update_attribute_type { > $attr_type->authorised_value_category($authorised_value_category); > my $display_checkout = $input->param('display_checkout'); > $attr_type->display_checkout($display_checkout); >- $attr_type->category_code($input->param('category_code')); >- $attr_type->class($input->param('class')); >+ $attr_type->category_code(scalar $input->param('category_code')); >+ $attr_type->class(scalar $input->param('class')); > my @branches = $input->multi_param('branches'); > $attr_type->branches( \@branches ); > >diff --git a/catalogue/image.pl b/catalogue/image.pl >index f347640..3d95b05 100755 >--- a/catalogue/image.pl >+++ b/catalogue/image.pl >@@ -64,7 +64,7 @@ if ( C4::Context->preference("LocalCoverImages") ) { > $imagenumber = $data->param('imagenumber'); > } > elsif ( defined $data->param('biblionumber') ) { >- my @imagenumbers = ListImagesForBiblio( $data->param('biblionumber') ); >+ my @imagenumbers = ListImagesForBiblio( $data->multi_param('biblionumber') ); > if (@imagenumbers) { > $imagenumber = $imagenumbers[0]; > } >diff --git a/circ/circulation.pl b/circ/circulation.pl >index e31c7c5..66b3dcf 100755 >--- a/circ/circulation.pl >+++ b/circ/circulation.pl >@@ -184,7 +184,7 @@ if ( @$barcodes ) { > $stickyduedate = $query->param('stickyduedate'); > $duedatespec = $query->param('duedatespec'); > } >- $session->param('auto_renew', $query->param('auto_renew')); >+ $session->param('auto_renew', scalar $query->param('auto_renew')); > } > else { > $session->clear('auto_renew'); >diff --git a/svc/config/systempreferences b/svc/config/systempreferences >index 021db3c..01cd782 100755 >--- a/svc/config/systempreferences >+++ b/svc/config/systempreferences >@@ -94,7 +94,7 @@ sub set_preferences { > > next if ( !defined( $pref ) ); > >- my $value = join( ',', $query->param( $param ) ); >+ my $value = join( ',', $query->multi_param( $param ) ); > > C4::Context->set_preference( $pref, $value ); > } >diff --git a/svc/members/search b/svc/members/search >index 1c2d644..2599519 100755 >--- a/svc/members/search >+++ b/svc/members/search >@@ -31,7 +31,7 @@ my $input = new CGI; > exit unless $input->param('template_path'); > > my ($template, $user, $cookie) = get_template_and_user({ >- template_name => $input->param('template_path'), >+ template_name => scalar $input->param('template_path'), > query => $input, > type => "intranet", > authnotrequired => 0, >diff --git a/svc/virtualshelves/search b/svc/virtualshelves/search >index 5e991fc..0c827c0 100755 >--- a/svc/virtualshelves/search >+++ b/svc/virtualshelves/search >@@ -13,7 +13,7 @@ my $input = new CGI; > exit unless $input->param('template_path'); > > my ($template, $user, $cookie) = get_template_and_user({ >- template_name => $input->param('template_path'), >+ template_name => scalar $input->param('template_path'), > query => $input, > type => "intranet", > authnotrequired => 0, >diff --git a/tools/marc_modification_templates.pl b/tools/marc_modification_templates.pl >index 10589e4..3689684 100755 >--- a/tools/marc_modification_templates.pl >+++ b/tools/marc_modification_templates.pl >@@ -42,7 +42,7 @@ my ($template, $loggedinuser, $cookie) > > if ( $op eq "create_template" ) { > $template_id = '' unless $cgi->param('duplicate_current_template'); >- $template_id = AddModificationTemplate( $cgi->param('template_name'), $template_id ); >+ $template_id = AddModificationTemplate( scalar $cgi->param('template_name'), $template_id ); > > } elsif ( $op eq "delete_template" ) { > >@@ -103,11 +103,11 @@ if ( $op eq "create_template" ) { > } > > } elsif ( $op eq "delete_action" ) { >- DelModificationTemplateAction( $cgi->param('mmta_id') ); >+ DelModificationTemplateAction( scalar $cgi->param('mmta_id') ); > > } elsif ( $op eq "move_action" ) { > >- MoveModificationTemplateAction( $cgi->param('mmta_id'), $cgi->param('where') ); >+ MoveModificationTemplateAction( scalar $cgi->param('mmta_id'), scalar $cgi->param('where') ); > > } > >-- >1.9.1
<b>You cannot view the attachment while viewing its details because your browser does not support IFRAMEs. <a href="attachment.cgi?id=51307">View the attachment on a separate page</a>.</b>
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 16259
:
50256
|
50279
|
50986
| 51307