Bug 40958 - Move patron_to_html (from js-patron-format.inc) to a standalone JS file
Summary: Move patron_to_html (from js-patron-format.inc) to a standalone JS file
Status: Signed Off
Alias: None
Product: Koha
Classification: Unclassified
Component: Architecture, internals, and plumbing (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low enhancement
Assignee: Jonathan Druart
QA Contact: Testopia
URL:
Keywords:
Depends on: 40699
Blocks: 32176 40995
  Show dependency treegraph
 
Reported: 2025-10-06 13:45 UTC by Jonathan Druart
Modified: 2025-10-10 15:07 UTC (History)
1 user (show)

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


Attachments
Bug 40958: Move patron_to_html to a standalone JS file (18.01 KB, patch)
2025-10-06 13:48 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 40958: Move patron_to_html to a standalone JS file (18.07 KB, patch)
2025-10-07 15:31 UTC, Andrew Fuerste-Henry
Details | Diff | Splinter Review
Bug 40958: Remove remaining occurrence in course_reserves/course.tt (973 bytes, patch)
2025-10-09 09:08 UTC, Jonathan Druart
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Druart 2025-10-06 13:45:13 UTC
We have to explicitly include the TT js-patron-format.inc in all the .tt files we are going to format a patron.

It's also hard to use from JS files, which is a bit ridiculous.
Comment 1 Jonathan Druart 2025-10-06 13:48:08 UTC
Created attachment 187476 [details] [review]
Bug 40958: Move patron_to_html to a standalone JS file

It will be easier to reuse patron_to_html

Test plan:
Search for patron and confirm that that patron are formatted correctly.
Global patron search and a couple of other places should be enough.
Comment 2 Andrew Fuerste-Henry 2025-10-07 15:31:29 UTC
Created attachment 187533 [details] [review]
Bug 40958: Move patron_to_html to a standalone JS file

It will be easier to reuse patron_to_html

Test plan:
Search for patron and confirm that that patron are formatted correctly.
Global patron search and a couple of other places should be enough.

Signed-off-by: Andrew Fuerste Henry <andrew@bywatersolutions.com>
Comment 3 Jonathan Druart 2025-10-09 09:08:13 UTC
Created attachment 187646 [details] [review]
Bug 40958: Remove remaining occurrence in course_reserves/course.tt

From bug 40699