http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10029
--- Comment #2 from Chris Cormack <[email protected]> --- Comment on attachment 17371 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=17371 Proposed patch Review of attachment 17371: --> (http://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=10029&attachment=17371) ----------------------------------------------------------------- ::: C4/Auth_with_cas.pm @@ +95,5 @@ > my $casparam = $query->param('cas'); > # FIXME: This should be more generic and handle whatever parameters > there might be > + if (defined $casparam) { > + $uri .= ($uri =~ m/\?/ ) ? '&' : '?'; > + $uri .= "cas=$casparam" ; You shouldn't need this, because $query->url will already contain the casparam, ie, if it exists it will be in $query->url, so we will be appending another copy of it. So if you remove this and the FIXME then I'd be happy to sign off. -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
