Bugzilla – Attachment 78867 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.39 KB, created by
ByWater Sandboxes
on 2018-09-15 17:57:20 UTC
(
hide
)
Description:
Bug 21323: Handle multi params in redirect after login
Filename:
MIME Type:
Creator:
ByWater Sandboxes
Created:
2018-09-15 17:57:20 UTC
Size:
1.39 KB
patch
obsolete
>From 667bfabb15d27c40dcc50fa856a419a590c3653f 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> >--- > C4/Auth.pm | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > >diff --git a/C4/Auth.pm b/C4/Auth.pm >index 4854477807..5a71de21e8 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