From b4e42bf429a9982e42875e01d335e124dd584ab6 Mon Sep 17 00:00:00 2001 From: Fridolin Somers Date: Thu, 26 Feb 2026 11:03:49 +0100 Subject: [PATCH] Bug 41942: Fix lang hidding in opac patron modification Bug 34438 added lang in opac registration/modification form. But it is linked to primary contact method hidden or not. Looks like this is not on purpose. This patch removes this link. Test plan: 1) Verify system preference 'PatronSelfModificationBorrowerUnwantedField' is empty 2) Go to OPAC user account > Personal details 3) You see 'Preferred language for notices' 4) In system preference 'PatronSelfModificationBorrowerUnwantedField' check 'primary_contact_method' 5) Go to OPAC user account > Personal details => Without patch lang is also hidden => With patch lang is not hidden Signed-off-by: Owen Leonard --- .../bootstrap/en/modules/opac-memberentry.tt | 72 +++++++++---------- 1 file changed, 36 insertions(+), 36 deletions(-) diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-memberentry.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-memberentry.tt index 8a7563c7756..d8aac16cc2f 100644 --- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-memberentry.tt +++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-memberentry.tt @@ -836,43 +836,43 @@ [% IF ( mandatory.defined('primary_contact_method') ) %]Required[% END %] - [% UNLESS hidden.defined('lang') %] -
  • - [% IF ( mandatory.defined('lang') ) %] - - [% ELSE %] - - [% END %] - + + [% FOR language IN languages %] + [% FOR sublanguage IN language.sublanguages_loop %] + [% IF language.plural %] + [% IF sublanguage.rfc4646_subtag == borrower.lang %] + [% ELSE %] - [% IF sublanguage.rfc4646_subtag == borrower.lang %] - - [% ELSE %] - - [% END %] - [% END # /IF language.plural %] - [% END # /FOR sublanguage %] - [% END #/FOR language %] - - - [% IF ( mandatory.defined('lang') ) %]
    Required
    [% END %] -
  • - [% END %] + + [% END %] + [% ELSE %] + [% IF sublanguage.rfc4646_subtag == borrower.lang %] + + [% ELSE %] + + [% END %] + [% END # /IF language.plural %] + [% END # /FOR sublanguage %] + [% END #/FOR language %] + + + [% IF ( mandatory.defined('lang') ) %]
    Required
    [% END %] + [% END %] -- 2.39.5