Bugzilla – Attachment 78832 Details for
Bug 21323
Redirect page after login missing multiple params
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 21323: Handle multi params in redirect after login
Bug-21323-Handle-multi-params-in-redirect-after-lo.patch (text/plain), 1.24 KB, created by
Jonathan Druart
on 2018-09-14 23:12:26 UTC
(
hide
)
Description:
Bug 21323: Handle multi params in redirect after login
Filename:
MIME Type:
Creator:
Jonathan Druart
Created:
2018-09-14 23:12:26 UTC
Size:
1.24 KB
patch
obsolete
>From a6548a303b5c3cf65dd7c62f2b83160a1dd43016 Mon Sep 17 00:00:00 2001 >From: Jonathan Druart <jonathan.druart@bugs.koha-community.org> >Date: Fri, 14 Sep 2018 16:10:56 -0700 >Subject: [PATCH] Bug 21323: Handle multi params in redirect after login > >Multi params are not handle correctly after login > >Test plan: >[0] Use staff interface >[1] Use advance search and tick itemtype more than one checkbox >[2] Copy result url >[3] Logout >[4] Paste url in browser >[5] Type username and password >[6] In search result page, results limit description under breadcrumbs >will show all limits >--- > C4/Auth.pm | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > >diff --git a/C4/Auth.pm b/C4/Auth.pm >index 8fd97289ec..17df6b3e71 100644 >--- a/C4/Auth.pm >+++ b/C4/Auth.pm >@@ -1212,8 +1212,8 @@ sub checkauth { > my @inputs = (); > foreach my $name ( param $query) { > (next) if ( $name eq 'userid' || $name eq 'password' || $name eq 'ticket' ); >- my $value = $query->param($name); >- push @inputs, { name => $name, value => $value }; >+ my @value = $query->multi_param($name); >+ push @inputs, { name => $name, value => $_ } for @value; > } > > my $patron = Koha::Patrons->find({ userid => $q_userid }); # Not necessary logged in! >-- >2.11.0
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 21323
:
78832
|
78867
|
79291