Bug 38984 - DataTables's columnDefs cleanup
Summary: DataTables's columnDefs cleanup
Status: Pushed to main
Alias: None
Product: Koha
Classification: Unclassified
Component: Templates (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement
Assignee: Jonathan Druart
QA Contact: Martin Renvoize (ashimema)
URL:
Keywords: release-notes-needed
Depends on: 26553
Blocks: 39015
  Show dependency treegraph
 
Reported: 2025-01-28 10:57 UTC by Jonathan Druart
Modified: 2025-04-23 07:40 UTC (History)
2 users (show)

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


Attachments
Bug 38984: Remove NoSort def (22.80 KB, patch)
2025-01-28 11:01 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38984: Replace nosort with NoSort (11.56 KB, patch)
2025-01-28 11:01 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38984: Remove anti-the (25.05 KB, patch)
2025-01-28 11:01 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38984: Add NoSort when can replace targets (49.55 KB, patch)
2025-01-28 11:01 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38984: Allow sort on all fields - club-enrollments (1.06 KB, patch)
2025-01-28 11:01 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38984: Also make 'Actions' translatable (1.95 KB, patch)
2025-01-28 11:01 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38984: Specific - tools/batch_extend_due_dates.tt (2.13 KB, patch)
2025-01-28 11:01 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38984: Add NoSort when can replace targets (49.55 KB, patch)
2025-01-28 15:19 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38984: Allow sort on all fields - club-enrollments (1.06 KB, patch)
2025-01-28 15:19 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38984: Also make 'Actions' translatable (1.95 KB, patch)
2025-01-28 15:19 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38984: Specific - tools/batch_extend_due_dates.tt (2.13 KB, patch)
2025-01-28 15:19 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38984: Also make 'Actions' translatable (3.02 KB, patch)
2025-02-06 20:15 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38984: Specific - tools/batch_extend_due_dates.tt (2.13 KB, patch)
2025-02-06 20:15 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38984: Remove NoSort def (22.79 KB, patch)
2025-02-07 16:52 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 38984: Replace nosort with NoSort (11.60 KB, patch)
2025-02-07 16:52 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 38984: Remove anti-the (24.45 KB, patch)
2025-02-07 16:52 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 38984: Add NoSort when can replace targets (49.52 KB, patch)
2025-02-07 16:52 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 38984: Allow sort on all fields - club-enrollments (1.11 KB, patch)
2025-02-07 16:52 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 38984: Also make 'Actions' translatable (3.07 KB, patch)
2025-02-07 16:52 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 38984: Specific - tools/batch_extend_due_dates.tt (2.18 KB, patch)
2025-02-07 16:52 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 38984: (follow-up) Fix syntax error in invoice-files table (981 bytes, patch)
2025-02-07 16:52 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 38984: (follow-up) Fix nosort class on first column (2.25 KB, patch)
2025-02-07 16:52 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 38984: (follow-up) Replace obsolete "paginate" option with "paging" (25.40 KB, patch)
2025-02-07 17:21 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 38984: Remove NoSort def (24.20 KB, patch)
2025-02-12 16:01 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38984: Replace nosort with NoSort (11.38 KB, patch)
2025-02-12 16:02 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38984: Remove anti-the (25.46 KB, patch)
2025-02-12 16:02 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38984: Add NoSort when can replace targets (49.29 KB, patch)
2025-02-12 16:02 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38984: Allow sort on all fields - club-enrollments (1.06 KB, patch)
2025-02-12 16:02 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38984: Also make 'Actions' translatable (2.30 KB, patch)
2025-02-12 16:02 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38984: Specific - tools/batch_extend_due_dates.tt (2.23 KB, patch)
2025-02-12 16:02 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38984: (follow-up) Fix syntax error in invoice-files table (1.64 KB, patch)
2025-02-12 16:02 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38984: (follow-up) Fix nosort class on first column (2.30 KB, patch)
2025-02-12 16:02 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38984: (follow-up) Replace obsolete "paginate" option with "paging" (25.62 KB, patch)
2025-02-12 16:02 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38984: Remove NoSort def (24.19 KB, patch)
2025-03-07 11:35 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 38984: Replace nosort with NoSort (11.38 KB, patch)
2025-03-07 11:35 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 38984: Remove anti-the (25.23 KB, patch)
2025-03-07 11:35 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 38984: Add NoSort when can replace targets (48.20 KB, patch)
2025-03-07 11:35 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 38984: Allow sort on all fields - club-enrollments (1.06 KB, patch)
2025-03-07 11:35 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 38984: Also make 'Actions' translatable (2.30 KB, patch)
2025-03-07 11:35 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 38984: Specific - tools/batch_extend_due_dates.tt (2.23 KB, patch)
2025-03-07 11:35 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 38984: (follow-up) Fix syntax error in invoice-files table (1.64 KB, patch)
2025-03-07 11:36 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 38984: (follow-up) Fix nosort class on first column (2.30 KB, patch)
2025-03-07 11:36 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 38984: (follow-up) Replace obsolete "paginate" option with "paging" (24.96 KB, patch)
2025-03-07 11:36 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 38984: Uniformize noSort NoExport NoExport (149.19 KB, patch)
2025-04-02 11:50 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38984: Remove NoSort def (24.30 KB, patch)
2025-04-02 13:37 UTC, Martin Renvoize (ashimema)
Details | Diff | Splinter Review
Bug 38984: Replace nosort with NoSort (11.50 KB, patch)
2025-04-02 13:38 UTC, Martin Renvoize (ashimema)
Details | Diff | Splinter Review
Bug 38984: Remove anti-the (25.50 KB, patch)
2025-04-02 13:38 UTC, Martin Renvoize (ashimema)
Details | Diff | Splinter Review
Bug 38984: Add NoSort when can replace targets (48.32 KB, patch)
2025-04-02 13:38 UTC, Martin Renvoize (ashimema)
Details | Diff | Splinter Review
Bug 38984: Allow sort on all fields - club-enrollments (1.17 KB, patch)
2025-04-02 13:38 UTC, Martin Renvoize (ashimema)
Details | Diff | Splinter Review
Bug 38984: Also make 'Actions' translatable (2.41 KB, patch)
2025-04-02 13:38 UTC, Martin Renvoize (ashimema)
Details | Diff | Splinter Review
Bug 38984: Specific - tools/batch_extend_due_dates.tt (2.35 KB, patch)
2025-04-02 13:38 UTC, Martin Renvoize (ashimema)
Details | Diff | Splinter Review
Bug 38984: (follow-up) Fix syntax error in invoice-files table (1.75 KB, patch)
2025-04-02 13:38 UTC, Martin Renvoize (ashimema)
Details | Diff | Splinter Review
Bug 38984: (follow-up) Fix nosort class on first column (2.41 KB, patch)
2025-04-02 13:38 UTC, Martin Renvoize (ashimema)
Details | Diff | Splinter Review
Bug 38984: (follow-up) Replace obsolete "paginate" option with "paging" (25.07 KB, patch)
2025-04-02 13:38 UTC, Martin Renvoize (ashimema)
Details | Diff | Splinter Review
Bug 38984: Uniformize noSort NoExport NoExport (149.21 KB, patch)
2025-04-02 13:38 UTC, Martin Renvoize (ashimema)
Details | Diff | Splinter Review
Bug 38984: Remove NoSort def (24.30 KB, patch)
2025-04-16 16:03 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 38984: Replace nosort with NoSort (11.55 KB, patch)
2025-04-16 16:03 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 38984: Remove anti-the (25.52 KB, patch)
2025-04-16 16:03 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 38984: Add NoSort when can replace targets (48.32 KB, patch)
2025-04-16 16:03 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 38984: Allow sort on all fields - club-enrollments (1.17 KB, patch)
2025-04-16 16:03 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 38984: Also make 'Actions' translatable (2.41 KB, patch)
2025-04-16 16:03 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 38984: Specific - tools/batch_extend_due_dates.tt (2.35 KB, patch)
2025-04-16 16:03 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 38984: (follow-up) Fix syntax error in invoice-files table (1.75 KB, patch)
2025-04-16 16:03 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 38984: (follow-up) Fix nosort class on first column (2.41 KB, patch)
2025-04-16 16:03 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 38984: (follow-up) Replace obsolete "paginate" option with "paging" (25.07 KB, patch)
2025-04-16 16:04 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 38984: Uniformize noSort NoExport NoExport (149.24 KB, patch)
2025-04-16 16:04 UTC, Owen Leonard
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-01-28 10:57:49 UTC
We don't need to have a columnDefs for NoSort or anti-the, they are already defined in the global datatables.js
Comment 1 Jonathan Druart 2025-01-28 11:01:07 UTC Comment hidden (obsolete)
Comment 2 Jonathan Druart 2025-01-28 11:01:10 UTC Comment hidden (obsolete)
Comment 3 Jonathan Druart 2025-01-28 11:01:12 UTC Comment hidden (obsolete)
Comment 4 Jonathan Druart 2025-01-28 11:01:15 UTC Comment hidden (obsolete)
Comment 5 Jonathan Druart 2025-01-28 11:01:17 UTC Comment hidden (obsolete)
Comment 6 Jonathan Druart 2025-01-28 11:01:20 UTC Comment hidden (obsolete)
Comment 7 Jonathan Druart 2025-01-28 11:01:22 UTC Comment hidden (obsolete)
Comment 8 Jonathan Druart 2025-01-28 11:02:27 UTC
Also tempted to rename NoSort with no-sort and NoExport with no-export, to match string-sort, anti-the, etc.
Comment 9 Jonathan Druart 2025-01-28 11:03:29 UTC
(In reply to Jonathan Druart from comment #8)
> Also tempted to rename NoSort with no-sort and NoExport with no-export, to
> match string-sort, anti-the, etc.

And it's "noExport"...
Comment 10 Jonathan Druart 2025-01-28 15:19:32 UTC Comment hidden (obsolete)
Comment 11 Jonathan Druart 2025-01-28 15:19:34 UTC Comment hidden (obsolete)
Comment 12 Jonathan Druart 2025-01-28 15:19:37 UTC Comment hidden (obsolete)
Comment 13 Jonathan Druart 2025-01-28 15:19:39 UTC Comment hidden (obsolete)
Comment 14 Jonathan Druart 2025-02-06 20:15:41 UTC Comment hidden (obsolete)
Comment 15 Jonathan Druart 2025-02-06 20:15:44 UTC Comment hidden (obsolete)
Comment 16 Owen Leonard 2025-02-07 16:52:10 UTC Comment hidden (obsolete)
Comment 17 Owen Leonard 2025-02-07 16:52:13 UTC Comment hidden (obsolete)
Comment 18 Owen Leonard 2025-02-07 16:52:15 UTC Comment hidden (obsolete)
Comment 19 Owen Leonard 2025-02-07 16:52:17 UTC Comment hidden (obsolete)
Comment 20 Owen Leonard 2025-02-07 16:52:19 UTC Comment hidden (obsolete)
Comment 21 Owen Leonard 2025-02-07 16:52:21 UTC Comment hidden (obsolete)
Comment 22 Owen Leonard 2025-02-07 16:52:23 UTC Comment hidden (obsolete)
Comment 23 Owen Leonard 2025-02-07 16:52:24 UTC Comment hidden (obsolete)
Comment 24 Owen Leonard 2025-02-07 16:52:26 UTC Comment hidden (obsolete)
Comment 25 Owen Leonard 2025-02-07 17:21:51 UTC Comment hidden (obsolete)
Comment 26 Jonathan Druart 2025-02-12 16:01:57 UTC Comment hidden (obsolete)
Comment 27 Jonathan Druart 2025-02-12 16:02:00 UTC Comment hidden (obsolete)
Comment 28 Jonathan Druart 2025-02-12 16:02:03 UTC Comment hidden (obsolete)
Comment 29 Jonathan Druart 2025-02-12 16:02:06 UTC Comment hidden (obsolete)
Comment 30 Jonathan Druart 2025-02-12 16:02:09 UTC Comment hidden (obsolete)
Comment 31 Jonathan Druart 2025-02-12 16:02:11 UTC Comment hidden (obsolete)
Comment 32 Jonathan Druart 2025-02-12 16:02:14 UTC Comment hidden (obsolete)
Comment 33 Jonathan Druart 2025-02-12 16:02:16 UTC Comment hidden (obsolete)
Comment 34 Jonathan Druart 2025-02-12 16:02:18 UTC Comment hidden (obsolete)
Comment 35 Jonathan Druart 2025-02-12 16:02:22 UTC Comment hidden (obsolete)
Comment 36 Owen Leonard 2025-03-07 11:35:45 UTC Comment hidden (obsolete)
Comment 37 Owen Leonard 2025-03-07 11:35:47 UTC Comment hidden (obsolete)
Comment 38 Owen Leonard 2025-03-07 11:35:49 UTC Comment hidden (obsolete)
Comment 39 Owen Leonard 2025-03-07 11:35:51 UTC Comment hidden (obsolete)
Comment 40 Owen Leonard 2025-03-07 11:35:53 UTC Comment hidden (obsolete)
Comment 41 Owen Leonard 2025-03-07 11:35:56 UTC Comment hidden (obsolete)
Comment 42 Owen Leonard 2025-03-07 11:35:58 UTC Comment hidden (obsolete)
Comment 43 Owen Leonard 2025-03-07 11:36:01 UTC Comment hidden (obsolete)
Comment 44 Owen Leonard 2025-03-07 11:36:03 UTC Comment hidden (obsolete)
Comment 45 Owen Leonard 2025-03-07 11:36:06 UTC Comment hidden (obsolete)
Comment 46 Martin Renvoize (ashimema) 2025-04-02 10:09:16 UTC
I'd love to also see the suggested NoSort -> no-sort and NoExport to no-export change.. that would be nice
Comment 47 Jonathan Druart 2025-04-02 11:39:14 UTC
We are missing noExport from datatables.js
Comment 48 Jonathan Druart 2025-04-02 11:43:23 UTC
(In reply to Jonathan Druart from comment #47)
> We are missing noExport from datatables.js

Forget that, it's something else.
Comment 49 Jonathan Druart 2025-04-02 11:50:21 UTC Comment hidden (obsolete)
Comment 50 Martin Renvoize (ashimema) 2025-04-02 13:37:59 UTC Comment hidden (obsolete)
Comment 51 Martin Renvoize (ashimema) 2025-04-02 13:38:03 UTC Comment hidden (obsolete)
Comment 52 Martin Renvoize (ashimema) 2025-04-02 13:38:06 UTC Comment hidden (obsolete)
Comment 53 Martin Renvoize (ashimema) 2025-04-02 13:38:09 UTC Comment hidden (obsolete)
Comment 54 Martin Renvoize (ashimema) 2025-04-02 13:38:13 UTC Comment hidden (obsolete)
Comment 55 Martin Renvoize (ashimema) 2025-04-02 13:38:16 UTC Comment hidden (obsolete)
Comment 56 Martin Renvoize (ashimema) 2025-04-02 13:38:19 UTC Comment hidden (obsolete)
Comment 57 Martin Renvoize (ashimema) 2025-04-02 13:38:22 UTC Comment hidden (obsolete)
Comment 58 Martin Renvoize (ashimema) 2025-04-02 13:38:25 UTC Comment hidden (obsolete)
Comment 59 Martin Renvoize (ashimema) 2025-04-02 13:38:28 UTC Comment hidden (obsolete)
Comment 60 Martin Renvoize (ashimema) 2025-04-02 13:38:32 UTC Comment hidden (obsolete)
Comment 61 Martin Renvoize (ashimema) 2025-04-02 13:41:26 UTC
Thanks for the tidy up here Jonathan :)
Comment 62 Katrin Fischer 2025-04-11 06:31:05 UTC
Can you please rebase? There is a conflict in sco-main I am not sure about.
Comment 63 Owen Leonard 2025-04-16 16:03:36 UTC
Created attachment 181021 [details] [review]
Bug 38984: Remove NoSort def

No need to pass it to the constructor as it's defined already in
datatables.js

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk>
Comment 64 Owen Leonard 2025-04-16 16:03:39 UTC
Created attachment 181022 [details] [review]
Bug 38984: Replace nosort with NoSort

We don't need nosort and NoSort, use NoSort everywhere

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk>
Comment 65 Owen Leonard 2025-04-16 16:03:42 UTC
Created attachment 181023 [details] [review]
Bug 38984: Remove anti-the

Same as previouly for anti-the

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk>
Comment 66 Owen Leonard 2025-04-16 16:03:45 UTC
Created attachment 181024 [details] [review]
Bug 38984: Add NoSort when can replace targets

In those occurrences we can simply use NoSort. It fixes some bugs, where
the display of the column depends on some pref's values

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk>
Comment 67 Owen Leonard 2025-04-16 16:03:47 UTC
Created attachment 181025 [details] [review]
Bug 38984: Allow sort on all fields - club-enrollments

it does not make sense to remove sorting only for the last 2 fields

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk>
Comment 68 Owen Leonard 2025-04-16 16:03:50 UTC
Created attachment 181026 [details] [review]
Bug 38984: Also make 'Actions' translatable

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk>
Comment 69 Owen Leonard 2025-04-16 16:03:52 UTC
Created attachment 181027 [details] [review]
Bug 38984: Specific - tools/batch_extend_due_dates.tt

Why no sort on item type? Why no sort on #checkouts_result?

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk>
Comment 70 Owen Leonard 2025-04-16 16:03:55 UTC
Created attachment 181028 [details] [review]
Bug 38984: (follow-up) Fix syntax error in invoice-files table

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk>
Comment 71 Owen Leonard 2025-04-16 16:03:58 UTC
Created attachment 181029 [details] [review]
Bug 38984: (follow-up) Fix nosort class on first column

Changing "nosort" to "NoSort". Also adding an "order" option to the
table configuration so that the table isn't sorted by default on the
first column (overriding the NoSort option).

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk>
Comment 72 Owen Leonard 2025-04-16 16:04:03 UTC
Created attachment 181030 [details] [review]
Bug 38984: (follow-up) Replace obsolete "paginate" option with "paging"

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk>
Comment 73 Owen Leonard 2025-04-16 16:04:07 UTC
Created attachment 181031 [details] [review]
Bug 38984: Uniformize noSort NoExport NoExport

Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk>
Comment 74 Katrin Fischer 2025-04-22 06:39:09 UTC
Maybe we could add a little summary for devs on what to use when now?
Comment 75 Katrin Fischer 2025-04-22 06:49:47 UTC
Pushed for 25.05!

Well done everyone, thank you!
Comment 76 Owen Leonard 2025-04-22 16:38:21 UTC
(In reply to Katrin Fischer from comment #74)
> Maybe we could add a little summary for devs on what to use when now?

I've done a little updating to this page: https://wiki.koha-community.org/wiki/DataTables_HowTo. There's probably more to be done.
Comment 77 Katrin Fischer 2025-04-22 20:06:58 UTC
(In reply to Owen Leonard from comment #76)
> (In reply to Katrin Fischer from comment #74)
> > Maybe we could add a little summary for devs on what to use when now?
> 
> I've done a little updating to this page:
> https://wiki.koha-community.org/wiki/DataTables_HowTo. There's probably more
> to be done.

That's awesome thanks :)

Wondering if we can delete this line and fix the file name?

WARNING: H::T::P has been dropped from the official codebase so what follows is not applicable to current coding standards... you have been warned.

koha-tmpl/intranet-tmpl/prog/en/modules/members/tables/issues.tmpl:
Comment 78 Jonathan Druart 2025-04-23 07:40:21 UTC
(In reply to Katrin Fischer from comment #77)
> (In reply to Owen Leonard from comment #76)
> > (In reply to Katrin Fischer from comment #74)
> > > Maybe we could add a little summary for devs on what to use when now?
> > 
> > I've done a little updating to this page:
> > https://wiki.koha-community.org/wiki/DataTables_HowTo. There's probably more
> > to be done.
> 
> That's awesome thanks :)
> 
> Wondering if we can delete this line and fix the file name?
> 
> WARNING: H::T::P has been dropped from the official codebase so what follows
> is not applicable to current coding standards... you have been warned.
> 
> koha-tmpl/intranet-tmpl/prog/en/modules/members/tables/issues.tmpl:

Yes, and add an "OBSOLETE" flag at the top of the page as well... It needs to be rewritten.