Bug 30617 - [OMNIBUS] We should consistently use includes for patron name display
Summary: [OMNIBUS] We should consistently use includes for patron name display
Status: NEW
Alias: None
Product: Koha
Classification: Unclassified
Component: Staff Client (show other bugs)
Version: master
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Bugs List
QA Contact: Testopia
URL:
Keywords: Academy
: 22279 (view as bug list)
Depends on: 21978 30651 30806 30807
Blocks: 30568
  Show dependency treegraph
 
Reported: 2022-04-26 07:18 UTC by Martin Renvoize
Modified: 2022-05-19 11:52 UTC (History)
7 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Renvoize 2022-04-26 07:18:55 UTC
Our templates are littered with various invocations of `[% patron.surname %], [% patron.firstname %]` and similar in varying orders and varying levels of detail (with or without cardnumber, with or without title, surname first or firstname first, linked and unlinked).

We have two ways of constructing a patron display block, patron-title.inc tt include and js-patron-format.inc for use in JS based displays.

We should congregate our templates around these two includes so that we can make our display consistent throughout with perhaps just a couple of display variations.. short and long forms perhaps?
Comment 1 Katrin Fischer 2022-04-26 11:41:45 UTC
I guess this calls for a lot of smaller bugs in a way trying to clean up various templates.
Comment 2 Martin Renvoize 2022-04-27 06:50:24 UTC
*** Bug 22279 has been marked as a duplicate of this bug. ***