From 664f287d078c1a6a8b8d4f9c80651fd2507c20c5 Mon Sep 17 00:00:00 2001 From: Andrew Fuerste Henry Date: Mon, 9 Dec 2024 21:58:17 +0000 Subject: [PATCH] Bug 30568: Add classes to patron title components to test: 1 - have a patron with data in firstname, surname, middle name, othernames, and preferred name 2 - view patron name in patron record and other places where patron.title is used, confirm lack of HTML markup 3 - apply patch, restart services 4 - reload patron record and other pages around Koha, confirm name display hasn't changed but the composite parts of the name are now marked with classes --- .../intranet-tmpl/prog/en/includes/patron-title.inc | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/patron-title.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/patron-title.inc index 72d4472a83..3a9661020f 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/includes/patron-title.inc +++ b/koha-tmpl/intranet-tmpl/prog/en/includes/patron-title.inc @@ -67,16 +67,16 @@ [%- END -%] [%- IF display_patron_name -%] [%- IF data.category_type == 'I' -%] - [%- data.surname | html -%] [%- IF data.othernames -%] ([%- data.othernames | html -%])[%- END -%] + [%- data.surname | html -%] [%- IF data.othernames -%] ([%- data.othernames | html -%])[%- END -%] [%- ELSIF invert_name -%] - [%- data.title | $raw -%][%- data.surname | html -%][%- IF ( data.preferred_name ) -%], [% data.preferred_name | html -%][%- END -%][%- IF data.middle_name -%] [% data.middle_name | html -%][%- END -%][%- IF data.othernames -%] ([%- data.othernames | html -%])[%- END -%] + [%- data.title | $raw -%][%- data.surname | html -%][%- IF ( data.preferred_name ) -%], [% data.preferred_name | html -%][%- END -%][%- IF data.middle_name -%] [% data.middle_name | html -%][%- END -%][%- IF data.othernames -%] ([%- data.othernames | html -%])[%- END -%] [%- ELSE -%] - [%- data.title | $raw -%][%- data.preferred_name | html %][%- IF data.middle_name -%] [% data.middle_name | html -%][%- END -%][%- IF data.othernames -%] ([%- data.othernames | html -%]) [%- END -%] [% data.surname | html -%] + [%- data.title | $raw -%][%- data.preferred_name | html %][%- IF data.middle_name -%] [% data.middle_name | html -%][%- END -%][%- IF data.othernames -%] ([%- data.othernames | html -%]) [%- END -%] [% data.surname | html -%] [%- END -%] - [%- IF display_cardnumber AND data.cardnumber -%] ([%- data.cardnumber | html -%])[%- END -%] + [%- IF display_cardnumber AND data.cardnumber -%] ([%- data.cardnumber | html -%])[%- END -%] [%- ELSIF display_cardnumber -%] [%- IF data.cardnumber -%][%# FIXME Cardnumber should always be defined, right? -%] - [%- data.cardnumber | html -%] + [%- data.cardnumber | html -%] [%- END -%] [%- ELSE -%] A patron from library [%- Branches.GetName( patron.branchcode ) | html -%] -- 2.39.5