Bugzilla – Attachment 78286 Details for
Bug 21096
Garbled username on intranet login page
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 21096: Do not display loggedinusername in template
Bug-21096-Do-not-display-loggedinusername-in-templ.patch (text/plain), 8.10 KB, created by
Jonathan Druart
on 2018-08-29 19:09:12 UTC
(
hide
)
Description:
Bug 21096: Do not display loggedinusername in template
Filename:
MIME Type:
Creator:
Jonathan Druart
Created:
2018-08-29 19:09:12 UTC
Size:
8.10 KB
patch
obsolete
>From 6497c6b8e6c15fce8902bbe3757db1beb7ea4c59 Mon Sep 17 00:00:00 2001 >From: Jonathan Druart <jonathan.druart@bugs.koha-community.org> >Date: Wed, 25 Jul 2018 12:25:58 -0300 >Subject: [PATCH] Bug 21096: Do not display loggedinusername in template >MIME-Version: 1.0 >Content-Type: text/plain; charset=UTF-8 >Content-Transfer-Encoding: 8bit > >This patch is a little fix for a much bigger hidden issues. > >The original issue: >1. Set the firstname and surname values of a paontr to utf-8 characters >("wide characters"), for instance ææ >2. Use this patron to login at the staff interface >=> In the header the logged in patron's info (concat of firstname and >surname) are displayed correctly >3. Hit whatever link >=> In the header the info are now displayed incorrectly >("ææ") > >What happens? >After that the user loggin, loggedinusername is set with the value from >the DB (borrowers.userid) >On next hits it is picked from the session (which contains the decoded >utf8 value, see first lines of C4::Context->set_userenv) >From C4::Auth::checkauth: >834 $s_userid = $session->param('id') // '' > >The quick fix is to use the logged_in_user variable in the template, but >it seems that issues may occurred if external authentication is used >(ldap, shib, cas). Could someone test this? > >Test plan: >Make sure the original issue is fixed > >Signed-off-by: Owen Leonard <oleonard@myacpl.org> > >Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> >--- > C4/Auth.pm | 2 +- > koha-tmpl/intranet-tmpl/prog/en/includes/header.inc | 4 ++-- > koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcel.tt | 2 +- > koha-tmpl/intranet-tmpl/prog/en/modules/circ/offline.tt | 2 +- > koha-tmpl/intranet-tmpl/prog/en/modules/patron_lists/add-modify.tt | 2 +- > koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-add.tt | 2 +- > koha-tmpl/intranet-tmpl/prog/en/modules/virtualshelves/shelves.tt | 4 ++-- > 7 files changed, 9 insertions(+), 9 deletions(-) > >diff --git a/C4/Auth.pm b/C4/Auth.pm >index f0f3eb8611..8fd97289ec 100644 >--- a/C4/Auth.pm >+++ b/C4/Auth.pm >@@ -261,7 +261,7 @@ sub get_template_and_user { > } > > # user info >- $template->param( loggedinusername => $user ); # FIXME Should be replaced with something like patron-title.inc >+ $template->param( loggedinusername => $user ); # OBSOLETE - Do not reuse this in template, use logged_in_user.userid instead > $template->param( loggedinusernumber => $borrowernumber ); # FIXME Should be replaced with logged_in_user.borrowernumber > $template->param( logged_in_user => $patron ); > $template->param( sessionID => $sessionID ); >diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/header.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/header.inc >index 8b47028545..7e3c17812a 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/includes/header.inc >+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/header.inc >@@ -59,7 +59,7 @@ > <i class="fa fa-user"></i> > </span> > <span id="logged-in-info-full"> >- <span class="loggedinusername">[% loggedinusername | html %]</span> >+ <span class="loggedinusername">[% logged_in_user.userid | html %]</span> > <span class="separator">|</span> > [% IF ( AutoLocation ) %] > <brand> >@@ -82,7 +82,7 @@ > > <li class="loggedin-menu-label"> > Logged in as:<br /> >- <span class="loggedinusername">[% loggedinusername | html %]</span> >+ <span class="loggedinusername">[% logged_in_user.userid | html %]</span> > </li> > <li class="loggedin-menu-label"> > [% IF ( AutoLocation ) %] >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcel.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcel.tt >index a2ac542406..7f302accfc 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcel.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcel.tt >@@ -88,7 +88,7 @@ > > [% UNLESS no_orders_to_display %] > <div id="acqui_receive_summary"> >-<p><strong>Invoice number:</strong> [% invoice | html %] <strong>Received by:</strong> [% loggedinusername | html %] <strong>On:</strong> [% datereceived | $KohaDates %]</p> >+<p><strong>Invoice number:</strong> [% invoice | html %] <strong>Received by:</strong> [% logged_in_user.userid | html %]<strong>On:</strong> [% datereceived | $KohaDates %]</p> > </div> > [% UNLESS (invoiceclosedate) %] > <div id="acqui_receive_search"> >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/offline.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/offline.tt >index be7c32674b..3bd92c2f95 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/offline.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/offline.tt >@@ -309,7 +309,7 @@ > } > > function synchronize() { >- kohadb.saveSetting("userid", "[% loggedinusername | html %]"); >+ kohadb.saveSetting("userid", "[% logged_in_user.userid | html %]"); > kohadb.saveSetting("branchcode", "[% LoginBranchcode | html %]"); > showSyncInfo(); > [% UNLESS (AllowOfflineCirculation) %] >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/patron_lists/add-modify.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/patron_lists/add-modify.tt >index cfb1e7c5b7..4bfa1fa127 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/patron_lists/add-modify.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/patron_lists/add-modify.tt >@@ -62,7 +62,7 @@ > </li> > > <li> >- <span class="label">Owner: </span>[% loggedinusername | html %] >+ <span class="label">Owner: </span>[% logged_in_user.userid | html %] > </li> > </ol> > >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-add.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-add.tt >index 846cbc37b0..7f1586c71c 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-add.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-add.tt >@@ -31,7 +31,7 @@ fieldset.rows li.radio { width: 100%; } /* override staff-global.css */ > [% ELSE %] > <input type="hidden" name="op" value="addsubscription" /> > [% END %] >- <input type="hidden" name="user" value="[% loggedinusername | html %]" /> >+ <input type="hidden" name="user" value="[% logged_in_user.userid | html %]" /> > <input type="hidden" name="irreg_check" value="0" /> > > <div id="page_1"> >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/virtualshelves/shelves.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/virtualshelves/shelves.tt >index d213c7e5cf..b89db7687e 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/virtualshelves/shelves.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/virtualshelves/shelves.tt >@@ -282,12 +282,12 @@ > <li> > <span class="label">Owner: </span> > [% IF op == 'add_form' %] >- <input type="hidden" name="owner" id="owner" value="[% loggedinusernumber | html %]" />[% loggedinusername | html %]</li> >+ <input type="hidden" name="owner" id="owner" value="[% loggedinusernumber | html %]" />[% logged_in_user.userid | html %] > [% ELSE %] > [% IF owner %] > <input type="hidden" id="owner" name="owner" value="[% owner.borrowernumber | html %]" />[% owner.firstname _ ' ' _ owner.surname | html %] > [% ELSE %] >- <input type="hidden" id="owner" name="owner" value="[% loggedinusernumber | html %]" />[% loggedinusername | html %] >+ <input type="hidden" id="owner" name="owner" value="[% loggedinusernumber | html %]" />[% logged_in_user.userid | html %] > [% END %] > [% END %] > </li> >-- >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 21096
:
77149
|
77270
|
77271
|
77415
| 78286