Bug 40287 - Fix untranslatable strings in more statistics wizards
Summary: Fix untranslatable strings in more statistics wizards
Status: Needs Signoff
Alias: None
Product: Koha
Classification: Unclassified
Component: I18N/L10N (show other bugs)
Version: Main
Hardware: All All
: P5 - low normal
Assignee: Owen Leonard
QA Contact: Testopia
URL:
Keywords:
Depends on: 20601
Blocks:
  Show dependency treegraph
 
Reported: 2025-07-01 18:42 UTC by Owen Leonard
Modified: 2025-07-21 17:12 UTC (History)
2 users (show)

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


Attachments
Bug 40287: Fix untranslatable strings in more statistics wizards (21.25 KB, patch)
2025-07-17 13:45 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 40287: (follow-up) Use GetDescriptionByKohaField for item fields (4.69 KB, patch)
2025-07-21 17:12 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 Owen Leonard 2025-07-01 18:42:22 UTC
The acquisitions, patrons, and catalog circulation statistics wizards have the same translation issues fixed in Bug 20601
Comment 1 Owen Leonard 2025-07-17 13:45:21 UTC
Created attachment 184201 [details] [review]
Bug 40287: Fix untranslatable strings in more statistics wizards

This patch makes some updates to the acquisitions, patrons, and catalog
statistics wizards in order to make more information translatable.

To test apply the patch and go to Reports.

- Under "Statistics wizards," test the acquisitions, patrons,
  circulation, and catalog statistics reports:
  - Test setting some or all options in the "Filter column" and running
    the report.
  - At the top of the results there should be a list of the filter
    options you selected and their values.
  - All the filter labels should be correct, and the filter values
    should be in their full/description form rather than in code form
    (e.g. library, item type, collection code, sorting field, etc.)

Sponsored-by: Athens County Public Libraries
Comment 2 Jonathan Druart 2025-07-21 08:28:08 UTC
I think tou are supposed to use GetDescriptionByKohaField.

AuthorisedValues.GetByCode( 'CCODE', loopfilte.filter )

=>

AuthorisedValues.GetDescriptionByKohaField( kohafield => 'items.ccode', authorised_value => loopfilte.filter )

In case items.ccode would be linked to another AV category than CCODE.

Same for LOC, NOT_LOAN, etc.
Comment 3 Owen Leonard 2025-07-21 17:12:52 UTC
Created attachment 184451 [details] [review]
Bug 40287: (follow-up) Use GetDescriptionByKohaField for item fields