Comment 5 for bug 1682292

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

Tested the fix from Mike, but couldn't tell if it was working till I found the following path to success...

With that fix in place on the fm_IDL.xml file, we should expect that any database values matching the crad.label or crad.description in config.i18n_core (of which there aren't many) to display the relevant translated value as part of the filter entry.

So if I were to see a row like in config.i18n_core like:

4693 | crad.label | item_type | fr-CA | Type de document

That ought to show me the translated value for "item_type" as "Type de document" when looking at French's translation using the item type filter. However, I didn't see it, and it's because the base code says crad.description || crad.label. So if you only have the label, and not the description, it will still display the description as english only.

Inserting a new row for crad.description with a translated value made things display as I expected.

So not 100% sure if we should also swap around the advanced search filter to be crad.label || crad.description (since I think label ought to be more commonly employed than descriptions in this area), but that can be the subject of another bug in the future.

Pushing fix to master and backporting to rel_2_12. Thanks Mike!