Bug 41157 - Inconsistency between patron JS display and patron-title.inc
Summary: Inconsistency between patron JS display and patron-title.inc
Status: NEW
Alias: None
Product: Koha
Classification: Unclassified
Component: Patrons (show other bugs)
Version: Main
Hardware: All All
: P5 - low normal
Assignee: Bugs List
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2025-10-31 11:34 UTC by Nick Clemens (kidclamp)
Modified: 2025-10-31 11:34 UTC (History)
2 users (show)

See Also:
GIT URL:
Initiative type: ---
Sponsorship status: ---
Crowdfunding goal: 0
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Nick Clemens (kidclamp) 2025-10-31 11:34:58 UTC
Bug 30568 comment 39:
I would like to note that we have currently an interesting tree in the queue (under bug 40995) which improve the JS counterpart (patron-format). The inc file is moved to a standalone JS file, Cypress tests are added, and more places are using it.

I do understand that we want to make the display more flexible, but here we need to keep the 2 versions in sync: they have the same purpose. However we don't include html tags in the JS function yet (why?)

I don't want to block it, but keep in mind that you are not affecting all the occurrences, which introduces more inconsistencies IMO.

A better solution seems to have a new syspref to let libraries provide their desire format, you could pick patron's fields, and add punctuation.

The both (tt and js) versions of the code will use it.

Maybe for a follow-up bug?

Hope that makes sense!