Database strings not marked for translation

Bug #1160347 reported by Pasi Kallinen
14
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Medium
Unassigned

Bug Description

*EG master

Strings not marked for translation in 950.data.seed-values.sql:

* 'Maximum visible age of User Trigger Events in Staff Interfaces'
* 'If this is unset, staff can view User Trigger Events regardless of age. When this is set to an interval, it represents the age of the oldest possible User Trigger Event that can be viewed.'
* 'Kid''s OPAC Search Filter'
* 'Children''s Materials'
* 'Young Adult Materials'
* 'General/Adult Materials'
* container.copy_bucket_type with label 'Circulation History'

* authority.browse_axis inserts
* config.index_normalizer inserts
* config.marc21_physical_characteristic_value_map inserts
* config.marc21_physical_characteristic_type_map inserts
* some of the config.usr_setting_type inserts
* vandelay.merge_profile inserts

Ben Shum (bshum)
Changed in evergreen:
status: New → Triaged
Revision history for this message
Ben Shum (bshum) wrote :

Using this bug to track other problems with the database seed values and i18n issues.

There are mismatches in the table for entries like:

INSERT INTO config.record_attr_definition (name,label,fixed_field,description) values ('audience','Audn','Audn', oils_i18n_gettext('audience', 'Audience', 'crad', 'label'));

Where the "description" is being translated with an ID of "label" instead, leading to messiness.

Let's make this bug a better overhaul of what's in the seed data values at large and getting things in better shape for i18n purposes.

Changed in evergreen:
status: Triaged → Confirmed
assignee: nobody → Ben Shum (bshum)
importance: Undecided → Medium
milestone: none → 3.0-alpha
Revision history for this message
Ben Shum (bshum) wrote :

I have finished a working branch that addresses most of the issues noted.

See: user/bshum/lp1160347-i18n-fixes-for-db.seed

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/bshum/lp1160347-i18n-fixes-for-db.seed

The following three areas I was unsure about though:

* config.index_normalizer inserts
* config.marc21_physical_characteristic_value_map inserts
* config.marc21_physical_characteristic_type_map inserts

My question is how would these be visible to end-users?

Changed in evergreen:
assignee: Ben Shum (bshum) → nobody
Revision history for this message
Ben Shum (bshum) wrote :

Okay, in discussion with gmcharlt, determined that the config.marc21_* stuff is visible to end users as part of the catalog editors in the web client. That said, introducing that many new translations that are specific to cataloging only, I'm not 100% sure on yet.

Rebased the existing branch back onto master for more relevant testing of the fixes thus far.

Revision history for this message
Ben Shum (bshum) wrote :

Oh, good, there's bug 1661327 (https://bugs.launchpad.net/evergreen/+bug/1661327) that's already tracking the coded value map translations not being done. I'll use that to track that future work and consider the fixes already on here as ready for pullrequest then.

tags: added: pullrequest
Changed in evergreen:
assignee: nobody → Chris Sharp (chrissharp123)
Revision history for this message
Chris Sharp (chrissharp123) wrote :
tags: added: signedoff
Revision history for this message
Galen Charlton (gmc) wrote :

Pushed to master. Thanks, Ben and Chris!

Changed in evergreen:
assignee: Chris Sharp (chrissharp123) → nobody
status: Confirmed → Fix Committed
Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.