Bug 40287

Summary: Fix untranslatable strings in more statistics wizards
Product: Koha Reporter: Owen Leonard <oleonard>
Component: I18N/L10NAssignee: Owen Leonard <oleonard>
Status: Needs Signoff --- QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: f.demians, jonathan.druart
Version: Main   
Hardware: All   
OS: All   
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:
Bug Depends on: 20601    
Bug Blocks:    
Attachments: Bug 40287: Fix untranslatable strings in more statistics wizards
Bug 40287: (follow-up) Use GetDescriptionByKohaField for item fields

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