Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.The attached patch improves l10n_client's presence in the installer the following ways:
(a) it exposes a checkbox which lets you decide to enable it or not
(b) if you enable it, it lets you enable sharing with localize.drupal.org
(c) if you enable sharing, it lets you enter your API key for user 1
(a) and (b) were hardwired before, and some people got surprised l10n_client is even there. Now they can disable it on install. (c) was not even there, so people still needed to do manual steps to configure it after installation, and were probably not aware.
Especially good is that the attached code uses Drupal 7 form API #states, so (b) and (c) items are hidden if (a) is unchecked and (c) is hidden if (b) is unchecked. Kinda neat. (a) and (b) are checked by default. I also tried to make the textbox appear required when sharing is enabled, but that did not display nice due to a core bug (submitted at #1017882: Required elements buggy with #states).
I tested this a lot, and it all seems to work, so committing it right away.
| Comment | File | Size | Author |
|---|---|---|---|
| make-l10n-client-configurable.patch | 4.81 KB | gábor hojtsy |
Comments
Comment #1
gábor hojtsyNeeds to be backported to Drupal 6 if at all possible (probably not with the same lovely UI though unless we want to fill it all in with custom JS).
Comment #2
asrobI would close it because Drupal 6 EOL.