Bug 40111 - Fix title sorting on two reports
Summary: Fix title sorting on two reports
Status: Pushed to oldstable
Alias: None
Product: Koha
Classification: Unclassified
Component: Templates (show other bugs)
Version: Main
Hardware: All All
: P5 - low minor
Assignee: Owen Leonard
QA Contact: Marcel de Rooy
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2025-06-10 15:47 UTC by Owen Leonard
Modified: 2025-07-31 12:51 UTC (History)
3 users (show)

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


Attachments
Bug 40111: Fix title sorting on two reports (2.24 KB, patch)
2025-06-10 15:56 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 40111: Fix title sorting on two reports (2.39 KB, patch)
2025-06-11 13:05 UTC, OpenFifth Sandboxes
Details | Diff | Splinter Review
Bug 40111: Fix title sorting on two reports (2.48 KB, patch)
2025-07-09 14:28 UTC, Marcel de Rooy
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Owen Leonard 2025-06-10 15:47:58 UTC
Two reports output titles but don't use the DataTable "anti-the" option for sorting titles while excluding articles: Items lost and orders by fund.
Comment 1 Owen Leonard 2025-06-10 15:56:28 UTC
Created attachment 183140 [details] [review]
Bug 40111: Fix title sorting on two reports

This patch modifies two reports in order to correct sorting on title
columns.

To test, apply the patch and got to reports -> Lost items.

- Run the report with parameters that will return multiple results.
  - Test sorting the title column. The articles 'a', 'an', and 'the'
    should be ignored.
- Perform the same test with the "Orders by fund" report.

Sponsored-by: Athens County Public Libraries
Comment 2 OpenFifth Sandboxes 2025-06-11 13:05:24 UTC
Created attachment 183160 [details] [review]
Bug 40111: Fix title sorting on two reports

This patch modifies two reports in order to correct sorting on title
columns.

To test, apply the patch and got to reports -> Lost items.

- Run the report with parameters that will return multiple results.
  - Test sorting the title column. The articles 'a', 'an', and 'the'
    should be ignored.
- Perform the same test with the "Orders by fund" report.

Sponsored-by: Athens County Public Libraries
Signed-off-by: Anneli Österman <anneli.osterman@koha-suomi.fi>
Comment 3 Marcel de Rooy 2025-07-09 14:28:40 UTC
Created attachment 183894 [details] [review]
Bug 40111: Fix title sorting on two reports

This patch modifies two reports in order to correct sorting on title
columns.

To test, apply the patch and got to reports -> Lost items.

- Run the report with parameters that will return multiple results.
  - Test sorting the title column. The articles 'a', 'an', and 'the'
    should be ignored.
- Perform the same test with the "Orders by fund" report.

Sponsored-by: Athens County Public Libraries
Signed-off-by: Anneli Österman <anneli.osterman@koha-suomi.fi>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 4 Lucas Gass (lukeg) 2025-07-11 21:29:35 UTC
Nice work everyone!

Pushed to main for 25.11
Comment 5 Paul Derscheid 2025-07-24 08:20:16 UTC
Nice work everyone!

Pushed to 25.05.x
Comment 6 Fridolin Somers 2025-07-31 12:48:56 UTC
Pushed to 24.11.x for 24.11.08