With #2311585: Regression: 'Show more/fewer' links are no longer translatable, facetapi module implements hook_i18n_string_info().

In admin/config/regional/translate/i18n_string, 'Facetapi' checkbox label is not display anymore since both facetapi and facetapi_i18n implement hook_i18n_string_info() with same group name 'facetapi'. Groups are merged and all values are array instead of string.

We should remove or rename facetapi group in facetapi_i18n_i18n_string_info().

Note: Refresh is still working thanks to #2466099: Enable "Refresh strings" for Facetapi

CommentFileSizeAuthor
facetapi-i18n-group.png140.81 KBGoZ
facetapi-label-array.png10.35 KBGoZ
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

GoZ created an issue. See original summary.

GoZ’s picture

GoZ CreditAttribution: GoZ at Centarro commented
Issue summary: View changes
donquixote’s picture

donquixote CreditAttribution: donquixote as a volunteer commented

Yes, renaming the group would be a solution.

Groups are merged and all values are array instead of string.

It is sad that module_invoke_all() works this way.
Just for the record: We can blame array_merge_recursive(). See
#791860: array_merge_recursive() is never what we want in Drupal: add a drupal_array_merge_recursive() function instead.
#1356170: Remove all uses of array_merge_recursive, or document why they are being used instead of NestedArray::mergeDeep()
This has not been fixed in D7, unfortunately. And doing so would be risky I assume, some modules might depend on this weird behavior.