Bugzilla – Attachment 79291 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.45 KB, created by
Julian Maurice
on 2018-09-24 10:16:50 UTC
(
hide
)
Description:
Bug 21323: Handle multi params in redirect after login
Filename:
MIME Type:
Creator:
Julian Maurice
Created:
2018-09-24 10:16:50 UTC
Size:
1.45 KB
patch
obsolete
>From 0e4615c7ad736230f0a9af3067565ef02c8a232a 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 >MIME-Version: 1.0 >Content-Type: text/plain; charset=UTF-8 >Content-Transfer-Encoding: 8bit > >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 > >Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr> >Signed-off-by: Julian Maurice <julian.maurice@biblibre.com> >--- > C4/Auth.pm | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > >diff --git a/C4/Auth.pm b/C4/Auth.pm >index 64f8823c85..243aada413 100644 >--- a/C4/Auth.pm >+++ b/C4/Auth.pm >@@ -1229,8 +1229,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.17.1
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