When enabling the i18n Internationalization module (https://www.drupal.org/project/i18n), some functionalities don't work as expected:

i18n_menu:

  • multi-language menu: filtering language-specific items per language doesn't work (shows all items for all languages)
  • language-specific pages can be viewed by uri in other languages, e.g.: /en/this-page-is-in-english is also viewable at /nl/this-page-is-in-english, also showing the English content. This creates unwanted duplicate content.

i18n_variable:

  • Site custom footer texts cannot be made translatable with i18n_variable, because the variable is not declared with hook_variable_info().

CT Task

  • product_code_appointment unique fieldsettings should be changed to "language", instead of "all" to make the field unique per language.
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Tom Voesenek created an issue. See original summary.

TVoesenek’s picture

Added a patch to fix this.

TVoesenek’s picture

Status: Active » Needs review
TVoesenek’s picture

Issue summary: View changes
Status: Needs review » Needs work

Content type Task has an unique field setting for the field product_code_appointment, currently set to scope "all". This means the product code can only be used once for all languages, but it should be set to once per language so the product code can be linked to a translated version of the task.

TVoesenek’s picture

FileSize
2.28 KB

Updated the patch to fix the unique fields settings.

TVoesenek’s picture

Status: Needs work » Needs review
TVoesenek’s picture

Status: Needs review » Needs work

When adding translations to variables, e.g. used for the footer texts, the user should by default have 'administer site configuration'. For DvG this acces level is too high, because you don't want to give this permission to the super editor role.
The patch in this topic #2268415: Accessing multilanguage variables depends on 'administer site configuration' sets the permission to edit these variables to 'access administration pages', which is already assigned to the 'super editor'.

TVoesenek’s picture

Status: Needs work » Needs review
FileSize
2.76 KB

modified the previous patch to add the permissions patch to the make file.

TVoesenek’s picture

ruudvanoijen’s picture

Tested the patch with the latest dev branch. Looks good.

ralphvdhoudt’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.