Fatal error: [] operator not supported for strings in includes/common.inc on line 2320

Although I'm unable to reproduce the fatal error, the current code in translation.module is indeed not correct. The language switch links get themed by theme_links() and theme_links() expects an array for the class value.

Setting status to needs review.

Tests are probably using the CSS classes. Since attribues['class'] is now an array, some links will get an extra 'language-link' class. I'll look into that.

I have the same problem as described.
I think that Patch 1408636-translation-attributes-class-array-1.patch is a good patch. c31ck has described that the patch failed testing because attributes['class'] is now an array. Following code in translations.test triggers the failures:
$xpath = '//div[contains(@class, :type)]//span[@class="locale-untranslated"]';

Trying this one, passes tests locally fine.

Yes, that one will probably pass the tests, but it doesn't respect any classes that have already been set on the link. Is that the behaviour we want here?

Hmm, you're right, let me try again locally.

Ok, better patch + test fixing

+++ b/core/modules/translation/translation.testundefined
@@ -437,7 +437,7 @@ class TranslationTestCase extends DrupalWebTestCase {
+        $xpath = '//div[contains(@class, :type)]//span[@class="language-link locale-untranslated"]';

Since we no longer replace all classes with "locale-untranslated", but rather add it as an additional class, this could still fail if classes other than language-link are added, no?

What about

$xpath = '//div[contains(@class, :type)]//span[contains(@class, "locale-untranslated")]';

-18 days to next Drupal core point release.

Absolutely right, better patch.

Looks great. Setting to RTBC based on #5, #10 and my own review.
Adding a D7 patch.

I agree that this is a bug. Committed to 8.x. Moving to 7.x for backporting.

Patch for D7.

Patch #14 also works for 7.12 :-)

#14 works. Considering the simplicity of the patch, it seems safe to mark it as reviewed.

i can confirm both : the bug and the solution #14.

#14 works fine. Please commit this patch. Related issue in i18n issue queue: #1406594: Fatal error: [] operator not supported for strings in includes/common.inc on line 2320.

Committed and pushed to 7.x. Thanks!

#1450298: Error locale-untranslated in translation module / hook language_switch_links_alter was marked as a duplicate of this issue.

Thx guys ! A hour ago this error hit me. Good to know it's ALREADY fixed :)

It's cool. 1408636-9.patch is worked.

I get the same error on all user pages for anonymous users:
»user«,
»user/register«, etc

I applied the patch in #14, but this patch did not solve my problem completely. Know the »user« site seems to work but not the other sites like »user/register« etc.

I do not get any further reports (on admin/reports/dblog). Don't know how to debug. What could I do?

post scriptum: Know I figured out, that the issue's cause are these lines in one of my custom modules:

    $account = l(t('Login'), 'user/login',          array('attributes' => array('class' => 'light login')));
    $login      = l(t('Register'), 'user/register', array('attributes' => array('class' => 'light register')));

The link creation is wrong, must be:

    $account = l(t('Login'), 'user/login',     array('attributes' => array('class' => array('light account'))));
    $login = l(t('Register'), 'user/register', array('attributes' => array('class' => array('light logout'))));

I'am sorry to bother you, was my fault!